From 69d6613be1f3229bea00d8395653accbbd1d9f62 Mon Sep 17 00:00:00 2001 From: Mart Raudsepp Date: Thu, 20 Nov 2008 12:20:35 +0000 Subject: [PATCH] Be silent in ram_check in non-debug loglevels As DBE61 support now runs ram_check for non-debug purposes and has expected failures on DBE61A, downgrade the per-address looped fail notification printk and other messages from BIOS_ERR to BIOS_DEBUG. Document that if something is wanted to be reported in non-debug loglevels, one should do so in the caller based on the return value. Tweak a debug string in ram_verify to be more descriptive. Signed-off-by: Mart Raudsepp Acked-by: Peter Stuge git-svn-id: svn://coreboot.org/repository/coreboot-v3@1047 f3766cd6-281f-0410-b1cd-43a5c92072e9 --- lib/ramtest.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/ramtest.c b/lib/ramtest.c index 02a4bdb171..8d5c419395 100644 --- a/lib/ramtest.c +++ b/lib/ramtest.c @@ -97,11 +97,11 @@ static int ram_verify(unsigned long start, unsigned long stop) value = ram_read_phys(addr); if (value != addr) { /* Display address with error. */ - printk(BIOS_ERR, "Fail @%lx Read value=%lx\n", + printk(BIOS_DEBUG, "Fail @%lx Read value=%lx\n", addr, value); /* Abort after 256 verify errors. */ if (++i > 256) { - printk(BIOS_ERR, "Aborting.\n"); + printk(BIOS_DEBUG, "256 errors seen, aborting.\n"); break; } } @@ -111,7 +111,7 @@ static int ram_verify(unsigned long start, unsigned long stop) printk(BIOS_DEBUG, "%lx\r", addr); /* Print whether or not the verify failed. */ - printk(BIOS_DEBUG, "\nDRAM range %sverified.", i ? "_NOT_ " : ""); + printk(BIOS_DEBUG, "\nDRAM range %sverified.\n", i ? "_NOT_ " : ""); return i; } @@ -121,6 +121,8 @@ static int ram_verify(unsigned long start, unsigned long stop) * * This is much more of a "Is my DRAM properly configured?" test than * a "Is my DRAM faulty?" test, though. + * If you want to show a message of the result in non-debug loglevels, be sure + * to do so yourself based on the return value. * * @param start The beginning of the RAM area. * @param stop The end of the RAM area.