diff mbox series

pwm: lpc18xx-sct: remove unneeded semicolon

Message ID 1612320605-6430-1-git-send-email-yang.lee@linux.alibaba.com
State Accepted
Headers show
Series pwm: lpc18xx-sct: remove unneeded semicolon | expand

Commit Message

Yang Li Feb. 3, 2021, 2:50 a.m. UTC
Eliminate the following coccicheck warning:
./drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/pwm/pwm-lpc18xx-sct.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Uwe Kleine-König Feb. 3, 2021, 7:43 a.m. UTC | #1
On Wed, Feb 03, 2021 at 10:50:05AM +0800, Yang Li wrote:
> Eliminate the following coccicheck warning:
> ./drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>

Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

But this is prior art:
https://lore.kernel.org/r/20210128134537.GA54687@068c7b848bbb

This patch is better though as it doesn't have a bogus Fixes: line.

*shrug*, I let Thierry decide which one to pick up.

Best regards
Uwe
Vladimir Zapolskiy Feb. 3, 2021, 5:52 p.m. UTC | #2
On 2/3/21 4:50 AM, Yang Li wrote:
> Eliminate the following coccicheck warning:
> ./drivers/pwm/pwm-lpc18xx-sct.c:292:2-3: Unneeded semicolon
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
> ---

Acked-by: Vladimir Zapolskiy <vz@mleia.com>

--
Best wishes,
Vladimir
diff mbox series

Patch

diff --git a/drivers/pwm/pwm-lpc18xx-sct.c b/drivers/pwm/pwm-lpc18xx-sct.c
index dc5133b..7ef4024 100644
--- a/drivers/pwm/pwm-lpc18xx-sct.c
+++ b/drivers/pwm/pwm-lpc18xx-sct.c
@@ -289,7 +289,7 @@  static int lpc18xx_pwm_request(struct pwm_chip *chip, struct pwm_device *pwm)
 		dev_err(lpc18xx_pwm->dev,
 			"maximum number of simultaneous channels reached\n");
 		return -EBUSY;
-	};
+	}
 
 	set_bit(event, &lpc18xx_pwm->event_map);
 	lpc18xx_data->duty_event = event;