[coreboot] New patch to review for coreboot: b5cef35 cbfs: Change false ERROR print to a WARNING.

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Mar 19 20:53:47 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/2833

-gerrit

commit b5cef35b4264dab340dd27b7a09c9a66be33e48c
Author: Shawn Nematbakhsh <shawnn at google.com>
Date:   Mon Feb 25 12:12:05 2013 -0800

    cbfs: Change false ERROR print to a WARNING.
    
    Change "ERROR" to "WARNING" -- not finding the indicated file is usually
    not a fatal error.
    
    Change-Id: I0600964360ee27484c393125823e833f29aaa7e7
    Signed-off-by: Shawn Nematbakhsh <shawnn at google.com>
---
 payloads/libpayload/libcbfs/cbfs_core.c | 2 +-
 src/lib/cbfs_core.c                     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/payloads/libpayload/libcbfs/cbfs_core.c b/payloads/libpayload/libcbfs/cbfs_core.c
index 2ea2771..a714b91 100644
--- a/payloads/libpayload/libcbfs/cbfs_core.c
+++ b/payloads/libpayload/libcbfs/cbfs_core.c
@@ -168,7 +168,7 @@ struct cbfs_file *cbfs_get_file(struct cbfs_media *media, const char *name)
 			offset += align - (offset % align);
 	}
 	media->close(media);
-	ERROR("ERROR: Not found.\n");
+	LOG("WARNING: Not found.\n");
 	return NULL;
 }
 
diff --git a/src/lib/cbfs_core.c b/src/lib/cbfs_core.c
index 604d5f4..9732b82 100644
--- a/src/lib/cbfs_core.c
+++ b/src/lib/cbfs_core.c
@@ -168,7 +168,7 @@ struct cbfs_file *cbfs_get_file(struct cbfs_media *media, const char *name)
 			offset += align - (offset % align);
 	}
 	media->close(media);
-	ERROR("ERROR: Not found.\n");
+	LOG("WARNING: Not found.\n");
 	return NULL;
 }
 



More information about the coreboot mailing list