[coreboot-gerrit] New patch to review for coreboot: a2b97a4 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 09:57:43 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 a2b97a44bc25780226467fd492ad577393e56bc0
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 'hd' instead of 'hhd' when printing absolute year of manufacture. This
    is the correct type for a 16-bit integer.
    
    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..f14ab44 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 %hd\n",
 					       edid[0x10], edid[0x11] + 1990);
 					out->week = edid[0x10];
 					out->year = edid[0x11] + 1990;



More information about the coreboot-gerrit mailing list