Hi, On Tue, Dec 18, 2012 at 06:02:09PM +0530, Hebbar Gururaja wrote: > From: "Hebbar, Gururaja" <gururaja.heb...@ti.com> > > omap4_cminst_wait_module_ready() checks if register offset is NULL. > > int omap4_cminst_wait_module_ready(u8 part, u16 inst, s16 cdoffs, > u16 clkctrl_offs) > { > int i = 0; > > if (!clkctrl_offs) > return 0; > > In case of AM33xx, not all clock domains have CLKSTCTRL at 0x00. An > example of this would be the RTC clock domain. In such cases the > module ready check is skipped which leads to a data abort during bootup > when RTC registers is accessed. > > Since the actual base address is verified in > omap4_cminst_read_inst_reg(), this check here is not required at all > and hence can be removed. > > Signed-off-by: Hebbar, Gururaja <gururaja.heb...@ti.com> > --- > :100644 100644 7f9a464... 40545ff... M arch/arm/mach-omap2/cminst44xx.c > arch/arm/mach-omap2/cminst44xx.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/arch/arm/mach-omap2/cminst44xx.c > b/arch/arm/mach-omap2/cminst44xx.c > index 7f9a464..40545ff 100644 > --- a/arch/arm/mach-omap2/cminst44xx.c > +++ b/arch/arm/mach-omap2/cminst44xx.c > @@ -271,9 +271,6 @@ int omap4_cminst_wait_module_ready(u8 part, u16 inst, s16 > cdoffs, > { > int i = 0; > > - if (!clkctrl_offs) > - return 0;
looks like commit log has the wrong argument. I believe you meant to say that AM33xx has CLKSTCTRL exactly at 0x00 and that's why this check has to be removed. No ? -- balbi
signature.asc
Description: Digital signature