diff mbox series

[1/1] crypto/fsl: unused value in caam_hash_update()

Message ID 20200627081355.12334-1-xypron.glpk@gmx.de
State Awaiting Upstream
Delegated to: Priyanka Jain
Headers show
Series [1/1] crypto/fsl: unused value in caam_hash_update() | expand

Commit Message

Heinrich Schuchardt June 27, 2020, 8:13 a.m. UTC
The value 0 assigned to final is overwritten before ever being used.

Remove the assignment.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 drivers/crypto/fsl/fsl_hash.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.20.1

Comments

Simon Glass June 29, 2020, 5:26 p.m. UTC | #1
On Sat, 27 Jun 2020 at 02:14, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> The value 0 assigned to final is overwritten before ever being used.
>
> Remove the assignment.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  drivers/crypto/fsl/fsl_hash.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox series

Patch

diff --git a/drivers/crypto/fsl/fsl_hash.c b/drivers/crypto/fsl/fsl_hash.c
index 953deec9ff..61f953e8a6 100644
--- a/drivers/crypto/fsl/fsl_hash.c
+++ b/drivers/crypto/fsl/fsl_hash.c
@@ -86,7 +86,7 @@  static int caam_hash_update(void *hash_ctx, const void *buf,
 			    unsigned int size, int is_last,
 			    enum caam_hash_algos caam_algo)
 {
-	uint32_t final = 0;
+	uint32_t final;
 	phys_addr_t addr = virt_to_phys((void *)buf);
 	struct sha_ctx *ctx = hash_ctx;