Re[2]: [PATCH] ARM: dts: imx: fix clocks for cspi

[shc_work@mail.ru (Alexander Shiyan)]

> On Thu, May 23, 2013 at 04:10:59PM +0400, Alexander Shiyan wrote:
> > > > The CSPI controller has only one clock, but the driver spi-imx.c needs clock "per" to calculate bitrate divisor.
> > > > 
> > > > Signed-off-by: Jonas Andersson <jonas@microbit.se>
> > > > ---
> > > >  arch/arm/boot/dts/imx25.dtsi | 12 ++++++------
> > > >  arch/arm/boot/dts/imx27.dtsi |  6 +++---
> > > >  arch/arm/boot/dts/imx51.dtsi |  2 +-
> > > >  arch/arm/boot/dts/imx53.dtsi |  2 +-
> > > >  4 files changed, 11 insertions(+), 11 deletions(-)
> > > > 
> > > > diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
> > > > index d2550e0..7011539 100644
> > > > --- a/arch/arm/boot/dts/imx25.dtsi
> > > > +++ b/arch/arm/boot/dts/imx25.dtsi
> > > > @@ -141,8 +141,8 @@
> > > >  				#size-cells = <0>;
> > > >  				compatible = "fsl,imx25-cspi", "fsl,imx35-cspi";
> > > >  				reg = <0x43fa4000 0x4000>;
> > > > -				clocks = <&clks 62>;
> > > > -				clock-names = "ipg";
> > > > +				clocks = <&clks 62>, <&clks 62>;
> > > > +				clock-names = "ipg", "per";
> > > >  				interrupts = <14>;
> > > >  				status = "disabled";
> > > >  			};
> > 
> > Sorry, typo.
> > 
> > > Why you do not use "dummy" clock for "per" here?
> > 
> > *for "ipg"
> 
> Because the same clock is used for the register clock and the baudrate
> clock. You have to enable the ipg clock to access registers.

I am not see any "ipg" clock usage in the spi driver. If "ipg" clock is
a dependency for "per" clock, it should be registered in ccm as parent.
Is not it?

---