* [PATCH 20/20] arm64: dts: Update cache properties for tesla
@ 2022-10-31 9:21 ` Pierre Gondois
2022-11-02 20:21 ` (subset) " Krzysztof Kozlowski
2022-11-03 5:18 ` Alim Akhtar
0 siblings, 2 replies; 4+ messages in thread
From: Pierre Gondois @ 2022-10-31 9:21 UTC (permalink / raw)
To: linux-kernel
Cc: pierre.gondois, Rob.Herring, Rob Herring, Krzysztof Kozlowski,
Alim Akhtar, Aswani Reddy, Shashank Prashar, Sriranjani P,
devicetree
The DeviceTree Specification v0.3 specifies that the cache node
'compatible' and 'cache-level' properties are 'required'. Cf.
s3.8 Multi-level and Shared Cache Nodes
The recently added init_of_cache_level() function checks
these properties. Add them if missing.
Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
---
arch/arm64/boot/dts/tesla/fsd.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/tesla/fsd.dtsi b/arch/arm64/boot/dts/tesla/fsd.dtsi
index f35bc5a288c2..bfab040fc1e4 100644
--- a/arch/arm64/boot/dts/tesla/fsd.dtsi
+++ b/arch/arm64/boot/dts/tesla/fsd.dtsi
@@ -284,6 +284,7 @@ cpucl_l2: l2-cache0 {
cache-size = <0x400000>;
cache-line-size = <64>;
cache-sets = <4096>;
+ cache-level = <2>;
};
idle-states {
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: (subset) [PATCH 20/20] arm64: dts: Update cache properties for tesla
2022-10-31 9:21 ` [PATCH 20/20] arm64: dts: Update cache properties for tesla Pierre Gondois
@ 2022-11-02 20:21 ` Krzysztof Kozlowski
2022-11-07 18:13 ` Krzysztof Kozlowski
2022-11-03 5:18 ` Alim Akhtar
1 sibling, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-02 20:21 UTC (permalink / raw)
To: Pierre Gondois, linux-kernel
Cc: Krzysztof Kozlowski, Sriranjani P, Shashank Prashar, Rob Herring,
Aswani Reddy, devicetree, Alim Akhtar, Krzysztof Kozlowski,
Rob.Herring
On Mon, 31 Oct 2022 10:21:19 +0100, Pierre Gondois wrote:
> The DeviceTree Specification v0.3 specifies that the cache node
> 'compatible' and 'cache-level' properties are 'required'. Cf.
> s3.8 Multi-level and Shared Cache Nodes
>
> The recently added init_of_cache_level() function checks
> these properties. Add them if missing.
>
> [...]
Applied, thanks!
[20/20] arm64: dts: Update cache properties for tesla
https://git.kernel.org/krzk/linux/c/3f0d4a02c7490f1aa15764ed85dcf0881bf66b9e
Best regards,
--
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH 20/20] arm64: dts: Update cache properties for tesla
2022-10-31 9:21 ` [PATCH 20/20] arm64: dts: Update cache properties for tesla Pierre Gondois
2022-11-02 20:21 ` (subset) " Krzysztof Kozlowski
@ 2022-11-03 5:18 ` Alim Akhtar
1 sibling, 0 replies; 4+ messages in thread
From: Alim Akhtar @ 2022-11-03 5:18 UTC (permalink / raw)
To: 'Pierre Gondois', linux-kernel
Cc: Rob.Herring, 'Rob Herring', 'Krzysztof Kozlowski',
'Aswani Reddy', 'Shashank Prashar',
'Sriranjani P',
devicetree
>-----Original Message-----
>From: Pierre Gondois [mailto:pierre.gondois@arm.com]
>Sent: Monday, October 31, 2022 2:51 PM
>To: linux-kernel@vger.kernel.org
>Cc: pierre.gondois@arm.com; Rob.Herring@arm.com; Rob Herring
><robh+dt@kernel.org>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@linaro.org>; Alim Akhtar
><alim.akhtar@samsung.com>; Aswani Reddy <aswani.reddy@samsung.com>;
>Shashank Prashar <s.prashar@samsung.com>; Sriranjani P
><sriranjani.p@samsung.com>; devicetree@vger.kernel.org
>Subject: [PATCH 20/20] arm64: dts: Update cache properties for tesla
>
>The DeviceTree Specification v0.3 specifies that the cache node
'compatible'
>and 'cache-level' properties are 'required'. Cf.
>s3.8 Multi-level and Shared Cache Nodes
>
>The recently added init_of_cache_level() function checks these properties.
>Add them if missing.
>
>Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
>---
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
> arch/arm64/boot/dts/tesla/fsd.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/arch/arm64/boot/dts/tesla/fsd.dtsi
>b/arch/arm64/boot/dts/tesla/fsd.dtsi
>index f35bc5a288c2..bfab040fc1e4 100644
>--- a/arch/arm64/boot/dts/tesla/fsd.dtsi
>+++ b/arch/arm64/boot/dts/tesla/fsd.dtsi
>@@ -284,6 +284,7 @@ cpucl_l2: l2-cache0 {
> cache-size = <0x400000>;
> cache-line-size = <64>;
> cache-sets = <4096>;
>+ cache-level = <2>;
> };
>
> idle-states {
>--
>2.25.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: (subset) [PATCH 20/20] arm64: dts: Update cache properties for tesla
2022-11-02 20:21 ` (subset) " Krzysztof Kozlowski
@ 2022-11-07 18:13 ` Krzysztof Kozlowski
0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-07 18:13 UTC (permalink / raw)
To: Pierre Gondois, linux-kernel
Cc: Sriranjani P, Shashank Prashar, Rob Herring, Aswani Reddy,
devicetree, Alim Akhtar, Krzysztof Kozlowski, Rob.Herring
On 02/11/2022 21:21, Krzysztof Kozlowski wrote:
> On Mon, 31 Oct 2022 10:21:19 +0100, Pierre Gondois wrote:
>> The DeviceTree Specification v0.3 specifies that the cache node
>> 'compatible' and 'cache-level' properties are 'required'. Cf.
>> s3.8 Multi-level and Shared Cache Nodes
>>
>> The recently added init_of_cache_level() function checks
>> these properties. Add them if missing.
>>
>> [...]
>
> Applied, thanks!
>
> [20/20] arm64: dts: Update cache properties for tesla
> https://git.kernel.org/krzk/linux/c/3f0d4a02c7490f1aa15764ed85dcf0881bf66b9e
Now dropped. I understand this is the intention/request of v2.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-11-07 18:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20221031092138epcas5p3070cc1775dafebe8d4abffcecaa860b0@epcas5p3.samsung.com>
2022-10-31 9:21 ` [PATCH 20/20] arm64: dts: Update cache properties for tesla Pierre Gondois
2022-11-02 20:21 ` (subset) " Krzysztof Kozlowski
2022-11-07 18:13 ` Krzysztof Kozlowski
2022-11-03 5:18 ` Alim Akhtar
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).