[PATCH] media: marvell: cafe: drop unneeded semicolon

Julia Lawall posted 1 patch an hour ago
drivers/media/platform/marvell/cafe-driver.c |    2 +-
drivers/media/platform/marvell/mcam-core.c   |    6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
[PATCH] media: marvell: cafe: drop unneeded semicolon
Posted by Julia Lawall an hour ago
When a function-like macro expands to an expression, that expression
doesn't need a semicolon after it.  All uses have been verified to
have their own semicolons.

This was found using the following Coccinelle semantic patch:

@r@
identifier i : script:ocaml() { String.lowercase_ascii i = i };
expression e;
@@

*#define i(...) e;

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>

---
 drivers/media/platform/marvell/cafe-driver.c |    2 +-
 drivers/media/platform/marvell/mcam-core.c   |    6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/media/platform/marvell/cafe-driver.c b/drivers/media/platform/marvell/cafe-driver.c
index 22034df6c..13d8faaab 100644
--- a/drivers/media/platform/marvell/cafe-driver.c
+++ b/drivers/media/platform/marvell/cafe-driver.c
@@ -127,7 +127,7 @@ struct cafe_camera {
  * Debugging and related.
  */
 #define cam_err(cam, fmt, arg...) \
-	dev_err(&(cam)->pdev->dev, fmt, ##arg);
+	dev_err(&(cam)->pdev->dev, fmt, ##arg)
 #define cam_warn(cam, fmt, arg...) \
 	dev_warn(&(cam)->pdev->dev, fmt, ##arg);
 
diff --git a/drivers/media/platform/marvell/mcam-core.c b/drivers/media/platform/marvell/mcam-core.c
index b8360d370..c1e8a2a30 100644
--- a/drivers/media/platform/marvell/mcam-core.c
+++ b/drivers/media/platform/marvell/mcam-core.c
@@ -224,11 +224,11 @@ static void mcam_buffer_done(struct mcam_camera *cam, int frame,
  * Debugging and related.
  */
 #define cam_err(cam, fmt, arg...) \
-	dev_err((cam)->dev, fmt, ##arg);
+	dev_err((cam)->dev, fmt, ##arg)
 #define cam_warn(cam, fmt, arg...) \
-	dev_warn((cam)->dev, fmt, ##arg);
+	dev_warn((cam)->dev, fmt, ##arg)
 #define cam_dbg(cam, fmt, arg...) \
-	dev_dbg((cam)->dev, fmt, ##arg);
+	dev_dbg((cam)->dev, fmt, ##arg)
 
 
 /*