Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/77065?usp=email )
Change subject: superio/ite/it8721f/acpi/superio: use IO instead of FixedIO resource ......................................................................
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 felix-coreboot@felixheld.de Change-Id: I6183d625fb7968fb33caf396f19feef8917ba4fe --- M src/superio/ite/it8721f/acpi/superio.asl 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/77065/1
diff --git a/src/superio/ite/it8721f/acpi/superio.asl b/src/superio/ite/it8721f/acpi/superio.asl index 21be5a7..065ec52 100644 --- a/src/superio/ite/it8721f/acpi/superio.asl +++ b/src/superio/ite/it8721f/acpi/superio.asl @@ -80,7 +80,7 @@ { /* 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) }) }