On 05/04, Sascha Hauer wrote: > On Thu, Apr 30, 2015 at 06:20:39PM -0700, Stephen Boyd wrote: > > On 04/23, Sascha Hauer wrote: > > > +The available clocks are defined in dt-bindings/clock/mt*-clk.h. > > > + > > > +Example: > > > + > > > +apmixedsys: apmixedsys@10209000 { > > > > apmixedsys: clock-controller@10209000 { > > > > would be more standard. The same comment applies throughout this > > patch. Otherwise it looks good to me. > > For apmixed this I agree, but the others are also reset controllers, so > I'm not sure if clock-controller is appropriate. Personally I don't care > much, I'll change to whatever you like. I've already applied the patch, so if you like you can send a follow up. Perhaps power-controller is more appropriate? I'm not too worried about it though. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- 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