Hi Rob,

On 17/11/2020 18:19, Sekhar Nori wrote:
With dtc 1.6.0, building TI device-tree files with W=2 results in warnings
like below for all interrupt controllers.

/bus@100000/bus@30000000/interrupt-controller1: Missing #address-cells in 
interrupt provider

Fix these by adding #address-cells = <0>; for all interrupt controllers in
TI device-tree files. Any other #address-cells value is really only needed
if interrupt-map property is being used (which is not the case for existing
TI device-tree files)

Signed-off-by: Sekhar Nori <[email protected]>
---
  arch/arm64/boot/dts/ti/k3-am65-main.dtsi              |  5 +++++
  arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi            |  2 ++
  arch/arm64/boot/dts/ti/k3-am654-base-board.dts        |  1 +
  arch/arm64/boot/dts/ti/k3-j7200-main.dtsi             |  3 +++
  arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi       |  1 +
  arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts |  1 +
  arch/arm64/boot/dts/ti/k3-j721e-main.dtsi             | 11 +++++++++++
  arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi       |  3 +++
  8 files changed, 27 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi 
b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index aa8725db0187..55aaa1404d7d 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -440,6 +440,7 @@
                interrupt-controller;
                interrupt-parent = <&gic500>;
                #interrupt-cells = <1>;
+               #address-cells = <0>;
Does it really required or mandatory to have #address-cells = <0>; defined for 
interrupt-controller DT nodes which
do not have child nodes and no "interrupt-map"?

--
Best regards,
grygorii

Reply via email to