Re: [PATCH] ARM: dts: zynq: Add address-cells property to interrupt controllers

From: Rob Herring
Date: Mon Feb 01 2021 - 12:42:56 EST


On Mon, Feb 1, 2021 at 8:27 AM Michal Simek <michal.simek@xxxxxxxxxx> wrote:
>
> The commit 3eb619b2f7d8 ("scripts/dtc: Update to upstream version
> v1.6.0-11-g9d7888cbf19c") updated dtc version which also contained DTC
> commit
> "81e0919a3e21 checks: Add interrupt provider test"
> where reasons for this checking are mentioned as
> "A missing #address-cells property is less critical, but creates
> ambiguities when used in interrupt-map properties, so warn about this as
> well now."
>
> Add address-cells property to gic and gpio nodes to get rid of this warning.
> The similar change has been done for ZynqMP too.

FYI, we're going to make this check dependent on having an
interrupt-map property. So adding these isn't necessary.

>
> CC: Andre Przywara <andre.przywara@xxxxxxx>
> Signed-off-by: Michal Simek <michal.simek@xxxxxxxxxx>
> ---
>
> arch/arm/boot/dts/zynq-7000.dtsi | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi
> index df9ad831cf05..c4810d58540b 100644
> --- a/arch/arm/boot/dts/zynq-7000.dtsi
> +++ b/arch/arm/boot/dts/zynq-7000.dtsi
> @@ -133,6 +133,7 @@ can1: can@e0009000 {
>
> gpio0: gpio@e000a000 {
> compatible = "xlnx,zynq-gpio-1.0";
> + #address-cells = <0>;
> #gpio-cells = <2>;
> clocks = <&clkc 42>;
> gpio-controller;
> @@ -168,6 +169,7 @@ i2c1: i2c@e0005000 {
> intc: interrupt-controller@f8f01000 {
> compatible = "arm,cortex-a9-gic";
> #interrupt-cells = <3>;
> + #address-cells = <0>;
> interrupt-controller;
> reg = <0xF8F01000 0x1000>,
> <0xF8F00100 0x100>;
> --
> 2.30.0
>