[coreboot-gerrit] Change in coreboot[master]: sb/intel/lynxpoint: Only generate SerialIO SSDT for PCH-LP

[Angel Pons (Code Review)]

Attention is currently required from: Patrick Rudolph.
Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/50477 )

Change subject: sb/intel/lynxpoint: Only generate SerialIO SSDT for PCH-LP
......................................................................

sb/intel/lynxpoint: Only generate SerialIO SSDT for PCH-LP

Lynxpoint PCH-H does not have SerialIO, so do not generate its SSDT.

Change-Id: Ie816ebd470df93a45826498bf21be59ff0a813bf
Signed-off-by: Angel Pons 
---
M src/southbridge/intel/lynxpoint/lpc.c
1 file changed, 8 insertions(+), 7 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/50477/1

diff --git a/src/southbridge/intel/lynxpoint/lpc.c b/src/southbridge/intel/lynxpoint/lpc.c
index b0ff545..3685a41 100644
--- a/src/southbridge/intel/lynxpoint/lpc.c
+++ b/src/southbridge/intel/lynxpoint/lpc.c
@@ -692,7 +692,6 @@
    					   struct acpi_rsdp *rsdp)
 {
    unsigned long current;
-	acpi_header_t *ssdt;

current = start;

@@ -706,12 +705,14 @@

current = acpi_align_current(current);

-	printk(BIOS_DEBUG, "ACPI:     * SSDT2\n");
-	ssdt = (acpi_header_t *)current;
-	acpi_create_serialio_ssdt(ssdt);
-	current += ssdt->length;
-	acpi_add_table(rsdp, ssdt);
-	current = acpi_align_current(current);
+	if (pch_is_lp()) {
+		printk(BIOS_DEBUG, "ACPI:     * SSDT2\n");
+		acpi_header_t *ssdt = (acpi_header_t *)current;
+		acpi_create_serialio_ssdt(ssdt);
+		current += ssdt->length;
+		acpi_add_table(rsdp, ssdt);
+		current = acpi_align_current(current);
+	}

printk(BIOS_DEBUG, "current = %lx\n", current);
    return current;