On Tue, Jan 27, 2015 at 09:16:45PM +0100, Arnd Bergmann wrote: > On Tuesday 27 January 2015 15:11:29 Zubair Lutfullah Kakakhel wrote: > > +clocks: phandle to rtcclk > > +clock-names: must be "rtc" > > + > > +Example: > > + > > +watchdog: jz47xx-watchdog@0x10002000 { > > + compatible = "ingenic,jz4780-watchdog"; > > + reg = <0x10002000 0x100>; > > + > > + clocks = <&rtclk>; > > + clock-names = "rtc"; > > +}; > > Why is the name "rtc"? are you sure you are not confusing the output > name of the clock controller with the input of the watchdog device? > Driver does this (today): drvdata->rtc_clk = clk_get(&pdev->dev, "rtc"); Isn't that the name to use ? Just wondering. Thanks, Guenter -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html