diff options
| author |   Yihao Han <hanyihao@vivo.com> | 2022-04-08 07:23:49 -0700 |
|---|---|---|
| committer |   Herbert Xu <herbert@gondor.apana.org.au> | 2022-04-15 16:34:28 +0800 |
| commit | 10299073bc35ce051530fef318598b2f8b36e383 (patch) | |
| tree | 380a5566c330df563ab37a476e3269b08c5a1d08 /drivers/crypto/ux500/hash/hash_core.c | |
| parent | hwrng: mpfs - add polarfire soc hwrng support (diff) | |
| download | linux-dev-10299073bc35ce051530fef318598b2f8b36e383.tar.xz linux-dev-10299073bc35ce051530fef318598b2f8b36e383.zip | |
crypto: ux500/hash - simplify if-if to if-else
Replace `if (!req_ctx->updated)` with `else` for simplification
and add curly brackets according to the kernel coding style:
"Do not unnecessarily use braces where a single statement will do."
...
"This does not apply if only one branch of a conditional statement is
a single statement; in the latter case use braces in both branches"
Please refer to:
https://www.kernel.org/doc/html/v5.17-rc8/process/coding-style.html
Signed-off-by: Yihao Han <hanyihao@vivo.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'drivers/crypto/ux500/hash/hash_core.c')
| -rw-r--r-- | drivers/crypto/ux500/hash/hash_core.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/crypto/ux500/hash/hash_core.c b/drivers/crypto/ux500/hash/hash_core.c index 5157c118d642..265ef3e96fdd 100644 --- a/drivers/crypto/ux500/hash/hash_core.c +++ b/drivers/crypto/ux500/hash/hash_core.c @@ -877,9 +877,7 @@ static int hash_dma_final(struct ahash_request *req) __func__); goto out; } - } - - if (!req_ctx->updated) { + } else { ret = hash_setconfiguration(device_data, &ctx->config); if (ret) { dev_err(device_data->dev, |