From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Geert Uytterhoeven <geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org>
Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: gpio-generic DT bindings?
Date: Tue, 19 Nov 2013 15:38:20 -0700 [thread overview]
Message-ID: <528BE85C.60101@wwwdotorg.org> (raw)
In-Reply-To: <CAMuHMdVuC5nJizr9Q2JXQCC=FtJkEvLa5iftBoa6Zd7B3XVojA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On 11/19/2013 03:17 PM, Geert Uytterhoeven wrote:
> On Tue, Nov 19, 2013 at 10:59 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> On 11/19/2013 02:45 PM, Geert Uytterhoeven wrote:
>>> This topic seems to come up from time to time.
>>> Unfortunately the last time it coincided with the move of the mailing list
>>> from ozlabs to vger, causing the mailing list archives not to have captured
>>> the full discussion.
>>>
>>> Is there anything definitive/usable out there?
>>
>> What do you mean by "gpio-generic DT bindings"? A generic binding for a
>> controller, or something else? I think it's best to have a specific
>> binding for each individual controller, so it's always possible to know
>> exactly which controller is present. Now, all the binding definitions
>> should all look the same or as similar as possible for consistency...
>
> I mean DT bindings and DT support for drivers/gpio/gpio-generic.c.
We should have DT bindings for particular HW, not for a driver. After
all, DT describes HW, not a particular OS's driver.
The path to adding DT support to gpio-generic.c is to define a binding
for the particular HW you're interested in (which would quite likely
only contain compatible, reg, and perhaps some other standard properties
like interrupts, clocks, power domains, etc.). There would be one
binding and compatible value for each different HW block you want to
support, although they could all share the same schema and definition.
Then update gpio-generic.c to bind to that (those) compatible value(s),
and have some kind of table that maps from compatible value to whatever
configuration structure gpio-generic.c uses internally.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2013-11-19 22:38 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-19 21:45 gpio-generic DT bindings? Geert Uytterhoeven
[not found] ` <CAMuHMdU9F_EjA+ZF0mn4PW3HDJ9rkM_84SHiDKkWwWr7MLJKJA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-11-19 21:59 ` Stephen Warren
[not found] ` <528BDF57.8080400-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-11-19 22:17 ` Geert Uytterhoeven
[not found] ` <CAMuHMdVuC5nJizr9Q2JXQCC=FtJkEvLa5iftBoa6Zd7B3XVojA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-11-19 22:38 ` Stephen Warren [this message]
[not found] ` <528BE85C.60101-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-11-20 15:44 ` Tom Rini
2013-11-20 16:15 ` Stephen Warren
[not found] ` <528CE010.3090301-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-11-20 18:03 ` Tom Rini
2013-11-20 17:20 ` Geert Uytterhoeven
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=528BE85C.60101@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.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).