[coreboot-gerrit] Patch merged into coreboot/master: tpm: report firmware version

[gerrit at coreboot.org]

the following patch was just integrated into master:
commit 58826fc85e63dde8f5171dfb37224bb1b7ceb9b2
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Thu Jun 23 18:17:33 2016 -0700

    tpm: report firmware version
    
    Some devices allow to retrieve firmware version by reading the same 4
    byte register repeatedly until the entire version string is read.
    
    Let's print out TPM firmware version when available. Just in case
    something goes wrong limit the version string length to 200 bytes.
    
    CQ-DEPEND=CL:355701
    BRANCH=none
    BUG=chrome-os-partner:54723
    TEST=built the new firmware and ran it on Gru, observed the following
         in the coreboot console log:
    
      Connected to device vid:did:rid of 1ae0:0028:00
      Firmware version: cr50_v1.1.4792-7a44484
    
    Original-Commit-Id: 1f54a30cebe808abf1b09478b47924bb722a0ca6
    Original-Change-Id: Idb069dabb80d34a0efdf04c3c40a42ab0c8a3f94
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/355704
    Original-Reviewed-by: Scott Collyer <scollyer at chromium.org>
    
    Squashed with:
    
    tpm: use 4 byte quantities when retrieving firmware version
    
    The CR50 device is capable of reporting its firmware version in 4 byte
    quantities, but the recently introduced code retrieves the version one
    byte at a time.
    
    With this fix the version is retrieved in 4 byte chunks.
    
    BRANCH=none
    BUG=none
    TEST=the version is still reported properly, as reported by the AP
         firmware console log:
    
    localhost ~ # grep cr50 /sys/firmware/log
    Firmware version: cr50_v1.1.4804-c64cf24
    localhost ~ #
    
    Original-Commit-Id: 3111537e7b66d8507b6608ef665e4cde76403818
    Original-Change-Id: I04116881a30001e35e989e51ec1567263f9149a6
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/356542
    Original-Reviewed-by: Andrey Pronin <apronin at chromium.org>
    
    Change-Id: Ia9f13a5bf1c34292b866f57c0d14470fe6ca9853
    Signed-off-by: Martin Roth <martinroth at chromium.org>
    Reviewed-on: https://review.coreboot.org/15573
    Tested-by: build bot (Jenkins)
    Reviewed-by: Furquan Shaikh <furquan at google.com>


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

-gerrit