[coreboot-gerrit] Patch merged into coreboot/master: edid: Add helper function to calculate bits-per-pixel dependent values

gerrit at coreboot.org gerrit at coreboot.org
Thu Mar 24 20:25:15 CET 2016


the following patch was just integrated into master:
commit 6911219ccc445f833264cf5a5a4b9439b9670b40
Author: Julius Werner <jwerner at chromium.org>
Date:   Mon Mar 14 17:29:55 2016 -0700

    edid: Add helper function to calculate bits-per-pixel dependent values
    
    Coreboot and most payloads support three basic pixel widths for the
    framebuffer. It assumes 32 by default, but several chipsets need to
    override that value with whatever else they're supporting. Our struct
    edid contains multiple convenience values that are directly derived from
    this (and other properties), so changing the bits per pixel always
    requires recalculating all those dependents in the chipset code. This
    patch provides a small convenience wrapper that can be used to
    consistently update the whole struct edid with a new pixel width
    instead, so we no longer need to duplicate those calculations
    everywhere.
    
    BUG=None
    TEST=Booted Oak in all three pixel widths (which it conveniently all
    supports), confirmed that images looked good.
    
    Change-Id: I5376dd4e28cf107ac2fba1dc418f5e1c5a2e2de6
    Signed-off-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: https://review.coreboot.org/14158
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list