RE: [PATCH V3 4/9] Documentation: devicetree: mfd: da9062/61 MFD binding

From: Steve Twiss
Date: Mon Nov 07 2016 - 10:28:52 EST


On 02 November 2016 14:32, Lee Jones wrote:

> On Mon, 31 Oct 2016, Steve Twiss wrote:
> > From: Steve Twiss <stwiss.opensource@xxxxxxxxxxx>
> >
> > [...]
> >
> > +- onkey : See ../input/da9062-onkey.txt
> > +
> > +
> > +- watchdog: See ../watchdog/da9062-watchdog.txt
> > +
> > +
> > +- thermal : See ../thermal/da9062-thermal.txt
>
> Remove all those extra lines in between.

Ok.

> > Example:
> > @@ -64,10 +96,6 @@ Example:
> > compatible = "dlg,da9062-rtc";
> > };
> >
> > - watchdog {
> > - compatible = "dlg,da9062-watchdog";
> > - };
> > -
>
> Why don't you include a sub-node per child device and use
> of_platform_populate() to register the devices?

In this case I'm just moving the binding descriptions into a
different file. I'm not sure I understand what you mean here.
What do you have in mind?

Regards,
Steve