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

Daniel Schwierzeck daniel.schwierzeck at googlemail.com
Tue Apr 3 17:49:20 CEST 2012


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.

-- 
Best regards,
Daniel


More information about the U-Boot mailing list