Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79565?usp=email )
(
3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: arch/x86/acpi: call pm_acpi_smi_cmd_port to get APMC SMI IO port ......................................................................
arch/x86/acpi: call pm_acpi_smi_cmd_port to get APMC SMI IO port
Instead of hard-coding the APMC SMI command IO port in the FADT, call pm_acpi_smi_cmd_port() to get the APMC SMI command IO port.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I731c780bc6db7e7fd59688340bab1da86fc93c11 Reviewed-on: https://review.coreboot.org/c/coreboot/+/79565 Reviewed-by: Jérémy Compostella jeremy.compostella@intel.com Reviewed-by: Patrick Rudolph patrick.rudolph@9elements.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/arch/x86/acpi.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Patrick Rudolph: Looks good to me, approved build bot (Jenkins): Verified Jérémy Compostella: Looks good to me, but someone else must approve
diff --git a/src/arch/x86/acpi.c b/src/arch/x86/acpi.c index 018cdf5..2497143 100644 --- a/src/arch/x86/acpi.c +++ b/src/arch/x86/acpi.c @@ -39,7 +39,7 @@ }
if (permanent_smi_handler()) { - fadt->smi_cmd = APM_CNT; + fadt->smi_cmd = pm_acpi_smi_cmd_port(); fadt->acpi_enable = APM_CNT_ACPI_ENABLE; fadt->acpi_disable = APM_CNT_ACPI_DISABLE; }