[coreboot-gerrit] Patch set updated for coreboot: b1ac548 lib/edid.c: Fix mismatch between format string and variable type

Alexandru Gagniuc (mr.nuke.me@gmail.com) gerrit at coreboot.org
Sun Dec 21 10:29:15 CET 2014


Alexandru Gagniuc (mr.nuke.me at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7885

-gerrit

commit b1ac5489a5348e828aac84a4cc494e64bef126c8
Author: Alexandru Gagniuc <mr.nuke.me at gmail.com>
Date:   Sun Dec 21 02:54:33 2014 -0600

    lib/edid.c: Fix mismatch between format string and variable type
    
    Use 'd' instead of 'hhd' when printing absolute year of manufacture. This
    is the correct type in this case, as the result is autoatically promoted
    to int.
    
    Change-Id: Ice4155bb1a04f206ae55c45c260089d6971b77d1
    Signed-off-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>
---
 src/lib/edid.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/edid.c b/src/lib/edid.c
index 7db1bc3..c047b51 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -1011,7 +1011,7 @@ int decode_edid(unsigned char *edid, int size, struct edid *out)
 				/* we know it's at least 2013, when this code was written */
 				if (edid[0x11] + 90 <= 2013) {
 					c.has_valid_year = 1;
-					printk(BIOS_SPEW, "Made week %hhd of %hhd\n",
+					printk(BIOS_SPEW, "Made week %hhd of %d\n",
 					       edid[0x10], edid[0x11] + 1990);
 					out->week = edid[0x10];
 					out->year = edid[0x11] + 1990;



More information about the coreboot-gerrit mailing list