On 12/7/21 6:58 AM, Rob Herring wrote: > On Mon, 06 Dec 2021 10:26:07 -0800, Florian Fainelli wrote: >> Convert the Broadcom STB GPIO Device Tree binding to YAML to help with >> validation. >> >> Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx> >> --- >> .../bindings/gpio/brcm,brcmstb-gpio.txt | 83 -------------- >> .../bindings/gpio/brcm,brcmstb-gpio.yaml | 105 ++++++++++++++++++ >> MAINTAINERS | 2 +- >> 3 files changed, 106 insertions(+), 84 deletions(-) >> delete mode 100644 Documentation/devicetree/bindings/gpio/brcm,brcmstb-gpio.txt >> create mode 100644 Documentation/devicetree/bindings/gpio/brcm,brcmstb-gpio.yaml >> > > 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/patch/1564132 > > > gpio@4172c0: interrupts-extended: [[6, 6], [7, 5]] is too long > arch/arm/boot/dts/bcm7445-bcm97445svmb.dt.yaml The property is correct AFAICT, we have: interrupts-extended = <&irq0_aon_intc 0x6>, <&aon_pm_l2_intc 0x5>; with both interrupt controllers having #interrupt-cells = <1>. I tried documenting the interrupts-extended for brcm,brcmstb-gpio.yaml to have maxItems: 2 but that does not eliminate the warning. Do you have any suggestions? -- Florian
