New patch to review for coreboot: 94ca95d edid: fill reserved bits fields in cb_framebuffer

[Patrick Georgi]

Patrick Georgi ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7674

-gerrit

commit 94ca95d9efa13d21f27366f6727b87ac6a44e187
Author: Patrick Georgi 
Date:   Sat Dec 6 17:35:25 2014 +0100

edid: fill reserved bits fields in cb_framebuffer

If it's a 4 byte format (as per documentation), there
    are some reserved bits, so let's mark them as such...

Change-Id: I50f12cfff2c9bb9d082a5f3c3ac54c0d514d862b
    Signed-off-by: Patrick Georgi 
---
 src/lib/edid.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/lib/edid.c b/src/lib/edid.c
index c8e3203..2028c70 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -1446,10 +1446,14 @@ void set_vbe_mode_info_valid(struct edid *edid, uintptr_t fb_addr)
     * I'm not sure with grahpics you ever can.
     */
    edid_fb.bits_per_pixel = edid->framebuffer_bits_per_pixel;
+	edid_fb.reserved_mask_pos = 0;
+	edid_fb.reserved_mask_size = 0;
    switch(edid->framebuffer_bits_per_pixel){
    case 32:
    case 24:
    	/* packed into 4-byte words */
+		edid_fb.reserved_mask_pos = 24;
+		edid_fb.reserved_mask_size = 8;
    	edid_fb.red_mask_pos = 16;
    	edid_fb.red_mask_size = 8;
    	edid_fb.green_mask_pos = 8;
@@ -1472,8 +1476,6 @@ void set_vbe_mode_info_valid(struct edid *edid, uintptr_t fb_addr)
    	return;
    }

-	edid_fb.reserved_mask_pos = 0;
-	edid_fb.reserved_mask_size = 0;
    vbe_valid = 1;
 }