[1/3] arm64: dts: renesas: r9a09g011: Fix unit address format error

Message ID 20221103230648.53748-2-fabrizio.castro.jz@renesas.com
State New
Headers
Series Minor fixes for RZ/V2M |

Commit Message

Fabrizio Castro Nov. 3, 2022, 11:06 p.m. UTC
  Although the HW User Manual for RZ/V2M states in the "Address Map"
section that the interrupt controller is assigned addresses starting
from 0x82000000, the memory locations from 0x82000000 0x0x8200FFFF
are marked as reserved in the "Interrupt Controller (GIC)" section
and are currently not used by the device tree, leading to the below
warning:

arch/arm64/boot/dts/renesas/r9a09g011.dtsi:51.38-63.5: Warning
(simple_bus_reg): /soc/interrupt-controller@82000000: simple-bus unit
address format error, expected "82010000"

Fix the unit address accordingly.

Fixes: fb1929b98f2e ("arm64: dts: renesas: Add initial DTSI for RZ/V2M SoC")
Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
---
 arch/arm64/boot/dts/renesas/r9a09g011.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Geert Uytterhoeven Nov. 4, 2022, 8:46 a.m. UTC | #1
On Fri, Nov 4, 2022 at 12:07 AM Fabrizio Castro
<fabrizio.castro.jz@renesas.com> wrote:
> Although the HW User Manual for RZ/V2M states in the "Address Map"
> section that the interrupt controller is assigned addresses starting
> from 0x82000000, the memory locations from 0x82000000 0x0x8200FFFF
> are marked as reserved in the "Interrupt Controller (GIC)" section
> and are currently not used by the device tree, leading to the below
> warning:
>
> arch/arm64/boot/dts/renesas/r9a09g011.dtsi:51.38-63.5: Warning
> (simple_bus_reg): /soc/interrupt-controller@82000000: simple-bus unit
> address format error, expected "82010000"
>
> Fix the unit address accordingly.
>
> Fixes: fb1929b98f2e ("arm64: dts: renesas: Add initial DTSI for RZ/V2M SoC")
> Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v6.2.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
  

Patch

diff --git a/arch/arm64/boot/dts/renesas/r9a09g011.dtsi b/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
index 9859c717bd10..1948d2e6bcae 100644
--- a/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
@@ -48,7 +48,7 @@  soc: soc {
 		#size-cells = <2>;
 		ranges;
 
-		gic: interrupt-controller@82000000 {
+		gic: interrupt-controller@82010000 {
 			compatible = "arm,gic-400";
 			#interrupt-cells = <3>;
 			#address-cells = <0>;