Re: [PATCH 11/12] dt-bindings: pcie: convert amlogic,meson-pcie.txt to dt-schema

Rob Herring
2022-11-18T18:51:12Z

On Fri, 18 Nov 2022 15:33:37 +0100, Neil Armstrong wrote:
> Convert the Amlogic Meson AXG DWC PCIE SoC controller bindings to
> dt-schema.
>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> .../devicetree/bindings/pci/amlogic,axg-pcie.yaml | 129 +++++++++++++++++++++
> .../devicetree/bindings/pci/amlogic,meson-pcie.txt | 70 -----------
> 2 files changed, 129 insertions(+), 70 deletions(-)
>

Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20221117-b4-amlogic-bindings-convert-v1-11-3f025599b968@linaro.org

pcie@f9800000: clock-names:0: 'pclk' was expected
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@f9800000: clock-names:1: 'port' was expected
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@f9800000: clock-names:2: 'general' was expected
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@f9800000: Unevaluated properties are not allowed ('clock-names' was unexpected)
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@fa000000: clock-names:0: 'pclk' was expected
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@fa000000: clock-names:1: 'port' was expected
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@fa000000: clock-names:2: 'general' was expected
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@fa000000: Unevaluated properties are not allowed ('clock-names' was unexpected)
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j100.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-2.dtb
arch/arm64/boot/dts/amlogic/meson-axg-jethome-jethub-j110-rev-3.dtb
arch/arm64/boot/dts/amlogic/meson-axg-s400.dtb

pcie@fc000000: clock-names:0: 'pclk' was expected
arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-u200.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-a311d-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking-pro.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2-plus.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-s922x-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air-gbit.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-h96-max.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-c4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air-gbit.dtb

pcie@fc000000: clock-names:1: 'port' was expected
arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-u200.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-a311d-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking-pro.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2-plus.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-s922x-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air-gbit.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-h96-max.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-c4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air-gbit.dtb

pcie@fc000000: clock-names:2: 'general' was expected
arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-u200.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-a311d-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking-pro.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2-plus.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-s922x-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air-gbit.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-h96-max.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-c4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air-gbit.dtb

pcie@fc000000: Unevaluated properties are not allowed ('clock-names', 'power-domains' were unexpected)
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-a95xf3-air-gbit.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-bananapi-m5.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-h96-max.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-khadas-vim3l.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-c4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-odroid-hc4.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air.dtb
arch/arm64/boot/dts/amlogic/meson-sm1-x96-air-gbit.dtb

pcie@fc000000: Unevaluated properties are not allowed ('clock-names' was unexpected)
arch/arm64/boot/dts/amlogic/meson-g12a-radxa-zero.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-u200.dtb
arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-a311d-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-gtking-pro.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2-plus.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-s922x-khadas-vim3.dtb
arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dtb