Re: [PATCH 8/8] dt-bindings: pinctrl: Add Amlogic Meson pinctrl binding
From: Heiner Kallweit <hkallweit1@gmail.com>
Date: 2023-01-24 07:14:18
Also in:
linux-amlogic, linux-arm-kernel, linux-gpio, linux-pwm, linux-rtc
On 24.01.2023 02:22, Rob Herring wrote:
On Mon, 23 Jan 2023 22:30:44 +0100, Heiner Kallweit wrote:quoted
Add Amlogic Meson pinctrl binding. Tested with make targets dt_binding_check and dtbs_check. Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> --- .../pinctrl/amlogic,meson-pinctrl.yaml | 121 ++++++++++++++++++ .../bindings/pinctrl/meson,pinctrl.txt | 94 -------------- 2 files changed, 121 insertions(+), 94 deletions(-) create mode 100644 Documentation/devicetree/bindings/pinctrl/amlogic,meson-pinctrl.yaml delete mode 100644 Documentation/devicetree/bindings/pinctrl/meson,pinctrl.txt
Thanks, I didn't catch these warnings because Meson8 support isn't enabled on my system. Will look into it.
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/69188960-9d88-4163-8a87-1820fb673eb9@gmail.com pinctrl@84: #address-cells:0:0: 2 was expected arch/arm/boot/dts/meson8b-ec100.dtb arch/arm/boot/dts/meson8b-mxq.dtb arch/arm/boot/dts/meson8b-odroidc1.dtb arch/arm/boot/dts/meson8m2-mxiii-plus.dtb arch/arm/boot/dts/meson8-minix-neo-x8.dtb pinctrl@84: compatible: ['amlogic,meson8m2-aobus-pinctrl', 'amlogic,meson8-aobus-pinctrl'] is too long arch/arm/boot/dts/meson8m2-mxiii-plus.dtb pinctrl@84: reg: 'anyOf' conditional failed, one must be fixed: arch/arm/boot/dts/meson8b-ec100.dtb arch/arm/boot/dts/meson8b-mxq.dtb arch/arm/boot/dts/meson8b-odroidc1.dtb arch/arm/boot/dts/meson8m2-mxiii-plus.dtb arch/arm/boot/dts/meson8-minix-neo-x8.dtb pinctrl@84: #size-cells:0:0: 2 was expected arch/arm/boot/dts/meson8b-ec100.dtb arch/arm/boot/dts/meson8b-mxq.dtb arch/arm/boot/dts/meson8b-odroidc1.dtb arch/arm/boot/dts/meson8m2-mxiii-plus.dtb arch/arm/boot/dts/meson8-minix-neo-x8.dtb pinctrl@9880: #address-cells:0:0: 2 was expected arch/arm/boot/dts/meson8b-ec100.dtb arch/arm/boot/dts/meson8b-mxq.dtb arch/arm/boot/dts/meson8b-odroidc1.dtb arch/arm/boot/dts/meson8m2-mxiii-plus.dtb arch/arm/boot/dts/meson8-minix-neo-x8.dtb pinctrl@9880: compatible: ['amlogic,meson8m2-cbus-pinctrl', 'amlogic,meson8-cbus-pinctrl'] is too long arch/arm/boot/dts/meson8m2-mxiii-plus.dtb pinctrl@9880: reg: 'anyOf' conditional failed, one must be fixed: arch/arm/boot/dts/meson8b-ec100.dtb arch/arm/boot/dts/meson8b-mxq.dtb arch/arm/boot/dts/meson8b-odroidc1.dtb arch/arm/boot/dts/meson8m2-mxiii-plus.dtb arch/arm/boot/dts/meson8-minix-neo-x8.dtb pinctrl@9880: #size-cells:0:0: 2 was expected arch/arm/boot/dts/meson8b-ec100.dtb arch/arm/boot/dts/meson8b-mxq.dtb arch/arm/boot/dts/meson8b-odroidc1.dtb arch/arm/boot/dts/meson8m2-mxiii-plus.dtb arch/arm/boot/dts/meson8-minix-neo-x8.dtb