Re: [PATCH 05/12] dt-bindings: media: convert meson-ir.txt to dt-schema

Rob Herring
2022-11-18T18:58:56Z

On Fri, 18 Nov 2022 15:33:31 +0100, Neil Armstrong wrote:
> Convert the Amlogic Meson IR remote control receiver bindings to
> dt-schema.
>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> .../bindings/media/amlogic,meson6-ir.yaml | 43 ++++++++++++++++++++++
> .../devicetree/bindings/media/meson-ir.txt | 20 ----------
> 2 files changed, 43 insertions(+), 20 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-5-3f025599b968@linaro.org

ir@580: compatible:0: 'amlogic,meson-gx-ir' is not one of ['amlogic,meson6-ir', 'amlogic,meson8b-ir', 'amlogic,meson-gxbb-ir']
arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-nanopi-k2.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-meta.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-telos.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-hub.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s805x-libretech-ac.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s805x-p241.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-libretech-pc.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-mecool-kii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p230.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p231.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-sml5442tw.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-vero4k-plus.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-jethome-jethub-j80.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-p281.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-tx3-mini.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-hwacom-amazetv.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-libretech-cc.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-libretech-cc-v2.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-nexbox-a95x.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-gt1-ultimate.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-mecool-kiii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-minix-neo-u9h.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-q200.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-q201.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-s912-libretech-pc.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-vega-s96.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-wetek-core2.dtb

ir@580: compatible: ['amlogic,meson-gx-ir', 'amlogic,meson-gxbb-ir'] is too long
arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-nanopi-k2.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-meta.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-telos.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-hub.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s805x-libretech-ac.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s805x-p241.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-libretech-pc.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-mecool-kii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p230.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p231.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-sml5442tw.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-vero4k-plus.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-jethome-jethub-j80.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-p281.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-tx3-mini.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-hwacom-amazetv.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-libretech-cc.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-libretech-cc-v2.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-nexbox-a95x.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-gt1-ultimate.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-mecool-kiii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-minix-neo-u9h.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-q200.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-q201.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-s912-libretech-pc.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-vega-s96.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-wetek-core2.dtb

ir@580: Unevaluated properties are not allowed ('compatible' was unexpected)
arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-nanopi-k2.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-nexbox-a95x.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-p201.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-meta.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95-telos.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-hub.dtb
arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s805x-libretech-ac.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s805x-p241.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-libretech-pc.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-mecool-kii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p230.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p231.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-sml5442tw.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905d-vero4k-plus.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-jethome-jethub-j80.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-p281.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905w-tx3-mini.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-hwacom-amazetv.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-libretech-cc.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-libretech-cc-v2.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-nexbox-a95x.dtb
arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-gt1-ultimate.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-mecool-kiii-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-minix-neo-u9h.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-q200.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-q201.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-s912-libretech-pc.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-vega-s96.dtb
arch/arm64/boot/dts/amlogic/meson-gxm-wetek-core2.dtb