From: Bean Huo <bean...@micron.com>

The memory allocated with devm_kzalloc() is freed automatically
no need to explicitly call devm_kfree.

Reviewed-by: Eric Biggers <ebigg...@google.com>
Signed-off-by: Bean Huo <bean...@micron.com>
---
 drivers/scsi/ufs/ufshcd-crypto.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/scsi/ufs/ufshcd-crypto.c b/drivers/scsi/ufs/ufshcd-crypto.c
index 07310b12a5dc..ec80ec83cf85 100644
--- a/drivers/scsi/ufs/ufshcd-crypto.c
+++ b/drivers/scsi/ufs/ufshcd-crypto.c
@@ -182,7 +182,7 @@ int ufshcd_hba_init_crypto_capabilities(struct ufs_hba *hba)
        err = blk_ksm_init(&hba->ksm,
                           hba->crypto_capabilities.config_count + 1);
        if (err)
-               goto out_free_caps;
+               goto out;
 
        hba->ksm.ksm_ll_ops = ufshcd_ksm_ops;
        /* UFS only supports 8 bytes for any DUN */
@@ -208,8 +208,6 @@ int ufshcd_hba_init_crypto_capabilities(struct ufs_hba *hba)
 
        return 0;
 
-out_free_caps:
-       devm_kfree(hba->dev, hba->crypto_cap_array);
 out:
        /* Indicate that init failed by clearing UFSHCD_CAP_CRYPTO */
        hba->caps &= ~UFSHCD_CAP_CRYPTO;
-- 
2.17.1

Reply via email to