[coreboot-gerrit] New patch to review for coreboot: b6eb00b libpayload: Fix baseaddr access in serial/tegra.c

Marc Jones (marc.jones@se-eng.com) gerrit at coreboot.org
Mon Jan 5 23:37:31 CET 2015


Marc Jones (marc.jones at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8123

-gerrit

commit b6eb00bc4fc6fe62906be2949e9e25158f66c50f
Author: Furquan Shaikh <furquan at google.com>
Date:   Tue Jun 17 17:08:58 2014 -0700

    libpayload: Fix baseaddr access in serial/tegra.c
    
    Fix baseaddr typecast to allow use in 32- and 64-bit systems
    
    BUG=None
    BRANCH=None
    TEST=Compiles successfully for rush
    
    Original-Change-Id: Ie5ded744d75a0ae4d1428d04ff2478bdfe54d146
    Original-Signed-off-by: Furquan Shaikh <furquan at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/204424
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Commit-Queue: Furquan Shaikh <furquan at chromium.org>
    Original-Tested-by: Furquan Shaikh <furquan at chromium.org>
    (cherry picked from commit b56814fe5fc7d3a2fdfb324d4baafb27a9d3ffd6)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: Icedc1c819e39b92a0dd92e98f848e15b3039dfbe
---
 payloads/libpayload/drivers/serial/tegra.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/payloads/libpayload/drivers/serial/tegra.c b/payloads/libpayload/drivers/serial/tegra.c
index fa5ff16..a3dad2b 100644
--- a/payloads/libpayload/drivers/serial/tegra.c
+++ b/payloads/libpayload/drivers/serial/tegra.c
@@ -99,7 +99,7 @@ void serial_init(void)
 	if (!lib_sysinfo.serial || !lib_sysinfo.serial->baseaddr)
 		return;
 
-	uart_regs = (struct tegra_uart *)lib_sysinfo.serial->baseaddr;
+	uart_regs = (struct tegra_uart *)(uintptr_t)lib_sysinfo.serial->baseaddr;
 }
 
 void serial_console_init(void)



More information about the coreboot-gerrit mailing list