[coreboot-gerrit] New patch to review for coreboot: intel/i82801gx 82801ix: Remove OpRegion of SMBus host

[Kyösti Mälkki (kyosti.malkki@gmail.com)]

Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16266

-gerrit

commit 2b488beef5db6faaad9b12afc26be23c5690cc37
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Fri Aug 19 23:41:32 2016 +0300

    intel/i82801gx 82801ix: Remove OpRegion of SMBus host
    
    Defining this OpRegion for SMBus controller prevents linux kernel
    driver i2c-i801 from registering SMBus under sysfs, with following
    error in dmesg:
    
      ACPI Warning: SystemIO range .. conflicts with OpRegion .. (\_SB.PCI0.SBUS.SMBI)
    
    Solution taken from intel/bd82x6x. Worth noting we do not
    define ENABLE_SMBUS_METHODS anywhere currently.
    
    Change-Id: I7c13d344b0343387681b46019cc5061b1435b46b
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/southbridge/intel/i82801gx/acpi/smbus.asl | 2 +-
 src/southbridge/intel/i82801ix/acpi/smbus.asl | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/southbridge/intel/i82801gx/acpi/smbus.asl b/src/southbridge/intel/i82801gx/acpi/smbus.asl
index 3c4337f..cad9cbc 100644
--- a/src/southbridge/intel/i82801gx/acpi/smbus.asl
+++ b/src/southbridge/intel/i82801gx/acpi/smbus.asl
@@ -20,6 +20,7 @@ Device (SBUS)
 {
 	Name (_ADR, 0x001f0003)
 
+ifdef ENABLE_SMBUS_METHODS
 	OperationRegion (SMBP, PCI_Config, 0x00, 0x100)
 	Field(SMBP, DWordAcc, NoLock, Preserve)
 	{
@@ -53,7 +54,6 @@ Device (SBUS)
 		NDLH,	8,	// Notify Data High Byte
 	}
 
-#ifdef ENABLE_SMBUS_METHODS
 	// Kill all SMBus communication
 	Method (KILL, 0, Serialized)
 	{
diff --git a/src/southbridge/intel/i82801ix/acpi/smbus.asl b/src/southbridge/intel/i82801ix/acpi/smbus.asl
index 3c4337f..3b314e4 100644
--- a/src/southbridge/intel/i82801ix/acpi/smbus.asl
+++ b/src/southbridge/intel/i82801ix/acpi/smbus.asl
@@ -20,6 +20,7 @@ Device (SBUS)
 {
 	Name (_ADR, 0x001f0003)
 
+#ifdef ENABLE_SMBUS_METHODS
 	OperationRegion (SMBP, PCI_Config, 0x00, 0x100)
 	Field(SMBP, DWordAcc, NoLock, Preserve)
 	{
@@ -53,7 +54,6 @@ Device (SBUS)
 		NDLH,	8,	// Notify Data High Byte
 	}
 
-#ifdef ENABLE_SMBUS_METHODS
 	// Kill all SMBus communication
 	Method (KILL, 0, Serialized)
 	{