Messages in this thread | | | Subject | Re: [PATCH v9 05/17] ARM: tegra: Propagate error from tegra_idle_lp2_last() | From | Dmitry Osipenko <> | Date | Fri, 21 Feb 2020 20:21:41 +0300 |
| |
21.02.2020 18:16, Daniel Lezcano пишет: > 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 ?
Yes, because suspend_cpu_complex() didn't fail. I don't see any reason why restore_cpu_complex() shouldn't be called, please clarify yours thought.
| |