Commit c31b4ade authored by Colin Ian King's avatar Colin Ian King Committed by Herbert Xu

crypto: caam/qi2 - remove redundant assignment to ret

The variable ret is being assigned a value that is never read, the
error exit path via label 'unmap' returns -ENOMEM anyhow, so assigning
ret with -ENOMEM is redundamt.

Addresses-Coverity: ("Unused value")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 29195232
...@@ -4044,7 +4044,6 @@ static int ahash_finup_no_ctx(struct ahash_request *req) ...@@ -4044,7 +4044,6 @@ static int ahash_finup_no_ctx(struct ahash_request *req)
DMA_TO_DEVICE); DMA_TO_DEVICE);
if (dma_mapping_error(ctx->dev, edesc->qm_sg_dma)) { if (dma_mapping_error(ctx->dev, edesc->qm_sg_dma)) {
dev_err(ctx->dev, "unable to map S/G table\n"); dev_err(ctx->dev, "unable to map S/G table\n");
ret = -ENOMEM;
goto unmap; goto unmap;
} }
edesc->qm_sg_bytes = qm_sg_bytes; edesc->qm_sg_bytes = qm_sg_bytes;
...@@ -4055,7 +4054,6 @@ static int ahash_finup_no_ctx(struct ahash_request *req) ...@@ -4055,7 +4054,6 @@ static int ahash_finup_no_ctx(struct ahash_request *req)
if (dma_mapping_error(ctx->dev, state->ctx_dma)) { if (dma_mapping_error(ctx->dev, state->ctx_dma)) {
dev_err(ctx->dev, "unable to map ctx\n"); dev_err(ctx->dev, "unable to map ctx\n");
state->ctx_dma = 0; state->ctx_dma = 0;
ret = -ENOMEM;
goto unmap; goto unmap;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment