diff options
| author | Akshay Adiga <akshay.adiga@linux.vnet.ibm.com> | 2018-01-04 16:58:04 +0530 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-01-14 21:05:51 -0600 |
| commit | e5c663c3f6debd5164cfced486fa926ab57b3656 (patch) | |
| tree | a3c457cac9854fde1c2fc4442313fa8c145aa1fd | |
| parent | a523298bc1e34466bfc4477e4d9aa4352703298a (diff) | |
| download | skiboot-e5c663c3f6debd5164cfced486fa926ab57b3656.zip skiboot-e5c663c3f6debd5164cfced486fa926ab57b3656.tar.gz skiboot-e5c663c3f6debd5164cfced486fa926ab57b3656.tar.bz2 | |
SLW: Call p9_stop_api only if deep_states are enabled
All init time p9_stop_api calls have been isolated to slw_late_init. If
p9_stop_api fails, then the deep states can be excluded from device tree.
For p9_stop_api called after device-tree for cpuidle is created ,
has_deep_states will be used to check if this call is even required.
Signed-off-by: Akshay Adiga <akshay.adiga@linux.vnet.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
| -rw-r--r-- | hw/slw.c | 9 |
1 files changed, 9 insertions, 0 deletions
@@ -924,6 +924,10 @@ void add_cpu_idle_state_properties(void) } } + if ((wakeup_engine_state == WAKEUP_ENGINE_PRESENT) && has_deep_states) + slw_late_init_p9(chip); + if (wakeup_engine_state != WAKEUP_ENGINE_PRESENT) + has_deep_states = false; } else if (chip->type == PROC_CHIP_P8_MURANO || chip->type == PROC_CHIP_P8_VENICE || chip->type == PROC_CHIP_P8_NAPLES) { @@ -1487,6 +1491,11 @@ int64_t opal_slw_set_reg(uint64_t cpu_pir, uint64_t sprn, uint64_t val) assert(chip); if (proc_gen == proc_gen_p9) { + if (!has_deep_states) { + prlog(PR_INFO, "SLW: Deep states not enabled\n"); + return OPAL_SUCCESS; + } + if (wakeup_engine_state != WAKEUP_ENGINE_PRESENT) { log_simple_error(&e_info(OPAL_RC_SLW_REG), "SLW: wakeup_engine in bad state=%d chip=%x\n", |