[coreboot-gerrit] Patch merged into coreboot/master: 7cb01e0 drivers: Add I2C TPM driver to coreboot

gerrit at coreboot.org gerrit at coreboot.org
Sun Aug 10 22:25:52 CEST 2014


the following patch was just integrated into master:
commit 7cb01e0bcfe4287df83b0bc07928dae33e29a9c8
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Thu Aug 29 16:05:02 2013 -0700

    drivers: Add I2C TPM driver to coreboot
    
    On ARM platforms the TPM is not attached through LPC but through I2C.
    This patch adds an I2C TPM driver that supports the following chips:
     * Infineon SLB9635
     * Infineon SLB9645
    In order to select the correct TPM implementation cleanly, CONFIG_TPM
    is moved to src/Kconfig and does the correct choice.
    
    Old-Change-Id: I2def0e0f86a869d6fcf56fc4ccab0bc935de2bf1
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/167543
    Reviewed-by: ron minnich <rminnich at chromium.org>
    (cherry picked from commit b4049a0e96f6335a93877e1e884f9a440487c421)
    
    i2c tpm: Remove mostly useless delay code/tables.
    
    I assume from the code in the TPM driver that the TPM spec defines
    different types of delays and timeouts which each have a particular
    duration, and that the TPM can tell you how long each type is if you ask
    it. There was a large table, some members of a data structure, and a
    function or two which managed the timeouts and figured their value for
    different operations.  The timeout values for the various "ordinals"
    were never set in the vendor specific data structure, however, and
    always defaulted to 2 minutes.  Similarly the timeouts a, b, c, and d
    were never overridden from their defaults.  This change gets rid of all
    the timeout management code and makes the "ordinal" timeout 2 minutes
    and the a, b, c, and d timeouts 2 seconds, the larger of the two default
    values.
    
    This is a port from depthcharge to coreboot, original change:
    https://chromium-review.googlesource.com/#/c/168363/
    
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    
    Old-Change-Id: I79696d6329184ca07f6a1be4f6ca85e1655a7aaf
    Reviewed-on: https://chromium-review.googlesource.com/168583
    Reviewed-by: Gabe Black <gabeblack at chromium.org>
    Tested-by: Stefan Reinauer <reinauer at google.com>
    Commit-Queue: Stefan Reinauer <reinauer at google.com>
    (cherry picked from commit b22395a73f361c38626911808332a3706b2334fe)
    
    TPM: Stop requesting/releasing the TPM locality.
    
    The locality is requested when the TPM is initialized and released when
    it's cleaned up. There's no reason to set it to the same thing again and
    restore it back to the same value before and after every transaction.
    
    forward ported from https://chromium-review.googlesource.com/#/c/168400
    
    Old-Change-Id: I291d1f86f220ef0eff6809c6cb00459bf95aa5e0
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/168584
    Reviewed-by: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit cc866c20c6f936f349d2f1773dd492dca9bbf0c1)
    
    Squashed three commits for the i2c tpm driver.
    
    Change-Id: Ie7a50c50fda8ee986c02de7fe27551666998229d
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6519
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list