[PATCH] scsi/arm: Clean up coding style violations
From: Darshan Rathod <hidden>
Date: 2025-11-07 12:33:31
Also in:
linux-scsi, lkml
Subsystem:
arm/riscpc architecture, scsi subsystem, the rest · Maintainers:
Russell King, "James E.J. Bottomley", "Martin K. Petersen", Linus Torvalds
Addressed checkpatch warnings by separating assignment from a conditional statement and documenting the empty for loop. These updates improve code clarity and maintain style consistency. No functional change intended. Signed-off-by: Darshan Rathod <redacted> --- drivers/scsi/arm/msgqueue.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/arm/msgqueue.c b/drivers/scsi/arm/msgqueue.c
index 58115831362f..7e09624e5505 100644
--- a/drivers/scsi/arm/msgqueue.c
+++ b/drivers/scsi/arm/msgqueue.c@@ -23,7 +23,8 @@ static struct msgqueue_entry *mqe_alloc(MsgQueue_t *msgq) { struct msgqueue_entry *mq; - if ((mq = msgq->free) != NULL) + mq = msgq->free; + if (mq) msgq->free = mq->next; return mq;
@@ -99,7 +100,8 @@ struct message *msgqueue_getmsg(MsgQueue_t *msgq, int msgno) { struct msgqueue_entry *mq; - for (mq = msgq->qe; mq && msgno; mq = mq->next, msgno--); + for (mq = msgq->qe; mq && msgno; mq = mq->next, msgno--) + ; /* intentional: iterate to the msgno-th entry */ return mq ? &mq->msg : NULL; }
--
2.43.0