[coreboot-gerrit] New patch to review for coreboot: 361f727 ELOG: Fix warning to fix the compilation.

Denis Carikli (GNUtoo@no-log.org) gerrit at coreboot.org
Wed Jun 26 21:17:59 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

-gerrit

commit 361f72705bae7a0edc2c390cc18af53f69bee9b7
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..102dc37 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 != %lu\n",
 		       header->header_size, sizeof(*header));
 		return 0;
 	}



More information about the coreboot-gerrit mailing list