| commit | 141955ac74b8fbbefd1f94155a236f0e0ca5eaff | [log] [tgz] |
|---|---|---|
| author | Felix Held <felix-coreboot@felixheld.de> | Fri Jan 05 22:15:46 2024 +0100 |
| committer | Felix Held <felix-coreboot@felixheld.de> | Wed Jan 10 14:44:51 2024 +0000 |
| tree | 121e82fe58799b57b737e5b3477dcc9ff15a86bf | |
| parent | 21710ea3cdff6de647a1f507f1e50bf382c2cf14 [diff] |
soc/amd: use apm_get_apmc() in APMC SMI handler Instead of open-coding this functionality, call the apm_get_apmc() helper function. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: Iac6b614d900e51d91a0c155116a5edc29775ea99 Reviewed-on: https://review.coreboot.org/c/coreboot/+/79836 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
diff --git a/src/soc/amd/common/block/cpu/smm/smi_apmc.c b/src/soc/amd/common/block/cpu/smm/smi_apmc.c index 0eab86c..9df6ae3 100644 --- a/src/soc/amd/common/block/cpu/smm/smi_apmc.c +++ b/src/soc/amd/common/block/cpu/smm/smi_apmc.c
@@ -91,7 +91,7 @@ void fch_apmc_smi_handler(void) { - const uint8_t cmd = inb(pm_acpi_smi_cmd_port()); + const uint8_t cmd = apm_get_apmc(); switch (cmd) { case APM_CNT_ACPI_ENABLE:
diff --git a/src/soc/amd/stoneyridge/smihandler.c b/src/soc/amd/stoneyridge/smihandler.c index a8521c5..f962fa0 100644 --- a/src/soc/amd/stoneyridge/smihandler.c +++ b/src/soc/amd/stoneyridge/smihandler.c
@@ -21,7 +21,7 @@ */ static void stoneyridge_fch_apmc_smi_handler(void) { - const uint8_t cmd = inb(pm_acpi_smi_cmd_port()); + const uint8_t cmd = apm_get_apmc(); switch (cmd) { case APM_CNT_ACPI_ENABLE: