[coreboot-gerrit] Patch merged into coreboot/master: a5aac76 drivers/spi: Pass flash parameters from coreboot to payload

gerrit at coreboot.org gerrit at coreboot.org
Fri Apr 17 09:21:10 CEST 2015


the following patch was just integrated into master:
commit a5aac76ac6be23448c164b0bc8047facb7238cdf
Author: Dan Ehrenberg <dehrenberg at chromium.org>
Date:   Thu Jan 8 10:29:19 2015 -0800

    drivers/spi: Pass flash parameters from coreboot to payload
    
    A payload may want to run erase operations on SPI NOR flash without
    re-probing the device to get its properties. This patch passes up
    three properties of flash to achieve that:
    - The size of the flash device
    - The sector size, i.e., the granularity of erase
    - The command used for erase
    The patch sends the parameters through coreboot and then libpayload.
    The patch also includes a minor refactoring of the flash erase code.
    Parameters are sent up for just one flash device. If multiple SPI
    flash devices are probed, the second one will "win" and its
    parameters will be sent up to the payload.
    
    TEST=Observed parameters to be passed up to depthcharge through
    libpayload and be used to correctly initialize flash and do an erase.
    TEST=Winbond and Gigadevices spi flash drivers compile with the changes;
    others don't, for seemingly unrelated reasons.
    BRANCH=none
    BUG=chromium:446377
    
    Change-Id: Ib8be86494b5a3d1cfe1d23d3492e3b5cba5f99c6
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 988c8c68bbfcdfa69d497ea5f806567bc80f8126
    Original-Change-Id: Ie2b3a7f5b6e016d212f4f9bac3fabd80daf2ce72
    Original-Signed-off-by: Dan Ehrenberg <dehrenberg at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/239570
    Original-Reviewed-by: Vadim Bendebury <vbendeb at chromium.org>
    Reviewed-on: http://review.coreboot.org/9726
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list