Re: [PATCH] ARM: dts: i.MX51: Add WEIM node

[Alexander Shiyan]

> On Mon, Jul 08, 2013 at 08:29:43AM +0400, Alexander Shiyan wrote:
> > > > @@ -745,6 +745,23 @@
> > > >  				status = "disabled";
> > > >  			};
> > > >  
> > > > +			weim: weim at 83fda000 {
> > > > +				#address-cells = <2>;
> > > > +				#size-cells = <1>;
> > > > +				compatible = "fsl,imx51-weim";
> > > > +				reg = <0x83fda000 0x1000>;
> > > > +				clocks = <&clks 0>;
> > > 
> > > Is it correct?  I think there should be a physical clock for weim.
> > 
> > Yes, but properly clock for weim is missing in the clk subsystem,
> > so I think at this point it is OK. Adding a properly clock should be a separate patch.
> 
> I've seen emi_slow and emi_fast clocks implemented in clk-imx51-imx53.c.
> Neither is the clock for weim?

I reviewed the documentation, emi_fast clock is used for DDR, emi_slow for WEIM,
so will post the revised patch.
Thanks.

---