On Tue, Apr 21, 2015 at 08:34:45PM +0100, Zhi Li wrote: > >>> + > >>> + clocks { > >>> + #address-cells = <1>; > >>> + #size-cells = <0>; > >>> + > >>> + ckil: clock@0 { > >>> + compatible = "fixed-clock"; > >>> + reg = <0>; > >>> + #clock-cells = <0>; > >>> + clock-frequency = <32768>; > >>> + clock-output-names = "ckil"; > >>> + }; > >>> + > >>> + osc: clock@1 { > >>> + compatible = "fixed-clock"; > >>> + reg = <1>; > >>> + #clock-cells = <0>; > >>> + clock-frequency = <24000000>; > >>> + clock-output-names = "osc"; > >>> + }; > >>> + > >>> + }; > >> > >> Get rid of the clocks container node, and put these under the root. > > > > Okay. > > After remove clocks node. > There will be warning > > Warning (reg_format): "reg" property in /clock@0 has invalid length (4 > bytes) (#address-cells == 1, #size-cells == 1) > Warning (reg_format): "reg" property in /clock@1 has invalid length (4 > bytes) (#address-cells == 1, #size-cells == 1) The reg property is meainingless, so just remove it and give the clocks different names. Mark. -- 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