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

Alim Akhtar
2025-02-05T16:11:02Z

Hi Krzysztof

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Wednesday, February 5, 2025 9:22 PM
> To: Sudeep Holla <sudeep.holla@arm.com>; Alim Akhtar
> <alim.akhtar@samsung.com>
> Cc: 'Devang Tailor' <dev.tailor@samsung.com>; robh@kernel.org;
> krzk+dt@kernel.org; conor+dt@kernel.org; devicetree@vger.kernel.org;
> linux-arm-kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org;
> linux-kernel@vger.kernel.org; faraz.ata@samsung.com
> 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