From: Lee Jones <lee.jones@linaro.org>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: linus.walleij@stericsson.com, arnd@arndb.de,
Linus Walleij <linus.walleij@linaro.org>,
linux-kernel@vger.kernel.org, grant.likely@secretlab.ca,
linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 09/14] i2c: Add Device Tree support to the Nomadik I2C driver
Date: Thu, 14 Jun 2012 19:59:49 +0100 [thread overview]
Message-ID: <4FDA34A5.7070704@linaro.org> (raw)
In-Reply-To: <20120614184656.GD30185@sirena.org.uk>
On 14/06/12 19:46, Mark Brown wrote:
> On Thu, Jun 14, 2012 at 07:36:36PM +0100, Mark Brown wrote:
>
>> You're not understanding Linus' point. The compatible string isn't
>> useful here because properties like the maximum clock rate of the bus
>> depend on the board design, not the silicon. The controller may be
>> perfectly happy to run at a given rate but other devices on the bus or
>> the electrical engineering of the PCB itself may restrict this further.
>
> Sorry, I read the next revision and see this was actually resolved OK.
Yes, I just went ahead created the bindings anyway. I figured it would
be neater (if no more functional) to keep all the variations in DT.
Especially if we had devices which only varied by one or two settings,
which would still require a complete new struct using the previous method.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
M: +44 77 88 633 515
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2012-06-14 18:59 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1339428307-3850-1-git-send-email-lee.jones@linaro.org>
2012-06-11 15:25 ` [PATCH 09/14] i2c: Add Device Tree support to the Nomadik I2C driver Lee Jones
2012-06-11 19:05 ` Wolfram Sang
2012-06-12 7:23 ` Lee Jones
[not found] ` <20120611190550.GK3887-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2012-06-12 8:52 ` Lee Jones
[not found] ` <4FD70357.9060905-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-12 8:57 ` Wolfram Sang
[not found] ` <1339428307-3850-10-git-send-email-lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-11 20:37 ` Linus Walleij
[not found] ` <CACRpkdaMDbH4NkiHRLAfbJZ_j4QXbwg94bvaWsdYRPH+dSfc8Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-12 7:34 ` Lee Jones
[not found] ` <4FD6F0E8.5040606-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-13 5:40 ` Linus Walleij
[not found] ` <CACRpkdYxiT-0x4aetMxB2x8SkB+Mmy8rk8uSzgU_anemSayhMw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-13 7:01 ` Lee Jones
2012-06-13 8:12 ` Linus Walleij
[not found] ` <CACRpkdZC8E6izDpdnWy6DMjOdA6KsqTaPoaq9pErWoNW0Ewytw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-13 12:28 ` Lee Jones
[not found] ` <4FD88761.9050703-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-14 18:36 ` Mark Brown
[not found] ` <20120614183636.GB30185-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2012-06-14 18:46 ` Mark Brown
2012-06-14 18:59 ` Lee Jones [this message]
2012-06-14 18:57 ` Lee Jones
[not found] ` <4FDA341C.8010501-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-15 9:32 ` Mark Brown
2012-06-15 10:00 ` Lee Jones
2012-06-15 10:32 ` Russell King - ARM Linux
[not found] ` <20120615103233.GA19046-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2012-06-15 11:43 ` Lee Jones
2012-06-13 16:07 ` [PATCH 1/3] " Lee Jones
[not found] ` <4FD8BAD2.50703-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-14 17:12 ` Linus Walleij
[not found] ` <CACRpkdZFi34nChnMEo6yik67zk9owZLk-6zcdP7mmOXyLz8uqQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-15 9:02 ` Lee Jones
[not found] ` <CAE2-_9rRYxjU9QQgtBv9ReMY5x+oRiJG1cDQahYHanjDrwVUYA@mail.gmail.com>
[not found] ` <CAE2-_9rRYxjU9QQgtBv9ReMY5x+oRiJG1cDQahYHanjDrwVUYA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-15 11:50 ` Fwd: " Srinidhi Kasagar
2012-06-15 12:45 ` Lee Jones
[not found] ` <4FDB2E57.4030904-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-15 13:05 ` Srinidhi Kasagar
2012-06-15 13:18 ` Lee Jones
[not found] ` <4FDB3642.5030804-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-15 13:37 ` Srinidhi Kasagar
2012-06-15 13:58 ` Lee Jones
2012-06-17 17:43 ` Linus Walleij
[not found] ` <CACRpkdZ7ESKhokk8Z+6sC9Kq+jPztFNwzger0Db7nCD1fPnG1A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-18 7:18 ` Lee Jones
2012-06-18 7:58 ` Srinidhi Kasagar
2012-06-18 8:41 ` Lee Jones
2012-06-13 16:08 ` [PATCH 3/3] Documentation: Device Tree binding information for i2c-nomadik driver Lee Jones
[not found] ` <4FD8BAF8.10806-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-14 17:13 ` Linus Walleij
2012-06-13 16:08 ` [PATCH 2/3] ARM: ux500: Add i2c configurations to the Device Tree for DB8500 based devices Lee Jones
[not found] ` <4FD8BAE4.4050606-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2012-06-14 17:13 ` Linus Walleij
2012-06-18 11:29 ` Linus Walleij
[not found] ` <CACRpkda2pH1by2hajpP20CJgdg+mWFF=QaGbeNdY=c8iOVtLUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-18 11:37 ` Lee Jones
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FDA34A5.7070704@linaro.org \
--to=lee.jones@linaro.org \
--cc=arnd@arndb.de \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=grant.likely@secretlab.ca \
--cc=linus.walleij@linaro.org \
--cc=linus.walleij@stericsson.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).