[coreboot-gerrit] New patch to review for coreboot: 7126490 src/lib/edid.c: missing break statement

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Sat Aug 9 17:16:54 CEST 2014


Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6563

-gerrit

commit 7126490e24846ccaefc2beab01f6fc453f5b3ff7
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sat Aug 9 17:16:24 2014 +0200

    src/lib/edid.c: missing break statement
    
    While vendor specific extension blocks are mostly opaque to us,
    they're not exactly "unknown".
    
    Change-Id: I9136c04d12045ad13ef4f942c0814c4df88bdf6b
    Found-by: Coverity Scan
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 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 623081e..5c2d964 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -905,7 +905,7 @@ parse_extension(struct edid *out, unsigned char *x)
 	case 0x50: printk(BIOS_SPEW, "LS extension block\n"); break;
 	case 0x60: printk(BIOS_SPEW, "DPVL extension block\n"); break;
 	case 0xF0: printk(BIOS_SPEW, "Block map\n"); break;
-	case 0xFF: printk(BIOS_SPEW, "Manufacturer-specific extension block\n");
+	case 0xFF: printk(BIOS_SPEW, "Manufacturer-specific extension block\n"); break;
 	default:
 		printk(BIOS_SPEW, "Unknown extension block\n");
 		break;



More information about the coreboot-gerrit mailing list