> -----Original Message----- > From: Jean-Christophe PLAGNIOL-VILLARD [mailto:plagn...@jcrosoft.com] > Sent: Saturday, April 04, 2009 9:46 PM > To: Prafulla Wadaskar > Cc: u-boot@lists.denx.de; Ronen Shitrit; Nicolas Pitre > Subject: Re: [U-Boot] [PATCH] Marvell > Feroceon-FR131/Sheeva-88SV131 cpu core support > > On 04:09 Sat 04 Apr , Prafulla Wadaskar wrote: > > This is ARM v5TE-compliant processor core with MMU and L1/L2 Cache > > > > Signed-off-by: prafulla_wadaskar <prafu...@marvell.com> > Reviewed by: > > Ronen Shitrit <rshit...@marvell.com> > > --- > > cpu/arm926ejs/start.S | 5 +++++ > > 1 files changed, 5 insertions(+), 0 deletions(-) > > > > diff --git a/cpu/arm926ejs/start.S b/cpu/arm926ejs/start.S index > > ed4932a..08ec955 100644 > > --- a/cpu/arm926ejs/start.S > > +++ b/cpu/arm926ejs/start.S > > @@ -214,10 +214,15 @@ cpu_init_crit: > > * disable MMU stuff and caches > > */ > > mrc p15, 0, r0, c1, c0, 0 > > +#if defined (CONFIG_FEROCEON_88FR131) || defined > (CONFIG_SHEEVA_88SV131) > > + bic r0, r0, #0x00000007 /* clear bits 2:0 (CAM) */ > > + orr r0, r0, #0x00000002 /* set bit 2 (A) Align */ > > +#else > > bic r0, r0, #0x00002300 /* clear bits 13, 9:8 > (--V- --RS) */ > > bic r0, r0, #0x00000087 /* clear bits 7, 2:0 > (B--- -CAM) */ > > orr r0, r0, #0x00000002 /* set bit 2 (A) Align */ > > orr r0, r0, #0x00001000 /* set bit 12 (I) I-Cache */ > > +#endif > I've plan to clean up the asm mmu and cache management as > I've done for the C part. The idea is to avoid to duplicated > code and #ifdef every where. So please create asm macro to handle this > > and please explain a few more why you need it
The issue was of 'order of operations'. In our devices we have L2 cache and the L2 must be enabled before the L1. this change was with reference to older u-boot and earlier SOC versions. I have tested the code without this patch and is functional. We can add code for disabling the L1 before enabling the L2 in SOC specific . This patch is not needed now, We can skip this patch. > > Best Regards, > J. > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
