Re: [PATCH v5] dt-bindings: iio: adc: Convert xilinx-xadc bindings to YAML schema

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: iio: adc: xlnx,xadc: commit log describes
  inter-revision changes
--

commit d68c424577a43ba2a1a0878e11b74b4e72fd8ee2
Author: Pramod Maurya <pramod.nexgen@gmail.com>

dt-bindings: iio: adc: Convert xilinx-xadc bindings to YAML schema

This commit converts the Xilinx XADC and UltraScale System Monitor device
tree binding from plain text to a YAML schema and adds a MAINTAINERS entry
for the driver.

> Move xlnx,channels under properties, as dt-schema now allows node names
> containing a comma for long-established bindings.Constrain the reg
> property of the channel subnodes to the valid channel range of 0 to
> 16 with minimum and maximum.
>
> Drop the redundant "type: boolean" from xlnx,bipolar, since its $ref to
> the flag type already implies it.

[Severity: Low]
This isn't a bug, but these two paragraphs describe changes between earlier
versions of this patch rather than changes against the repository, where no
prior YAML schema existed. They belong in the revision changelog below the
"---" line rather than the permanent commit message (and there is a missing
space before "Constrain").

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261107134955.1423727-1-dileepkumar.nagavarapu@amd.com?part=1