Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44198 )
Change subject: soc/intel/cannonlake/acpi/serialio.asl: Don't advertise unavailable devices ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44198/1/src/soc/intel/cannonlake/ac... File src/soc/intel/cannonlake/acpi/serialio.asl:
https://review.coreboot.org/c/coreboot/+/44198/1/src/soc/intel/cannonlake/ac... PS1, Line 62: Device (UAR0) : { : Name (_ADR, 0x001e0000) : Name (_DDN, "Serial IO UART Controller 0") : } : : Device (UAR1) : { : Name (_ADR, 0x001e0001) : Name (_DDN, "Serial IO UART Controller 1") : }
Good catch. That's also wrong. Both device operate fine in PCI mode.
Done