summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorThorsten Blum <thorsten.blum@linux.dev>2026-07-06 17:04:07 +0200
committerHerbert Xu <herbert@gondor.apana.org.au>2026-07-17 18:09:21 +1000
commitae69d42541633bdf5edef0ab02acd208e482ec62 (patch)
treec5a9b67bfaddd403e96e4937dc6c21d43335f4da /drivers
parentea4a50c0b5bb2d3b0c98b900b753dec9ad6fa457 (diff)
downloadlinux-next-ae69d42541633bdf5edef0ab02acd208e482ec62.tar.gz
linux-next-ae69d42541633bdf5edef0ab02acd208e482ec62.zip
crypto: atmel-tdes - drop redundant return variable in crypt_pdc_stop
In atmel_tdes_crypt_pdc_stop(), remove the redundant return variable and return the error directly. Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/crypto/atmel-tdes.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/crypto/atmel-tdes.c b/drivers/crypto/atmel-tdes.c
index 32f0acae46e3..00a0b9694a26 100644
--- a/drivers/crypto/atmel-tdes.c
+++ b/drivers/crypto/atmel-tdes.c
@@ -285,7 +285,6 @@ static int atmel_tdes_write_ctrl(struct atmel_tdes_dev *dd)
static int atmel_tdes_crypt_pdc_stop(struct atmel_tdes_dev *dd)
{
- int err = 0;
size_t count;
atmel_tdes_write(dd, TDES_PTCR, TDES_PTCR_TXTDIS|TDES_PTCR_RXTDIS);
@@ -297,16 +296,15 @@ static int atmel_tdes_crypt_pdc_stop(struct atmel_tdes_dev *dd)
dma_sync_single_for_cpu(dd->dev, dd->dma_addr_out,
dd->dma_size, DMA_FROM_DEVICE);
- /* copy data */
count = atmel_tdes_sg_copy(&dd->out_sg, &dd->out_offset,
dd->buf_out, dd->buflen, dd->dma_size, 1);
if (count != dd->dma_size) {
- err = -EINVAL;
dev_dbg(dd->dev, "not all data converted: %zu\n", count);
+ return -EINVAL;
}
}
- return err;
+ return 0;
}
static int atmel_tdes_buff_init(struct atmel_tdes_dev *dd)