[PATCH v4 2/3] ARM: dts: qcom: include cpu in idle-state node names

[PATCH v4 2/3] ARM: dts: qcom: include cpu in idle-state node names

[David Heidelberg]

Required for dt-schema validation.

Signed-off-by: David Heidelberg <david@ixit.cz>
---
 arch/arm/boot/dts/qcom/qcom-apq8064.dtsi | 2 +-
 arch/arm/boot/dts/qcom/qcom-apq8084.dtsi | 2 +-
 arch/arm/boot/dts/qcom/qcom-msm8974.dtsi | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
index 59fd86b9fb47..d2ae13a67382 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-apq8064.dtsi
@@ -87,7 +87,7 @@ L2: l2-cache {
 		};
 
 		idle-states {
-			CPU_SPC: spc {
+			CPU_SPC: cpu-spc {
 				compatible = "qcom,idle-state-spc",
 						"arm,idle-state";
 				entry-latency-us = <400>;
diff --git a/arch/arm/boot/dts/qcom/qcom-apq8084.dtsi b/arch/arm/boot/dts/qcom/qcom-apq8084.dtsi
index 2b1f9d0fb510..bdf64895c55c 100644
--- a/arch/arm/boot/dts/qcom/qcom-apq8084.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-apq8084.dtsi
@@ -79,7 +79,7 @@ L2: l2-cache {
 		};
 
 		idle-states {
-			CPU_SPC: spc {
+			CPU_SPC: cpu-spc {
 				compatible = "qcom,idle-state-spc",
 						"arm,idle-state";
 				entry-latency-us = <150>;
diff --git a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
index 76006c3c4af2..e82b2d184735 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8974.dtsi
@@ -85,7 +85,7 @@ L2: l2-cache {
 		};
 
 		idle-states {
-			CPU_SPC: spc {
+			CPU_SPC: cpu-spc {
 				compatible = "qcom,idle-state-spc",
 						"arm,idle-state";
 				entry-latency-us = <150>;
-- 
2.42.0

[PATCH v4 3/3] ARM: dts: qcom: msm8916: idle-state compatible require the generic idle-state

[David Heidelberg]

compatible must be qcom,idle-state-spc AND arm,idle-state.

Fixes: d468f825b3fd ("ARM: dts: qcom: msm8916: Add include for SMP without PSCI on ARM32")

Signed-off-by: David Heidelberg <david@ixit.cz>
---
 arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi b/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi
index 36328dbe4212..1ba403b83cb1 100644
--- a/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi
+++ b/arch/arm/boot/dts/qcom/qcom-msm8916-smp.dtsi
@@ -26,7 +26,7 @@ psci {
 };
 
 &CPU_SLEEP_0 {
-	compatible = "qcom,idle-state-spc";
+	compatible = "qcom,idle-state-spc", "arm,idle-state";
 };
 
 &cpu0_acc {
-- 
2.42.0

Re: [PATCH v4 2/3] ARM: dts: qcom: include cpu in idle-state node names

[Konrad Dybcio]

On 3.12.2023 00:47, David Heidelberg wrote:
> Required for dt-schema validation.
> 
> Signed-off-by: David Heidelberg <david@ixit.cz>
> ---
Only patches 2 and 3 made it to my inbox and linux-arm-msm

Konrad

Re: [PATCH v4 2/3] ARM: dts: qcom: include cpu in idle-state node names

[David Heidelberg]

Hello Konrad,

v4 should be outside, 
https://patchwork.kernel.org/project/linux-hwmon/patch/20231209171653.85468-1-david@ixit.cz/

Sadly, v3 probably got filtered by SPAM filter :(

David

On 04/12/2023 12:12, Konrad Dybcio wrote:
> On 3.12.2023 00:47, David Heidelberg wrote:
>> Required for dt-schema validation.
>>
>> Signed-off-by: David Heidelberg <david@ixit.cz>
>> ---
> Only patches 2 and 3 made it to my inbox and linux-arm-msm
>
> Konrad

-- 
David Heidelberg

Re: [PATCH v4 2/3] ARM: dts: qcom: include cpu in idle-state node names

[David Heidelberg]

Ignore the link and v4, wrong patchset. At least linux-riscv accepted 
the patch:

https://patchwork.kernel.org/project/linux-riscv/list/?series=806279&state=*&archive=both

David

On 09/12/2023 18:31, David Heidelberg wrote:
> Hello Konrad,
>
> v4 should be outside, 
> https://patchwork.kernel.org/project/linux-hwmon/patch/20231209171653.85468-1-david@ixit.cz/
>
> Sadly, v3 probably got filtered by SPAM filter :(
>
> David
>
> On 04/12/2023 12:12, Konrad Dybcio wrote:
>> On 3.12.2023 00:47, David Heidelberg wrote:
>>> Required for dt-schema validation.
>>>
>>> Signed-off-by: David Heidelberg <david@ixit.cz>
>>> ---
>> Only patches 2 and 3 made it to my inbox and linux-arm-msm
>>
>> Konrad
>
-- 
David Heidelberg