summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVadim Bendebury <vbendeb@chromium.org>2017-11-21 17:45:08 -0800
committerChromeOS Commit Bot <chromeos-commit-bot@chromium.org>2018-01-03 16:58:34 +0000
commit161d426f34918a0b978a9653f4ab9fa45b6104c9 (patch)
treeda539655037a4bfcb69d1a2e12cc848fea72276f
parent76adb75f9a6d14582bd585baad6d9903a4d5996d (diff)
downloadchrome-ec-161d426f34918a0b978a9653f4ab9fa45b6104c9.tar.gz
cr50: ccd: add vendor command to disable RMA mode
The new vendor command takes the CCD state machine through necessary transitions leaving it in the CCD locked state. It succeeds only if user password is not set and CCD capabilities are right, which is guaranteed to be the case after an RMA unlock. BRANCH=cr50 BUG=b:68213540 TEST=tested using the modified gsctool utility. Change-Id: Ic2cce34e74b1ff476841cfa1a99f50d6a947c315 Signed-off-by: Vadim Bendebury <vbendeb@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/784352 Reviewed-by: Randall Spangler <rspangler@chromium.org> (cherry picked from commit baff7ae621d88929c0ab808eee54c081e44b2710) Reviewed-on: https://chromium-review.googlesource.com/848240 Commit-Queue: YH Lin <yueherngl@chromium.org> Tested-by: YH Lin <yueherngl@chromium.org>
-rw-r--r--common/ccd_config.c69
-rw-r--r--include/tpm_vendor_cmds.h1
2 files changed, 70 insertions, 0 deletions
diff --git a/common/ccd_config.c b/common/ccd_config.c
index f850a36ff5..dd46b1b1db 100644
--- a/common/ccd_config.c
+++ b/common/ccd_config.c
@@ -1153,3 +1153,72 @@ static enum vendor_cmd_rc ccd_password(enum vendor_cmd_cc code,
return VENDOR_RC_SUCCESS;
}
DECLARE_VENDOR_COMMAND(VENDOR_CC_CCD_PASSWORD, ccd_password);
+
+
+static enum vendor_cmd_rc ccd_disable_rma(enum vendor_cmd_cc code,
+ void *buf,
+ size_t input_size,
+ size_t *response_size)
+{
+ int rv;
+ int error_line;
+
+ do {
+ if (raw_has_password()) {
+ error_line = __LINE__;
+ break;
+ }
+
+ /* Check if physical presence is required to unlock. */
+ if (!ccd_is_cap_enabled(CCD_CAP_REMOVE_BATTERY_BYPASSES_PP) ||
+ board_battery_is_present()) {
+ const uint8_t required_capabilities[] = {
+ CCD_CAP_OPEN_WITHOUT_TPM_WIPE,
+ CCD_CAP_UNLOCK_WITHOUT_AP_REBOOT,
+ CCD_CAP_OPEN_WITHOUT_LONG_PP,
+ CCD_CAP_UNLOCK_WITHOUT_SHORT_PP
+ };
+ unsigned int i;
+
+ for (i = 0;
+ i < ARRAY_SIZE(required_capabilities);
+ i++) {
+ if (!ccd_is_cap_enabled
+ (required_capabilities[i]))
+ break;
+ }
+
+ if (i < ARRAY_SIZE(required_capabilities)) {
+ CPRINTF("Capability %d is not present\n",
+ required_capabilities[i]);
+ error_line = __LINE__;
+ break;
+ }
+ }
+
+ ccd_set_state(CCD_STATE_OPENED);
+
+ rv = command_ccd_reset(0, NULL);
+ if (rv != EC_SUCCESS) {
+ error_line = __LINE__;
+ break;
+ }
+
+
+ rv = command_ccd_lock();
+ if (rv != EC_SUCCESS) {
+ error_line = __LINE__;
+ break;
+ }
+
+ *response_size = 0;
+ return VENDOR_RC_SUCCESS;
+ } while (0);
+
+ CPRINTF("%s: error in line %d\n", __func__, error_line);
+
+ ((uint8_t *)buf)[0] = (uint8_t)rv;
+ *response_size = 1;
+ return VENDOR_RC_INTERNAL_ERROR;
+}
+DECLARE_VENDOR_COMMAND(VENDOR_CC_DISABLE_RMA, ccd_disable_rma);
diff --git a/include/tpm_vendor_cmds.h b/include/tpm_vendor_cmds.h
index 93961e793d..1a9682be69 100644
--- a/include/tpm_vendor_cmds.h
+++ b/include/tpm_vendor_cmds.h
@@ -45,6 +45,7 @@ enum vendor_cmd_cc {
VENDOR_CC_GET_REC_BTN = 29,
VENDOR_CC_RMA_CHALLENGE_RESPONSE = 30,
VENDOR_CC_CCD_PASSWORD = 31,
+ VENDOR_CC_DISABLE_RMA = 32,
LAST_VENDOR_COMMAND = 65535,
};