[coreboot-gerrit] Patch merged into coreboot/master: adcb095 Provide a common CBFS wrapper for SPI storage

gerrit at coreboot.org gerrit at coreboot.org
Tue Dec 30 19:29:51 CET 2014


the following patch was just integrated into master:
commit adcb095e9e1e535b9331648dab0d628afd1336ca
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Thu May 1 12:23:09 2014 -0700

    Provide a common CBFS wrapper for SPI storage
    
    Coreboot has all necessary infrastructure to use the proper SPI flash
    interface in bootblock for CBFS. This patch creates a common CBFS
    wrapper which can be enabled on different platforms as required.
    
    COMMON_CBFS_SPI_WRAPPER, a new configuration option, enables the
    common CBFS interface and prevents default inclusion of all SPI chip
    drivers, only explicitly configured ones will be included when the new
    feature is enabled. Since the wrapper uses the same driver at all
    stages, enabling the new feature will also make it necessary to
    include the SPI chip drivers in bootblock and romstage images.
    
    init_default_cbfs_media() can now be common for different platforms,
    and as such is defined in the library.
    
    BUG=none
    TEST=manual
       . with this change and the rest of the patches coreboot on AP148
         comes up all the way to attempting to boot the payload (reading
         earlier stages from the SPI flash along the way).
    
    Original-Change-Id: Ia887bb7f386a0e23a110e38001d86f9d43fadf2c
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/197800
    Original-Tested-by: Vadim Bendebury <vbendeb at google.com>
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
    (cherry picked from commit 60eb16ebe624f9420c6191afa6ba239b8e83a6e6)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I7b0bf3dda915c227659ab62743e405312dedaf41
    Reviewed-on: http://review.coreboot.org/7932
    Tested-by: build bot (Jenkins)
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>


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

-gerrit



More information about the coreboot-gerrit mailing list