2022-11-03 23:24:46

by Fabrizio Castro

[permalink] [raw]
Subject: [PATCH 0/3] Minor fixes for RZ/V2M

Dear All,

this series aims at improving a couple of couple of things
found in r9a09g011.dtsi:
* unit address error for the gic node
* SoC specific compatible string for I2C nodes

Since it's early days for r9a09g011.dtsi, and compatible
renesas,i2c-r9a09g011 isn't being actively used at the moment,
I think it's safe to change to make compatible strings less
confusing.

Thanks,
Fab

Fabrizio Castro (3):
arm64: dts: renesas: r9a09g011: Fix unit address format error
dt-bindings: i2c: renesas,rzv2m: Fix SoC specific string
arm64: dts: renesas: r9a09g011: Fix I2C SoC specific strings

Documentation/devicetree/bindings/i2c/renesas,rzv2m.yaml | 4 ++--
arch/arm64/boot/dts/renesas/r9a09g011.dtsi | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)

--
2.34.1



2022-11-03 23:32:02

by Fabrizio Castro

[permalink] [raw]
Subject: [PATCH 1/3] arm64: dts: renesas: r9a09g011: Fix unit address format error

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 <[email protected]>
---
arch/arm64/boot/dts/renesas/r9a09g011.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

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>;
--
2.34.1


2022-11-04 09:00:34

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH 1/3] arm64: dts: renesas: r9a09g011: Fix unit address format error

On Fri, Nov 4, 2022 at 12:07 AM Fabrizio Castro
<[email protected]> 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 <[email protected]>

Reviewed-by: Geert Uytterhoeven <[email protected]>
i.e. will queue in renesas-devel for v6.2.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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