Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63244 )
Change subject: src/mb/facebook/monolith/acpi/superio.asl: Remove _PRS ......................................................................
src/mb/facebook/monolith/acpi/superio.asl: Remove _PRS
IASL reports warning on missing _SRS.
Devices have fixed configuration which is always enabled. Remove _PRS for this fixed configuration.
BUG=N/A TEST=built facebook monolith and verify no IASL warning is reported.
Change-Id: I554d3497255c1e50cdbe74b1cffc9f2c59fbae77 Signed-off-by: Frans Hendriks fhendriks@eltan.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/63244 Reviewed-by: Erik van den Bogaert ebogaert@eltan.com Reviewed-by: Elyes Haouas ehaouas@noos.fr Reviewed-by: Angel Pons th3fanbus@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/facebook/monolith/acpi/superio.asl 1 file changed, 0 insertions(+), 19 deletions(-)
Approvals: build bot (Jenkins): Verified Elyes Haouas: Looks good to me, but someone else must approve Angel Pons: Looks good to me, approved Erik van den Bogaert: Looks good to me, but someone else must approve
diff --git a/src/mainboard/facebook/monolith/acpi/superio.asl b/src/mainboard/facebook/monolith/acpi/superio.asl index 4ffd557..ce5595f 100644 --- a/src/mainboard/facebook/monolith/acpi/superio.asl +++ b/src/mainboard/facebook/monolith/acpi/superio.asl @@ -17,16 +17,6 @@ FixedIO (0x6E, 0x02) IRQNoFlags () {4} }) - - Name (_PRS, ResourceTemplate () - { - StartDependentFn (0, 0) { - FixedIO (0x03F8, 0x08) - FixedIO (0x6E, 0x02) - IRQNoFlags () {4} - } - EndDependentFn () - }) }
Device (COM2) { @@ -43,13 +33,4 @@ FixedIO (0x03E8, 0x08) IRQNoFlags () {3} }) - - Name (_PRS, ResourceTemplate () - { - StartDependentFn (0, 0) { - FixedIO (0x03E8, 0x08) - IRQNoFlags () {3} - } - EndDependentFn () - }) }