RE: [PATCH v2] arm64: dts: add cpu cache information to ExynosAuto-v920

[Alim Akhtar]

Hi Krzysztof

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Wednesday, February 5, 2025 9:22 PM
> To: Sudeep Holla <sudeep.holla@xxxxxxx>; Alim Akhtar
> <alim.akhtar@xxxxxxxxxxx>
> Cc: 'Devang Tailor' <dev.tailor@xxxxxxxxxxx>; robh@xxxxxxxxxx;
> krzk+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx; faraz.ata@xxxxxxxxxxx
> Subject: Re: [PATCH v2] arm64: dts: add cpu cache information to
> ExynosAuto-v920
> 
> On 31/01/2025 14:27, Sudeep Holla wrote:
> >>>
> >> [snip]
> >>  > +		l3_cache_cl0: l3-cache0 {
> >> You can add one node for cl0 and cl1, say "l3_cache_cl0_cl1" and
> >> Remove the specific node for CL1, because both are same.
> >>
> >
> > What do you mean by "both are same" ?
> > Do you mean both have exact same properties but are physically
> > different caches ? OR Do you mean it is just one shared cache ?
> >
> > If former, we still need distinct node to get the cacheinfo about
> > shareability correct. If this is about avoiding duplication of errors,
> > you can probably define some macro and avoid it, but we need 2 nodes
> > in the devicetree.
> >
> > If latter, you suggestion is correct.
> 
> No answers here, so I drop this patch from my queue.
> 
It took sometime to get the confirmation internally (because of new year holiday)
Just replied to Sudeep. It will be great if you can consider this patch for this cycle. 
Thanks!

> Best regards,
> Krzysztof