Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33287 )
Change subject: payloads/libpayload: Update the Makefile in the sample directory and made an example configuration for QEMU/ARM. ......................................................................
Patch Set 9:
(1 comment)
https://review.coreboot.org/#/c/33287/9/payloads/libpayload/include/arm/arch... File payloads/libpayload/include/arm/arch/io.h:
https://review.coreboot.org/#/c/33287/9/payloads/libpayload/include/arm/arch... PS9, Line 43: #define outl(v, a) write32((void *) (&v), a) Yes, Arm only uses MMIO. in/out I/O ports are a totally x86-specific thing. Please remove these. Building any driver that straight-up calls inb()/outb() for an Arm board is an error.
(Note that CONFIG_LP_IO_ADDRESS_SPACE exists to model this difference and should not be set on Arm, so you should be able to build the 8250 driver without providing these. We're doing that only plenty of boards already, e.g. configs/config.gru.)