summaryrefslogtreecommitdiffstats
path: root/src/superio/ite
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2023-08-07 20:12:19 +0200
committerFelix Held <felix-coreboot@felixheld.de>2023-08-11 23:11:44 +0000
commite566e1547b0ac7be36e87b6a629355b80241250b (patch)
treeec2bdc338a12cfb1324b61cb43433659b214f0b3 /src/superio/ite
parent431c0b487c651d07af0061a2497141a796327dfd (diff)
downloadcoreboot-e566e1547b0ac7be36e87b6a629355b80241250b.tar.gz
coreboot-e566e1547b0ac7be36e87b6a629355b80241250b.tar.bz2
coreboot-e566e1547b0ac7be36e87b6a629355b80241250b.zip
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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/77065 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin L Roth <gaumless@gmail.com>
Diffstat (limited to 'src/superio/ite')
-rw-r--r--src/superio/ite/it8721f/acpi/superio.asl2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/superio/ite/it8721f/acpi/superio.asl b/src/superio/ite/it8721f/acpi/superio.asl
index 21be5a75b61a..065ec52a47b4 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)
})
}