From 7b654a9702640c2d9fb8c37e4ae7f6b27ca949a0 Mon Sep 17 00:00:00 2001 From: Hung-Te Lin Date: Mon, 18 Feb 2013 18:35:00 +0800 Subject: cbfstool: Fix compile warnings caused by incorrect data types. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The "offset" in cbfs-mkpayload should be printed as type %lu instead of %d as `gcc` rightfully warns about. gcc -g -Wall -D_7ZIP_ST -c -o /srv/filme/src/coreboot/util/cbfstool/cbfs-mkpayload.o cbfs-mkpayload.c cbfs-mkpayload.c: In function ‘parse_fv_to_payload’: cbfs-mkpayload.c:284:3: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long unsigned int’ [-Wformat] cbfs-mkpayload.c:296:3: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘long unsigned int’ [-Wformat] This warning was introduced in the following commit. commit 4610247ef1744ccabbcc6bfc441a3583aa49f7b5 Author: Patrick Georgi Date: Sat Feb 9 13:26:19 2013 +0100 cbfstool: Handle alignment in UEFI payloads Reviewed-on: http://review.coreboot.org/2334 Change-Id: I50c26a314723d45fcc6ff9ae2f08266cb7969a12 Signed-off-by: Hung-Te Lin Reviewed-on: http://review.coreboot.org/2440 Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) --- util/cbfstool/cbfs-mkpayload.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'util') diff --git a/util/cbfstool/cbfs-mkpayload.c b/util/cbfstool/cbfs-mkpayload.c index 6115e492c3..9f3dabf40f 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) { -- cgit v1.2.3