Re: [RESEND PATCH] dt-bindings: sifive-ccache: fix cache level for l3 cache

From: Conor.Dooley
Date: Wed Aug 31 2022 - 02:26:06 EST


On 31/08/2022 06:17, Zong Li wrote:
> [Some people who received this message don't often get email from zongbox@xxxxxxxxx. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
>
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> <Conor.Dooley@xxxxxxxxxxxxx> 於 2022年8月31日 週三 凌晨1:09寫道:
>> That would keep the enforcement for existing caches and allow you
>> the freedome to do w/e you want for the ccache0 compatible.
>
> Thanks you all for bring me here, we actually have some core series
> with 4096 cache set in ccache, should we need to extend the cache set
> as follow? or we only need to focus on the DTS which is already in
> mainline.
>
> cache-sets:
> - enum: [1024, 2048]
> + enum: [1024, 2048, 4096]

Until a user shows up, I think we are better off not adding 4096.

>>> Do we need someone to take charge of this series?
>>>
>>
>> Can I volunteer Zong? (since all but two of the patches are theirs)
>>
>
> It is ok to me, but I'm still refining the patchset for V2, and I'm
> not sure if we will need the V3. Do you prefer to take V2 patch set
> first and replace the dt-binding patch?

If you could incorporate it for v2 it would make reviewing easier
I think.

Thanks,
Conor.