* [PATCH] Staging: rts5208: rtsx: Moved else statement to same line with else if
@ 2023-05-19 10:08 Deepanshu Kartikey
2023-05-19 10:08 ` [PATCH v2] " Deepanshu Kartikey
0 siblings, 1 reply; 3+ messages in thread
From: Deepanshu Kartikey @ 2023-05-19 10:08 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, Deepanshu Kartikey
Moved else statement to same line with else if,
this warning was given by checkpatch.pl
Signed-off-by: Deepanshu Kartikey <kartikey406@gmail.com>
...
Changes in V2:
- Moved else to the same line with else if
- Moved comment on else statement to inside of it
---
drivers/staging/rts5208/rtsx.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx.c b/drivers/staging/rts5208/rtsx.c
index 2284a96abcff..47dacef0aaf8 100644
--- a/drivers/staging/rts5208/rtsx.c
+++ b/drivers/staging/rts5208/rtsx.c
@@ -399,10 +399,8 @@ static int rtsx_control_thread(void *__dev)
chip->srb->device->id,
(u8)chip->srb->device->lun);
chip->srb->result = DID_BAD_TARGET << 16;
- }
-
- /* we've got a command, let's do it! */
- else {
+ } else {
+ /* we've got a command, let's do it! */
scsi_show_command(chip);
rtsx_invoke_transport(chip->srb, chip);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH v2] Staging: rts5208: rtsx: Moved else statement to same line with else if
2023-05-19 10:08 [PATCH] Staging: rts5208: rtsx: Moved else statement to same line with else if Deepanshu Kartikey
@ 2023-05-19 10:08 ` Deepanshu Kartikey
0 siblings, 0 replies; 3+ messages in thread
From: Deepanshu Kartikey @ 2023-05-19 10:08 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, Deepanshu Kartikey
Moved else statement to same line with else if,
this warning was given by checkpatch.pl
Signed-off-by: Deepanshu Kartikey <kartikey406@gmail.com>
...
Changes in V2:
- Moved else to the same line with else if
- Moved comment on else statement to inside of it
---
drivers/staging/rts5208/rtsx.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx.c b/drivers/staging/rts5208/rtsx.c
index 2284a96abcff..47dacef0aaf8 100644
--- a/drivers/staging/rts5208/rtsx.c
+++ b/drivers/staging/rts5208/rtsx.c
@@ -399,10 +399,8 @@ static int rtsx_control_thread(void *__dev)
chip->srb->device->id,
(u8)chip->srb->device->lun);
chip->srb->result = DID_BAD_TARGET << 16;
- }
-
- /* we've got a command, let's do it! */
- else {
+ } else {
+ /* we've got a command, let's do it! */
scsi_show_command(chip);
rtsx_invoke_transport(chip->srb, chip);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] Staging: rts5208: rtsx: Moved else statement to same line with else if
@ 2023-05-19 10:19 Deepanshu Kartikey
0 siblings, 0 replies; 3+ messages in thread
From: Deepanshu Kartikey @ 2023-05-19 10:19 UTC (permalink / raw)
To: gregkh; +Cc: linux-kernel, linux-staging, Deepanshu Kartikey
Moved else statement to same line with else if,
this warning was given by checkpatch.pl
Signed-off-by: Deepanshu Kartikey <kartikey406@gmail.com>
---
Changes in V2:
- Moved else to the same line with else if
- Moved comment on else statement to inside of it
---
drivers/staging/rts5208/rtsx.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx.c b/drivers/staging/rts5208/rtsx.c
index 2284a96abcff..47dacef0aaf8 100644
--- a/drivers/staging/rts5208/rtsx.c
+++ b/drivers/staging/rts5208/rtsx.c
@@ -399,10 +399,8 @@ static int rtsx_control_thread(void *__dev)
chip->srb->device->id,
(u8)chip->srb->device->lun);
chip->srb->result = DID_BAD_TARGET << 16;
- }
-
- /* we've got a command, let's do it! */
- else {
+ } else {
+ /* we've got a command, let's do it! */
scsi_show_command(chip);
rtsx_invoke_transport(chip->srb, chip);
}
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-05-19 10:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-19 10:08 [PATCH] Staging: rts5208: rtsx: Moved else statement to same line with else if Deepanshu Kartikey
2023-05-19 10:08 ` [PATCH v2] " Deepanshu Kartikey
-- strict thread matches above, loose matches on Subject: below --
2023-05-19 10:19 [PATCH] " Deepanshu Kartikey
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox