From e566e1547b0ac7be36e87b6a629355b80241250b Mon Sep 17 00:00:00 2001 From: Felix Held Date: Mon, 7 Aug 2023 20:12:19 +0200 Subject: [PATCH] superio/ite/it8721f/acpi/superio: use IO instead of FixedIO resource The fixed I/O resource descriptor macro implies that the device will only decode 10 of the 16 IO port bits causing aliasing. Use an I/O port descriptor instead and use Decode16 to tell the OS that this I/O resource will decode all I/O address bits. Signed-off-by: Felix Held Change-Id: I6183d625fb7968fb33caf396f19feef8917ba4fe Reviewed-on: https://review.coreboot.org/c/coreboot/+/77065 Tested-by: build bot (Jenkins) Reviewed-by: Martin L Roth --- src/superio/ite/it8721f/acpi/superio.asl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/superio/ite/it8721f/acpi/superio.asl b/src/superio/ite/it8721f/acpi/superio.asl index 21be5a75b6..065ec52a47 100644 --- a/src/superio/ite/it8721f/acpi/superio.asl +++ b/src/superio/ite/it8721f/acpi/superio.asl @@ -80,7 +80,7 @@ Device(SUPERIO_DEV) { { /* Announce the used I/O ports to the OS */ Return (ResourceTemplate () { - FixedIO (SUPERIO_PNP_BASE, 0x02) + IO (Decode16, SUPERIO_PNP_BASE, SUPERIO_PNP_BASE, 0x01, 0x02) }) }