| Message ID | 20240315091436.2430227-1-colin.i.king@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [next] iio: accel: adxl367: Remove second semicolon | expand |
On Fri, 2024-03-15 at 09:14 +0000, Colin Ian King wrote: > There is a statement with two semicolons. Remove the second one, it > is redundant. > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> > --- Reviewed-by: Nuno Sa <nuno.sa@analog.com> > drivers/iio/accel/adxl367.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/accel/adxl367.c b/drivers/iio/accel/adxl367.c > index 210228affb80..5cf4828a5eb5 100644 > --- a/drivers/iio/accel/adxl367.c > +++ b/drivers/iio/accel/adxl367.c > @@ -621,7 +621,7 @@ static int _adxl367_set_odr(struct adxl367_state *st, enum > adxl367_odr odr) > static int adxl367_set_odr(struct iio_dev *indio_dev, enum adxl367_odr odr) > { > iio_device_claim_direct_scoped(return -EBUSY, indio_dev) { > - struct adxl367_state *st = iio_priv(indio_dev);; > + struct adxl367_state *st = iio_priv(indio_dev); > int ret; > > guard(mutex)(&st->lock);
On Fri, 15 Mar 2024 12:51:07 +0100 Nuno Sá <noname.nuno@gmail.com> wrote: > On Fri, 2024-03-15 at 09:14 +0000, Colin Ian King wrote: > > There is a statement with two semicolons. Remove the second one, it > > is redundant. > > > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> > > --- > > Reviewed-by: Nuno Sa <nuno.sa@analog.com> Thanks Colin. oops. Applied to the togreg-normal branch of iio.git, but note I'll not rush this in, so will be the 6.10 merge window now. Jonathan > > > drivers/iio/accel/adxl367.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/iio/accel/adxl367.c b/drivers/iio/accel/adxl367.c > > index 210228affb80..5cf4828a5eb5 100644 > > --- a/drivers/iio/accel/adxl367.c > > +++ b/drivers/iio/accel/adxl367.c > > @@ -621,7 +621,7 @@ static int _adxl367_set_odr(struct adxl367_state *st, enum > > adxl367_odr odr) > > static int adxl367_set_odr(struct iio_dev *indio_dev, enum adxl367_odr odr) > > { > > iio_device_claim_direct_scoped(return -EBUSY, indio_dev) { > > - struct adxl367_state *st = iio_priv(indio_dev);; > > + struct adxl367_state *st = iio_priv(indio_dev); > > int ret; > > > > guard(mutex)(&st->lock); >
diff --git a/drivers/iio/accel/adxl367.c b/drivers/iio/accel/adxl367.c index 210228affb80..5cf4828a5eb5 100644 --- a/drivers/iio/accel/adxl367.c +++ b/drivers/iio/accel/adxl367.c @@ -621,7 +621,7 @@ static int _adxl367_set_odr(struct adxl367_state *st, enum adxl367_odr odr) static int adxl367_set_odr(struct iio_dev *indio_dev, enum adxl367_odr odr) { iio_device_claim_direct_scoped(return -EBUSY, indio_dev) { - struct adxl367_state *st = iio_priv(indio_dev);; + struct adxl367_state *st = iio_priv(indio_dev); int ret; guard(mutex)(&st->lock);
There is a statement with two semicolons. Remove the second one, it is redundant. Signed-off-by: Colin Ian King <colin.i.king@gmail.com> --- drivers/iio/accel/adxl367.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)