summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorElyes HAOUAS <ehaouas@noos.fr>2019-10-25 10:53:20 +0200
committerMartin Roth <martinroth@google.com>2019-11-29 19:23:05 +0000
commit01787608670adec26fcea48173e18395e51c790e (patch)
tree6a60df15f88244034578f80e92aa32725539fd53
parent179da7fb5cff3c9034dc3203086c84342560c600 (diff)
downloadcoreboot-01787608670adec26fcea48173e18395e51c790e.tar.xz
{northbridge,soc,southbridge}: Don't use both of _ADR and _HID
ACPI Version 6.3 Section 6.1: "A device object must contain either an _HID object or an _ADR object, but should not contain both." Change-Id: Ifb777c09aeef09a6a4cbee254b081519f5b6c457 Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr> Reviewed-on: https://review.coreboot.org/c/coreboot/+/36318 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com>
-rw-r--r--src/northbridge/amd/agesa/family14/acpi/northbridge.asl3
-rw-r--r--src/northbridge/amd/agesa/family15tn/acpi/northbridge.asl2
-rw-r--r--src/northbridge/amd/agesa/family16kb/acpi/northbridge.asl2
-rw-r--r--src/northbridge/amd/pi/00630F01/acpi/northbridge.asl2
-rw-r--r--src/northbridge/amd/pi/00660F01/acpi/northbridge.asl2
-rw-r--r--src/northbridge/amd/pi/00730F01/acpi/northbridge.asl3
-rw-r--r--src/soc/amd/stoneyridge/acpi/northbridge.asl2
-rw-r--r--src/soc/intel/broadwell/acpi/serialio.asl9
-rw-r--r--src/southbridge/intel/lynxpoint/acpi/serialio.asl9
9 files changed, 9 insertions, 25 deletions
diff --git a/src/northbridge/amd/agesa/family14/acpi/northbridge.asl b/src/northbridge/amd/agesa/family14/acpi/northbridge.asl
index 06199a1b07..fad157da29 100644
--- a/src/northbridge/amd/agesa/family14/acpi/northbridge.asl
+++ b/src/northbridge/amd/agesa/family14/acpi/northbridge.asl
@@ -16,7 +16,7 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A08")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A08")) // PCI Express Root Bridge */
Name(_CID, EISAID("PNP0A03")) /* PCI Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
@@ -125,7 +125,6 @@ Device(PE23) {
/* Northbridge function 3 */
Device(NBF3) {
- Name(_ADR, 0x00180003)
/* k10temp thermal zone */
#include "thermal_mixin.asl"
diff --git a/src/northbridge/amd/agesa/family15tn/acpi/northbridge.asl b/src/northbridge/amd/agesa/family15tn/acpi/northbridge.asl
index 9a1fa9ed88..96c2d8bfac 100644
--- a/src/northbridge/amd/agesa/family15tn/acpi/northbridge.asl
+++ b/src/northbridge/amd/agesa/family15tn/acpi/northbridge.asl
@@ -16,7 +16,7 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A03")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A03")) // PCI Express Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
/* Describe the Northbridge devices */
diff --git a/src/northbridge/amd/agesa/family16kb/acpi/northbridge.asl b/src/northbridge/amd/agesa/family16kb/acpi/northbridge.asl
index f74b31a080..a7e8307349 100644
--- a/src/northbridge/amd/agesa/family16kb/acpi/northbridge.asl
+++ b/src/northbridge/amd/agesa/family16kb/acpi/northbridge.asl
@@ -16,7 +16,7 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A08")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A08")) // PCI Express Root Bridge */
Name(_CID, EISAID("PNP0A03")) /* PCI Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
diff --git a/src/northbridge/amd/pi/00630F01/acpi/northbridge.asl b/src/northbridge/amd/pi/00630F01/acpi/northbridge.asl
index c2b3aac4c5..de47bc2151 100644
--- a/src/northbridge/amd/pi/00630F01/acpi/northbridge.asl
+++ b/src/northbridge/amd/pi/00630F01/acpi/northbridge.asl
@@ -16,7 +16,7 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A03")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A03")) // PCI Express Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
/* Describe the Northbridge devices */
diff --git a/src/northbridge/amd/pi/00660F01/acpi/northbridge.asl b/src/northbridge/amd/pi/00660F01/acpi/northbridge.asl
index d54f985e90..4a48aaf401 100644
--- a/src/northbridge/amd/pi/00660F01/acpi/northbridge.asl
+++ b/src/northbridge/amd/pi/00660F01/acpi/northbridge.asl
@@ -16,7 +16,7 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A08")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A08")) // PCI Express Root Bridge */
Name(_CID, EISAID("PNP0A03")) /* PCI Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
diff --git a/src/northbridge/amd/pi/00730F01/acpi/northbridge.asl b/src/northbridge/amd/pi/00730F01/acpi/northbridge.asl
index f74b31a080..b317ccf1ea 100644
--- a/src/northbridge/amd/pi/00730F01/acpi/northbridge.asl
+++ b/src/northbridge/amd/pi/00730F01/acpi/northbridge.asl
@@ -16,10 +16,9 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A08")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A08")) // PCI Express Root Bridge */
Name(_CID, EISAID("PNP0A03")) /* PCI Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
-
/* Describe the Northbridge devices */
Method(_BBN, 0, NotSerialized) /* Bus number = 0 */
diff --git a/src/soc/amd/stoneyridge/acpi/northbridge.asl b/src/soc/amd/stoneyridge/acpi/northbridge.asl
index fe78534403..09bf2e18c2 100644
--- a/src/soc/amd/stoneyridge/acpi/northbridge.asl
+++ b/src/soc/amd/stoneyridge/acpi/northbridge.asl
@@ -17,7 +17,7 @@
/* Note: Only need HID on Primary Bus */
External (TOM1)
External (TOM2)
-Name(_HID, EISAID("PNP0A08")) /* PCI Express Root Bridge */
+/* Name(_HID, EISAID("PNP0A08")) // PCI Express Root Bridge */
Name(_CID, EISAID("PNP0A03")) /* PCI Root Bridge */
Name(_ADR, 0x00180000) /* Dev# = BSP Dev#, Func# = 0 */
diff --git a/src/soc/intel/broadwell/acpi/serialio.asl b/src/soc/intel/broadwell/acpi/serialio.asl
index 1b44e9566a..fd25b0d8a8 100644
--- a/src/soc/intel/broadwell/acpi/serialio.asl
+++ b/src/soc/intel/broadwell/acpi/serialio.asl
@@ -157,7 +157,7 @@ Device (SIOR)
Device (SDMA)
{
// Serial IO DMA Controller
- Name (_HID, "INTL9C60")
+ /* Name (_HID, "INTL9C60") */
Name (_UID, 1)
Name (_ADR, 0x00150000)
@@ -205,7 +205,6 @@ Device (I2C0)
Return ("INT33C2")
}
Name (_UID, 1)
- Name (_ADR, 0x00150001)
Name (SSCN, Package () { 432, 507, 30 })
Name (FMCN, Package () { 72, 160, 30 })
@@ -276,7 +275,6 @@ Device (I2C1)
Return ("INT33C3")
}
Name (_UID, 1)
- Name (_ADR, 0x00150002)
Name (SSCN, Package () { 432, 507, 30 })
Name (FMCN, Package () { 72, 160, 30 })
@@ -347,7 +345,6 @@ Device (SPI0)
Return ("INT33C0")
}
Name (_UID, 1)
- Name (_ADR, 0x00150003)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -403,7 +400,6 @@ Device (SPI1)
Return ("INT33C1")
}
Name (_UID, 1)
- Name (_ADR, 0x00150004)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -471,7 +467,6 @@ Device (UAR0)
Return ("INT33C4")
}
Name (_UID, 1)
- Name (_ADR, 0x00150005)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -539,7 +534,6 @@ Device (UAR1)
Return ("INT33C5")
}
Name (_UID, 1)
- Name (_ADR, 0x00150006)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -596,7 +590,6 @@ Device (SDIO)
}
Name (_CID, "PNP0D40")
Name (_UID, 1)
- Name (_ADR, 0x00170000)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
diff --git a/src/southbridge/intel/lynxpoint/acpi/serialio.asl b/src/southbridge/intel/lynxpoint/acpi/serialio.asl
index 9323b91cac..0eebe32dd6 100644
--- a/src/southbridge/intel/lynxpoint/acpi/serialio.asl
+++ b/src/southbridge/intel/lynxpoint/acpi/serialio.asl
@@ -123,7 +123,7 @@ Device (SIOR)
Device (SDMA)
{
// Serial IO DMA Controller
- Name (_HID, "INTL9C60")
+ /* Name (_HID, "INTL9C60") */
Name (_UID, 1)
Name (_ADR, 0x00150000)
@@ -163,7 +163,6 @@ Device (I2C0)
Name (_HID, "INT33C2")
Name (_CID, "INT33C2")
Name (_UID, 1)
- Name (_ADR, 0x00150001)
Name (SSCN, Package () { 432, 507, 30 })
Name (FMCN, Package () { 72, 160, 30 })
@@ -245,7 +244,6 @@ Device (I2C1)
Name (_HID, "INT33C3")
Name (_CID, "INT33C3")
Name (_UID, 1)
- Name (_ADR, 0x00150002)
Name (SSCN, Package () { 432, 507, 30 })
Name (FMCN, Package () { 72, 160, 30 })
@@ -327,7 +325,6 @@ Device (SPI0)
Name (_HID, "INT33C0")
Name (_CID, "INT33C0")
Name (_UID, 1)
- Name (_ADR, 0x00150003)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -365,7 +362,6 @@ Device (SPI1)
Name (_HID, "INT33C1")
Name (_CID, "INT33C1")
Name (_UID, 1)
- Name (_ADR, 0x00150004)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -416,7 +412,6 @@ Device (UAR0)
Name (_HID, "INT33C4")
Name (_CID, "INT33C4")
Name (_UID, 1)
- Name (_ADR, 0x00150005)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -467,7 +462,6 @@ Device (UAR1)
Name (_HID, "INT33C5")
Name (_CID, "INT33C5")
Name (_UID, 1)
- Name (_ADR, 0x00150006)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()
@@ -505,7 +499,6 @@ Device (SDIO)
Name (_HID, "INT33C6")
Name (_CID, "PNP0D40")
Name (_UID, 1)
- Name (_ADR, 0x00170000)
// BAR0 is assigned during PCI enumeration and saved into NVS
Name (RBUF, ResourceTemplate ()