diff options
| author | Patrick Rudolph <patrick.rudolph@9elements.com> | 2019-12-13 12:16:06 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2020-01-09 14:22:51 +0000 |
| commit | c83bab62b3657d97299c8368b6c610e4cbf994b6 (patch) | |
| tree | f568d7f66b4a3491c6292040db29905111771de4 /src/drivers/net | |
| parent | 5c8ff794a8dcb0760c137d3b2dff1879a4a8dad6 (diff) | |
| download | coreboot-c83bab62b3657d97299c8368b6c610e4cbf994b6.tar.xz | |
acpi: Be more ACPI compliant when generating _UID
* Add function to generate unique _UID using CRC32
* Add function to write the _UID based on a device's ACPI path
ACPI devices that have the same _HID must use different _UID.
Linux doesn't care about _UID if it's not used.
Windows 10 verifies the ACPI code on boot and BSODs if two devices
with the same _HID share the same _UID.
Fixes BSOD seen on Windows 10.
Change-Id: I47cd5396060d325f9ce338afced6af021e7ff2b4
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37695
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Philipp Deppenwiese <zaolin.daisuki@gmail.com>
Diffstat (limited to 'src/drivers/net')
| -rw-r--r-- | src/drivers/net/r8168.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/drivers/net/r8168.c b/src/drivers/net/r8168.c index bc0132fc13..1bca8799aa 100644 --- a/src/drivers/net/r8168.c +++ b/src/drivers/net/r8168.c @@ -312,7 +312,8 @@ static void r8168_net_fill_ssdt(struct device *dev) acpigen_write_scope(path); acpigen_write_device(acpi_device_name(dev)); acpigen_write_name_string("_HID", R8168_ACPI_HID); - acpigen_write_name_integer("_UID", 0); + acpi_device_write_uid(dev); + if (dev->chip_ops) acpigen_write_name_string("_DDN", dev->chip_ops->name); |