* [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi
@ 2024-09-26 7:49 Dragan Simic
2024-09-26 8:24 ` Heiko Stuebner
2024-09-26 9:24 ` Robin Murphy
0 siblings, 2 replies; 6+ messages in thread
From: Dragan Simic @ 2024-09-26 7:49 UTC (permalink / raw)
To: linux-rockchip
Cc: heiko, linux-arm-kernel, linux-kernel, devicetree, robh, krzk+dt,
conor+dt, stable
Move the "l3_cache" node under the "cpus" node in the dtsi file for Rockchip
RK356x SoCs. There's no need for this cache node to be at the higher level.
Fixes: 8612169a05c5 ("arm64: dts: rockchip: Add cache information to the SoC dtsi for RK356x")
Cc: stable@vger.kernel.org
Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---
arch/arm64/boot/dts/rockchip/rk356x.dtsi | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
index 4690be841a1c..9f7136e5d553 100644
--- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
@@ -113,19 +113,19 @@ cpu3: cpu@300 {
d-cache-sets = <128>;
next-level-cache = <&l3_cache>;
};
- };
- /*
- * There are no private per-core L2 caches, but only the
- * L3 cache that appears to the CPU cores as L2 caches
- */
- l3_cache: l3-cache {
- compatible = "cache";
- cache-level = <2>;
- cache-unified;
- cache-size = <0x80000>;
- cache-line-size = <64>;
- cache-sets = <512>;
+ /*
+ * There are no private per-core L2 caches, but only the
+ * L3 cache that appears to the CPU cores as L2 caches
+ */
+ l3_cache: l3-cache {
+ compatible = "cache";
+ cache-level = <2>;
+ cache-unified;
+ cache-size = <0x80000>;
+ cache-line-size = <64>;
+ cache-sets = <512>;
+ };
};
cpu0_opp_table: opp-table-0 {
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi
2024-09-26 7:49 [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi Dragan Simic
@ 2024-09-26 8:24 ` Heiko Stuebner
2024-09-26 8:32 ` Dragan Simic
2024-09-26 9:24 ` Robin Murphy
1 sibling, 1 reply; 6+ messages in thread
From: Heiko Stuebner @ 2024-09-26 8:24 UTC (permalink / raw)
To: linux-rockchip, Dragan Simic
Cc: linux-arm-kernel, linux-kernel, devicetree, robh, krzk+dt,
conor+dt, stable
Am Donnerstag, 26. September 2024, 09:49:18 CEST schrieb Dragan Simic:
> Move the "l3_cache" node under the "cpus" node in the dtsi file for Rockchip
> RK356x SoCs. There's no need for this cache node to be at the higher level.
>
> Fixes: 8612169a05c5 ("arm64: dts: rockchip: Add cache information to the SoC dtsi for RK356x")
> Cc: stable@vger.kernel.org
I think the commit message needs a bit more rationale on why this is a
stable-worthy fix. Because from the move and commit message it reads
like a styling choice ;-) .
I do agree that it makes more sense as child of cpus, but the commit
message should also elaborate on why that would matter for stable.
Heiko
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> ---
> arch/arm64/boot/dts/rockchip/rk356x.dtsi | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
> index 4690be841a1c..9f7136e5d553 100644
> --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
> @@ -113,19 +113,19 @@ cpu3: cpu@300 {
> d-cache-sets = <128>;
> next-level-cache = <&l3_cache>;
> };
> - };
>
> - /*
> - * There are no private per-core L2 caches, but only the
> - * L3 cache that appears to the CPU cores as L2 caches
> - */
> - l3_cache: l3-cache {
> - compatible = "cache";
> - cache-level = <2>;
> - cache-unified;
> - cache-size = <0x80000>;
> - cache-line-size = <64>;
> - cache-sets = <512>;
> + /*
> + * There are no private per-core L2 caches, but only the
> + * L3 cache that appears to the CPU cores as L2 caches
> + */
> + l3_cache: l3-cache {
> + compatible = "cache";
> + cache-level = <2>;
> + cache-unified;
> + cache-size = <0x80000>;
> + cache-line-size = <64>;
> + cache-sets = <512>;
> + };
> };
>
> cpu0_opp_table: opp-table-0 {
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi
2024-09-26 8:24 ` Heiko Stuebner
@ 2024-09-26 8:32 ` Dragan Simic
2024-09-26 8:48 ` Heiko Stuebner
0 siblings, 1 reply; 6+ messages in thread
From: Dragan Simic @ 2024-09-26 8:32 UTC (permalink / raw)
To: Heiko Stuebner
Cc: linux-rockchip, linux-arm-kernel, linux-kernel, devicetree, robh,
krzk+dt, conor+dt, stable
Hello Heiko,
On 2024-09-26 10:24, Heiko Stuebner wrote:
> Am Donnerstag, 26. September 2024, 09:49:18 CEST schrieb Dragan Simic:
>> Move the "l3_cache" node under the "cpus" node in the dtsi file for
>> Rockchip
>> RK356x SoCs. There's no need for this cache node to be at the higher
>> level.
>>
>> Fixes: 8612169a05c5 ("arm64: dts: rockchip: Add cache information to
>> the SoC dtsi for RK356x")
>> Cc: stable@vger.kernel.org
>
> I think the commit message needs a bit more rationale on why this is a
> stable-worthy fix. Because from the move and commit message it reads
> like a styling choice ;-) .
>
> I do agree that it makes more sense as child of cpus, but the commit
> message should also elaborate on why that would matter for stable.
Thanks for your feedback! Perhaps it would be the best to simply drop
the
submission to stable kernels... Believe it or not, :) I spent a fair
amount
of time deliberating over the submission to stable, but now I think it's
simply better to omit that and not increase the amount of patches that
go
into stable unnecessary.
Would you like me to send the v2 with no Cc to stable, or would you
prefer
to drop that line yourself?
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>> arch/arm64/boot/dts/rockchip/rk356x.dtsi | 24
>> ++++++++++++------------
>> 1 file changed, 12 insertions(+), 12 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> index 4690be841a1c..9f7136e5d553 100644
>> --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> @@ -113,19 +113,19 @@ cpu3: cpu@300 {
>> d-cache-sets = <128>;
>> next-level-cache = <&l3_cache>;
>> };
>> - };
>>
>> - /*
>> - * There are no private per-core L2 caches, but only the
>> - * L3 cache that appears to the CPU cores as L2 caches
>> - */
>> - l3_cache: l3-cache {
>> - compatible = "cache";
>> - cache-level = <2>;
>> - cache-unified;
>> - cache-size = <0x80000>;
>> - cache-line-size = <64>;
>> - cache-sets = <512>;
>> + /*
>> + * There are no private per-core L2 caches, but only the
>> + * L3 cache that appears to the CPU cores as L2 caches
>> + */
>> + l3_cache: l3-cache {
>> + compatible = "cache";
>> + cache-level = <2>;
>> + cache-unified;
>> + cache-size = <0x80000>;
>> + cache-line-size = <64>;
>> + cache-sets = <512>;
>> + };
>> };
>>
>> cpu0_opp_table: opp-table-0 {
>>
>
>
>
>
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi
2024-09-26 8:32 ` Dragan Simic
@ 2024-09-26 8:48 ` Heiko Stuebner
0 siblings, 0 replies; 6+ messages in thread
From: Heiko Stuebner @ 2024-09-26 8:48 UTC (permalink / raw)
To: Dragan Simic
Cc: linux-rockchip, linux-arm-kernel, linux-kernel, devicetree, robh,
krzk+dt, conor+dt, stable
Am Donnerstag, 26. September 2024, 10:32:17 CEST schrieb Dragan Simic:
> Hello Heiko,
>
> On 2024-09-26 10:24, Heiko Stuebner wrote:
> > Am Donnerstag, 26. September 2024, 09:49:18 CEST schrieb Dragan Simic:
> >> Move the "l3_cache" node under the "cpus" node in the dtsi file for
> >> Rockchip
> >> RK356x SoCs. There's no need for this cache node to be at the higher
> >> level.
> >>
> >> Fixes: 8612169a05c5 ("arm64: dts: rockchip: Add cache information to
> >> the SoC dtsi for RK356x")
> >> Cc: stable@vger.kernel.org
> >
> > I think the commit message needs a bit more rationale on why this is a
> > stable-worthy fix. Because from the move and commit message it reads
> > like a styling choice ;-) .
> >
> > I do agree that it makes more sense as child of cpus, but the commit
> > message should also elaborate on why that would matter for stable.
>
> Thanks for your feedback! Perhaps it would be the best to simply drop
> the
> submission to stable kernels... Believe it or not, :) I spent a fair
> amount
> of time deliberating over the submission to stable, but now I think it's
> simply better to omit that and not increase the amount of patches that
> go
> into stable unnecessary.
>
> Would you like me to send the v2 with no Cc to stable, or would you
> prefer
> to drop that line yourself?
I'm hopeful that I'll remember to drop it :-), so I guess no need
to resend for that.
Heiko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi
2024-09-26 7:49 [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi Dragan Simic
2024-09-26 8:24 ` Heiko Stuebner
@ 2024-09-26 9:24 ` Robin Murphy
2024-09-26 9:36 ` Dragan Simic
1 sibling, 1 reply; 6+ messages in thread
From: Robin Murphy @ 2024-09-26 9:24 UTC (permalink / raw)
To: Dragan Simic, linux-rockchip
Cc: heiko, linux-arm-kernel, linux-kernel, devicetree, robh, krzk+dt,
conor+dt, stable
On 2024-09-26 8:49 am, Dragan Simic wrote:
> Move the "l3_cache" node under the "cpus" node in the dtsi file for Rockchip
> RK356x SoCs. There's no need for this cache node to be at the higher level.
Except it does arguably represent the physical topology - the L3 cache
doesn't belong to the CPUs, it belongs to the DSU, which very much is
"outside" the CPUs.
Thanks,
Robin.
>
> Fixes: 8612169a05c5 ("arm64: dts: rockchip: Add cache information to the SoC dtsi for RK356x")
> Cc: stable@vger.kernel.org
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> ---
> arch/arm64/boot/dts/rockchip/rk356x.dtsi | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
> index 4690be841a1c..9f7136e5d553 100644
> --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
> @@ -113,19 +113,19 @@ cpu3: cpu@300 {
> d-cache-sets = <128>;
> next-level-cache = <&l3_cache>;
> };
> - };
>
> - /*
> - * There are no private per-core L2 caches, but only the
> - * L3 cache that appears to the CPU cores as L2 caches
> - */
> - l3_cache: l3-cache {
> - compatible = "cache";
> - cache-level = <2>;
> - cache-unified;
> - cache-size = <0x80000>;
> - cache-line-size = <64>;
> - cache-sets = <512>;
> + /*
> + * There are no private per-core L2 caches, but only the
> + * L3 cache that appears to the CPU cores as L2 caches
> + */
> + l3_cache: l3-cache {
> + compatible = "cache";
> + cache-level = <2>;
> + cache-unified;
> + cache-size = <0x80000>;
> + cache-line-size = <64>;
> + cache-sets = <512>;
> + };
> };
>
> cpu0_opp_table: opp-table-0 {
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi
2024-09-26 9:24 ` Robin Murphy
@ 2024-09-26 9:36 ` Dragan Simic
0 siblings, 0 replies; 6+ messages in thread
From: Dragan Simic @ 2024-09-26 9:36 UTC (permalink / raw)
To: Robin Murphy
Cc: linux-rockchip, heiko, linux-arm-kernel, linux-kernel, devicetree,
robh, krzk+dt, conor+dt, stable
Hello Robin,
On 2024-09-26 11:24, Robin Murphy wrote:
> On 2024-09-26 8:49 am, Dragan Simic wrote:
>> Move the "l3_cache" node under the "cpus" node in the dtsi file for
>> Rockchip
>> RK356x SoCs. There's no need for this cache node to be at the higher
>> level.
>
> Except it does arguably represent the physical topology - the L3 cache
> doesn't belong to the CPUs, it belongs to the DSU, which very much is
> "outside" the CPUs.
That's a very good point, thanks! I knew there must have been
a very good reason why I placed the L3 cache outside the CPUs
originally, in the commit 8612169a05c5 referenced below, but I
also somehow managed to forget that reason for a moment. :)
Let's drop this patch, and I'll submit another patch for the
RK3588 SoC dtsi files that moves the L3 cache outside the CPUs,
to reflect the physical topology better.
>> Fixes: 8612169a05c5 ("arm64: dts: rockchip: Add cache information to
>> the SoC dtsi for RK356x")
>> Cc: stable@vger.kernel.org
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>> arch/arm64/boot/dts/rockchip/rk356x.dtsi | 24
>> ++++++++++++------------
>> 1 file changed, 12 insertions(+), 12 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> index 4690be841a1c..9f7136e5d553 100644
>> --- a/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> +++ b/arch/arm64/boot/dts/rockchip/rk356x.dtsi
>> @@ -113,19 +113,19 @@ cpu3: cpu@300 {
>> d-cache-sets = <128>;
>> next-level-cache = <&l3_cache>;
>> };
>> - };
>> - /*
>> - * There are no private per-core L2 caches, but only the
>> - * L3 cache that appears to the CPU cores as L2 caches
>> - */
>> - l3_cache: l3-cache {
>> - compatible = "cache";
>> - cache-level = <2>;
>> - cache-unified;
>> - cache-size = <0x80000>;
>> - cache-line-size = <64>;
>> - cache-sets = <512>;
>> + /*
>> + * There are no private per-core L2 caches, but only the
>> + * L3 cache that appears to the CPU cores as L2 caches
>> + */
>> + l3_cache: l3-cache {
>> + compatible = "cache";
>> + cache-level = <2>;
>> + cache-unified;
>> + cache-size = <0x80000>;
>> + cache-line-size = <64>;
>> + cache-sets = <512>;
>> + };
>> };
>> cpu0_opp_table: opp-table-0 {
>>
>> _______________________________________________
>> Linux-rockchip mailing list
>> Linux-rockchip@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-rockchip
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-09-26 9:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-26 7:49 [PATCH] arm64: dts: rockchip: Move L3 cache under CPUs in RK356x SoC dtsi Dragan Simic
2024-09-26 8:24 ` Heiko Stuebner
2024-09-26 8:32 ` Dragan Simic
2024-09-26 8:48 ` Heiko Stuebner
2024-09-26 9:24 ` Robin Murphy
2024-09-26 9:36 ` Dragan Simic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).