| Message ID | 20201027201436.1597726-1-trix@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | iio: light: apds9960: remove unneeded semicolon | expand |
On Tue, Oct 27, 2020 at 1:14 PM <trix@redhat.com> wrote: > > From: Tom Rix <trix@redhat.com> > > A semicolon is not needed after a switch statement. > Acked-by: Matt Ranostay <matt.ranostay@konsulko.com> > Signed-off-by: Tom Rix <trix@redhat.com> > --- > 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 >
On Tue, 27 Oct 2020 21:36:17 -0700 Matt Ranostay <matt.ranostay@konsulko.com> wrote: > On Tue, Oct 27, 2020 at 1:14 PM <trix@redhat.com> wrote: > > > > From: Tom Rix <trix@redhat.com> > > > > A semicolon is not needed after a switch statement. > > > > Acked-by: Matt Ranostay <matt.ranostay@konsulko.com> 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 <trix@redhat.com> > > --- > > 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 > >
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; }