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 ��.n��������+%������w��{.n�����{��)��^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�