[coreboot] New patch to review for coreboot: ae7f0ed cbfstool: Fix compile warnings caused by incorrect data types.

Hung-Te Lin (hungte@chromium.org) gerrit at coreboot.org
Mon Feb 18 11:37:57 CET 2013


Hung-Te Lin (hungte at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2440

-gerrit

commit ae7f0ed2d1fc3f96a9adec408d9906c1fb31bb05
Author: Hung-Te Lin <hungte at chromium.org>
Date:   Mon Feb 18 18:35:00 2013 +0800

    cbfstool: Fix compile warnings caused by incorrect data types.
    
    The "offset" in cbfs-mkpayload should be printed as type %lu instead of %d.
    
    Change-Id: I50c26a314723d45fcc6ff9ae2f08266cb7969a12
    Signed-off-by: Hung-Te Lin <hungte at chromium.org>
---
 util/cbfstool/cbfs-mkpayload.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/cbfstool/cbfs-mkpayload.c b/util/cbfstool/cbfs-mkpayload.c
index 6115e49..9f3dabf 100644
--- a/util/cbfstool/cbfs-mkpayload.c
+++ b/util/cbfstool/cbfs-mkpayload.c
@@ -281,7 +281,7 @@ int parse_fv_to_payload(const struct buffer *input,
 	fh = (ffs_file_header_t *)(input->data + fv->header_length);
 	while (fh->file_type == FILETYPE_PAD) {
 		unsigned long offset = (fh->size[2] << 16) | (fh->size[1] << 8) | fh->size[0];
-		ERROR("skipping %d bytes of FV padding\n", offset);
+		ERROR("skipping %lu bytes of FV padding\n", offset);
 		fh = (ffs_file_header_t *)(((void*)fh) + offset);
 	}
 	if (fh->file_type != FILETYPE_SEC) {
@@ -293,7 +293,7 @@ int parse_fv_to_payload(const struct buffer *input,
 	cs = (common_section_header_t *)&fh[1];
 	while (cs->section_type == SECTION_RAW) {
 		unsigned long offset = (cs->size[2] << 16) | (cs->size[1] << 8) | cs->size[0];
-		ERROR("skipping %d bytes of section padding\n", offset);
+		ERROR("skipping %lu bytes of section padding\n", offset);
 		cs = (common_section_header_t *)(((void*)cs) + offset);
 	}
 	if (cs->section_type != SECTION_PE32) {



More information about the coreboot mailing list