On Sat Aug 1 21:09:53 2026 +0200, Julia Lawall wrote:
> 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 <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
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 22034df6cba9..13d8faaabb50 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 b8360d37000a..c1e8a2a3012d 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)
/*
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
