[coreboot] New patch to review for coreboot: f1776cd AMD SB800 early console use fix

Frank Vibrans III (frank.vibrans@amd.com) gerrit at coreboot.org
Thu Sep 15 00:22:46 CEST 2011


Frank Vibrans III (frank.vibrans at amd.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/216

-gerrit

commit f1776cd4f022aee49fad497dc0adcc6c224d62fb
Author: efdesign98 <efdesign98 at gmail.com>
Date:   Wed Sep 14 16:22:31 2011 -0600

    AMD SB800 early console use fix
    
    This change removes printk's that occur before
    console init is called.  In the best case, these
    would cause an extremely slow boot, and in the
    worst case would cause a complete post failure.
    
    Change-Id: I50388e71225e95db602aa45835c39126c1c920a3
    Signed-off-by: Frank Vibrans <frank.vibrans at amd.com>
    Signed-off-by: efdesign98 <efdesign98 at gmail.com>
---
 src/southbridge/amd/cimx/sb800/cfg.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/src/southbridge/amd/cimx/sb800/cfg.c b/src/southbridge/amd/cimx/sb800/cfg.c
index 45a460b..a7801a8 100644
--- a/src/southbridge/amd/cimx/sb800/cfg.c
+++ b/src/southbridge/amd/cimx/sb800/cfg.c
@@ -31,10 +31,8 @@
 void sb800_cimx_config(AMDSBCFG *sb_config)
 {
 	if (!sb_config) {
-		printk(BIOS_DEBUG, "SB800 - Cfg.c - sb800_cimx_config - No sb_config.\n");
 		return;
 	}
-	printk(BIOS_INFO, "SB800 - Cfg.c - sb800_cimx_config - Start.\n");
 	//memset(sb_config, 0, sizeof(AMDSBCFG));
 
 	/* header */
@@ -128,6 +126,5 @@ void sb800_cimx_config(AMDSBCFG *sb_config)
 		sb_config->StdHeader.CALLBACK.CalloutPtr = sb800_callout_entry;
 	}
 #endif //!__PRE_RAM__
-	printk(BIOS_INFO, "SB800 - Cfg.c - sb800_cimx_config - End.\n");
 }
 




More information about the coreboot mailing list