Re: [PATCH v2 09/14] dt-bindings: interrupt-controller: Convert Broadcom STB L2 to YAML

From: Rob Herring <robh@kernel.org>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: "Alexandre Belloni" <alexandre.belloni@bootlin.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Doug Berger" <opendmb@gmail.com>,
	linux-gpio@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	"Ulf Hansson" <ulf.hansson@linaro.org>,
	linux-pm@vger.kernel.org, "Gregory Fong" <gregory.0xf0@gmail.com>,
	"Marc Zyngier" <maz@kernel.org>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Ray Jui" <rjui@broadcom.com>,
	linux-usb@vger.kernel.org,
	"Thierry Reding" <thierry.reding@gmail.com>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	"Alessandro Zummo" <a.zummo@towertech.it>,
	"Scott Branden" <sbranden@broadcom.com>,
	"Zhang Rui" <rui.zhang@intel.com>,
	"Amit Kucheria" <amitk@kernel.org>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	"Damien Le Moal" <damien.lemoal@opensource.wdc.com>,
	"Daniel Lezcano" <daniel.lezcano@linaro.org>,
	"Al Cooper" <alcooperx@gmail.com>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	linux-pwm@vger.kernel.org, linux-ide@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-rtc@vger.kernel.org,
	linux-crypto@vger.kernel.org, "Lee Jones" <lee.jones@linaro.org>,
	devicetree@vger.kernel.org,
	bcm-kernel-feedback-list@broadcom.com,
	"Bartosz Golaszewski" <brgl@bgdev.pl>,
	"Linus Walleij" <linus.walleij@linaro.org>,
	"Herbert Xu" <herbert@gondor.apana.org.au>,
	"Markus Mayer" <mmayer@broadcom.com>,
	linux-mmc@vger.kernel.org
Subject: Re: [PATCH v2 09/14] dt-bindings: interrupt-controller: Convert Broadcom STB L2 to YAML
Date: Tue, 07 Dec 2021 08:58:10 -0600	[thread overview]
Message-ID: <1638889090.711166.5801.nullmailer@robh.at.kernel.org> (raw)
In-Reply-To: <20211206182616.2089677-10-f.fainelli@gmail.com>

On Mon, 06 Dec 2021 10:26:11 -0800, Florian Fainelli wrote:
> Convert the Broadcom STB L2 generic Level 2 interrupt controller Device
> Tree binding to YAML to help with validation.
> 
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
>  .../interrupt-controller/brcm,l2-intc.txt     | 31 ---------
>  .../interrupt-controller/brcm,l2-intc.yaml    | 64 +++++++++++++++++++
>  2 files changed, 64 insertions(+), 31 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/interrupt-controller/brcm,l2-intc.txt
>  create mode 100644 Documentation/devicetree/bindings/interrupt-controller/brcm,l2-intc.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/1564135

interrupt-controller@3e1000: 'interrupt-names' does not match any of the regexes: 'pinctrl-[0-9]+'
	arch/arm/boot/dts/bcm7445-bcm97445svmb.dt.yaml

interrupt-controller@7ef00100: compatible: 'oneOf' conditional failed, one must be fixed:
	arch/arm64/boot/dts/broadcom/bcm2711-rpi-400.dt.yaml
	arch/arm64/boot/dts/broadcom/bcm2711-rpi-4-b.dt.yaml
	arch/arm64/boot/dts/broadcom/bcm2711-rpi-cm4-io.dt.yaml
	arch/arm/boot/dts/bcm2711-rpi-400.dt.yaml
	arch/arm/boot/dts/bcm2711-rpi-4-b.dt.yaml
	arch/arm/boot/dts/bcm2711-rpi-cm4-io.dt.yaml