diff options
| author | Simon Glass <sjg@chromium.org> | 2018-03-08 11:00:39 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-03-09 07:17:51 -0800 |
| commit | 526cc1e4edacf23450380c5e3469576ade196acc (patch) | |
| tree | 44076aa2f286c9db92275765ad5d58572f50a59a /include | |
| parent | 49f76d0c863146cce707e724fa79de629b912a65 (diff) | |
| download | chrome-ec-526cc1e4edacf23450380c5e3469576ade196acc.tar.gz | |
i2c: npcx: Check that unwedging succeeds
If the I2C port is completely broken then there is no point in trying to
continue with the transaction. This can happen if the pull-ups on the bus
are not working because they are in a power domain which is not currently
enabled.
Check the return value from i2c_unwedge() and return if it doesn't work.
Tidy up a few comments at the same time.
BUG=b:72129419
BRANCH=none
TEST=manually remove the check AP being off in sb_tsi_get_val() and see
that it quickly fails now, rather than hanging the EC for a while trying
the transaction.
Change-Id: I66ab52943459726537dfc74355f8aeaa5817dba5
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/956017
Reviewed-by: Edward Hill <ecgh@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@google.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions