[coreboot-gerrit] Patch merged into coreboot/master: 584e048 libpayload arm64: don't modify cbtable entries

gerrit at coreboot.org gerrit at coreboot.org
Sat Mar 21 13:43:11 CET 2015


the following patch was just integrated into master:
commit 584e048185f2e456bc67a86cb54feac4dda8084c
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Tue Nov 18 11:55:33 2014 -0600

    libpayload arm64: don't modify cbtable entries
    
    The framebuffer structure lives in the coreboot tables. Those
    tables have a checksum calculation applied over all the entries.
    Therefore, one shouldnot be modifying fields within the coreboot
    table entries because the calculated checksum would be wrong.
    
    BRANCH=none
    BUG=chrome-os-partner:31936
    TEST=On ryu, confirmed dev screen still works as well as cbmem utility
         once booted.
    
    Change-Id: I93830a8efe98aa848f2b0f8388688de0e93b2f82
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 6026ca5ad0254c14c30412882dc63550656c7d16
    Original-Change-Id: Ic9c164ded03d10d6f6f3ce15e9b38b1f6ce61a91
    Original-Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/230471
    Original-Reviewed-by: Furquan Shaikh <furquan at chromium.org>
    Original-Commit-Queue: Furquan Shaikh <furquan at chromium.org>
    Reviewed-on: http://review.coreboot.org/8797
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See http://review.coreboot.org/8797 for details.

-gerrit



More information about the coreboot-gerrit mailing list