From be5ba870e006e8e7fe0f1ace2952b540c3aec3d9 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Mon, 8 Feb 2016 21:17:12 +0100 Subject: [PATCH 2/4] libpayload: use 32bit access when accessing 4byte wide uart registers This fixes serial on rk3288. Change-Id: I3dbf3cc165e516ed7b0132332624f882c0c9b27f Signed-off-by: Patrick Georgi Reviewed-on: https://review.coreboot.org/13636 Reviewed-by: Stefan Reinauer Tested-by: build bot (Jenkins) --- payloads/libpayload/drivers/serial/8250.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/payloads/libpayload/drivers/serial/8250.c b/payloads/libpayload/drivers/serial/8250.c index 7fe9920..0386f23 100644 --- a/payloads/libpayload/drivers/serial/8250.c +++ b/payloads/libpayload/drivers/serial/8250.c @@ -46,7 +46,10 @@ static uint8_t serial_read_reg(int offset) return inb(IOBASE + offset); else #endif - return readb(MEMBASE + offset); + if (lib_sysinfo.serial->regwidth == 4) + return readl(MEMBASE + offset) & 0xff; + else + return readb(MEMBASE + offset); } static void serial_write_reg(uint8_t val, int offset) @@ -58,7 +61,10 @@ static void serial_write_reg(uint8_t val, int offset) outb(val, IOBASE + offset); else #endif - writeb(val, MEMBASE + offset); + if (lib_sysinfo.serial->regwidth == 4) + writel(val & 0xff, MEMBASE + offset); + else + writeb(val, MEMBASE + offset); } #if IS_ENABLED(CONFIG_LP_SERIAL_SET_SPEED) -- 2.8.0