Re: [PATCH] dt-bindings: iio: accel: adi,adis16201: add adis16203 compatible
Rob Herring <[email protected]>
| Newsgroups | org.kernel.vger.linux-iio,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
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]. > > 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/[email protected]/ > Link: https://lore.kernel.org/all/[email protected]/ > 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 <[email protected]> > --- > .../devicetree/bindings/iio/accel/adi,adis16201.yaml | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) Reviewed-by: Rob Herring (Arm) <[email protected]>