[U-Boot] [PATCH 2/3] MIPS: fix inconsistency in config option for cache operation mode

Marek Vasut marex at denx.de
Tue Apr 3 18:09:33 CEST 2012


Dear Daniel Schwierzeck,

> Hi,
> 
> On Tue, Apr 3, 2012 at 3:40 PM, Marek Vasut <marex at denx.de> wrote:
> > Mike, there was some issue with this patch?
> > 
> >> Commit ab2a98b11716364bc5a8c43cdfa7fee176cda1d8 missed to
> >> use the new config option in dcache_enable().
> 
> this commit refers to a patch that I sent a while ago as preparation
> for supporting Lantiq SoCs.
> Some Lantiq SoCs wants CONF_CM_CACHABLE_NO_WA  instead of the
> hard-coded CONF_CM_CACHABLE_NONCOHERENT.
> So I created a config option for the required cache mode.
> 
> >> Fix this to avoid inconsistencies if someone wants to disable
> >> and enable D-caches.
> >> 
> >> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck at googlemail.com>
> >> ---
> >>  arch/mips/cpu/mips32/cache.S |    6 +++++-
> >>  1 file changed, 5 insertions(+), 1 deletion(-)
> >> 
> >> diff --git a/arch/mips/cpu/mips32/cache.S b/arch/mips/cpu/mips32/cache.S
> >> index 5ce0ec4..e683e8b 100644
> >> --- a/arch/mips/cpu/mips32/cache.S
> >> +++ b/arch/mips/cpu/mips32/cache.S
> >> @@ -30,6 +30,10 @@
> >>  #include <asm/addrspace.h>
> >>  #include <asm/cacheops.h>
> >> 
> >> +#ifndef CONFIG_SYS_MIPS_CACHE_MODE
> >> +#define CONFIG_SYS_MIPS_CACHE_MODE CONF_CM_CACHABLE_NONCOHERENT
> >> +#endif
> >> +
> >>  #define RA           t8
> >> 
> >>  /*
> >> @@ -224,7 +228,7 @@ LEAF(dcache_enable)
> >>       mfc0    t0, CP0_CONFIG
> >>       ori     t0, CONF_CM_CMASK
> >>       xori    t0, CONF_CM_CMASK
> >> -     ori     t0, CONF_CM_CACHABLE_NONCOHERENT
> >> +     ori     t0, CONFIG_SYS_MIPS_CACHE_MODE
> >>       mtc0    t0, CP0_CONFIG
> >>       jr      ra
> >>       END(dcache_enable)
> > 
> > Best regards,
> > Marek Vasut
> 
> I only changed start.S and forgot to change this code part too. That is why
> I created this patch to fix this inconsistency.

I see ... so this fixes some other commit. What was Shinya-san's concern, that 
you only changed this file?

Best regards,
Marek Vasut


More information about the U-Boot mailing list