[coreboot-gerrit] Patch merged into coreboot/master: T210: Add 128MB VPR allocation/carveout

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


the following patch was just integrated into master:
commit 50967870a9b0a55a5c973785803d7fe8921c1107
Author: Tom Warren <twarren at nvidia.com>
Date:   Tue Aug 4 13:08:50 2015 -0700

    T210: Add 128MB VPR allocation/carveout
    
    The NV security team requested that coreboot allocate a 128MB
    region in SDRAM for VPR (Video Protection Region). We had
    previously just disabled the VPR by setting BOM/SIZE to 0.
    
    Once allocated, the VPR will be locked from further access.
    The ALLOW_TZ_WRITE_ACCESS bit is _not_ set, as dynamic VPR config
    is not supported at this time (i.e. trusted code can _not_ remap
    or resize the VPR).
    
    BUG=None
    BRANCH=None
    TEST=Built and booted on my P5 A44. Saw the VPR region in the
    boot spew (ID:3 [f6800000 - fe800000]). Dumped the MC VideoProtect
    registers and verified their values.
    
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
    Original-Commit-Id: a7481dba31dc39f482f8a7bfdaba1d1f4fc3cb81
    Original-Change-Id: Ia19af485430bc09dbba28fcef5de16de851f81aa
    Original-Signed-off-by: Tom Warren <twarren at nvidia.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/290475
    Original-Reviewed-by: Hyung Taek Ryoo <hryoo at nvidia.com>
    Original-Reviewed-by: Furquan Shaikh <furquan at chromium.org>
    Original-Reviewed-by: Hridya Valsaraju <hvalsaraju at nvidia.com>
    Original-(cherry picked from commit 9629b318eb17b145315531509f950da02483114f)
    Original-Reviewed-on: https://chromium-review.googlesource.com/291095
    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: I19a93c915990644177c491c8212f2cf356d4d17d
    Reviewed-on: http://review.coreboot.org/11384
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list