[PATCH 1/2] nvme-auth: remove unnecessary semicolon

Chaitanya Kulkarni kch at nvidia.com
Tue Jan 14 19:01:14 PST 2025


Remove the unnecessary semicolon to get rid of the warning:-

cocci warnings: (new ones prefixed by >>)
   drivers/nvme/host/auth.c:742:2-3: Unneeded semicolon
   drivers/nvme/host/auth.c:751:2-3: Unneeded semicolon

Also, use this opportunity to align the following dev_dbg function
parameters since they fit into one line.

Signed-off-by: Chaitanya Kulkarni <kch at nvidia.com>
---
 drivers/nvme/host/auth.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/nvme/host/auth.c b/drivers/nvme/host/auth.c
index 000c0a4197ba..896223e24850 100644
--- a/drivers/nvme/host/auth.c
+++ b/drivers/nvme/host/auth.c
@@ -739,9 +739,9 @@ static int nvme_auth_secure_concat(struct nvme_ctrl *ctrl,
 			 "%s: qid %d failed to generate digest, error %d\n",
 			 __func__, chap->qid, ret);
 		goto out_free_psk;
-	};
-	dev_dbg(ctrl->device, "%s: generated digest %s\n",
-		 __func__, digest);
+	}
+	dev_dbg(ctrl->device, "%s: generated digest %s\n", __func__, digest);
+
 	ret = nvme_auth_derive_tls_psk(chap->hash_id, psk, psk_len, digest, &tls_psk);
 	if (ret) {
 		dev_warn(ctrl->device,
-- 
2.40.0




More information about the Linux-nvme mailing list