[coreboot-gerrit] Patch merged into coreboot/master: Smaug: Add NVDEC and TSEC carveouts

gerrit at coreboot.org gerrit at coreboot.org
Fri Aug 28 08:40:13 CEST 2015


the following patch was just integrated into master:
commit 0bdb88b106ec65e3d1296d61545e09ecdab0794f
Author: Tom Warren <twarren at nvidia.com>
Date:   Mon Aug 3 14:58:11 2015 -0700

    Smaug: Add NVDEC and TSEC carveouts
    
    The NV security team requested that coreboot allocate the NVDEC
    and TSEC carveouts. Added code to set up NVDEC (1 region, 1MB)
    and TSEC (2 regions, splitting 2MB), and set their lock bits.
    Kernel/trusted code should be able to use the regions now.
    
    Note that this change sets the UNLOCKED bit in Carveout1Cfg0
    and Carveout4Cfg0/5Cfg0 (bit 1) to 0 in the BCT .inc files
    (both 3GB and 4GB BCTs) so that the BOMs can be written.
    Any future revisions to these BCT files should take this
    into account.
    
    BUG=None
    BRANCH=None
    TEST=Built and booted on my P5 A44. Saw the carveout regions
    in the boot spew, and CBMEM living just below the last region
    (TSEC). Dumped the MC GeneralizedCarveoutX registers and
    verified their values (same as BCT, with only BOM/CFG0 changed).
    
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Original-Commit-Id: a34b0772cd721193640b322768ce5fcbb4624f23
    Original-Change-Id: I2abc872fa1cc4ea669409ffc9f2e66dbbc4efcd0
    Original-Signed-off-by: Tom Warren <twarren at nvidia.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/290452
    Original-Reviewed-by: Furquan Shaikh <furquan at chromium.org>
    Original-(cherry picked from commit f3bbf25397db4d17044e9cfd135ecf73df0ffa60)
    Original-Reviewed-on: https://chromium-review.googlesource.com/291081
    Original-Commit-Queue: Furquan Shaikh <furquan at chromium.org>
    Original-Trybot-Ready: Furquan Shaikh <furquan at chromium.org>
    Original-Tested-by: Furquan Shaikh <furquan at chromium.org>
    
    Change-Id: I924dfdae7b7c9b877cb1c93fd94f0ef98b728ac5
    Reviewed-on: http://review.coreboot.org/11381
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list