diff options
author | Shawn Nematbakhsh <shawnn@chromium.org> | 2013-07-22 16:19:13 -0700 |
---|---|---|
committer | ChromeBot <chrome-bot@google.com> | 2013-07-23 12:37:04 -0700 |
commit | 472d93c146403c0f95b656e182b972e15ff11835 (patch) | |
tree | e1a5b95e604fc9eb42ab7493df10b92cd1a4a0a5 /firmware | |
parent | 964144bf2f3befe8c8a010000439cb5e5dccf00d (diff) | |
download | vboot-472d93c146403c0f95b656e182b972e15ff11835.tar.gz |
rollback_index: Remove recovery_mode parameter to SetupTPM.
SetupTPM no longer uses recovery_mode parameter for anything other than
a debug print. This change moves the debug print to a caller function,
then removes recovery_mode from SetupTPM and some caller functions that
no longer have a use for it.
BUG=chrome-os-partner:20913.
TEST=Manual. Boot factory install shim in recovery mode and verify TPM
clear operations succeed. Boot in dev mode and verify "Lock physical
presence" print on UART.
BRANCH=None.
Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
Change-Id: I2f671f6680a6e67cf722855e659e99752bc0783c
Reviewed-on: https://gerrit.chromium.org/gerrit/62916
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'firmware')
-rw-r--r-- | firmware/lib/include/rollback_index.h | 7 | ||||
-rw-r--r-- | firmware/lib/mocked_rollback_index.c | 7 | ||||
-rw-r--r-- | firmware/lib/rollback_index.c | 14 | ||||
-rw-r--r-- | firmware/lib/vboot_api_init.c | 4 | ||||
-rw-r--r-- | firmware/linktest/main.c | 2 |
5 files changed, 15 insertions, 19 deletions
diff --git a/firmware/lib/include/rollback_index.h b/firmware/lib/include/rollback_index.h index 386ad77f..7c410eec 100644 --- a/firmware/lib/include/rollback_index.h +++ b/firmware/lib/include/rollback_index.h @@ -83,7 +83,7 @@ uint32_t RollbackS3Resume(void); /** * This must be called. */ -uint32_t RollbackFirmwareSetup(int recovery_mode, int is_hw_dev, +uint32_t RollbackFirmwareSetup(int is_hw_dev, int disable_dev_request, int clear_tpm_owner_request, /* two outputs on success */ @@ -161,9 +161,8 @@ uint32_t OneTimeInitializeTPM(RollbackSpaceFirmware *rsf, * Start the TPM and establish the root of trust for the anti-rollback * mechanism. */ -uint32_t SetupTPM(int recovery_mode, int developer_mode, - int disable_dev_request, int clear_tpm_owner_request, - RollbackSpaceFirmware *rsf); +uint32_t SetupTPM(int developer_mode, int disable_dev_request, + int clear_tpm_owner_request, RollbackSpaceFirmware *rsf); /** * Utility function to turn the virtual dev-mode flag on or off. 0=off, 1=on. diff --git a/firmware/lib/mocked_rollback_index.c b/firmware/lib/mocked_rollback_index.c index 6f026a33..f83e0378 100644 --- a/firmware/lib/mocked_rollback_index.c +++ b/firmware/lib/mocked_rollback_index.c @@ -23,9 +23,8 @@ uint32_t TPMClearAndReenable(void) { } -uint32_t SetupTPM(int recovery_mode, int developer_mode, - int disable_dev_request, int clear_tpm_owner_request, - RollbackSpaceFirmware* rsf) { +uint32_t SetupTPM(int developer_mode, int disable_dev_request, + int clear_tpm_owner_request, RollbackSpaceFirmware* rsf) { return TPM_SUCCESS; } @@ -35,7 +34,7 @@ uint32_t RollbackS3Resume(void) { } -uint32_t RollbackFirmwareSetup(int recovery_mode, int is_hw_dev, +uint32_t RollbackFirmwareSetup(int is_hw_dev, int disable_dev_request, int clear_tpm_owner_request, int *is_virt_dev, uint32_t *version) { diff --git a/firmware/lib/rollback_index.c b/firmware/lib/rollback_index.c index 619ba013..e372d9b6 100644 --- a/firmware/lib/rollback_index.c +++ b/firmware/lib/rollback_index.c @@ -341,9 +341,8 @@ uint32_t OneTimeInitializeTPM(RollbackSpaceFirmware *rsf, * to the TPM flashram at every reboot or wake-up, because of concerns about * the durability of the NVRAM. */ -uint32_t SetupTPM(int recovery_mode, int developer_mode, - int disable_dev_request, int clear_tpm_owner_request, - RollbackSpaceFirmware* rsf) +uint32_t SetupTPM(int developer_mode, int disable_dev_request, + int clear_tpm_owner_request, RollbackSpaceFirmware* rsf) { uint8_t in_flags; uint8_t disable; @@ -351,8 +350,6 @@ uint32_t SetupTPM(int recovery_mode, int developer_mode, uint32_t result; uint32_t versions; - VBDEBUG(("TPM: SetupTPM(r%d, d%d)\n", recovery_mode, developer_mode)); - RETURN_ON_FAILURE(TlclLibInit()); #ifdef TEGRA_SOFT_REBOOT_WORKAROUND @@ -494,7 +491,7 @@ uint32_t RollbackS3Resume(void) return TPM_SUCCESS; } -uint32_t RollbackFirmwareSetup(int recovery_mode, int is_hw_dev, +uint32_t RollbackFirmwareSetup(int is_hw_dev, int disable_dev_request, int clear_tpm_owner_request, int *is_virt_dev, uint32_t *version) @@ -556,7 +553,7 @@ uint32_t RollbackS3Resume(void) return result; } -uint32_t RollbackFirmwareSetup(int recovery_mode, int is_hw_dev, +uint32_t RollbackFirmwareSetup(int is_hw_dev, int disable_dev_request, int clear_tpm_owner_request, int *is_virt_dev, uint32_t *version) @@ -566,8 +563,7 @@ uint32_t RollbackFirmwareSetup(int recovery_mode, int is_hw_dev, /* Set version to 0 in case we fail */ *version = 0; - RETURN_ON_FAILURE(SetupTPM(recovery_mode, is_hw_dev, - disable_dev_request, + RETURN_ON_FAILURE(SetupTPM(is_hw_dev, disable_dev_request, clear_tpm_owner_request, &rsf)); Memcpy(version, &rsf.fw_versions, sizeof(*version)); *is_virt_dev = (rsf.flags & FLAG_VIRTUAL_DEV_MODE_ON) ? 1 : 0; diff --git a/firmware/lib/vboot_api_init.c b/firmware/lib/vboot_api_init.c index 20502971..5174aa1f 100644 --- a/firmware/lib/vboot_api_init.c +++ b/firmware/lib/vboot_api_init.c @@ -175,7 +175,9 @@ VbError_t VbInit(VbCommonParams *cparams, VbInitParams *iparams) * TPM space is initialized by this call, the virtual * dev-switch will be disabled by default) */ - tpm_status = RollbackFirmwareSetup(recovery, is_hw_dev, + VBDEBUG(("TPM: Call RollbackFirmwareSetup(r%d, d%d)\n", + recovery, is_hw_dev)); + tpm_status = RollbackFirmwareSetup(is_hw_dev, disable_dev_request, clear_tpm_owner_request, /* two outputs on success */ diff --git a/firmware/linktest/main.c b/firmware/linktest/main.c index abbc165b..bdf1483a 100644 --- a/firmware/linktest/main.c +++ b/firmware/linktest/main.c @@ -31,7 +31,7 @@ int main(void) /* rollback_index.h */ RollbackS3Resume(); - RollbackFirmwareSetup(0, 0, 0, 0, 0, 0); + RollbackFirmwareSetup(0, 0, 0, 0, 0); RollbackFirmwareWrite(0); RollbackFirmwareLock(); RollbackKernelRead(0); |