Re: [PATCH v4 0/3] Convert Xilinx XADC binding to YAML and related cleanups

[Jonathan Cameron <jic23@kernel.org>]

On Fri, 15 May 2026 03:57:33 -0400
Pramod Maurya <pramod.nexgen@gmail.com> wrote:

> This series converts the Xilinx XADC device tree binding from the legacy
> plain-text format to a validated YAML schema, fixes a checkpatch alignment
> warning in the axis-fifo staging driver, and adds a YAML DT binding for
> the Xilinx AXI-Stream FIFO IP core.
> 
Hi Pramod

Small process thing.  Don't send new versions with the In-reply-to field
set to an earlier version.

1) Ends up with very hard to read threads if the discussion gets complex.
2) Time poor reviewers tend to read latest stuff in their email folders first.
   So by replying to an earlier thread you shunt your stuff way back in their
   pile of unread email - thus reducing the chance anyone reads it ;)

Not that important this time, but please keep in mind for the future.

Thanks,

Jonathan

> Patch 1 is v4 of the XADC binding conversion. Previous versions had the
> following issues that are now addressed:
>   - v3: xlnx,channels was placed in patternProperties to satisfy the
>     vendor-props.yaml meta-schema; dt-schema PR#195 (merged by Rob
>     Herring) now allows commas in nodenames so it correctly lives in
>     properties. A companion dt-schema update is needed to add
>     xlnx,channels to the vendor-props.yaml explicit object exceptions
>     (similar to the existing adi,channels entry).
>   - v3: reg constraints inside channel subnodes used bare minimum/maximum
>     keywords which are silently ignored on array types; the description
>     now documents the valid range and reg uses maxItems: 1 per cell.yaml.
>   - v3: xlnx,bipolar had a redundant type: boolean alongside the $ref
>     to /schemas/types.yaml#/definitions/flag; the $ref already implies
>     the boolean type so type: boolean is removed.
> 
> Patches 2 and 3 are new additions to this series.
> 
> Pramod Maurya (3):
>   dt-bindings: iio: adc: Convert xilinx-xadc bindings to YAML schema
>   staging: axis-fifo: Fix alignment of wait_event_interruptible
>     arguments
>   dt-bindings: misc: Add binding for Xilinx AXI-Stream FIFO
> 
>  .../bindings/iio/adc/xilinx-xadc.txt          | 141 ------------
>  .../bindings/iio/adc/xlnx,xadc.yaml           | 210 ++++++++++++++++++
>  .../bindings/misc/xlnx,axi-fifo-mm-s.yaml     |  93 ++++++++
>  MAINTAINERS                                   |  13 ++
>  drivers/staging/axis-fifo/axis-fifo.c         |   3 +-
>  5 files changed, 318 insertions(+), 142 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/xilinx-xadc.txt
>  create mode 100644 Documentation/devicetree/bindings/iio/adc/xlnx,xadc.yaml
>  create mode 100644 Documentation/devicetree/bindings/misc/xlnx,axi-fifo-mm-s.yaml
>