[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130915112755.GB9765@ulmo>
Date: Sun, 15 Sep 2013 13:27:56 +0200
From: Thierry Reding <thierry.reding@...il.com>
To: Lee Jones <lee.jones@...aro.org>
Cc: Milo Kim <milo.kim@...com>, Samuel Ortiz <sameo@...ux.intel.com>,
linux-kernel@...r.kernel.org, linux-pwm@...r.kernel.org,
Linus Walleij <linus.walleij@...aro.org>,
devicetree@...r.kernel.org
Subject: Re: [PATCH v3 4/4] Documentation: add LP3943 DT bindings and document
On Thu, Sep 12, 2013 at 04:24:55PM +0100, Lee Jones wrote:
> On Thu, 12 Sep 2013, Milo Kim wrote:
[...]
> > diff --git a/Documentation/lp3943.txt b/Documentation/lp3943.txt
> > new file mode 100644
> > index 0000000..576ebd0
> > --- /dev/null
> > +++ b/Documentation/lp3943.txt
[...]
> If this stuff is really important (and most of it really isn't), then
> put it either in the commit log or in the driver.
I can only speak for myself, but the initial driver commit message is
about the last place I look for this kind of information. I'd expect it
to be either somewhere in Documentation/ or in one of the source files.
In this case I'd probably look at the MFD core driver source file. More
likely, even, I'd just go google "LP3943" and grab a public datasheet
from TI where all of this can hopefully be found.
Thierry
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists