Received-SPF: pass (google.com: domain of linux-kernel+bounces-105197-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Sat, 16 Mar 2024 14:06:16 +0000
From: Jonathan Cameron <jic23@kernel.org>
To: Nuno =?UTF-8?B?U8Oh?= <noname.nuno@gmail.com>
Cc: Colin Ian King <colin.i.king@gmail.com>, Lars-Peter Clausen
 <lars@metafoo.de>, Michael Hennerich <Michael.Hennerich@analog.com>, Cosmin
 Tanislav <cosmin.tanislav@analog.com>, linux-iio@vger.kernel.org,
 kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH][next] iio: accel: adxl367: Remove second semicolon
Message-ID: <20240316140616.09301073@jic23-huawei>
In-Reply-To: <717ad48efa7ffc6cc1960be05558e9cbf0b6c4c8.camel@gmail.com>
References: <20240315091436.2430227-1-colin.i.king@gmail.com>
	<717ad48efa7ffc6cc1960be05558e9cbf0b6c4c8.camel@gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Fri, 15 Mar 2024 12:51:07 +0100
Nuno S=C3=A1 <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.
> >=20
> > Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> > --- =20
>=20
> 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


>=20
> > =C2=A0drivers/iio/accel/adxl367.c | 2 +-
> > =C2=A01 file changed, 1 insertion(+), 1 deletion(-)
> >=20
> > 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 *s=
t, enum
> > adxl367_odr odr)
> > =C2=A0static int adxl367_set_odr(struct iio_dev *indio_dev, enum adxl36=
7_odr odr)
> > =C2=A0{
> > =C2=A0	iio_device_claim_direct_scoped(return -EBUSY, indio_dev) {
> > -		struct adxl367_state *st =3D iio_priv(indio_dev);;
> > +		struct adxl367_state *st =3D iio_priv(indio_dev);
> > =C2=A0		int ret;
> > =C2=A0
> > =C2=A0		guard(mutex)(&st->lock); =20
>=20