[coreboot-gerrit] New patch to review for coreboot: 112ab6d cimx/sb800: fix pedantic gcc error

Ronald G. Minnich (rminnich@gmail.com) gerrit at coreboot.org
Sat Oct 18 13:20:15 CEST 2014


Ronald G. Minnich (rminnich at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7122

-gerrit

commit 112ab6d9abe49022c6b565553f7627a408abd8f0
Author: Ronald G. Minnich <rminnich at gmail.com>
Date:   Sat Oct 18 11:19:28 2014 +0000

    cimx/sb800: fix pedantic gcc error
    
    A cast did not work for me, but this variable did.
    This is one of the many issues with building e3501 I'm
    running into.
    
    Change-Id: Ifb19a17770604f2d63dfef762d08200add77ee34
    Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
---
 src/southbridge/amd/cimx/sb800/cfg.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/southbridge/amd/cimx/sb800/cfg.c b/src/southbridge/amd/cimx/sb800/cfg.c
index 91b6067..9f10dc5 100644
--- a/src/southbridge/amd/cimx/sb800/cfg.c
+++ b/src/southbridge/amd/cimx/sb800/cfg.c
@@ -76,6 +76,7 @@ unsigned long get_top_of_ram(void)
  */
 void sb800_cimx_config(AMDSBCFG *sb_config)
 {
+	uint16_t bios_size = BIOS_SIZE;
 	if (!sb_config)
 		return;
 
@@ -88,7 +89,7 @@ void sb800_cimx_config(AMDSBCFG *sb_config)
 	sb_config->StdHeader.PcieBasePtr = PCIEX_BASE_ADDRESS;
 
 	/* static Build Parameters */
-	sb_config->BuildParameters.BiosSize = BIOS_SIZE;
+	sb_config->BuildParameters.BiosSize = bios_size;
 	sb_config->BuildParameters.LegacyFree = LEGACY_FREE;
 	sb_config->BuildParameters.WatchDogTimerBase = WATCHDOG_TIMER_BASE_ADDRESS;
 	sb_config->BuildParameters.AcpiGpe0BlkAddr = GPE0_BLK_ADDRESS;



More information about the coreboot-gerrit mailing list