Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32088 )
Change subject: lib/edid.c: Add missing break statement ......................................................................
lib/edid.c: Add missing break statement
This was flagged as CID 1229647 (MISSING_BREAK). It was originally fixed in e211bd9, but then reverted in 1c8ee21. Hopefully fix it for good now.
Signed-off-by: Jacob Garber jgarber1@ualberta.ca Change-Id: Ib25b07b633ed02c466391050df15eaf9c36a3199 Reviewed-on: https://review.coreboot.org/c/coreboot/+/32088 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Julius Werner jwerner@chromium.org --- M src/lib/edid.c 1 file changed, 1 insertion(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Julius Werner: Looks good to me, approved
diff --git a/src/lib/edid.c b/src/lib/edid.c index d56099b..3087172 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -1002,6 +1002,7 @@ break; case 0xFF: printk(BIOS_SPEW, "Manufacturer-specific extension block\n"); + break; default: printk(BIOS_SPEW, "Unknown extension block\n"); break;