lkml.org 
[lkml]   [2020]   [Nov]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [EXT] [PATCH] scsi: bnx2fc: remove unneeded semicolon
Date
Hi Tom,

> -----Original Message-----
> From: trix@redhat.com <trix@redhat.com>
> Sent: Sunday, November 1, 2020 8:08 PM
> To: Saurav Kashyap <skashyap@marvell.com>; Javed Hasan
> <jhasan@marvell.com>; jejb@linux.ibm.com; martin.petersen@oracle.com
> Cc: GR-QLogic-Storage-Upstream <GR-QLogic-Storage-
> Upstream@marvell.com>; linux-scsi@vger.kernel.org; linux-
> kernel@vger.kernel.org; Tom Rix <trix@redhat.com>
> Subject: [EXT] [PATCH] scsi: bnx2fc: remove unneeded semicolon
>
> External Email
>
> ----------------------------------------------------------------------
> From: Tom Rix <trix@redhat.com>
>
> A semicolon is not needed after a switch statement.
>
> Signed-off-by: Tom Rix <trix@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 <skashyap@marvell.com>

Thanks,
~Saurav
>
> enum bnx2fc_create_link_state {
> --
> 2.18.1

\
 
 \ /
  Last update: 2020-11-02 05:44    [W:0.079 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site