From 3e6c71ea2740a4a1c4c156e43e4fc81a2587ceb5 Mon Sep 17 00:00:00 2001 From: Vadim Bendebury Date: Thu, 20 Jul 2017 09:40:23 -0700 Subject: cr50: modify tpm_endorse() error reporting logic Using zero to represent success make it cleaner when processing return values of this function. Also, let's report different failures using different error values. Note that nobody checks this function's return value yet. BRANCH=cr50 BUG=b:63686091 TEST=none Change-Id: If25d30e637701f6c37f3ce75e5a5d5315e0e86e9 Signed-off-by: Vadim Bendebury Reviewed-on: https://chromium-review.googlesource.com/579583 Reviewed-by: Marius Schilder Reviewed-by: Nick Sanders --- include/tpm_manufacture.h | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) (limited to 'include/tpm_manufacture.h') diff --git a/include/tpm_manufacture.h b/include/tpm_manufacture.h index f43fd9ec13..f12db82bf9 100644 --- a/include/tpm_manufacture.h +++ b/include/tpm_manufacture.h @@ -13,6 +13,22 @@ /* Returns non-zero if the TPM manufacture steps have been completed. */ int tpm_manufactured(void); -int tpm_endorse(void); + +/* Codes for success and various manufacturing error conditions. */ +enum manufacturing_status { + mnf_success = 0, + mnf_no_certs = 1, + mnf_eps_decr = 2, + mnf_bad_rsa_size = 3, + mnf_bad_total_size = 4, + mnf_bad_rsa_type = 5, + mnf_bad_ecc_type = 6, + mnf_hmac_mismatch = 7, + mnf_rsa_proc = 8, + mnf_ecc_proc = 9, + mnf_store = 10, +}; + +enum manufacturing_status tpm_endorse(void); #endif /* __CROS_EC_TPM_MANUFACTURE_H */ -- cgit v1.2.1