From 216e58a31288bd25802d280c81a0e8d01b3b11d1 Mon Sep 17 00:00:00 2001 From: Roberto Vargas Date: Mon, 4 Sep 2017 16:49:41 +0100 Subject: [PATCH] Reduce time lock in psci_do_cpu_off psci_set_power_off_state only initializes a local variable, so there isn't any reason why it should be done while the lock is held. Change-Id: I1c62f4cd5d860d102532e5a5350152180d41d127 Signed-off-by: Roberto Vargas --- lib/psci/psci_off.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/psci/psci_off.c b/lib/psci/psci_off.c index e7fb6532..231deea2 100644 --- a/lib/psci/psci_off.c +++ b/lib/psci/psci_off.c @@ -49,6 +49,9 @@ int psci_do_cpu_off(unsigned int end_pwrlvl) */ assert(psci_plat_pm_ops->pwr_domain_off); + /* Construct the psci_power_state for CPU_OFF */ + psci_set_power_off_state(&state_info); + /* * This function acquires the lock corresponding to each power * level so that by the time all locks are taken, the system topology @@ -68,9 +71,6 @@ int psci_do_cpu_off(unsigned int end_pwrlvl) goto exit; } - /* Construct the psci_power_state for CPU_OFF */ - psci_set_power_off_state(&state_info); - /* * This function is passed the requested state info and * it returns the negotiated state info for each power level upto -- 2.30.2