[PATCH] ARM: cache-l2x0: fix wrong print of AUX_CTRL register

[Catalin Marinas]

On 27 April 2011 13:42, Santosh Shilimkar <santosh.shilimkar at ti.com> wrote:
> On 4/27/2011 5:56 PM, srinidhi kasagar wrote:
>>
>> On platforms where l2x0 is already enabled and configured
>> by the secure code, printing the masked value of the
>> AUX_CTRL register seems to be wrong and differ from the
>> programmed value.
>>
>> Fix it by reading the AUX_CTRL register again.
>>
>> Signed-off-by: srinidhi kasagar<srinidhi.kasagar at stericsson.com>
>> Signed-off-by: Vijaya Kumar Kilari<vijay.kilari at stericsson.com>
>> ---
>>  arch/arm/mm/cache-l2x0.c |    2 ++
>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mm/cache-l2x0.c b/arch/arm/mm/cache-l2x0.c
>> index ef59099..d07f8cb 100644
>> --- a/arch/arm/mm/cache-l2x0.c
>> +++ b/arch/arm/mm/cache-l2x0.c
>> @@ -340,6 +340,8 @@ void __init l2x0_init(void __iomem *base, __u32
>> aux_val, __u32 aux_mask)
>>        outer_cache.disable = l2x0_disable;
>>        outer_cache.set_debug = l2x0_set_debug;
>>
>> +       aux = readl_relaxed(l2x0_base + L2X0_AUX_CTRL);
>> +
>
> Can you not read this register in your machine code and pass
> the right 'aux_val' as part of l2x0_init() ?

We already read this register in l2x0_init().

In theory, the Linux kernel shouldn't even touch this register but
leave it to the boot loader or other firmware (or just some sane reset
values that the hardware people should put). But no-one seems to be
doing this.

-- 
Catalin