Messages in this thread | | | Date | Fri, 21 Feb 2020 16:16:12 +0100 | From | Daniel Lezcano <> | Subject | Re: [PATCH v9 05/17] ARM: tegra: Propagate error from tegra_idle_lp2_last() |
| |
On Thu, Feb 13, 2020 at 02:51:22AM +0300, Dmitry Osipenko wrote: > 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> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com> > ---
[ ... ]
> cpu_cluster_pm_enter(); > suspend_cpu_complex(); > > - cpu_suspend(PHYS_OFFSET - PAGE_OFFSET, &tegra_sleep_cpu); > + err = cpu_suspend(PHYS_OFFSET - PAGE_OFFSET, &tegra_sleep_cpu); > > /* > * Resume L2 cache if it wasn't re-enabled early during resume, > @@ -208,6 +210,8 @@ void tegra_idle_lp2_last(void) > > restore_cpu_complex();
If the cpu_suspend fails, does restore_cpu_complex() need to be called ?
> cpu_cluster_pm_exit(); > + > + return err; > }
[ ... ]
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
| |