[coreboot-gerrit] Change in coreboot[master]: sb/intel/i82371eb: Claim less I/O ports in ACPI

[Keith Hui (Code Review)]

Keith Hui has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41093 )

Change subject: sb/intel/i82371eb: Claim less I/O ports in ACPI
......................................................................

sb/intel/i82371eb: Claim less I/O ports in ACPI

To avoid resource conflicts, this change leaves unclaimed:
- PM and SMBus ports (claimed by MBRS device written in SSDT)
- Ports 0x2e-0x2f (After reviewing Asus P3B-F OEM firmware)

Change-Id: Id5adb37d047621d7c8faf81607ceea4cbcac3d34
Signed-off-by: Keith Hui 
---
M src/southbridge/intel/i82371eb/acpi/i82371eb.asl
1 file changed, 4 insertions(+), 14 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/41093/1

diff --git a/src/southbridge/intel/i82371eb/acpi/i82371eb.asl b/src/southbridge/intel/i82371eb/acpi/i82371eb.asl
index 8b60edb..45e7a5e 100644
--- a/src/southbridge/intel/i82371eb/acpi/i82371eb.asl
+++ b/src/southbridge/intel/i82371eb/acpi/i82371eb.asl
@@ -31,15 +31,13 @@
    	{
    	Name (BUF1, ResourceTemplate ()
    	{
-			/* PM register ports */
-			IO (Decode16, 0x0000, 0x0000, 0x01, 0x40, _Y06)
-			/* SMBus register ports */
-			IO (Decode16, 0x0000, 0x0000, 0x01, 0x10, _Y07)
    		/* PIIX4E ports */
    		/* Aliased DMA ports */
    		IO (Decode16, 0x0010, 0x0010, 0x01, 0x10, )
    		/* Aliased PIC ports */
-			IO (Decode16, 0x0022, 0x0022, 0x01, 0x1E, )
+			/* Do not claim 0x2e-0x2f, per P3B-F vendor DSDT */
+			IO (Decode16, 0x0022, 0x0022, 0x01, 0x0C, )
+			IO (Decode16, 0x0030, 0x0030, 0x01, 0x10, )
    		/* Aliased timer ports */
    		IO (Decode16, 0x0050, 0x0050, 0x01, 0x04, )
    		IO (Decode16, 0x0062, 0x0062, 0x01, 0x02, )
@@ -49,18 +47,10 @@
    		IO (Decode16, 0x00A2, 0x00A2, 0x01, 0x1E, )
    		IO (Decode16, 0x00E0, 0x00E0, 0x01, 0x10, )
    		IO (Decode16, 0x0294, 0x0294, 0x01, 0x04, )
+			/* W83977TF/EF Super I/O config ports */
    		IO (Decode16, 0x03F0, 0x03F0, 0x01, 0x02, )
    		IO (Decode16, 0x04D0, 0x04D0, 0x01, 0x02, )
    	})
-		CreateWordField (BUF1, _Y06._MIN, PMLO)
-		CreateWordField (BUF1, _Y06._MAX, PMRL)
-		CreateWordField (BUF1, _Y07._MIN, SBLO)
-		CreateWordField (BUF1, _Y07._MAX, SBRL)
-
-		And (_SB.PCI0.PX43.PM00, 0xFFFE, PMLO)
-		And (_SB.PCI0.PX43.SB00, 0xFFFE, SBLO)
-		Store (PMLO, PMRL)
-		Store (SBLO, SBRL)
    	Return (BUF1)
    	}
    }