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

[Nico Huber (Code Review)]

Attention is currently required from: Patrick Rudolph.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39002 )

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

Patch Set 27:

(1 comment)

File src/lib/edid_fill_fb.c:

https://review.coreboot.org/c/coreboot/+/39002/comment/1d0a2190_1c017d6d 
PS27, Line 53: 		       " This is a driver bug.\n", __func__, fb->bits_per_pixel, bpp_mask);
> How should we go about restoring the picasso framebuffer?

Ideally, you would convince AMD to contribute to open-source graphics. Blobs
will always stay fragile.

Second best option is to ask them for a VGA BIOS that sets sane values.

I can't say that VBE really specifies how things should be. It basically
leaves the reserved_mask* fields undocumented :-/ under these circumstances
I'd say our VBE driver should sanitize the fields before passing them on
to common code?