All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
To: Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org"
	<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
	ksummit-2013-discuss-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
	Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
Subject: Re: [Ksummit-2013-discuss] [ATTEND] Handling of devicetree bindings
Date: Mon, 15 Jul 2013 09:56:20 -0700	[thread overview]
Message-ID: <20130715165620.GA29040@kroah.com> (raw)
In-Reply-To: <CACRpkdaJ5podVCrb7XuhhGF0Oco+eLrFH0EVGZM0MBbGP9m7WQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On Sun, Jul 14, 2013 at 01:46:45AM +0200, Linus Walleij wrote:
> On Sat, Jul 13, 2013 at 10:49 PM, Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> wrote:
> > On Sat, Jul 13, 2013 at 08:26:47PM +0100, Wolfram Sang wrote:
> 
> >> I think the KS would be a good opportunity to present the status quo,
> >> show some rules of thumb and finally discuss if we can improve the
> >> process even further. E.g., should first the bindings be accepted, then
> >> the driver? What could be the process if the need for a generic binding
> >> arises? And spreading the word, so at least the basic issues are
> >> understood by most maintainers.
> >>
> > Sounds like a good idea, but I think you'll need some deadline. When reviewing
> > hwmon drivers I usually wait for a couple of weeks if there is any feedback
> > from devicetree-discuss before I accept bindings, but what should I do
> > if there is no feedback ? Holding the driver hostage doesn't seem like a good
> > idea either.
> 
> Nobody rarely say anything on devicetree-discuss. And if something is
> said it will usually be about syntax rather than semantics. And if it's
> semantics, it's usually a subsystem or arch maintainer saying it.
> 
> I've been ranting a bit about this recently, and one of the problems with
> device tree now being driven by the Linux community (basically - it used
> to be a IEEE thing at one point in the past) is that as the bindings are
> merged by the subsystem maintainers, they alone get to decide when
> they are finished.
> 
> They are all in Documentation/devicetree/bindings/* so theoretically
> we could have a dedicated maintainer for it, but that requires someone
> to step up :-/
> 
> I wonder if some subsystem maintainers who are more used to things
> like ACPI or PCI where someone else has already done the thinking
> and written a large (non-perfect, mind you) specification even think that
> reviewing hardware descriptions is their problem at all?
> 
> Maybe some just consider this "some documentation", think it has been
> defined by someone who thought it over and merge it without looking
> closely.
> 
> There have been discussions on improving the situation recently,
> so maybe someone has a few words to add?

How about a hint for subsystem maintainers as to what exactly we should
be looking for with these bindings?  I for one have no idea what is
"right" vs. "wrong" with them, so a document explaining this would be
good to have.

Or if we already have it, a pointer to it perhaps?

thanks,

greg k-h

  parent reply	other threads:[~2013-07-15 16:56 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130713192647.GA3798@katana>
     [not found] ` <20130713204927.GA1124@roeck-us.net>
     [not found]   ` <20130713204927.GA1124-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2013-07-13 23:46     ` [Ksummit-2013-discuss] [ATTEND] Handling of devicetree bindings Linus Walleij
     [not found]       ` <CACRpkdaJ5podVCrb7XuhhGF0Oco+eLrFH0EVGZM0MBbGP9m7WQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-14 18:13         ` Guenter Roeck
     [not found]           ` <20130714181331.GC26513-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>
2013-07-15  8:36             ` Linus Walleij
     [not found]               ` <20130715082915.5142547a@lwn.net>
     [not found]                 ` <20130715082915.5142547a-T1hC0tSOHrs@public.gmane.org>
2013-07-15 14:59                   ` Guenter Roeck
2013-07-15 21:07                   ` Linus Walleij
     [not found]                     ` <CACRpkdaEswVPNqwQkMjsNWutCpOQxgMeGwsgemsF4SSZxXMocQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-20  5:51                       ` Grant Likely
     [not found]                     ` <20130719154004.GA3081@katana>
2013-07-21  7:42                       ` Guenter Roeck
2013-07-15 16:56         ` Greg KH [this message]
     [not found]           ` <20130715165620.GA29040-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2013-07-15 17:06             ` Mark Brown
     [not found]               ` <20130715170600.GW11538-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2013-07-15 18:41                 ` Greg KH
     [not found]                   ` <20130715184157.GB17220-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2013-07-15 19:58                     ` Mark Brown
2013-07-15 18:50             ` David Woodhouse
     [not found]               ` <1373914227.2128.24.camel-Fexsq3y4057IgHVZqg5X0TlWvGAXklZc@public.gmane.org>
2013-07-20  5:48                 ` Grant Likely
2013-07-20 16:16                 ` Linus Walleij
2013-07-21  8:05 Chaiken, Alison
     [not found] ` <60BA5429A0E1584BA3633194F6F993B502AF6D65-0dz9ie/QGrnnlEkxMdpx1dQH9K4/4qFeAL8bYrjMMd8@public.gmane.org>
2013-07-21 11:33   ` Linus Walleij
2013-07-21 16:53   ` David Woodhouse
     [not found]     ` <60BA5429A0E1584BA3633194F6F993B502B10DBC@NA-MBX-03.mgc.mentorg.com>
2013-07-22  6:37       ` Chaiken, Alison

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=20130715165620.GA29040@kroah.com \
    --to=greg-u8xffu+wg4eavxtiumwx3w@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=ksummit-2013-discuss-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=linux-0h96xk9xTtrk1uMJSBkQmQ@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.