[coreboot-gerrit] Change in coreboot[master]: mb/emulation/qemu-q35: Control SMI related FADT entries

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41907 )

Change subject: mb/emulation/qemu-q35: Control SMI related FADT entries
......................................................................

mb/emulation/qemu-q35: Control SMI related FADT entries

Never advertise SMI in FADT for the emulation.

Change-Id: I170058793798648c6713de1530d89ec2ac53e39a
Signed-off-by: Kyösti Mälkki 
---
M src/mainboard/emulation/qemu-q35/acpi_tables.c
1 file changed, 7 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/41907/1

diff --git a/src/mainboard/emulation/qemu-q35/acpi_tables.c b/src/mainboard/emulation/qemu-q35/acpi_tables.c
index b9e76a9..4236257 100644
--- a/src/mainboard/emulation/qemu-q35/acpi_tables.c
+++ b/src/mainboard/emulation/qemu-q35/acpi_tables.c
@@ -42,11 +42,13 @@
    fadt->reserved = 0x00;
    fadt->preferred_pm_profile = PM_MOBILE;
    fadt->sci_int = 0x9;
-	fadt->smi_cmd = 0;
-	fadt->acpi_enable = 0;
-	fadt->acpi_disable = 0;
-	fadt->s4bios_req = 0x0;
-	fadt->pstate_cnt = 0;
+
+	if (CONFIG(HAVE_SMI_HANDLER)) {
+		/* Do not advertise SMI even if installed. */
+		fadt->smi_cmd = 0;
+		fadt->acpi_enable = 0;
+		fadt->acpi_disable = 0;
+	}

fadt->pm1a_evt_blk = pmbase;
    fadt->pm1b_evt_blk = 0x0;
@@ -65,7 +67,6 @@
    fadt->gpe0_blk_len = 16;
    fadt->gpe1_blk_len = 0;
    fadt->gpe1_base = 0;
-	fadt->cst_cnt = 0;
    fadt->p_lvl2_lat = 1;
    fadt->p_lvl3_lat = 0x39;
    fadt->flush_size = 0;