[coreboot-gerrit] New patch to review for coreboot: 02ac134 libpayload: make cfbs searches even less verbose

Dave Frodin (dave.frodin@se-eng.com) gerrit at coreboot.org
Tue May 7 00:21:56 CEST 2013


Dave Frodin (dave.frodin at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3208

-gerrit

commit 02ac134448ce8ce59be7232164e972372f33c0ab
Author: Dave Frodin <dave.frodin at se-eng.com>
Date:   Mon May 6 15:51:39 2013 -0600

    libpayload: make cfbs searches even less verbose
    
    The cbfs core code would print out the name of the file it is
    searching for and when it is found would print out the name
    again. This contributes to a lot of unnecessary messages in a
    functioning payloads output. Change this message to a DEBUG one
    so that it will only be printed when CONFIG_DEBUG_CBFS is enabled.
    
    Change-Id: Ib238ff174bedba8eaaad8d1d452721fcac339b1a
    Signed-off-by: Dave Frodin <dave.frodin at se-eng.com>
---
 payloads/libpayload/libcbfs/cbfs_core.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/payloads/libpayload/libcbfs/cbfs_core.c b/payloads/libpayload/libcbfs/cbfs_core.c
index 3fe6971..9274736 100644
--- a/payloads/libpayload/libcbfs/cbfs_core.c
+++ b/payloads/libpayload/libcbfs/cbfs_core.c
@@ -123,7 +123,7 @@ struct cbfs_file *cbfs_get_file(struct cbfs_media *media, const char *name)
 #endif
 	DEBUG("CBFS location: 0x%x~0x%x, align: %d\n", offset, romsize, align);
 
-	LOG("Looking for '%s' starting from 0x%x.\n", name, offset);
+	DEBUG("Looking for '%s' starting from 0x%x.\n", name, offset);
 	media->open(media);
 	while (offset < romsize &&
 	       media->read(media, &file, offset, sizeof(file)) == sizeof(file)) {
@@ -150,7 +150,7 @@ struct cbfs_file *cbfs_get_file(struct cbfs_media *media, const char *name)
 		} else if (strcmp(file_name, name) == 0) {
 			int file_offset = ntohl(file.offset),
 			    file_len = ntohl(file.len);
-			LOG("Found file (offset=0x%x, len=%d).\n",
+			DEBUG("Found file (offset=0x%x, len=%d).\n",
 			    offset + file_offset, file_len);
 			media->unmap(media, file_name);
 			file_ptr = media->map(media, offset,



More information about the coreboot-gerrit mailing list