diff options
author | Julius Werner <jwerner@chromium.org> | 2016-11-30 17:46:17 -0800 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2017-01-03 22:13:19 +0100 |
commit | a69ac7861b03e438e62c57992c30d9c42996c56b (patch) | |
tree | a6080e8a43a2fdd1d61ccf35ce6b07391fa3a5ea /payloads/nvramcui | |
parent | b431a51086f89e74aa681b61744bc312255b6cb4 (diff) |
i2c/tpm: Ignore 0xFF bytes for status and burstCount
We've found that the SLB9645 TPM sometimes seems to randomly start
returning 0xFF bytes for all requests. The exact cause is yet unknown,
but we should try to write our TIS code such that it avoids bad
interactions with this kind of response (e.g. any wait_for_status()
immediately succeeds because all "status bits" are set in the response).
At least for status and burstCount readings we can say for sure that the
value is nonsensical and we're already reading those in a loop until we
get valid results anyway, so let's add code to explicitly discount 0xFF
bytes.
BRANCH=oak
BUG=chrome-os-partner:55764
TEST=None
Change-Id: I934d42c36d6847a22a185795cea49d282fa113d9
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/420470
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-on: https://review.coreboot.org/18006
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'payloads/nvramcui')
0 files changed, 0 insertions, 0 deletions