[U-Boot] [bug report]cmd: cache: output information

[U-Boot] [bug report]cmd: cache: output information

[jk]

Hi guys,

I think the output of command 'dcache' is improper. It always print 
'writethrough' regardless of the real policy, such as write back.

=> dcache
Data (writethrough) Cache is ON

Not only ARM but also other socs support cache, hence we can't 
differentiate policy by macro (i.e CONFIG_SYS_ARM_CACHE_WRITETHROUGH).
Can you think of anything better?

[U-Boot] [bug report]cmd: cache: output information

[Simon Glass]

Hi,

On 23 July 2016 at 00:31, jk <jk.kernel@gmail.com> wrote:
> Hi guys,
>
> I think the output of command 'dcache' is improper. It always print
> 'writethrough' regardless of the real policy, such as write back.
>
> => dcache
> Data (writethrough) Cache is ON
>
> Not only ARM but also other socs support cache, hence we can't differentiate
> policy by macro (i.e CONFIG_SYS_ARM_CACHE_WRITETHROUGH).
> Can you think of anything better?
>

I'm sure it could be improved. Perhaps it could also be done in a
generic way so it works on all archs. Do you want to send a patch?

Regards,
Simon

[U-Boot] [bug report]cmd: cache: output information

[jk]

Hi Simon,

I'm not very familiar with other archs except, hence I'm not able to 
implement it.:-(

Regards,
Ziyuan Xu

On 2016?07?26? 10:47, Simon Glass wrote:
> Hi,
>
> On 23 July 2016 at 00:31, jk <jk.kernel@gmail.com> wrote:
>> Hi guys,
>>
>> I think the output of command 'dcache' is improper. It always print
>> 'writethrough' regardless of the real policy, such as write back.
>>
>> => dcache
>> Data (writethrough) Cache is ON
>>
>> Not only ARM but also other socs support cache, hence we can't differentiate
>> policy by macro (i.e CONFIG_SYS_ARM_CACHE_WRITETHROUGH).
>> Can you think of anything better?
>>
> I'm sure it could be improved. Perhaps it could also be done in a
> generic way so it works on all archs. Do you want to send a patch?
>
> Regards,
> Simon

[U-Boot] [bug report]cmd: cache: output information

[Simon Glass]

Hi Ziyuan,

On 25 July 2016 at 20:56, jk <jk.kernel@gmail.com> wrote:
> Hi Simon,
>
> I'm not very familiar with other archs except, hence I'm not able to
> implement it.:-(

Well a patch for ARM would be useful. I'm sure other people could add
to it later.

Regards,
Simon

>
> Regards,
> Ziyuan Xu
>
>
> On 2016?07?26? 10:47, Simon Glass wrote:
>>
>> Hi,
>>
>> On 23 July 2016 at 00:31, jk <jk.kernel@gmail.com> wrote:
>>>
>>> Hi guys,
>>>
>>> I think the output of command 'dcache' is improper. It always print
>>> 'writethrough' regardless of the real policy, such as write back.
>>>
>>> => dcache
>>> Data (writethrough) Cache is ON
>>>
>>> Not only ARM but also other socs support cache, hence we can't
>>> differentiate
>>> policy by macro (i.e CONFIG_SYS_ARM_CACHE_WRITETHROUGH).
>>> Can you think of anything better?
>>>
>> I'm sure it could be improved. Perhaps it could also be done in a
>> generic way so it works on all archs. Do you want to send a patch?
>>
>> Regards,
>> Simon
>
>