c06fb25d1f
Some checks failed
Build Kernel / Build all affected Kernels (push) Has been cancelled
Build all core packages / Build all core packages for selected target (push) Has been cancelled
Build and Push prebuilt tools container / Build and Push all prebuilt containers (push) Has been cancelled
Build Toolchains / Build Toolchains for each target (push) Has been cancelled
Build host tools / Build host tools for linux and macos based systems (push) Has been cancelled
Coverity scan build / Coverity x86/64 build (push) Has been cancelled
41 lines
1.4 KiB
Diff
41 lines
1.4 KiB
Diff
From c28b2d367da8a471482e6a4aa8337ab6369a80c2 Mon Sep 17 00:00:00 2001
|
|
From: Russell King <rmk+kernel@arm.linux.org.uk>
|
|
Date: Sat, 3 Oct 2015 09:13:05 +0100
|
|
Subject: cpuidle: mvebu: indicate failure to enter deeper sleep states
|
|
|
|
The cpuidle ->enter method expects the return value to be the sleep
|
|
state we entered. Returning negative numbers or other codes is not
|
|
permissible since coupled CPU idle was merged.
|
|
|
|
At least some of the mvebu_v7_cpu_suspend() implementations return the
|
|
value from cpu_suspend(), which returns zero if the CPU vectors back
|
|
into the kernel via cpu_resume() (the success case), or the non-zero
|
|
return value of the suspend actor, or one (failure cases).
|
|
|
|
We do not want to be returning the failure case value back to CPU idle
|
|
as that indicates that we successfully entered one of the deeper idle
|
|
states. Always return zero instead, indicating that we slept for the
|
|
shortest amount of time.
|
|
|
|
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
|
|
---
|
|
drivers/cpuidle/cpuidle-mvebu-v7.c | 6 +++++-
|
|
1 file changed, 5 insertions(+), 1 deletion(-)
|
|
|
|
--- a/drivers/cpuidle/cpuidle-mvebu-v7.c
|
|
+++ b/drivers/cpuidle/cpuidle-mvebu-v7.c
|
|
@@ -42,8 +42,12 @@ static __cpuidle int mvebu_v7_enter_idle
|
|
|
|
cpu_pm_exit();
|
|
|
|
+ /*
|
|
+ * If we failed to enter the desired state, indicate that we
|
|
+ * slept lightly.
|
|
+ */
|
|
if (ret)
|
|
- return ret;
|
|
+ return 0;
|
|
|
|
return index;
|
|
}
|