2020-04-28 06:37:01

by Jason Yan

[permalink] [raw]
Subject: [PATCH] watchdog: riowd: remove unneeded semicolon

Fix the following coccicheck warning:

drivers/watchdog/riowd.c:144:2-3: Unneeded semicolon

Signed-off-by: Jason Yan <[email protected]>
---
drivers/watchdog/riowd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/watchdog/riowd.c b/drivers/watchdog/riowd.c
index dc3c06a92f93..1b9a6dc8f982 100644
--- a/drivers/watchdog/riowd.c
+++ b/drivers/watchdog/riowd.c
@@ -141,7 +141,7 @@ static long riowd_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)

default:
return -EINVAL;
- };
+ }

return 0;
}
--
2.21.1


2020-04-28 13:15:38

by Guenter Roeck

[permalink] [raw]
Subject: Re: [PATCH] watchdog: riowd: remove unneeded semicolon

On 4/27/20 11:33 PM, Jason Yan wrote:
> Fix the following coccicheck warning:
>
> drivers/watchdog/riowd.c:144:2-3: Unneeded semicolon
>
> Signed-off-by: Jason Yan <[email protected]>

Reviewed-by: Guenter Roeck <[email protected]>

> ---
> drivers/watchdog/riowd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/watchdog/riowd.c b/drivers/watchdog/riowd.c
> index dc3c06a92f93..1b9a6dc8f982 100644
> --- a/drivers/watchdog/riowd.c
> +++ b/drivers/watchdog/riowd.c
> @@ -141,7 +141,7 @@ static long riowd_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
>
> default:
> return -EINVAL;
> - };
> + }
>
> return 0;
> }
>