[PATCH 2/2] ARM: dts: Add support for the i.MX35.
Shawn Guo
shawn.guo at linaro.org
Thu Dec 19 04:06:57 EST 2013
On Thu, Dec 19, 2013 at 08:38:44AM +0100, Sascha Hauer wrote:
> On Thu, Dec 19, 2013 at 02:49:59PM +0800, Shawn Guo wrote:
> > On Wed, Dec 18, 2013 at 03:10:26PM +0100, Sascha Hauer wrote:
> > > + clocks {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + ckil {
> > > + compatible = "fsl,imx-ckil", "fixed-clock";
> > > + clock-frequency = <32768>;
> > > + };
> > > +
> > > + osc {
> > > + compatible = "fsl,imx-osc", "fixed-clock";
> > > + clock-frequency = <24000000>;
> > > + };
> >
> > I asked the question in the last round, but did not get any answer. Do
> > these two clocks exist on imx35? At least from the imx35 clock code,
> > I do not see them.
>
> ckil does exist, but it's optional, so I think if we specify it in the
> dtsi then the rate should be 0. It's unused in the code though and noone
> ever seemed to miss it, so we can equally well just drop it.
> osc does exist aswell, but in the code it's unused and a hardcoded 24MHz
> value is used, so specifying it in the devicetree is bogus.
> I think we should just drop the nodes.
Ok, I will fixed it up and apply.
Shawn
More information about the linux-arm-kernel
mailing list