Received-SPF: pass (google.com: domain of linux-kernel+bounces-76335-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Thu, 22 Feb 2024 10:29:51 +0000
From: Conor Dooley <conor@kernel.org>
To: Marco Felsch <m.felsch@pengutronix.de>
Cc: jic23@kernel.org, lars@metafoo.de, robh+dt@kernel.org,
	krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
	denis.ciocca@st.com, linus.walleij@linaro.org,
	linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, kernel@pengutronix.de
Subject: Re: [PATCH] dt-bindings: iio: st-sensors: Add IIS2MDC magnetometer
Message-ID: <20240222-fretted-riches-bf15f5d63018@spud>
References: <20240221175810.3581399-1-m.felsch@pengutronix.de>
 <20240221-undecided-union-4078db711693@spud>
 <20240221191644.5r3ylr5w3cnfnrzj@pengutronix.de>
 <20240221-imitate-molar-81d93285ac77@spud>
 <20240221194518.3sm4o5i274ldpvzf@pengutronix.de>
 <20240221-lubricant-machine-79054f117eb0@spud>
 <20240222084717.rcckoyjeh4shmcxr@pengutronix.de>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="ahPoXcnEvQcnarTl"
Content-Disposition: inline
In-Reply-To: <20240222084717.rcckoyjeh4shmcxr@pengutronix.de>


--ahPoXcnEvQcnarTl
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Feb 22, 2024 at 09:47:17AM +0100, Marco Felsch wrote:
> On 24-02-21, Conor Dooley wrote:
> > On Wed, Feb 21, 2024 at 08:45:18PM +0100, Marco Felsch wrote:
> > > On 24-02-21, Conor Dooley wrote:
> > > > On Wed, Feb 21, 2024 at 08:16:44PM +0100, Marco Felsch wrote:
> > > > > On 24-02-21, Conor Dooley wrote:
> > > > > > On Wed, Feb 21, 2024 at 06:58:10PM +0100, Marco Felsch wrote:

> > > > Besides, having fallback compatibles is the norm when one device ha=
s the
> > > > same programming model as another.
> > >=20
> > > Not for this binding according the driver.
> >=20
> > If they don't have the same programming model, then describing them as
> > "equivalent" wouldn't be correct. That said, they seem to use the same
> > sensor settings when alls said and done (see st_magn_sensors_settings),
> > so I think they are actually compatible even if the driver has separate
> > match data for each.
>=20
> I told you that I have checked the driver and skimed the datasheets and
> came to the exact same conclusion.

Did you tell me that? I did't see it.

Anyway, I think what happened is that I interpreted "Not for this
binding according the driver" as a comment about the programming model
being different, but you meant it in reference to the fallback. Since
they do have the same programming model the fallback is appropriate,
even if the driver knows about the iis2mdc compatible.
I'd squash in the following, similar to what was done for the
st,iis328dq that was added recently:

diff --git a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml b/Doc=
umentation/devicetree/bindings/iio/st,st-sensors.yaml
index ee593c8bbb65..ee6c3800436d 100644
--- a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
+++ b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml
@@ -64,7 +64,6 @@ properties:
           - st,lsm9ds0-gyro
       - description: STMicroelectronics Magnetometers
         enum:
-          - st,iis2mdc
           - st,lis2mdl
           - st,lis3mdl-magn
           - st,lsm303agr-magn
@@ -73,6 +72,9 @@ properties:
           - st,lsm303dlhc-magn
           - st,lsm303dlm-magn
           - st,lsm9ds1-magn
+      - items:
+          - const: st,iis2mdc
+          - const: st,lis2mdl
       - description: STMicroelectronics Pressure Sensors
         enum:
           - st,lps001wp-press

--ahPoXcnEvQcnarTl
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZdciHwAKCRB4tDGHoIJi
0rYLAP9zXNYPi9SZuObV9gQE3XitJbRNMbErzItFTaLr7pgDwgD+NA+JrD+s8dqv
AQAMzEvicv1YL6dcsEnWJJyJKjz4fgw=
=TXSE
-----END PGP SIGNATURE-----

--ahPoXcnEvQcnarTl--