diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2018-09-27 19:26:30 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-10-04 00:37:07 -0700 |
| commit | 9ef91389ca20f0dc50c2ab0d6d231ec6feb2802b (patch) | |
| tree | 50d6a99cc44b2ad5b0846b8b99438f92b8191157 /board | |
| parent | 14a70cf973458d73fa71e14d46351769ddf131da (diff) | |
| download | chrome-ec-9ef91389ca20f0dc50c2ab0d6d231ec6feb2802b.tar.gz | |
rma: when processing 'RMA open' do not reboot the device
Once RMA open is processed and CCD state is updated, the AP still
might require to perform some operations, even if TPM is not available
any more.
With this patch enable_ccd_factory_mode() does not trigger device
reset, if invoked by the RMA open handler.
Another modification is that WP is disabled immediately when factory
mode is enabled, there is no need to reset the H1 for WP status to
change.
BRANCH=cr50, cr50-mp
BUG=b:115495431
TEST=verified that running 'gsctool -a -r <authcode>' sets to 'Y' all
CCD properties, disables write protection, but does not reboot
the device.
Change-Id: I834a9e4b5ebbe4aaaf1caafad9c82424087d01f7
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1250037
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Diffstat (limited to 'board')
| -rw-r--r-- | board/cr50/factory_mode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/cr50/factory_mode.c b/board/cr50/factory_mode.c index 232f757a1a..6e07fe00f5 100644 --- a/board/cr50/factory_mode.c +++ b/board/cr50/factory_mode.c @@ -112,7 +112,7 @@ static enum vendor_cmd_rc vc_factory_reset(enum vendor_cmd_cc code, return VENDOR_RC_NOT_ALLOWED; CPRINTF("factory reset\n"); - enable_ccd_factory_mode(); + enable_ccd_factory_mode(1); return VENDOR_RC_SUCCESS; } |