linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] scsi: fcoe: remove unneeded semicolon
@ 2020-11-01 14:40 trix
  2020-11-05  3:05 ` Martin K. Petersen
  2020-11-11  2:58 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: trix @ 2020-11-01 14:40 UTC (permalink / raw)
  To: hare, jejb, martin.petersen; +Cc: linux-scsi, linux-kernel, Tom Rix

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/fcoe/fcoe.c       | 2 +-
 drivers/scsi/fcoe/fcoe_sysfs.c | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/scsi/fcoe/fcoe.c b/drivers/scsi/fcoe/fcoe.c
index a4be6f439c47..03bf49adaafe 100644
--- a/drivers/scsi/fcoe/fcoe.c
+++ b/drivers/scsi/fcoe/fcoe.c
@@ -2023,7 +2023,7 @@ static int fcoe_ctlr_enabled(struct fcoe_ctlr_device *cdev)
 	case FCOE_CTLR_UNUSED:
 	default:
 		return -ENOTSUPP;
-	};
+	}
 }
 
 /**
diff --git a/drivers/scsi/fcoe/fcoe_sysfs.c b/drivers/scsi/fcoe/fcoe_sysfs.c
index ffef2c8eddc6..af658aa38fed 100644
--- a/drivers/scsi/fcoe/fcoe_sysfs.c
+++ b/drivers/scsi/fcoe/fcoe_sysfs.c
@@ -312,7 +312,7 @@ static ssize_t store_ctlr_mode(struct device *dev,
 	default:
 		LIBFCOE_SYSFS_DBG(ctlr, "Mode change not supported.\n");
 		return -ENOTSUPP;
-	};
+	}
 }
 
 static FCOE_DEVICE_ATTR(ctlr, mode, S_IRUGO | S_IWUSR,
@@ -346,7 +346,7 @@ static ssize_t store_ctlr_enabled(struct device *dev,
 		break;
 	case FCOE_CTLR_UNUSED:
 		return -ENOTSUPP;
-	};
+	}
 
 	rc = ctlr->f->set_fcoe_ctlr_enabled(ctlr);
 	if (rc)
-- 
2.18.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] scsi: fcoe: remove unneeded semicolon
  2020-11-01 14:40 [PATCH] scsi: fcoe: remove unneeded semicolon trix
@ 2020-11-05  3:05 ` Martin K. Petersen
  2020-11-11  2:58 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2020-11-05  3:05 UTC (permalink / raw)
  To: trix; +Cc: hare, jejb, martin.petersen, linux-scsi, linux-kernel


Tom,

> A semicolon is not needed after a switch statement.

Applied to 5.11/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] scsi: fcoe: remove unneeded semicolon
  2020-11-01 14:40 [PATCH] scsi: fcoe: remove unneeded semicolon trix
  2020-11-05  3:05 ` Martin K. Petersen
@ 2020-11-11  2:58 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2020-11-11  2:58 UTC (permalink / raw)
  To: trix, jejb, hare; +Cc: Martin K . Petersen, linux-kernel, linux-scsi

On Sun, 1 Nov 2020 06:40:17 -0800, trix@redhat.com wrote:

> A semicolon is not needed after a switch statement.

Applied to 5.11/scsi-queue, thanks!

[1/1] scsi: fcoe: Remove unneeded semicolon
      https://git.kernel.org/mkp/scsi/c/00c00807a110

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-11-11  2:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-01 14:40 [PATCH] scsi: fcoe: remove unneeded semicolon trix
2020-11-05  3:05 ` Martin K. Petersen
2020-11-11  2:58 ` Martin K. Petersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).