From f3758b6738d2968980309257197ef4f1977dc974 Mon Sep 17 00:00:00 2001
From: Kyösti Mälkki <kyosti.malkki@gmail.com>
Date: Tue, 8 Oct 2019 19:25:57 +0300
Subject: AGESA,binaryPI: Move SCI enable outside table creation
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Preferably, coreboot tables creation is kept hardware-invariant.

Change-Id: I37810771090dd9b0377f9a72c7a17ef1564ccf68
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36812
Reviewed-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 src/southbridge/amd/pi/hudson/lpc.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

(limited to 'src/southbridge/amd/pi/hudson/lpc.c')

diff --git a/src/southbridge/amd/pi/hudson/lpc.c b/src/southbridge/amd/pi/hudson/lpc.c
index 1e080a0e8e..e65fd838b0 100644
--- a/src/southbridge/amd/pi/hudson/lpc.c
+++ b/src/southbridge/amd/pi/hudson/lpc.c
@@ -23,6 +23,7 @@
 #include <device/pci_def.h>
 #include <pc80/mc146818rtc.h>
 #include <pc80/isa-dma.h>
+#include <arch/io.h>
 #include <arch/ioapic.h>
 #include <arch/acpi.h>
 #include <pc80/i8254.h>
@@ -351,6 +352,16 @@ static const char *lpc_acpi_name(const struct device *dev)
 	return NULL;
 }
 
+static void lpc_final(struct device *dev)
+{
+	if (!acpi_is_wakeup_s3()) {
+		if (CONFIG(HAVE_SMI_HANDLER))
+			outl(0x0, ACPI_PM1_CNT_BLK);	/* clear SCI_EN */
+		else
+			outl(0x1, ACPI_PM1_CNT_BLK);	/* set SCI_EN */
+	}
+}
+
 static struct pci_operations lops_pci = {
 	.set_subsystem = pci_dev_set_subsystem,
 };
@@ -363,6 +374,7 @@ static struct device_operations lpc_ops = {
 	.write_acpi_tables = acpi_write_hpet,
 #endif
 	.init = lpc_init,
+	.final = lpc_final,
 	.scan_bus = scan_static_bus,
 	.ops_pci = &lops_pci,
 	.acpi_name = lpc_acpi_name,
-- 
cgit v1.2.3