linux-kernel - Re: [PATCH 3/3] dt-bindings: iio: adis16480: add devices to adis16480

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20241101151243.558a8577@jic23-huawei>
Date: Fri, 1 Nov 2024 15:12:43 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Darius Berghe <darius.berghe@...log.com>
Cc: <linux-iio@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 3/3] dt-bindings: iio: adis16480: add devices to
 adis16480

On Fri, 1 Nov 2024 15:07:42 +0200
Darius Berghe <darius.berghe@...log.com> wrote:

> Add the adis16486, adis16487 and adis16489 Six Degrees
> of Freedom Inertial Sensors to the list of compatible devices
> of the adis16480 iio subsystem driver.
Hi Darius

Please give a brief statement of what differs between these and the
many other compatibles in this binding (beyond the ID register)?

(usually scales, features etc)

In particularly you state in the docs patch that one of them
is functionally equivalent to an existing part so there a fallback
compatible is appropriate.

See how the st,lsm6dsx does it.  The reason to do this it allows
the potential for an old kernel driver to work fine with a newer
device tree.

Other than that the series looks good to me.

Given we have a case where data could be shared, longer term
breaking up the chip_info array into individual structures would
allow sharing of the data.

The name handling in here isn't ideal as it relies on matching
in the spi_device_id table which is fragile.  Doing that more
cleanly might break the sharing of structures.  Oh well, not
relevant for this set anyway.

Jonathan

> 
> Signed-off-by: Darius Berghe <darius.berghe@...log.com>
> ---
>  Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml
> index e3eec38897bf..5bef2878a0df 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16480.yaml
> @@ -15,7 +15,10 @@ properties:
>        - adi,adis16375
>        - adi,adis16480
>        - adi,adis16485
> +      - adi,adis16486
> +      - adi,adis16487
>        - adi,adis16488
> +      - adi,adis16489
>        - adi,adis16490
>        - adi,adis16495-1
>        - adi,adis16495-2