Re: [PATCH] dt-bindings: Fix phandle-array constraints, again

Rob Herring (Arm)
2026-04-21T21:33:32Z

On Tue, 21 Apr 2026 14:55:25 -0500, Rob Herring (Arm) wrote:
> The unfortunately named 'phandle-array' property type is really a matrix
> with phandle and fixed arg cells entries. A matrix property should have 2
> levels of items constraints.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
> Can someone from QCom provide some descriptions for 'qcom,smem-states'
> properties.
> ---
> .../display/rockchip/rockchip,rk3399-cdn-dp.yaml | 2 ++
> .../bindings/mmc/hisilicon,hi3798cv200-dw-mshc.yaml | 7 ++++---
> Documentation/devicetree/bindings/net/qcom,bam-dmux.yaml | 6 ++++++
> Documentation/devicetree/bindings/net/qcom,ipa.yaml | 6 ++++++
> .../devicetree/bindings/net/wireless/qcom,ath10k.yaml | 5 ++++-
> .../devicetree/bindings/net/wireless/qcom,ath11k.yaml | 5 ++++-
> .../bindings/net/wireless/qcom,ipq5332-wifi.yaml | 9 +++++++++
> .../devicetree/bindings/pci/toshiba,tc9563.yaml | 5 +++--
> .../bindings/remoteproc/qcom,msm8916-mss-pil.yaml | 3 +++
> .../bindings/remoteproc/qcom,msm8996-mss-pil.yaml | 3 +++
> .../devicetree/bindings/remoteproc/qcom,pas-common.yaml | 4 ++++
> .../bindings/remoteproc/qcom,qcs404-cdsp-pil.yaml | 4 ++++
> .../bindings/remoteproc/qcom,sc7180-mss-pil.yaml | 3 +++
> .../bindings/remoteproc/qcom,sc7280-adsp-pil.yaml | 3 +++
> .../bindings/remoteproc/qcom,sc7280-mss-pil.yaml | 3 +++
> .../bindings/remoteproc/qcom,sc7280-wpss-pil.yaml | 3 +++
> .../bindings/remoteproc/qcom,sdm845-adsp-pil.yaml | 3 +++
> .../devicetree/bindings/remoteproc/qcom,wcnss-pil.yaml | 3 +++
> Documentation/devicetree/bindings/sound/samsung,tm2.yaml | 2 ++
> .../devicetree/bindings/spi/st,stm32mp25-ospi.yaml | 5 +++--
> .../devicetree/bindings/usb/chipidea,usb2-common.yaml | 2 ++
> Documentation/devicetree/bindings/usb/ci-hdrc-usb2.yaml | 7 ++++---
> 22 files changed, 81 insertions(+), 12 deletions(-)
>

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:
./Documentation/devicetree/bindings/remoteproc/qcom,qcs404-cdsp-pil.yaml:99:1: [warning] too many blank lines (2 > 1) (empty-lines)
./Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml:67:1: [warning] too many blank lines (2 > 1) (empty-lines)

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/i2c/i2c-demux-pinctrl.example.dtb: i2c-mux3 (i2c-demux-pinctrl): i2c-parent:0: [2, 3, 4] is too long
from schema $id: http://devicetree.org/schemas/i2c/i2c-demux-pinctrl.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/sound/samsung,tm2.example.dtb: sound (samsung,tm2-audio): i2s-controller: [[4294967295], [0], [4294967295], [0]] is too long
from schema $id: http://devicetree.org/schemas/sound/samsung,tm2.yaml

doc reference errors (make refcheckdocs):

See https://patchwork.kernel.org/project/devicetree/patch/20260421195836.1547469-1-robh@kernel.org

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.