2020-10-28 21:33:33

by Tom Rix

[permalink] [raw]
Subject: [PATCH] iio: light: apds9960: remove unneeded semicolon

From: Tom Rix <[email protected]>

A semicolon is not needed after a switch statement.

Signed-off-by: Tom Rix <[email protected]>
---
drivers/iio/light/apds9960.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
index 9afb3fcc74e6..547e7f9d6920 100644
--- a/drivers/iio/light/apds9960.c
+++ b/drivers/iio/light/apds9960.c
@@ -561,7 +561,7 @@ static int apds9960_write_raw(struct iio_dev *indio_dev,
}
default:
return -EINVAL;
- };
+ }

return 0;
}
--
2.18.1


2020-10-28 21:44:11

by Matt Ranostay

[permalink] [raw]
Subject: Re: [PATCH] iio: light: apds9960: remove unneeded semicolon

On Tue, Oct 27, 2020 at 1:14 PM <[email protected]> wrote:
>
> From: Tom Rix <[email protected]>
>
> A semicolon is not needed after a switch statement.
>

Acked-by: Matt Ranostay <[email protected]>

> Signed-off-by: Tom Rix <[email protected]>
> ---
> drivers/iio/light/apds9960.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
> index 9afb3fcc74e6..547e7f9d6920 100644
> --- a/drivers/iio/light/apds9960.c
> +++ b/drivers/iio/light/apds9960.c
> @@ -561,7 +561,7 @@ static int apds9960_write_raw(struct iio_dev *indio_dev,
> }
> default:
> return -EINVAL;
> - };
> + }
>
> return 0;
> }
> --
> 2.18.1
>

2020-10-29 14:56:41

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] iio: light: apds9960: remove unneeded semicolon

On Tue, 27 Oct 2020 21:36:17 -0700
Matt Ranostay <[email protected]> wrote:

> On Tue, Oct 27, 2020 at 1:14 PM <[email protected]> wrote:
> >
> > From: Tom Rix <[email protected]>
> >
> > A semicolon is not needed after a switch statement.
> >
>
> Acked-by: Matt Ranostay <[email protected]>
Applied to the togreg branch of iio.git. I'll push that out later as
testing for the autobuilders to poke at it.

Thanks,

Jonathan

>
> > Signed-off-by: Tom Rix <[email protected]>
> > ---
> > drivers/iio/light/apds9960.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
> > index 9afb3fcc74e6..547e7f9d6920 100644
> > --- a/drivers/iio/light/apds9960.c
> > +++ b/drivers/iio/light/apds9960.c
> > @@ -561,7 +561,7 @@ static int apds9960_write_raw(struct iio_dev *indio_dev,
> > }
> > default:
> > return -EINVAL;
> > - };
> > + }
> >
> > return 0;
> > }
> > --
> > 2.18.1
> >