* [PATCH 2/2] dt-bindings: iio: imu: mpu6050: Add icm20600 bindings to mpu6050 [not found] <20230421071630.524822-1-chenhuiz@axis.com> @ 2023-04-21 7:16 ` Hermes Zhang 2023-04-21 8:23 ` Krzysztof Kozlowski 0 siblings, 1 reply; 4+ messages in thread From: Hermes Zhang @ 2023-04-21 7:16 UTC (permalink / raw) To: jic23, krzysztof.kozlowski+dt, robh+dt, Jean-Baptiste Maneyrol Cc: linux-iio, linux-kernel, kernel, Hermes Zhang, Lars-Peter Clausen, devicetree Adding the invensense ICM-20600 to the compatible list of the mpu6050 driver Signed-off-by: Hermes Zhang <chenhuiz@axis.com> --- .../devicetree/bindings/iio/imu/invensense,mpu6050.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml index ec64d7877fe5..67711bc0ee6c 100644 --- a/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml +++ b/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml @@ -20,6 +20,7 @@ properties: - invensense,icm20608 - invensense,icm20609 - invensense,icm20689 + - invensense,icm20600 - invensense,icm20602 - invensense,icm20690 - invensense,mpu6000 -- 2.30.2 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] dt-bindings: iio: imu: mpu6050: Add icm20600 bindings to mpu6050 2023-04-21 7:16 ` [PATCH 2/2] dt-bindings: iio: imu: mpu6050: Add icm20600 bindings to mpu6050 Hermes Zhang @ 2023-04-21 8:23 ` Krzysztof Kozlowski 2023-04-22 6:14 ` Hermes Zhang 0 siblings, 1 reply; 4+ messages in thread From: Krzysztof Kozlowski @ 2023-04-21 8:23 UTC (permalink / raw) To: Hermes Zhang, jic23, krzysztof.kozlowski+dt, robh+dt, Jean-Baptiste Maneyrol Cc: linux-iio, linux-kernel, kernel, Lars-Peter Clausen, devicetree On 21/04/2023 09:16, Hermes Zhang wrote: > Adding the invensense ICM-20600 to the compatible list of the mpu6050 > driver > > Signed-off-by: Hermes Zhang <chenhuiz@axis.com> > --- > .../devicetree/bindings/iio/imu/invensense,mpu6050.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml b/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml > index ec64d7877fe5..67711bc0ee6c 100644 > --- a/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml > +++ b/Documentation/devicetree/bindings/iio/imu/invensense,mpu6050.yaml > @@ -20,6 +20,7 @@ properties: > - invensense,icm20608 > - invensense,icm20609 > - invensense,icm20689 > + - invensense,icm20600 > - invensense,icm20602 Didn't we start switching to compatible groups of devices? This looks compatible with 20602. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] dt-bindings: iio: imu: mpu6050: Add icm20600 bindings to mpu6050 2023-04-21 8:23 ` Krzysztof Kozlowski @ 2023-04-22 6:14 ` Hermes Zhang 2023-04-22 17:19 ` Jonathan Cameron 0 siblings, 1 reply; 4+ messages in thread From: Hermes Zhang @ 2023-04-22 6:14 UTC (permalink / raw) To: Krzysztof Kozlowski, jic23, krzysztof.kozlowski+dt, robh+dt, Jean-Baptiste Maneyrol Cc: linux-iio, linux-kernel, kernel, Lars-Peter Clausen, devicetree 在 2023/4/21 16:23, Krzysztof Kozlowski 写道: > Didn't we start switching to compatible groups of devices? This looks > compatible with 20602. Hi Krzysztof, Sorry, I may not aware of this. Could you give some more info about the compatible groups? Yes, the icm20602 is quite similar as icm20600. Thanks & Best Regards, Hermes ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] dt-bindings: iio: imu: mpu6050: Add icm20600 bindings to mpu6050 2023-04-22 6:14 ` Hermes Zhang @ 2023-04-22 17:19 ` Jonathan Cameron 0 siblings, 0 replies; 4+ messages in thread From: Jonathan Cameron @ 2023-04-22 17:19 UTC (permalink / raw) To: Hermes Zhang Cc: Krzysztof Kozlowski, krzysztof.kozlowski+dt, robh+dt, Jean-Baptiste Maneyrol, linux-iio, linux-kernel, kernel, Lars-Peter Clausen, devicetree On Sat, 22 Apr 2023 14:14:49 +0800 Hermes Zhang <chenhuiz@axis.com> wrote: > 在 2023/4/21 16:23, Krzysztof Kozlowski 写道: > > Didn't we start switching to compatible groups of devices? This looks > > compatible with 20602. > > Hi Krzysztof, > > Sorry, I may not aware of this. Could you give some more info about the > compatible groups? Yes, the icm20602 is quite similar as icm20600. If it's compatible enough that a device tree that lists compatible = "invensense,icm20600", "invensense,icm20602" would work correctly with an old kernel by first checking for invensense,icm20600 then falling back to invense,icm20602 then it should be listed with that fallback. See the existing block that covers the fallback form icm20608d to icm20608 - items: - const: invensense,icm20608d - const: invensense,icm20608 Note it is fine if the driver 'warns' that it's found a WHOAMI value it doesn't know as long as it then goes on and works correctly. Thus there can't be any incompatibilities between the two parts when we consider the functionality the driver implements today. If we add features in future that aren't compatible, that's fine because we will be matching against the more specific compatible you are adding here. Jonathan > > Thanks & Best Regards, > > Hermes > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-04-22 17:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20230421071630.524822-1-chenhuiz@axis.com>
2023-04-21 7:16 ` [PATCH 2/2] dt-bindings: iio: imu: mpu6050: Add icm20600 bindings to mpu6050 Hermes Zhang
2023-04-21 8:23 ` Krzysztof Kozlowski
2023-04-22 6:14 ` Hermes Zhang
2023-04-22 17:19 ` Jonathan Cameron
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).