scsi: megaraid_sas_fusion: Mark expected switch fall-through

GustavoARSilva · martinkpetersen · commit 74e716f2a9d8 · 2018-11-28T21:47:30.000-05:00
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where
we are expecting to fall through.

Notice that, in this particular case, I replaced "Fall thru" with a "Fall
through" annotation and added a dash as a token in order to separate the
"Fall through" annotation from the rest of the comment on the same line,
which is what GCC is expecting to find.

Signed-off-by: Gustavo A. R. Silva &lt;gustavo@embeddedor.com&gt;
Acked-by: Sumit Saxena &lt;sumit.saxena@broadcom.com&gt;
Signed-off-by: Martin K. Petersen &lt;martin.petersen@oracle.com&gt;
diff --git a/drivers/scsi/megaraid/megaraid_sas_fusion.c b/drivers/scsi/megaraid/megaraid_sas_fusion.c
@@ -3453,7 +3453,7 @@ complete_cmd_fusion(struct megasas_instance *instance, u32 MSIxIndex)
 				atomic_dec(&lbinfo->scsi_pending_cmds[cmd_fusion->pd_r1_lb]);
 				cmd_fusion->scmd->SCp.Status &= ~MEGASAS_LOAD_BALANCE_FLAG;
 			}
-			//Fall thru and complete IO
+			/* Fall through - and complete IO */
 		case MEGASAS_MPI2_FUNCTION_LD_IO_REQUEST: /* LD-IO Path */
 			atomic_dec(&instance->fw_outstanding);
 			if (cmd_fusion->r1_alt_dev_handle == MR_DEVHANDLE_INVALID) {

Original file line number	Diff line number	Diff line change
`@@ -3453,7 +3453,7 @@ complete_cmd_fusion(struct megasas_instance *instance, u32 MSIxIndex)`
`3453`	`3453`	`atomic_dec(&lbinfo->scsi_pending_cmds[cmd_fusion->pd_r1_lb]);`
`3454`	`3454`	`cmd_fusion->scmd->SCp.Status &= ~MEGASAS_LOAD_BALANCE_FLAG;`
`3455`	`3455`	`}`
`3456`		`- //Fall thru and complete IO`
	`3456`	`+ /* Fall through - and complete IO */`
`3457`	`3457`	`case MEGASAS_MPI2_FUNCTION_LD_IO_REQUEST: /* LD-IO Path */`
`3458`	`3458`	`atomic_dec(&instance->fw_outstanding);`
`3459`	`3459`	`if (cmd_fusion->r1_alt_dev_handle == MR_DEVHANDLE_INVALID) {`