Received-SPF: pass (google.com: domain of linux-kernel+bounces-104367-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <717ad48efa7ffc6cc1960be05558e9cbf0b6c4c8.camel@gmail.com>
Subject: Re: [PATCH][next] iio: accel: adxl367: Remove second semicolon
From: Nuno =?ISO-8859-1?Q?S=E1?= <noname.nuno@gmail.com>
To: 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>, Jonathan Cameron <jic23@kernel.org>,
  linux-iio@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
Date: Fri, 15 Mar 2024 12:51:07 +0100
In-Reply-To: <20240315091436.2430227-1-colin.i.king@gmail.com>
References: <20240315091436.2430227-1-colin.i.king@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.50.4 
Precedence: bulk
MIME-Version: 1.0

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>
> ---

Reviewed-by: Nuno Sa <nuno.sa@analog.com>

> =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 *st,=
 enum
> adxl367_odr odr)
> =C2=A0static int adxl367_set_odr(struct iio_dev *indio_dev, enum adxl367_=
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);