Re: [PATCH 01/12] dt-bindings: firmware: convert meson_sm.txt to dt-schema

From: Rob Herring
Date: Fri Nov 18 2022 - 13:51:12 EST



On Fri, 18 Nov 2022 15:33:27 +0100, Neil Armstrong wrote:
> Convert the Amlogic Secure Monitor bindings to dt-schema.
>
> Signed-off-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx>
> ---
> .../bindings/firmware/amlogic,meson-gxbb-sm.yaml | 36 ++++++++++++++++++++++
> .../bindings/firmware/meson/meson_sm.txt | 15 ---------
> 2 files changed, 36 insertions(+), 15 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-1-3f025599b968@xxxxxxxxxx


secure-monitor: compatible:0: 'amlogic,meson-gxbb-sm' was expected
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

secure-monitor: compatible: ['amlogic,meson-gx-sm', 'amlogic,meson-gxbb-sm'] 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