[coreboot-gerrit] Patch merged into coreboot/master: tpm2: add SPI TPM driver

gerrit at coreboot.org gerrit at coreboot.org
Thu Jun 23 17:14:32 CEST 2016


the following patch was just integrated into master:
commit e31d24366ca5f344e8aed14bb6892ba1b8507f94
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Sat Apr 9 18:33:49 2016 -0700

    tpm2: add SPI TPM driver
    
    This introduces a SPI TPM driver compliant with the TCG issued "TPM
    Profile (PTP) Specification Revision 00.43" which can be found by
    googling its title.
    
    The driver implements both the hardware flow control protocol and the
    TPM state machine.
    
    The hardware flow control allows to map SPI based TPM devices to the
    LPC address space on x86 platforms, on all other platforms it needs to
    be implemented in the driver software.
    
    The tis layer is somewhat superficial, it might have to be expanded
    later.
    
    A lot more implementation details can be found in the code comments.
    
    Also, it is worth mentioning that this is not a complete version of
    the driver: its robustness needs to be improved, delay loops need to
    be bound, error conditions need to propagate up the call stack.
    
    BRANCH=none
    BUG=chrome-os-partner:52132, chrome-os-partner:50645, chrome-os-partner:54141
    TEST=with the rest of the patches applied coreboot is able complete
         Chrome OS factory initialization of the TPM2 device.
    
    Change-Id: I967bc5c689f6e6f345755f08cb088ad37abd5d1c
    Signed-off-by: Martin Roth <martinroth at chromium.org>
    Original-Commit-Id: 5611c6f7d7fe6d37da668f337f0e70263913d63e
    Original-Change-Id: I17d732e66bd231c2289ec289994dd819c6276855
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/350124
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: https://review.coreboot.org/15298
    Reviewed-by: Philipp Deppenwiese <zaolin.daisuki at googlemail.com>
    Tested-by: build bot (Jenkins)


See https://review.coreboot.org/15298 for details.

-gerrit



More information about the coreboot-gerrit mailing list