diff options
author | Vadim Bendebury <vbendeb@chromium.org> | 2017-08-17 20:19:08 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-08-23 12:19:07 -0700 |
commit | 52527acbfb4267db45df05a08ebb7a1a30682e61 (patch) | |
tree | 4c54e4c85dd0dfb965590e5e43dbf75a8d1d1378 | |
parent | e24bd63fae46ca1abe1d58257f3dbfe924e9a94b (diff) | |
download | chrome-ec-52527acbfb4267db45df05a08ebb7a1a30682e61.tar.gz |
cr50: log I2C slave 'wedged bus recovery' event
We want to be able to collect statistics of the i2c bus problems. This
patch logs an event each time the wedged bus recovery happens.
BRANCH=cr50
BUG=b:63760920
TEST=with the upcoming patches verified that i2c recovery events are
logged as expected.
Change-Id: I1241b2dece33f89cd724d53a48f94e17f4415c62
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/620114
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-by: Shawn N <shawnn@chromium.org>
-rw-r--r-- | chip/g/i2cs.c | 6 | ||||
-rw-r--r-- | include/tpm_log.h | 2 |
2 files changed, 7 insertions, 1 deletions
diff --git a/chip/g/i2cs.c b/chip/g/i2cs.c index a42122dae6..8c00c4ddea 100644 --- a/chip/g/i2cs.c +++ b/chip/g/i2cs.c @@ -71,6 +71,7 @@ #include "registers.h" #include "system.h" #include "task.h" +#include "tpm_log.h" #define REGISTER_FILE_SIZE (1 << 6) /* 64 bytes. */ #define REGISTER_FILE_MASK (REGISTER_FILE_SIZE - 1) @@ -175,6 +176,11 @@ static void poll_read_state(void) i2cs_read_recovery_count++; i2cs_register_write_complete_handler (write_complete_handler_); + +#ifdef CONFIG_TPM_LOGGING + tpm_log_event(TPM_I2C_RESET, + i2cs_read_recovery_count); +#endif return; } last_i2cs_read_irq_count = i2cs_read_irq_count; diff --git a/include/tpm_log.h b/include/tpm_log.h index d934560290..cb1b6e099a 100644 --- a/include/tpm_log.h +++ b/include/tpm_log.h @@ -10,7 +10,7 @@ enum tpm_event { TPM_EVENT_INIT, - /* TODO: Add log events */ + TPM_I2C_RESET, }; /* Log TPM event of given type with data payload. */ |