diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2020-07-01 15:59:20 +0300 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2021-02-10 01:31:09 +0000 |
commit | 2fec394913d2ef721623d796ac1db82d19ab5195 (patch) | |
tree | 560aef1e9d50d54fd9a4c60df6479313e83ae149 /src/soc/amd/picasso/smi.c | |
parent | 708e4cb596cea7603c1ccb25d13b1fa05c900caf (diff) |
soc/amd/picasso: Move APM_CNT_SMMINFO callsite
Triggering SMI is not part of the semantics of global_smi_enable(), so
move it to the post_mp_init handler. Even without the !acpi_is_wakeup_s3
check we don't get PSP warnings/errors during resume, so we can drop the
workaround introduced in commit 5dbe45e0f5608794e03634aed28530ddb2ab9ac5
in this patch.
Change-Id: Id0e7723c2bb9811f80fe36c38199a01445dc1d7d
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42987
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/soc/amd/picasso/smi.c')
-rw-r--r-- | src/soc/amd/picasso/smi.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/soc/amd/picasso/smi.c b/src/soc/amd/picasso/smi.c index 58221513a1..fb6d3484ad 100644 --- a/src/soc/amd/picasso/smi.c +++ b/src/soc/amd/picasso/smi.c @@ -4,8 +4,6 @@ * Utilities for SMM setup */ -#include <acpi/acpi.h> -#include <arch/io.h> #include <console/console.h> #include <cpu/x86/smm.h> #include <amdblocks/acpimmio.h> @@ -19,7 +17,4 @@ void global_smi_enable(void) reg &= ~SMITRG0_SMIENB; /* Enable SMI generation */ reg |= SMITRG0_EOS; /* Set EOS bit */ smi_write32(SMI_REG_SMITRIG0, reg); - - if (!acpi_is_wakeup_s3()) - outb(APM_CNT_SMMINFO, APM_CNT); } |