diff options
author | Vadim Bendebury <vbendeb@chromium.org> | 2021-11-23 22:05:55 -0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-11-29 17:51:41 +0000 |
commit | 6ba72e7fd312bf9b32bb42ee0afb93082fd91885 (patch) | |
tree | 58086970387974147f3c0f69415bc994b79cf3fc /test/nvmem.tasklist | |
parent | 0a7e3292803be9d61bafb8fa235eb57d5e01b1c9 (diff) | |
download | chrome-ec-6ba72e7fd312bf9b32bb42ee0afb93082fd91885.tar.gz |
ap_ro_check: fix error processing logic
In a situation where there is a failing V1 check and no V2 information
in the AP flash, the results of V2 check were overriding the results
of V2 check, replacing 'failed' with 'not found'.
This patch prevents the override and simplifies the verification logic
- always check for V2 if V1 check fails.
BUG=b:207545621
TEST=the DUT properly stops booting when a corrupted V1 structure is
detected and V2 structure is not present.
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Change-Id: I0abe19780bf34ed4455f1a1a61b9cf23ff83173f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3299280
Reviewed-by: Andrey Pronin <apronin@chromium.org>
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Diffstat (limited to 'test/nvmem.tasklist')
0 files changed, 0 insertions, 0 deletions