Re: [PATCH] dt-bindings: iio: accel: adi,adis16201: add adis16203 compatible

[Jonathan Cameron]

On Tue, 11 Aug 2026 20:54:21 -0500
Rob Herring <robh@xxxxxxxxxx> wrote:

> On Wed, Aug 05, 2026 at 03:22:24PM +0500, Shehryar Ahmad wrote:
> > The ADIS16203 has the same external interface as the already
> > documented ADIS16201/ADIS16209, for the properties covered by this
> > binding (compatible/reg, the DIO-based interrupt pin, and VDD
> > supply), verified by datasheets Table 5, Pin Function Descriptions,
> > in the respective Analog Devices datasheets. Some Auxiliary pins
> > differ (e.g. adis16209 marks a few pins DNC where adis16201/adis16203
> > use them as AUX COM), but these are not represented in this binding.
> > Add adis16203 to the existing enum rather than creating a separate
> > file.
> > 
> > Tested with checkpatch.pl --strict and dt_binding_check; both pass
> > with no warnings.  
> 
> This is assumed, no need to record it forever in the commit.
> 
> > Three prior attempts to document this part are visible on the
> > mailing list, all as separate files, each redirected toward this
> > existing binding approach instead: Saalim Quadri, March 2025 [2],
> > and Sheng Kun Chang, April 2026 [3]. See below for Jonathan
> > Cameron's earlier, related attempt [1].

Key thing here is that we generally don't take bindings for drivers
that are in staging and adis16203 remains there.

This patch looks good for when/if the driver is moved out of
staging.  There have been various series on list for that so
perhaps ask those authors what current status is (just reply
to that series).

e.g. 

https://lore.kernel.org/linux-iio/20260401162458.88110-1-nothingchang@xxxxxxxxxxxxxx/

Which is the parent of the last link below and I think the last of
various attempts to do this.

Jonathan

> > 
> > A richer version adding reset-gpios and a second interrupt
> > (interrupt-names: dio0/dio1) was proposed by Jonathan Cameron in
> > 2023 [1]. I tested the same additions locally for all three
> > parts; checkpatch and dt_binding_check both pass. It is not
> > included here, as it hasn't been verified against any real board
> > dts file, and none in-tree currently reference adis16201,
> > adis16203, or adis16209. Can send that as a follow-up if
> > wanted.
> > 
> > Link: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20230129155602.740743-13-jic23@xxxxxxxxxx/
> > Link: https://lore.kernel.org/all/5390c973-6a92-4de8-b00a-cd62af347ccb@xxxxxxxxxx/
> > Link: https://lore.kernel.org/all/20260402-fair-wakeful-okapi-3c6dba@quoll/  
> 
> Thanks for the history. It should be after the '---' so it too is not 
> recorded in the commit.
> 
> > 
> > Signed-off-by: Shehryar Ahmad <shehryar.amd@xxxxxxxxx>
> > ---
> >  .../devicetree/bindings/iio/accel/adi,adis16201.yaml        | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)  
> 
> Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>