Name of user not set #1003143 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 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG@7 PS1, Line 7: .
nit: drop trailing period in commit summary
Done
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG@9 PS1, Line 9: flah
sorry, no, that should be "flag" […]
Done
https://review.coreboot.org/c/coreboot/+/46694/1//COMMIT_MSG@11 PS1, Line 11:
nit: only one space is enough
Done
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 […]
I do not know of any bug number associated with this. I found this bug while creating tests for this module. I checked previous versions of lib/edid.c in repository and found the exact place where this flag had to be, but was scrapped in commit 2536c1ac76790b12554fe8e277ef2dcbc5f58242 probably by accident.