[coreboot-gerrit] Patch set updated for coreboot: effcfe2 ELOG: Fix warning to fix the compilation.

Denis Carikli (GNUtoo@no-log.org) gerrit at coreboot.org
Thu Jun 27 22:27:33 CEST 2013

Denis Carikli (GNUtoo at no-log.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3551


commit effcfe25c797cc1dbf448a261b41395bd2680140
Author: Denis 'GNUtoo' Carikli <GNUtoo at no-log.org>
Date:   Wed Jun 26 20:04:49 2013 +0200

    ELOG: Fix warning to fix the compilation.
    Without that fix we have:
      src/drivers/elog/elog.c: In function 'elog_is_header_valid':
      src/drivers/elog/elog.c:213:3: error: format '%u' expects argument of type 'unsigned int', but argument 4 has type 'long unsigned int' [-Werror=format]
    Change-Id: I71b80a94c03a04eedb688ae107d92c05a878315e
    Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo at no-log.org>
 src/drivers/elog/elog.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/drivers/elog/elog.c b/src/drivers/elog/elog.c
index 1a794d9..71e908b 100644
--- a/src/drivers/elog/elog.c
+++ b/src/drivers/elog/elog.c
@@ -210,7 +210,7 @@ static int elog_is_header_valid(struct elog_header *header)
 		return 0;
 	if (header->header_size != sizeof(*header)) {
-		printk(BIOS_ERR, "ELOG: header size mismatch %u != %u\n",
+		printk(BIOS_ERR, "ELOG: header size mismatch %u != %zu\n",
 		       header->header_size, sizeof(*header));
 		return 0;

More information about the coreboot-gerrit mailing list