Re: [PATCH 14/41] scsi: esas2r: esas2r_init: mark expected switch fall-throughs

From: Gustavo A. R. Silva
Date: Thu Jan 10 2019 - 15:09:10 EST


Hi,

Friendly ping (second one):

Who can ack/review/take this patch, please?

Thanks
--
Gustavo

On 12/19/18 9:39 AM, Gustavo A. R. Silva wrote:
Hi,

Friendly ping:

Who can ack or review this patch, please?

Thanks
--
Gustavo

On 11/27/18 10:28 PM, Gustavo A. R. Silva wrote:
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.

Signed-off-by: Gustavo A. R. Silva <gustavo@xxxxxxxxxxxxxx>
---
 drivers/scsi/esas2r/esas2r_init.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/scsi/esas2r/esas2r_init.c b/drivers/scsi/esas2r/esas2r_init.c
index 46b2c83ba21f..950cd92df2ff 100644
--- a/drivers/scsi/esas2r/esas2r_init.c
+++ b/drivers/scsi/esas2r/esas2r_init.c
@@ -1241,6 +1241,7 @@ static bool esas2r_format_init_msg(struct esas2r_adapter *a,
ÂÂÂÂÂÂÂÂÂÂÂÂÂ a->init_msg = ESAS2R_INIT_MSG_GET_INIT;
ÂÂÂÂÂÂÂÂÂÂÂÂÂ break;
ÂÂÂÂÂÂÂÂÂ }
+ÂÂÂÂÂÂÂ /* fall through */
ÂÂÂÂÂ case ESAS2R_INIT_MSG_GET_INIT:
ÂÂÂÂÂÂÂÂÂ if (msg == ESAS2R_INIT_MSG_GET_INIT) {
@@ -1254,7 +1255,7 @@ static bool esas2r_format_init_msg(struct esas2r_adapter *a,
ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ esas2r_hdebug("FAILED");
ÂÂÂÂÂÂÂÂÂÂÂÂÂ }
ÂÂÂÂÂÂÂÂÂ }
-ÂÂÂ /* fall through */
+ÂÂÂÂÂÂÂ /* fall through */
ÂÂÂÂÂ default:
ÂÂÂÂÂÂÂÂÂ rq->req_stat = RS_SUCCESS;