On Thu, Feb 04, 2016 at 06:57:44PM -0500, Akshay Bhat wrote: > >>+/ { > >>+ clocks { > >>+ #address-cells = <1>; > >>+ #size-cells = <0>; > > > >These are already specified in imx6qdl.dtsi. > > > > Will fix this. Once it is removed, should I have a new line before > the child node? > eg: > > clocks { > No, this new line is not needed. Shawn > mclk: clock@0 { > compatible = "fixed-clock"; > >>+ > >>+ mclk: clock@0 { > >>+ compatible = "fixed-clock"; > >>+ reg = <0>; > >>+ #clock-cells = <0>; > >>+ clock-frequency = <22000000>; > >>+ }; > >>+ }; -- 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