summaryrefslogtreecommitdiff
path: root/src/mainboard/google/samus/ec.h
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2015-02-20 13:36:11 -0800
committerStefan Reinauer <stefan.reinauer@coreboot.org>2015-04-10 20:31:36 +0200
commit02e847b919b4807440ddb02bdfa7c8ac74edd7f2 (patch)
tree1954ae29c208fc7d0a3ce293be6768b41e82e0cc /src/mainboard/google/samus/ec.h
parent1c4743f86b928400e097830eba21a30ddb23f741 (diff)
cros_ec: Retry failed VBNV transactions
This patch adds a few retries to NVRAM read/write transactions with the EC. Failing to read the NVRAM is not fatal to the boot, but it's still pretty bad... especially since a single initial read failure will cause vboot to blindly reinitialize the whole NVRAM with zeroes, destroying important configuration bits like dev_boot_usb. The current EC transaction timeout is one second, so the three retries added here can potentially increase boot time by three seconds per transaction... but this shouldn't happen in any normal case anyway, and if there are errors a little extra wait is probably preferrable to nuking your NVRAM. (Also, added a missing newline to an error message in the EC code.) BRANCH=veyron BUG=chrome-os-partner:36924 TEST=Booted a Jerry with the power button bug with a 2 second press, noticed that the first two transactions failed but the third one succeeded. Change-Id: I5d1cf29ac1c555ea2336ebb0b0e0a3f7cbb9c3fd Signed-off-by: Stefan Reinauer <reinauer@chromium.org> Original-Commit-Id: 894a8a0b4a9805e92544b5e3dfa90baf6d36649a Original-Change-Id: I6267cdda2be2bad34541b687404c2434d3be345b Original-Signed-off-by: Julius Werner <jwerner@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/251694 Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/9507 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'src/mainboard/google/samus/ec.h')
0 files changed, 0 insertions, 0 deletions