summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorMatt DeVillier <matt.devillier@gmail.com>2017-06-02 12:30:07 -0500
committerMartin Roth <martinroth@google.com>2017-06-09 16:29:10 +0200
commit6a67ffb6ea0e05d42222b6159802aa5e18b22f8f (patch)
treee328adff67f7564914d5bb17d0def95607eb4fd4 /src
parent397c7b341123ae832e3bb4c90b1934a0d3069645 (diff)
downloadcoreboot-6a67ffb6ea0e05d42222b6159802aa5e18b22f8f.tar.xz
soc/braswell: fix scope for I2C ACPI devices
For an unknown reason, the I2C ACPI devices were placed under \SB intead of \SB.PCI0, as with all other non-Atom based Intel platforms. While Linux is tolerant of this, Windows is not. Correct by moving I2C ACPI devices where they belong. Also, adjust I2C devices at board level for intel/strago and google/cyan as to not break compilation. Change-Id: Iaf8211bd86d6261ee8c4d9c4262338f7fe19ef43 Signed-off-by: Matt DeVillier <matt.devillier@gmail.com> Reviewed-on: https://review.coreboot.org/20055 Reviewed-by: Aaron Durbin <adurbin@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
-rw-r--r--src/mainboard/google/cyan/acpi/mainboard.asl18
-rw-r--r--src/mainboard/intel/strago/acpi/mainboard.asl16
-rw-r--r--src/soc/intel/braswell/acpi/southcluster.asl3
3 files changed, 20 insertions, 17 deletions
diff --git a/src/mainboard/google/cyan/acpi/mainboard.asl b/src/mainboard/google/cyan/acpi/mainboard.asl
index 218d81b7e8..31b9090809 100644
--- a/src/mainboard/google/cyan/acpi/mainboard.asl
+++ b/src/mainboard/google/cyan/acpi/mainboard.asl
@@ -26,7 +26,7 @@ Scope (\_SB)
}
}
-Scope (\_SB.I2C1)
+Scope (\_SB.PCI0.I2C1)
{
Device (ETSA)
{
@@ -44,7 +44,7 @@ Scope (\_SB.I2C1)
ControllerInitiated, /* SlaveMode */
400000, /* ConnectionSpeed */
AddressingMode7Bit, /* AddressingMode */
- "\\_SB.I2C1", /* ResourceSource */
+ "\\_SB.PCI0.I2C1", /* ResourceSource */
)
GpioInt (Edge, ActiveLow, ExclusiveAndWake, PullNone,,
"\\_SB.GPSW") { BOARD_TOUCH_GPIO_INDEX }
@@ -57,7 +57,7 @@ Scope (\_SB.I2C1)
ControllerInitiated, /* SlaveMode */
400000, /* ConnectionSpeed */
AddressingMode7Bit, /* AddressingMode */
- "\\_SB.I2C1", /* ResourceSource */
+ "\\_SB.PCI0.I2C1", /* ResourceSource */
)
GpioInt (Edge, ActiveLow, ExclusiveAndWake, PullNone,,
"\\_SB.GPNC") { BOARD_EVT_TOUCH_GPIO_INDEX }
@@ -86,7 +86,7 @@ Scope (\_SB.I2C1)
}
}
-Scope (\_SB.I2C2)
+Scope (\_SB.PCI0.I2C2)
{
/* Maxim Audio Codec */
Device (MAXM) /* Audio Codec driver I2C */
@@ -115,7 +115,7 @@ Scope (\_SB.I2C2)
ControllerInitiated, /* SlaveMode: default to ControllerInitiated */
400000, /* ConnectionSpeed: in Hz */
AddressingMode7Bit, /* Addressing Mode: default to 7 bit */
- "\\_SB.I2C2", /* ResourceSource: I2C bus controller name */
+ "\\_SB.PCI0.I2C2", /* ResourceSource: I2C bus controller name */
)
GpioInt (Edge, ActiveLow, ExclusiveAndWake, PullNone,,
@@ -159,7 +159,7 @@ Scope (\_SB.I2C2)
ControllerInitiated, /* SlaveMode: default to ControllerInitiated */
400000, /* ConnectionSpeed: in Hz */
AddressingMode7Bit, /* Addressing Mode: default to 7 bit */
- "\\_SB.I2C2", /* ResourceSource: I2C bus controller name */
+ "\\_SB.PCI0.I2C2", /* ResourceSource: I2C bus controller name */
)
GpioInt (Edge, ActiveLow, ExclusiveAndWake, PullNone,,
@@ -176,7 +176,7 @@ Scope (\_SB.I2C2)
}
}
-Scope (\_SB.I2C6)
+Scope (\_SB.PCI0.I2C6)
{
Device (ETPA)
{
@@ -192,7 +192,7 @@ Scope (\_SB.I2C6)
ControllerInitiated, /* SlaveMode */
400000, /* ConnectionSpeed */
AddressingMode7Bit, /* AddressingMode */
- "\\_SB.I2C6", /* ResourceSource */
+ "\\_SB.PCI0.I2C6", /* ResourceSource */
)
GpioInt (Edge, ActiveLow, ExclusiveAndWake, PullNone,,
"\\_SB.GPNC") { BOARD_TRACKPAD_GPIO_INDEX }
@@ -214,7 +214,7 @@ Scope (\_SB.I2C6)
}
}
-Scope (\_SB.LPEA)
+Scope (\_SB.PCI0.LPEA)
{
Name (GBUF, ResourceTemplate ()
{
diff --git a/src/mainboard/intel/strago/acpi/mainboard.asl b/src/mainboard/intel/strago/acpi/mainboard.asl
index c559296cc0..eaf9e854de 100644
--- a/src/mainboard/intel/strago/acpi/mainboard.asl
+++ b/src/mainboard/intel/strago/acpi/mainboard.asl
@@ -26,7 +26,7 @@ Scope (\_SB)
}
}
-Scope (\_SB.I2C1)
+Scope (\_SB.PCI0.I2C1)
{
Device (ATSB)
{
@@ -44,7 +44,7 @@ Scope (\_SB.I2C1)
ControllerInitiated, /* SlaveMode */
400000, /* ConnectionSpeed */
AddressingMode7Bit, /* AddressingMode */
- "\\_SB.I2C1", /* ResourceSource */
+ "\\_SB.PCI0.I2C1", /* ResourceSource */
)
Interrupt (ResourceConsumer, Edge, ActiveLow)
{
@@ -83,7 +83,7 @@ Scope (\_SB.I2C1)
ControllerInitiated, /* SlaveMode */
400000, /* ConnectionSpeed */
AddressingMode7Bit, /* AddressingMode */
- "\\_SB.I2C1", /* ResourceSource */
+ "\\_SB.PCI0.I2C1", /* ResourceSource */
)
Interrupt (ResourceConsumer, Edge, ActiveLow)
{
@@ -109,7 +109,7 @@ Scope (\_SB.I2C1)
}
}
-Scope (\_SB.I2C5)
+Scope (\_SB.PCI0.I2C5)
{
/* Realtek Audio Codec */
Device (RTEK) /* Audio Codec driver I2C */
@@ -129,7 +129,7 @@ Scope (\_SB.I2C5)
ControllerInitiated, /* SlaveMode: default to ControllerInitiated */
400000, /* ConnectionSpeed: in Hz */
AddressingMode7Bit, /* Addressing Mode: default to 7 bit */
- "\\_SB.I2C5", /* ResourceSource: I2C bus controller name */
+ "\\_SB.PCI0.I2C5" /* ResourceSource: I2C bus controller name */
)
/* Jack Detect (index 0) */
@@ -146,7 +146,7 @@ Scope (\_SB.I2C5)
}
}
-Scope (\_SB.I2C6)
+Scope (\_SB.PCI0.I2C6)
{
Device (ETPA)
{
@@ -162,7 +162,7 @@ Scope (\_SB.I2C6)
ControllerInitiated, /* SlaveMode */
400000, /* ConnectionSpeed */
AddressingMode7Bit, /* AddressingMode */
- "\\_SB.I2C6", /* ResourceSource */
+ "\\_SB.PCI0.I2C6", /* ResourceSource */
)
GpioInt (Edge, ActiveLow, ExclusiveAndWake, PullNone,,
"\\_SB.GPNC") { BOARD_TRACKPAD_GPIO_INDEX }
@@ -183,7 +183,7 @@ Scope (\_SB.I2C6)
}
}
-Scope (\_SB.LPEA)
+Scope (\_SB.PCI0.LPEA)
{
Name (GBUF, ResourceTemplate ()
{
diff --git a/src/soc/intel/braswell/acpi/southcluster.asl b/src/soc/intel/braswell/acpi/southcluster.asl
index 36250c154b..76176620d8 100644
--- a/src/soc/intel/braswell/acpi/southcluster.asl
+++ b/src/soc/intel/braswell/acpi/southcluster.asl
@@ -253,7 +253,10 @@ Scope (\_SB)
{
/* GPIO Devices */
#include "gpio.asl"
+}
+Scope (\_SB.PCI0)
+{
/* LPSS Devices */
#include "lpss.asl"