[coreboot-gerrit] Patch merged into coreboot/master: edid: Make framebuffer row alignment configurable

gerrit at coreboot.org gerrit at coreboot.org
Thu Apr 7 20:46:41 CEST 2016


the following patch was just integrated into master:
commit 2b6db9738ef6d09a068b65ef472c2d54f99abc37
Author: Julius Werner <jwerner at chromium.org>
Date:   Wed Apr 6 12:50:40 2016 -0700

    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 <jwerner at chromium.org>
    Reviewed-on: https://review.coreboot.org/14267
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>


See https://review.coreboot.org/14267 for details.

-gerrit



More information about the coreboot-gerrit mailing list