<div dir="ltr"><div dir="ltr"><div>From 52191d4f83fe9e3fa47ec2f9ee98aec83304a8f6 Mon Sep 17 00:00:00 2001</div><div>From: Matt DeVillier <<a href="mailto:matt.devillier@puri.sm">matt.devillier@puri.sm</a>></div><div>Date: Tue, 11 Sep 2018 16:54:53 -0500</div><div>Subject: [PATCH 1/1] SeaVGABios/cbvga: Fix bpp for coreboot framebuffer</div><div><br></div><div>Commit 4b42cc4 [SeaVGABios/cbvga: Advertise correct pixel format] neglected</div><div>to wrap the cbfb mask size components in GET_FARVAR(), which resulted in a</div><div>bogus value for bpp, breaking output on most/all devices.  Fix this by</div><div>adding GET_FARVAR() as appropriate.</div><div><br></div><div>TEST: build/boot a variety of devices (google/fizz, purism/librem_skl) using</div><div>coreboot for display init, verify SeaBIOS banner and boot menu prompt visible.</div><div><br></div><div>Signed-off-by: Matt DeVillier <<a href="mailto:matt.devillier@puri.sm">matt.devillier@puri.sm</a>></div><div>---</div><div> vgasrc/cbvga.c | 6 ++++--</div><div> 1 file changed, 4 insertions(+), 2 deletions(-)</div><div><br></div><div>diff --git a/vgasrc/cbvga.c b/vgasrc/cbvga.c</div><div>index 859524c..d64625e 100644</div><div>--- a/vgasrc/cbvga.c</div><div>+++ b/vgasrc/cbvga.c</div><div>@@ -312,8 +312,10 @@ cbvga_setup(void)</div><div>     }</div><div> </div><div>     u64 addr = GET_FARVAR(0, cbfb->physical_address);</div><div>-    u8 bpp = cbfb->blue_mask_size + cbfb->green_mask_size</div><div>-             + cbfb->red_mask_size + cbfb->reserved_mask_size;</div><div>+    u8 bpp = GET_FARVAR(0, cbfb->blue_mask_size)</div><div>+             + GET_FARVAR(0, cbfb->green_mask_size)</div><div>+             + GET_FARVAR(0, cbfb->red_mask_size)</div><div>+             + GET_FARVAR(0, cbfb->reserved_mask_size);</div><div>     u32 xlines = GET_FARVAR(0, cbfb->x_resolution);</div><div>     u32 ylines = GET_FARVAR(0, cbfb->y_resolution);</div><div>     u32 linelength = GET_FARVAR(0, cbfb->bytes_per_line);</div><div>-- </div><div>2.17.1</div><div><br></div></div></div>