| Message ID | 20241203-crypto-qce-refactor-v1-0-c5901d2dd45c@linaro.org |
|---|---|
| Headers | show |
| Series | crypto: qce - refactor the driver | expand |
On 03/12/2024 10:19, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > > If qce_check_version() fails, we should jump to err_dma as we already > called qce_dma_request() a couple lines before. > > Cc: stable@vger.kernel.org > Fixes: ec8f5d8f6f76 ("crypto: qce - Qualcomm crypto engine driver") > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- > drivers/crypto/qce/core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/crypto/qce/core.c b/drivers/crypto/qce/core.c > index e228a31fe28dc..58ea93220f015 100644 > --- a/drivers/crypto/qce/core.c > +++ b/drivers/crypto/qce/core.c > @@ -247,7 +247,7 @@ static int qce_crypto_probe(struct platform_device *pdev) > > ret = qce_check_version(qce); > if (ret) > - goto err_clks; > + goto err_dma; > > spin_lock_init(&qce->lock); > tasklet_init(&qce->done_tasklet, qce_tasklet_req_done, > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
On Tue, Dec 03, 2024 at 10:19:28AM +0100, Bartosz Golaszewski wrote: > This driver will soon be getting more features so show it some > refactoring love in the meantime. Switching to using a workqueue and > sleeping locks improves cryptsetup benchmark results for AES encryption. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- > Bartosz Golaszewski (9): > crypto: qce - fix goto jump in error path > crypto: qce - unregister previously registered algos in error path > crypto: qce - remove unneeded call to icc_set_bw() in error path > crypto: qce - shrink code with devres clk helpers > crypto: qce - convert qce_dma_request() to use devres > crypto: qce - make qce_register_algs() a managed interface > crypto: qce - use __free() for a buffer that's always freed > crypto: qce - convert tasklet to workqueue > crypto: qce - switch to using a mutex > > drivers/crypto/qce/core.c | 131 ++++++++++++++++------------------------------ > drivers/crypto/qce/core.h | 9 ++-- > drivers/crypto/qce/dma.c | 22 ++++---- > drivers/crypto/qce/dma.h | 3 +- > drivers/crypto/qce/sha.c | 6 +-- > 5 files changed, 68 insertions(+), 103 deletions(-) > --- > base-commit: f486c8aa16b8172f63bddc70116a0c897a7f3f02 > change-id: 20241128-crypto-qce-refactor-ab58869eec34 All applied. Thanks.
This driver will soon be getting more features so show it some refactoring love in the meantime. Switching to using a workqueue and sleeping locks improves cryptsetup benchmark results for AES encryption. Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> --- Bartosz Golaszewski (9): crypto: qce - fix goto jump in error path crypto: qce - unregister previously registered algos in error path crypto: qce - remove unneeded call to icc_set_bw() in error path crypto: qce - shrink code with devres clk helpers crypto: qce - convert qce_dma_request() to use devres crypto: qce - make qce_register_algs() a managed interface crypto: qce - use __free() for a buffer that's always freed crypto: qce - convert tasklet to workqueue crypto: qce - switch to using a mutex drivers/crypto/qce/core.c | 131 ++++++++++++++++------------------------------ drivers/crypto/qce/core.h | 9 ++-- drivers/crypto/qce/dma.c | 22 ++++---- drivers/crypto/qce/dma.h | 3 +- drivers/crypto/qce/sha.c | 6 +-- 5 files changed, 68 insertions(+), 103 deletions(-) --- base-commit: f486c8aa16b8172f63bddc70116a0c897a7f3f02 change-id: 20241128-crypto-qce-refactor-ab58869eec34 Best regards,