Re: [PATCH] dt-bindings: PCI: Remove obsolete .txt docs

From: Rob Herring (Arm)
Date: Fri Apr 04 2025 - 19:22:03 EST



On Fri, 04 Apr 2025 17:15:57 -0500, Rob Herring (Arm) wrote:
> The content in these files has been moved to the schemas in dtschema.
> pci.txt is covered by pci-bus-common.yaml and pci-host-bridge.yaml.
> pci-iommu.txt is covered by pci-iommu.yaml. pci-msi.txt is covered in
> msi-map property in pci-host-bridge.yaml.
>
> Cc: Frank Li <Frank.li@xxxxxxx>
> Signed-off-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> ---
> .../bindings/pci/nvidia,tegra194-pcie-ep.yaml | 2 +-
> .../devicetree/bindings/pci/pci-iommu.txt | 171 --------------
> .../devicetree/bindings/pci/pci-msi.txt | 220 ------------------
> Documentation/devicetree/bindings/pci/pci.txt | 84 -------
> 4 files changed, 1 insertion(+), 476 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/pci/pci-iommu.txt
> delete mode 100644 Documentation/devicetree/bindings/pci/pci-msi.txt
> delete mode 100644 Documentation/devicetree/bindings/pci/pci.txt
>

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

yamllint warnings/errors:

dtschema/dtc warnings/errors:


doc reference errors (make refcheckdocs):
Warning: Documentation/devicetree/bindings/virtio/pci-iommu.yaml references a file that doesn't exist: Documentation/devicetree/bindings/pci/pci.txt
Documentation/devicetree/bindings/virtio/pci-iommu.yaml: Documentation/devicetree/bindings/pci/pci.txt

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250404221559.552201-1-robh@xxxxxxxxxx

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.