diff options
| author |   Dmitry Osipenko <digetx@gmail.com> | 2020-02-25 01:40:45 +0300 |
|---|---|---|
| committer |   Thierry Reding <treding@nvidia.com> | 2020-03-12 10:54:05 +0100 |
| commit | 891e1286c138eafb26d1f36eee3f31b8cd2c2bd9 (patch) | |
| tree | b64239faed7373f54c4de088a0a630e0f98916e9 /arch/arm/mach-tegra/cpuidle-tegra30.c | |
| parent | ARM: tegra: Change tegra_set_cpu_in_lp2() type to void (diff) | |
| download | linux-dev-891e1286c138eafb26d1f36eee3f31b8cd2c2bd9.tar.xz linux-dev-891e1286c138eafb26d1f36eee3f31b8cd2c2bd9.zip | |
ARM: tegra: Propagate error from tegra_idle_lp2_last()
Technically cpu_suspend() may fail and it's never good to lose information
about failure. For example things like cpuidle core could correctly sample
idling time in the case of failure.
Acked-by: Peter De Schrijver <pdeschrijver@nvidia.com>
Tested-by: Peter Geis <pgwipeout@gmail.com>
Tested-by: Jasper Korten <jja2000@gmail.com>
Tested-by: David Heidelberg <david@ixit.cz>
Tested-by: Nicolas Chauvet <kwizart@gmail.com>
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Diffstat (limited to 'arch/arm/mach-tegra/cpuidle-tegra30.c')
| -rw-r--r-- | arch/arm/mach-tegra/cpuidle-tegra30.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/arm/mach-tegra/cpuidle-tegra30.c b/arch/arm/mach-tegra/cpuidle-tegra30.c index a3ce8dabfe18..17cbd118abee 100644 --- a/arch/arm/mach-tegra/cpuidle-tegra30.c +++ b/arch/arm/mach-tegra/cpuidle-tegra30.c @@ -68,9 +68,7 @@ static bool tegra30_cpu_cluster_power_down(struct cpuidle_device *dev, return false; } - tegra_idle_lp2_last(); - - return true; + return !tegra_idle_lp2_last(); } #ifdef CONFIG_SMP |