Hi Tom,
> -----Original Message-----
> From: t...@redhat.com <t...@redhat.com>
> Sent: Sunday, November 1, 2020 8:08 PM
> To: Saurav Kashyap <skash...@marvell.com>; Javed Hasan
> <jha...@marvell.com>; j...@linux.ibm.com; martin.peter...@oracle.com
> Cc: GR-QLogic-Storage-Upstream <GR-QLogic-Storage-
> upstr...@marvell.com>; linux-s...@vger.kernel.org; linux-
> ker...@vger.kernel.org; Tom Rix <t...@redhat.com>
> Subject: [EXT] [PATCH] scsi: bnx2fc: remove unneeded semicolon
>
> External Email
>
> ----------------------------------------------------------------------
> From: Tom Rix <t...@redhat.com>
>
> A semicolon is not needed after a switch statement.
>
> Signed-off-by: Tom Rix <t...@redhat.com>
> ---
> drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
> b/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
> index 6890bbe04a8c..a436adb6092d 100644
> --- a/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
> +++ b/drivers/scsi/bnx2fc/bnx2fc_fcoe.c
> @@ -2275,7 +2275,7 @@ static int bnx2fc_ctlr_enabled(struct
> fcoe_ctlr_device *cdev)
> case FCOE_CTLR_UNUSED:
> default:
> return -ENOTSUPP;
> - };
> + }
> }
Thanks for a patch.
Reviewed-by: Saurav Kashyap <skash...@marvell.com>
Thanks,
~Saurav
>
> enum bnx2fc_create_link_state {
> --
> 2.18.1
