Sean Rhodes has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86089?usp=email )
(
2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: soc/intel/alderlake: Fix incorrect reporting of S0ix ......................................................................
soc/intel/alderlake: Fix incorrect reporting of S0ix
If S0ix is not enabled, then it should not be reported that it is supported.
TEST=boot linux on starlabs/starlite_adl, check s2idle isn't listed under `/sys/power/mem_sleep`.
Change-Id: Ia31fbfd0b9795990b0ca98220bb002bf2c3857b2 Signed-off-by: Sean Rhodes sean@starlabs.systems Reviewed-on: https://review.coreboot.org/c/coreboot/+/86089 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Subrata Banik subratabanik@google.com --- M src/soc/intel/alderlake/cpu.c 1 file changed, 4 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Subrata Banik: Looks good to me, approved
diff --git a/src/soc/intel/alderlake/cpu.c b/src/soc/intel/alderlake/cpu.c index 00c683c..9e4ed52 100644 --- a/src/soc/intel/alderlake/cpu.c +++ b/src/soc/intel/alderlake/cpu.c @@ -323,6 +323,10 @@
uint8_t get_supported_lpm_mask(void) { + const config_t *conf = config_of_soc(); + if (!conf->s0ix_enable) + return 0; + enum adl_cpu_type type = get_adl_cpu_type(); switch (type) { case ADL_M: /* fallthrough */