Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46694 )
Change subject: lib/edid: Add missing name descriptor presence flag. ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG@9 PS1, Line 9: flah flash
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG@13 PS1, Line 13: Is there a bug number for this? I know that coreboot doesn't really care about bug numbers, but when the upstream CL lands in the chromium tree, it will be nice to have a bug number associated with it.
Even better is if there is a unit test for this module (I know there isn't one yet), then we add a test case that fails before this fix is applied and passes after it is applied.