* [PATCH] arm64: dts: qcom: hamoa/x1: fix idle exit latency
@ 2026-02-20 7:02 Daniel J Blueman
2026-02-20 9:41 ` Konrad Dybcio
0 siblings, 1 reply; 3+ messages in thread
From: Daniel J Blueman @ 2026-02-20 7:02 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel, Daniel J Blueman
Designs based on the Qualcomm X1 Hamoa reference platform report:
driver: Idle state 1 target residency too low
This is because the declared X1 idle entry plus exit latency of 680us
exceeds the declared minimum 600us residency time:
entry-latency-us = <180>;
exit-latency-us = <500>;
min-residency-us = <600>;
As 500us idle exit latency is overly conservative, fix this to be a more
realistic 200us, in range of similar silicon. Tested on a Lenovo Yoga Slim
7x with Qualcomm X1E-80-100.
Signed-off-by: Daniel J Blueman <daniel@quora.org>
---
arch/arm64/boot/dts/qcom/hamoa.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/hamoa.dtsi b/arch/arm64/boot/dts/qcom/hamoa.dtsi
index a17900eacb20..b870f4dc9c42 100644
--- a/arch/arm64/boot/dts/qcom/hamoa.dtsi
+++ b/arch/arm64/boot/dts/qcom/hamoa.dtsi
@@ -269,7 +269,7 @@ cluster_c4: cpu-sleep-0 {
idle-state-name = "ret";
arm,psci-suspend-param = <0x00000004>;
entry-latency-us = <180>;
- exit-latency-us = <500>;
+ exit-latency-us = <200>;
min-residency-us = <600>;
};
};
--
2.51.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] arm64: dts: qcom: hamoa/x1: fix idle exit latency
2026-02-20 7:02 [PATCH] arm64: dts: qcom: hamoa/x1: fix idle exit latency Daniel J Blueman
@ 2026-02-20 9:41 ` Konrad Dybcio
2026-02-20 11:10 ` Maulik Shah (mkshah)
0 siblings, 1 reply; 3+ messages in thread
From: Konrad Dybcio @ 2026-02-20 9:41 UTC (permalink / raw)
To: Daniel J Blueman, Bjorn Andersson, Konrad Dybcio, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Maulik Shah
Cc: linux-arm-msm, devicetree, linux-kernel
On 2/20/26 8:02 AM, Daniel J Blueman wrote:
> Designs based on the Qualcomm X1 Hamoa reference platform report:
> driver: Idle state 1 target residency too low
>
> This is because the declared X1 idle entry plus exit latency of 680us
> exceeds the declared minimum 600us residency time:
> entry-latency-us = <180>;
> exit-latency-us = <500>;
> min-residency-us = <600>;
>
> As 500us idle exit latency is overly conservative, fix this to be a more
> realistic 200us, in range of similar silicon. Tested on a Lenovo Yoga Slim
> 7x with Qualcomm X1E-80-100.
>
> Signed-off-by: Daniel J Blueman <daniel@quora.org>
> ---
> arch/arm64/boot/dts/qcom/hamoa.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/hamoa.dtsi b/arch/arm64/boot/dts/qcom/hamoa.dtsi
> index a17900eacb20..b870f4dc9c42 100644
> --- a/arch/arm64/boot/dts/qcom/hamoa.dtsi
> +++ b/arch/arm64/boot/dts/qcom/hamoa.dtsi
> @@ -269,7 +269,7 @@ cluster_c4: cpu-sleep-0 {
> idle-state-name = "ret";
> arm,psci-suspend-param = <0x00000004>;
> entry-latency-us = <180>;
> - exit-latency-us = <500>;
> + exit-latency-us = <200>;
> min-residency-us = <600>;
+Maulik?
Konrad
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] arm64: dts: qcom: hamoa/x1: fix idle exit latency
2026-02-20 9:41 ` Konrad Dybcio
@ 2026-02-20 11:10 ` Maulik Shah (mkshah)
0 siblings, 0 replies; 3+ messages in thread
From: Maulik Shah (mkshah) @ 2026-02-20 11:10 UTC (permalink / raw)
To: Konrad Dybcio, Daniel J Blueman, Bjorn Andersson, Konrad Dybcio,
Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-msm, devicetree, linux-kernel
On 2/20/2026 3:11 PM, Konrad Dybcio wrote:
> On 2/20/26 8:02 AM, Daniel J Blueman wrote:
>> Designs based on the Qualcomm X1 Hamoa reference platform report:
>> driver: Idle state 1 target residency too low
>>
>> This is because the declared X1 idle entry plus exit latency of 680us
>> exceeds the declared minimum 600us residency time:
>> entry-latency-us = <180>;
>> exit-latency-us = <500>;
>> min-residency-us = <600>;
>>
>> As 500us idle exit latency is overly conservative, fix this to be a more
>> realistic 200us, in range of similar silicon. Tested on a Lenovo Yoga Slim
>> 7x with Qualcomm X1E-80-100.
>>
>> Signed-off-by: Daniel J Blueman <daniel@quora.org>
>> ---
>> arch/arm64/boot/dts/qcom/hamoa.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/hamoa.dtsi b/arch/arm64/boot/dts/qcom/hamoa.dtsi
>> index a17900eacb20..b870f4dc9c42 100644
>> --- a/arch/arm64/boot/dts/qcom/hamoa.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/hamoa.dtsi
>> @@ -269,7 +269,7 @@ cluster_c4: cpu-sleep-0 {
>> idle-state-name = "ret";
>> arm,psci-suspend-param = <0x00000004>;
>> entry-latency-us = <180>;
>> - exit-latency-us = <500>;
>> + exit-latency-us = <200>;
>> min-residency-us = <600>;
>
> +Maulik?
>
> Konrad
Linux dt idle driver sums up entry & exit latency and displays final number as total exit latency at
/sys/devices/system/cpu/cpu*/cpuidle/state1/latency
Can you please keep 320 usec as exit-latency-us to match with downstream 500usec total exit latency number from cluster_c4 state.
Thanks,
Maulik
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-02-20 11:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-20 7:02 [PATCH] arm64: dts: qcom: hamoa/x1: fix idle exit latency Daniel J Blueman
2026-02-20 9:41 ` Konrad Dybcio
2026-02-20 11:10 ` Maulik Shah (mkshah)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox