On Mon Jun 29 11:30:42 2026 +0000, Ricardo Ribalda wrote:
> There is no need to add a semicolon after a switch statement.
> 
> It also makes cocci a bit uneasy. It triggers the following warnings:
> 
> ./usb/em28xx/em28xx-cards.c:4085:2-3: Unneeded semicolon
> ./usb/em28xx/em28xx-core.c:635:2-3: Unneeded semicolon
> 
> Signed-off-by: Ricardo Ribalda <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/usb/em28xx/em28xx-cards.c | 2 +-
 drivers/media/usb/em28xx/em28xx-core.c  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

---

diff --git a/drivers/media/usb/em28xx/em28xx-cards.c 
b/drivers/media/usb/em28xx/em28xx-cards.c
index e3c0f5e491e3..c71cce963850 100644
--- a/drivers/media/usb/em28xx/em28xx-cards.c
+++ b/drivers/media/usb/em28xx/em28xx-cards.c
@@ -4111,7 +4111,7 @@ static void em28xx_check_usb_descriptor(struct em28xx 
*dev,
                        dev->analog_ep_bulk = e->bEndpointAddress;
                }
                return;
-       };
+       }
 }
 
 /*
diff --git a/drivers/media/usb/em28xx/em28xx-core.c 
b/drivers/media/usb/em28xx/em28xx-core.c
index 5bbb082dbed9..d4197e37f637 100644
--- a/drivers/media/usb/em28xx/em28xx-core.c
+++ b/drivers/media/usb/em28xx/em28xx-core.c
@@ -632,7 +632,7 @@ void em2828X_decoder_vmux(struct em28xx *dev, unsigned int 
vin)
        default:
                dev_dbg(&dev->intf->dev, "EM2828X_SVIDEO\n");
                break;
-       };
+       }
 
        em28xx_write_reg(dev, 0x24, 0x00);
        em28xx_write_reg(dev, 0x25, 0x02);
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to