From 81ee8e2f2e859ace24bf2ecd8977079b87a4429c Mon Sep 17 00:00:00 2001 From: Fred Reitberger Date: Thu, 24 Mar 2022 07:42:40 -0400 Subject: [PATCH] soc/amd/picasso/cpu.c: Skip SMMINFO init in S3 resume SMMINFO is already set up in S5, so it should be skipped in S3 resume TEST=builds Change-Id: Ia58000ce9dac5ecb69ca39354f7775524e439bd0 Signed-off-by: Fred Reitberger Reviewed-on: https://review.coreboot.org/c/coreboot/+/63087 Tested-by: build bot (Jenkins) Reviewed-by: Raul Rangel --- src/soc/amd/picasso/cpu.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/soc/amd/picasso/cpu.c b/src/soc/amd/picasso/cpu.c index de6e9c035b..f14ee49648 100644 --- a/src/soc/amd/picasso/cpu.c +++ b/src/soc/amd/picasso/cpu.c @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -41,7 +42,10 @@ static void pre_mp_init(void) static void post_mp_init(void) { global_smi_enable(); - apm_control(APM_CNT_SMMINFO); + + /* SMMINFO only needs to be set up when booting from S5 */ + if (!acpi_is_wakeup_s3()) + apm_control(APM_CNT_SMMINFO); } static const struct mp_ops mp_ops = {