diff options
| author |   Antoine Tenart <antoine.tenart@bootlin.com> | 2019-05-27 16:50:56 +0200 |
|---|---|---|
| committer |   Herbert Xu <herbert@gondor.apana.org.au> | 2019-06-06 14:38:55 +0800 |
| commit | 709ecc101aea53d3625b688697b7c9dc4235716d (patch) | |
| tree | 3beea7162550642db8f96e43a4725edcf385d2f2 /drivers | |
| parent | crypto: inside-secure - fix coding style for a condition (diff) | |
| download | linux-dev-709ecc101aea53d3625b688697b7c9dc4235716d.tar.xz linux-dev-709ecc101aea53d3625b688697b7c9dc4235716d.zip | |
crypto: inside-secure - remove useless check
When sending an ahash request, the code checks for the extra variable
not to be 0. This check is useless as the extra variable can't be 0 at
this point (it is checked on the line just before).
This patch does not modify the driver behaviour in any way.
Signed-off-by: Antoine Tenart <antoine.tenart@bootlin.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/crypto/inside-secure/safexcel_hash.c | 24 |
1 files changed, 11 insertions, 13 deletions
diff --git a/drivers/crypto/inside-secure/safexcel_hash.c b/drivers/crypto/inside-secure/safexcel_hash.c index a7cee9ed3789..58ce480690eb 100644 --- a/drivers/crypto/inside-secure/safexcel_hash.c +++ b/drivers/crypto/inside-secure/safexcel_hash.c @@ -230,19 +230,17 @@ static int safexcel_ahash_send_req(struct crypto_async_request *async, int ring, if (!extra) extra = crypto_ahash_blocksize(ahash); - if (extra) { - sg_pcopy_to_buffer(areq->src, sg_nents(areq->src), - req->cache_next, extra, - areq->nbytes - extra); - - queued -= extra; - len -= extra; - - if (!queued) { - *commands = 0; - *results = 0; - return 0; - } + sg_pcopy_to_buffer(areq->src, sg_nents(areq->src), + req->cache_next, extra, + areq->nbytes - extra); + + queued -= extra; + len -= extra; + + if (!queued) { + *commands = 0; + *results = 0; + return 0; } } |