[coreboot-gerrit] New patch to review for coreboot: 0489e75 cimx/sb800/cfg.c: Cut out purposeless ROM reading noise.

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Sun Apr 13 08:49:58 CEST 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5495

-gerrit

commit 0489e750825d5aebec59f48d943ed3e65f353b28
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sun Apr 13 16:45:12 2014 +1000

    cimx/sb800/cfg.c: Cut out purposeless ROM reading noise.
    
    Follow along hudson, cut out "SLP_TYP type was 0" excessively filling
    the buffer. We could make this conditional on non-zero?
    
    Change-Id: Iffd4c146b2ac4f57dbc3a011a683c92b6e132e39
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/southbridge/amd/cimx/sb800/cfg.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/southbridge/amd/cimx/sb800/cfg.c b/src/southbridge/amd/cimx/sb800/cfg.c
index 8520548..78aab79 100644
--- a/src/southbridge/amd/cimx/sb800/cfg.c
+++ b/src/southbridge/amd/cimx/sb800/cfg.c
@@ -31,7 +31,7 @@ int acpi_get_sleep_type(void)
 {
 	u16 tmp = inw(PM1_CNT_BLK_ADDRESS);
 	tmp = ((tmp & (7 << 10)) >> 10);
-	printk(BIOS_DEBUG, "SLP_TYP type was %x\n", tmp);
+	/* printk(BIOS_DEBUG, "SLP_TYP type was %x\n", tmp); */
 	return (int)tmp;
 }
 #endif



More information about the coreboot-gerrit mailing list