From dc512f893f61e731504d6867f40de502acfd66ed Mon Sep 17 00:00:00 2001 From: Daniel Kurtz Date: Thu, 11 Jan 2018 18:13:32 -0800 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/23233 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Marc Jones --- src/soc/amd/stoneyridge/acpi/sb_fch.asl | 4 ++-- 1 file 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 9bf7decb49..2ffd951935 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)