diff mbox series

arm64: dts: qcom: qcm6490: Update PSCI suspend param for cluster sleep

Message ID 20240105070223.11415-1-quic_mkshah@quicinc.com
State New
Headers show
Series arm64: dts: qcom: qcm6490: Update PSCI suspend param for cluster sleep | expand

Commit Message

Maulik Shah Jan. 5, 2024, 7:02 a.m. UTC
QCM6490 uses Trustzone as firmware whereas SC7280 uses arm trusted
firmware. The PSCI suspend param is different in Trustzone for cluster
sleep. Update the same for qcm6490 based idp and fairphone fp5 boards.

Signed-off-by: Maulik Shah <quic_mkshah@quicinc.com>
---
 arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts | 4 ++++
 arch/arm64/boot/dts/qcom/qcm6490-idp.dts           | 4 ++++
 2 files changed, 8 insertions(+)

Comments

Konrad Dybcio Jan. 5, 2024, 10:05 a.m. UTC | #1
On 5.01.2024 08:02, Maulik Shah wrote:
> QCM6490 uses Trustzone as firmware whereas SC7280 uses arm trusted
> firmware. The PSCI suspend param is different in Trustzone for cluster
> sleep. Update the same for qcm6490 based idp and fairphone fp5 boards.
> 
> Signed-off-by: Maulik Shah <quic_mkshah@quicinc.com>
> ---

Nice spot!

For things that are chrome-specific, I'd prefer them to be moved to
-chrome.dtsi and then the "normal" (LA+LE+WP+friends) values to
stay as default.

>  arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts | 4 ++++
>  arch/arm64/boot/dts/qcom/qcm6490-idp.dts           | 4 ++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
> index 176898c9dbbd..b56f6f26d96f 100644
> --- a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
> +++ b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
> @@ -446,6 +446,10 @@
>  	};
>  };
>  
> +&CLUSTER_SLEEP_0 {
> +	arm,psci-suspend-param = <0x4100b344>;
>

I think the residency times should also be updated here.

Moreover, I'm seeing additional sleep states here (APSS PC, CX RET
and this one being AOSS SLEEP), with values 0x4<<4, 0x134<<4,
0xb34<<4 (the one you added here) respectively.

Could you please add them as well, similar to [1]?

Konrad

[1] https://lore.kernel.org/linux-arm-msm/20230619-topic-sc8280xp-idle-v2-1-cde50bf02f3c@linaro.org/
Maulik Shah Jan. 9, 2024, 3:52 p.m. UTC | #2
On 1/5/2024 3:35 PM, Konrad Dybcio wrote:
> On 5.01.2024 08:02, Maulik Shah wrote:
>> QCM6490 uses Trustzone as firmware whereas SC7280 uses arm trusted
>> firmware. The PSCI suspend param is different in Trustzone for cluster
>> sleep. Update the same for qcm6490 based idp and fairphone fp5 boards.
>>
>> Signed-off-by: Maulik Shah <quic_mkshah@quicinc.com>
>> ---
> 
> Nice spot!
> 
> For things that are chrome-specific, I'd prefer them to be moved to
> -chrome.dtsi and then the "normal" (LA+LE+WP+friends) values to
> stay as default.

Thanks for the review.
Moved the chrome specific changes to sc7280-chrome-common.dtsi in v2.

> 
>>   arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts | 4 ++++
>>   arch/arm64/boot/dts/qcom/qcm6490-idp.dts           | 4 ++++
>>   2 files changed, 8 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
>> index 176898c9dbbd..b56f6f26d96f 100644
>> --- a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
>> @@ -446,6 +446,10 @@
>>   	};
>>   };
>>   
>> +&CLUSTER_SLEEP_0 {
>> +	arm,psci-suspend-param = <0x4100b344>;
>>
> 
> I think the residency times should also be updated here.
> 
> Moreover, I'm seeing additional sleep states here (APSS PC, CX RET
> and this one being AOSS SLEEP), with values 0x4<<4, 0x134<<4,
> 0xb34<<4 (the one you added here) respectively.
> 
> Could you please add them as well, similar to [1]?
> 
> Konrad
> 
> [1] https://lore.kernel.org/linux-arm-msm/20230619-topic-sc8280xp-idle-v2-1-cde50bf02f3c@linaro.org/

Sure, added additional sleep stats with updated residency time in v2.

Thanks,
Maulik
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
index 176898c9dbbd..b56f6f26d96f 100644
--- a/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
+++ b/arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts
@@ -446,6 +446,10 @@ 
 	};
 };
 
+&CLUSTER_SLEEP_0 {
+	arm,psci-suspend-param = <0x4100b344>;
+};
+
 &dispcc {
 	/* Disable for now so simple-framebuffer continues working */
 	status = "disabled";
diff --git a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
index 03e97e27d16d..afac0ea0d297 100644
--- a/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
+++ b/arch/arm64/boot/dts/qcom/qcm6490-idp.dts
@@ -415,6 +415,10 @@ 
 	};
 };
 
+&CLUSTER_SLEEP_0 {
+	arm,psci-suspend-param = <0x4100b344>;
+};
+
 &qupv3_id_0 {
 	status = "okay";
 };