[PATCH 2/2] ARM: dts: Add support for the i.MX35.

Sascha Hauer s.hauer at pengutronix.de
Thu Dec 19 02:38:44 EST 2013


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.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



More information about the linux-arm-kernel mailing list