[coreboot] New patch to review for coreboot: d027fbc cbmem utility: unify debug output

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Jan 8 00:26:22 CET 2013


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2116

-gerrit

commit d027fbc3cbf4c32d8e1dd13ab22632dd5b7dbab8
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Jan 7 15:25:37 2013 -0800

    cbmem utility: unify debug output
    
    ... and indent it to make output more comprehensible.
    
    Change-Id: If321f3233b31be14b2723175b781e5dd60dd72b6
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 util/cbmem/cbmem.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/util/cbmem/cbmem.c b/util/cbmem/cbmem.c
index 4c0bd68..59d0540 100644
--- a/util/cbmem/cbmem.c
+++ b/util/cbmem/cbmem.c
@@ -158,12 +158,12 @@ static int parse_cbtable(u64 address)
 			debug("  coreboot table entry 0x%02x\n", lbr_p->tag);
 			switch (lbr_p->tag) {
 			case LB_TAG_TIMESTAMPS: {
-				debug("Found timestamp table\n");
+				debug("    Found timestamp table.\n");
 				timestamps = *(struct lb_cbmem_ref *) lbr_p;
 				continue;
 			}
 			case LB_TAG_CBMEM_CONSOLE: {
-				debug("Found cbmem console\n");
+				debug("    Found cbmem console.\n");
 				console = *(struct lb_cbmem_ref *) lbr_p;
 				continue;
 			}
@@ -174,6 +174,7 @@ static int parse_cbtable(u64 address)
 				 */
 				struct lb_forward lbf_p =
 					*(struct lb_forward *) lbr_p;
+				debug("    Found forwarding entry.\n");
 				unmap_memory();
 				return parse_cbtable(lbf_p.forward);
 			}



More information about the coreboot mailing list