On Sun, Mar 18, 2018 at 12:14 AM, Anson Huang <Anson.Huang@xxxxxxx> wrote: > diff --git a/arch/arm/boot/dts/imx6sx-sabreauto.dts b/arch/arm/boot/dts/imx6sx-sabreauto.dts > index 72da5ac..83f7cac 100644 > --- a/arch/arm/boot/dts/imx6sx-sabreauto.dts > +++ b/arch/arm/boot/dts/imx6sx-sabreauto.dts > @@ -18,6 +18,14 @@ > reg = <0x80000000 0x80000000>; > }; > > + clocks { > + codec_osc: anaclk2 { > + #clock-cells = <0>; > + compatible = "fixed-clock"; > + clock-frequency = <24576000>; > + }; > + }; The clocks container should be removed and the clock node could be put directly into the root node. Also, I do not see 'codec_osc' being used in this dts. -- 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