From 2b6db9738ef6d09a068b65ef472c2d54f99abc37 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Wed, 6 Apr 2016 12:50:40 -0700 Subject: edid: Make framebuffer row alignment configurable Our EDID code had always been aligning the framebuffer's bytes_per_line (and x_resolution dependent on that) to 64. It turns out that this is a controller-dependent parameter that seems to only really be necessary for Intel chipsets, and commit 6911219cc (edid: Add helper function to calculate bits-per-pixel dependent values) probably actually broke this for some other controllers by applying the alignment too widely. This patch makes it explicitly configurable and depends the default on ARCH_X86 (which seems to be the simplest and least intrusive way to make it fit most cases for now... boards where this doesn't apply can still override it manually by calling edid_set_framebuffer_bits_per_pixel() again). Change-Id: I1c565a72826fc5ddfbb1ae4a5db5e9063b761455 Signed-off-by: Julius Werner Reviewed-on: https://review.coreboot.org/14267 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich --- src/include/edid.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/include/edid.h') diff --git a/src/include/edid.h b/src/include/edid.h index f46c761935..73412a3304 100644 --- a/src/include/edid.h +++ b/src/include/edid.h @@ -93,7 +93,8 @@ struct edid { /* Defined in src/lib/edid.c */ int decode_edid(unsigned char *edid, int size, struct edid *out); -void edid_set_framebuffer_bits_per_pixel(struct edid *edid, int fb_bpp); +void edid_set_framebuffer_bits_per_pixel(struct edid *edid, int fb_bpp, + int row_byte_alignment); void set_vbe_mode_info_valid(struct edid *edid, uintptr_t fb_addr); int set_display_mode(struct edid *edid, enum edid_modes mode); -- cgit v1.2.3