[coreboot-gerrit] Change in coreboot[master]: lib/edid_fill_fb: Support multiple framebuffers

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39002 )

Change subject: lib/edid_fill_fb: Support multiple framebuffers
......................................................................

Patch Set 24: Code-Review+1

(4 comments)

https://review.coreboot.org/c/coreboot/+/39002/24//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/39002/24//COMMIT_MSG@17 
PS24, Line 17: of
nit: drop `of`

https://review.coreboot.org/c/coreboot/+/39002/24/src/lib/edid_fill_fb.c 
File src/lib/edid_fill_fb.c:

https://review.coreboot.org/c/coreboot/+/39002/24/src/lib/edid_fill_fb.c@70 
PS24, Line 70: 	list_insert_after(&info->node, &list);
Why not insert the new node in `fb_new_framebuffer_info`?

https://review.coreboot.org/c/coreboot/+/39002/24/src/lib/edid_fill_fb.c@77 
PS24, Line 77: supports
nit: no `s`: support

https://review.coreboot.org/c/coreboot/+/39002/24/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/mainboard.c:

https://review.coreboot.org/c/coreboot/+/39002/24/src/mainboard/google/kukui... 
PS24, Line 174: 	if (info)
Why not:

if (!info)
    return false;