Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 23 Jul 2023 13:50:35 +0100
From:   Jonathan Cameron <jic23@kernel.org>
To:     Lorenzo Bianconi <lorenzo@kernel.org>
Cc:     Alejandro Tafalla <atafalla@dnyon.com>,
        Lars-Peter Clausen <lars@metafoo.de>,
        Jonathan Cameron <Jonathan.Cameron@huawei.com>,
        linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] iio: imu: lsm6dsx: Fix mount matrix retrieval
Message-ID: <20230723135035.4a2f0909@jic23-huawei>
In-Reply-To: <ZLmm2fSt1wkttdLC@lore-desk>
References: <20230714153132.27265-1-atafalla@dnyon.com>
        <20230715183515.56deaa1d@jic23-huawei>
        <3017278.mvXUDI8C0e@alexpc>
        <20230720194037.60d109d7@jic23-huawei>
        <ZLmm2fSt1wkttdLC@lore-desk>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Thu, 20 Jul 2023 23:27:53 +0200
Lorenzo Bianconi <lorenzo@kernel.org> wrote:

> On Jul 20, Jonathan Cameron wrote:
> > Alejandro Tafalla <atafalla@dnyon.com> wrote:
> >  =20
> > > On s=C3=A1bado, 15 de julio de 2023 19:35:15 (CEST) Jonathan Cameron =
wrote: =20
> > > > On Fri, 14 Jul 2023 17:31:26 +0200
> > > >=20
> > > > Alejandro Tafalla <atafalla@dnyon.com> wrote:   =20
> > > > > The function lsm6dsx_get_acpi_mount_matrix should return an error=
 when
> > > > > ACPI
> > > > > support is not enabled to allow executing iio_read_mount_matrix i=
n the
> > > > > probe function.
> > > > >=20
> > > > > Fixes: dc3d25f22b88 ("iio: imu: lsm6dsx: Add ACPI mount matrix re=
trieval")   =20
> > > >=20
> > > > I can fix it up whilst applying if no other issues, but there must =
not be
> > > > a blank line here.  All tags need to be in a single block for some =
tooling
> > > > that is used with the kernel tree (and some of the checking scripts=
 warn
> > > > about this so it won't get applied with the blank line here).   =20
> > >=20
> > > Okay, i'll keep it in mind for new patches. Thank you. =20
> > Np.
> >=20
> > Just waiting for Lorenzo to have time to take a final look. =20
>=20
> I am fine with this patch.
>=20
> Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>
>=20
Ah. My filters broke so I'd lost this.  Thankfully b4 picked it up and made
me go look for where the missing emails had gone.  All recovered now
and the ack picked up.

Thanks,

Jonathan

> >=20
> > Jonathan
> >  =20
> > > >    =20
> > > > > Signed-off-by: Alejandro Tafalla <atafalla@dnyon.com>
> > > > > ---
> > > > > Changes in v3:
> > > > > - Removed unneeded check for err =3D=3D -EOPNOTSUPP.
> > > > >=20
> > > > > Changes in v2:
> > > > > - Use of error codes instead of true/false
> > > > >=20
> > > > >  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c | 2 +-
> > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >=20
> > > > > diff --git a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> > > > > b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c index
> > > > > 6a18b363cf73..b6e6b1df8a61 100644
> > > > > --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> > > > > +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c
> > > > > @@ -2687,7 +2687,7 @@ static int lsm6dsx_get_acpi_mount_matrix(st=
ruct
> > > > > device *dev,>=20
> > > > >  static int lsm6dsx_get_acpi_mount_matrix(struct device *dev,
> > > > > =20
> > > > >  					  struct    =20
> > > iio_mount_matrix *orientation) =20
> > > > > =20
> > > > >  {
> > > > >=20
> > > > > -	return false;
> > > > > +	return -EOPNOTSUPP;
> > > > >=20
> > > > >  }
> > > > > =20
> > > > >  #endif   =20
> > >=20
> > >=20
> > >=20
> > >  =20
> >  =20