crypto: caam - don't emit ICV check failures to dmesg

ICV check failures are part of normal operation;
leave user notification up to the higher levels,
as is done in s/w algorithm implementations.

Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
Tested-by: Cristian Stoica <cristian.stoica@freescale.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/drivers/crypto/caam/error.c b/drivers/crypto/caam/error.c
index 66d73bf54166e2e3beb3a0fe7ad564949bfee831..33e41ea83fcca2aa566990d86c512433d7729ef8 100644 (file)
--- a/drivers/crypto/caam/error.c
+++ b/drivers/crypto/caam/error.c
@@ -151,10 +151,15 @@ static void report_ccb_status(struct device *jrdev, const u32 status,
        else
                snprintf(err_err_code, sizeof(err_err_code), "%02x", err_id);
 
-       dev_err(jrdev, "%08x: %s: %s %d: %s%s: %s%s\n",
-               status, error, idx_str, idx,
-               cha_str, cha_err_code,
-               err_str, err_err_code);
+       /*
+        * CCB ICV check failures are part of normal operation life;
+        * we leave the upper layers to do what they want with them.
+        */
+       if (err_id != JRSTA_CCBERR_ERRID_ICVCHK)
+               dev_err(jrdev, "%08x: %s: %s %d: %s%s: %s%s\n",
+                       status, error, idx_str, idx,
+                       cha_str, cha_err_code,
+                       err_str, err_err_code);
 }
 
 static void report_jump_status(struct device *jrdev, const u32 status,