crypto: ccree - account for TEE not ready to report
authorGilad Ben-Yossef <gilad@benyossef.com>
Tue, 2 Jul 2019 11:39:19 +0000 (14:39 +0300)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 26 Jul 2019 04:51:55 +0000 (14:51 +1000)
When ccree driver runs it checks the state of the Trusted Execution
Environment CryptoCell driver before proceeding. We did not account
for cases where the TEE side is not ready or not available at all.
Fix it by only considering TEE error state after sync with the TEE
side driver.

Signed-off-by: Gilad Ben-Yossef <gilad@benyossef.com>
Fixes: ab8ec9658f5a ("crypto: ccree - add FIPS support")
CC: stable@vger.kernel.org # v4.17+
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/ccree/cc_fips.c

index 5ad3ffb7acaa8973a9f9fcc09c87c6eb8878f339..040e09c0e1af609c966d3d167abcba8e7f93b463 100644 (file)
@@ -21,7 +21,13 @@ static bool cc_get_tee_fips_status(struct cc_drvdata *drvdata)
        u32 reg;
 
        reg = cc_ioread(drvdata, CC_REG(GPR_HOST));
-       return (reg == (CC_FIPS_SYNC_TEE_STATUS | CC_FIPS_SYNC_MODULE_OK));
+       /* Did the TEE report status? */
+       if (reg & CC_FIPS_SYNC_TEE_STATUS)
+               /* Yes. Is it OK? */
+               return (reg & CC_FIPS_SYNC_MODULE_OK);
+
+       /* No. It's either not in use or will be reported later */
+       return true;
 }
 
 /*