[RFC v2 2/5] tps6586x: Add device tree support
Thierry Reding
thierry.reding at avionic-design.de
Wed Apr 25 20:14:29 EST 2012
* Mark Brown wrote:
> On Wed, Apr 25, 2012 at 11:44:59AM +0200, Thierry Reding wrote:
> > This commit adds device tree support for the TPS6586x regulator.
> >
> > Signed-off-by: Thierry Reding <thierry.reding at avionic-design.de>
>
> This looks basically good from a quick scan through but the pattern of
> looking up regulator nodes by name is very common so should be factored
> out - I made a similar comment in response to a recent patch from
> Rhyland Klein and earlier today he posted a patch "regulator: add
> generic of node parsing for regulators" which does just that. Can you
> please redo this on top of his code? I'll probably apply it later
> today, though I didn't properly read the code yet.
I'll take a look.
> I guess it should be possible to apply this patch independantly of the
> rest of the series? It shouldn't break bisection if it's missing as
> it's a new driver that's being added as the consumer.
Yes, it can be applied independently.
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/devicetree-discuss/attachments/20120425/3dc20287/attachment-0001.sig>
More information about the devicetree-discuss
mailing list