diff options
| author | Daniel Kurtz <djkurtz@chromium.org> | 2018-01-11 18:13:32 -0800 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-01-13 23:43:05 +0000 |
| commit | dc512f893f61e731504d6867f40de502acfd66ed (patch) | |
| tree | ac75c26e8342e0501e1a7ded268c2e34a99d0f95 | |
| parent | a273753c729eeb2bde30ad8886649a5058430461 (diff) | |
| download | coreboot-dc512f893f61e731504d6867f40de502acfd66ed.tar.gz coreboot-dc512f893f61e731504d6867f40de502acfd66ed.tar.bz2 coreboot-dc512f893f61e731504d6867f40de502acfd66ed.zip | |
soc/amd/stonyridge: Give I2C devices unique _UIDs
The ACPI unique identifier (_UID) should be unique.
This doesn't actually matter much for Linux, though, since the kernel
can handle it when the BIOS doesn't get this right.
See:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=b4b6cae2f36d92b31788f10816709d5290a1119a
b4b6cae2f36d ACPI / platform: use ACPI device name instead of _HID._UID
Change-Id: I8b1b3143174584a93f3d45bf482b8922b3f0ec12
Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
Reviewed-on: https://review.coreboot.org/23233
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
| -rw-r--r-- | src/soc/amd/stoneyridge/acpi/sb_fch.asl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/amd/stoneyridge/acpi/sb_fch.asl b/src/soc/amd/stoneyridge/acpi/sb_fch.asl index 9bf7decb49a7..2ffd951935f6 100644 --- a/src/soc/amd/stoneyridge/acpi/sb_fch.asl +++ b/src/soc/amd/stoneyridge/acpi/sb_fch.asl @@ -111,7 +111,7 @@ Device (I2CB) Device (I2CC) { Name (_HID, "AMD0010") - Name (_UID, 0x0) + Name (_UID, 0x2) Name (_CRS, ResourceTemplate() { IRQ (Edge, ActiveHigh, Exclusive) { 6 } @@ -127,7 +127,7 @@ Device (I2CC) { Device (I2CD) { Name (_HID, "AMD0010") - Name (_UID, 0x1) + Name (_UID, 0x3) Name (_CRS, ResourceTemplate() { IRQ (Edge, ActiveHigh, Exclusive) { 14 } Memory32Fixed(ReadWrite, 0xFEDC5000, 0x1000) |