Change in coreboot[master]: lib/edid_fill_fb: Relax bits_per_pixel constraint

[Raul Rangel (Code Review)]

Raul Rangel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/49404 )

Change subject: lib/edid_fill_fb: Relax bits_per_pixel constraint
......................................................................

lib/edid_fill_fb: Relax bits_per_pixel constraint

The Picasso VBIOS is not setting the reserved_mask_size correctly. This
change relaxes the constraint to allow bpp_mask <= bits_per_pixel. This
is how the code previously used to work before CB:39002.

BUG=b:177094598, b:177422379
TEST=boot zork and see depthcharge working

Signed-off-by: Raul E Rangel 
Change-Id: I2e67532fa949fbd673269d8d7f1c0d8af6124ac9
---
M src/lib/edid_fill_fb.c
1 file changed, 10 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/49404/1

diff --git a/src/lib/edid_fill_fb.c b/src/lib/edid_fill_fb.c
index 712cd0f..cfd7776 100644
--- a/src/lib/edid_fill_fb.c
+++ b/src/lib/edid_fill_fb.c
@@ -48,10 +48,17 @@

bpp_mask = fb->blue_mask_size + fb->green_mask_size + fb->red_mask_size +
    	fb->reserved_mask_size;
-	if (fb->bits_per_pixel != bpp_mask) {
-		printk(BIOS_ERR, "%s: BPP=%d and channel bit mask=%d doesn't match."
-		       " This is a driver bug.\n", __func__, fb->bits_per_pixel, bpp_mask);
+	if (bpp_mask > fb->bits_per_pixel) {
+		printk(BIOS_ERR,
+		       "%s: channel bit mask=%d is greater than BPP=%d ."
+		       " This is a driver bug. Framebuffer is invalid.\n",
+		       __func__, bpp_mask, fb->bits_per_pixel);
    	return NULL;
+	} else if (fb->bits_per_pixel != bpp_mask) {
+		printk(BIOS_WARNING,
+		       "%s: BPP=%d and channel bit mask=%d doesn't match."
+		       " This is a driver bug.\n",
+		       __func__, fb->bits_per_pixel, bpp_mask);
    }

info = fb_new_framebuffer_info();