[PATCH 0/5] v3u: add support for RWDT

[PATCH 0/5] v3u: add support for RWDT

[Wolfram Sang]

Here is the series to enable the RWDT on V3U. I took the DTS patches
from the BSP, the rest was developed on mainline tree. Note that
currently, the bootloader does not unmask the RWDT reset for us. This is
why patch #5 is needed if you want a reboot. We need to see if this is
going to be added to the bootloader later. Also, while the normal
watchdog trigger works flawlessly, the 'reboot' command will triger a
OOPS. This is a known issue also present on other Gen3 SoCs and will
be addressed seperately soon.

Other than that, patch #1 should go in via watchdog, the rest via
Geert's various trees :)

Looking forward to comments.

Happy hacking and have a nice weekend!


Hoang Vo (2):
  arm64: dts: renesas: r8a779a0: Add RWDT node
  arm64: dts: renesas: falcon: Enable watchdog timer

Wolfram Sang (3):
  dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support
  clk: renesas: r8a779a0: Add RWDT clocks
  WIP soc: v3u: allow WDT reset

 .../devicetree/bindings/watchdog/renesas,wdt.yaml      |  1 +
 arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts        |  5 +++++
 arch/arm64/boot/dts/renesas/r8a779a0.dtsi              | 10 ++++++++++
 drivers/clk/renesas/r8a779a0-cpg-mssr.c                |  9 +++++++++
 drivers/soc/renesas/rcar-rst.c                         |  8 ++++++++
 5 files changed, 33 insertions(+)

-- 
2.29.2

[PATCH 1/5] dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support

[Wolfram Sang]

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---

Please apply it to the watchdog-tree.

 Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
index 6933005b52bd..ab66d3f0c476 100644
--- a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
@@ -50,6 +50,7 @@ properties:
               - renesas,r8a77980-wdt     # R-Car V3H
               - renesas,r8a77990-wdt     # R-Car E3
               - renesas,r8a77995-wdt     # R-Car D3
+              - renesas,r8a779a0-wdt     # R-Car V3U
           - const: renesas,rcar-gen3-wdt # R-Car Gen3 and RZ/G2
 
   reg:
-- 
2.29.2

Re: [PATCH 1/5] dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support

[Geert Uytterhoeven]

On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

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

Re: [PATCH 1/5] dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support

[Rob Herring]

On Fri, 18 Dec 2020 18:37:26 +0100, Wolfram Sang wrote:
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
> 
> Please apply it to the watchdog-tree.
> 
>  Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH 1/5] dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 1022 bytes --]

On Fri, Dec 18, 2020 at 06:37:26PM +0100, Wolfram Sang wrote:
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
> 
> Please apply it to the watchdog-tree.

Guenter, could you pick/ack it, please?

> 
>  Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
> index 6933005b52bd..ab66d3f0c476 100644
> --- a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
> @@ -50,6 +50,7 @@ properties:
>                - renesas,r8a77980-wdt     # R-Car V3H
>                - renesas,r8a77990-wdt     # R-Car E3
>                - renesas,r8a77995-wdt     # R-Car D3
> +              - renesas,r8a779a0-wdt     # R-Car V3U
>            - const: renesas,rcar-gen3-wdt # R-Car Gen3 and RZ/G2
>  
>    reg:
> -- 
> 2.29.2
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH 1/5] dt-bindings: watchdog: renesas,wdt: add r8a779a0 (V3U) support

[Guenter Roeck]

On Fri, Dec 18, 2020 at 06:37:26PM +0100, Wolfram Sang wrote:
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> Acked-by: Rob Herring <robh@kernel.org>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
> 
> Please apply it to the watchdog-tree.
> 
>  Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
> index 6933005b52bd..ab66d3f0c476 100644
> --- a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml
> @@ -50,6 +50,7 @@ properties:
>                - renesas,r8a77980-wdt     # R-Car V3H
>                - renesas,r8a77990-wdt     # R-Car E3
>                - renesas,r8a77995-wdt     # R-Car D3
> +              - renesas,r8a779a0-wdt     # R-Car V3U
>            - const: renesas,rcar-gen3-wdt # R-Car Gen3 and RZ/G2
>  
>    reg:

[PATCH 3/5] arm64: dts: renesas: r8a779a0: Add RWDT node

[Wolfram Sang]

From: Hoang Vo <hoang.vo.eb@renesas.com>

Add a device node for the Watchdog Timer (WDT) controller on the
R8A779A0 SoC.

Signed-off-by: Hoang Vo <hoang.vo.eb@renesas.com>
[wsa: rebased to mainline]
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 arch/arm64/boot/dts/renesas/r8a779a0.dtsi | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a779a0.dtsi b/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
index 6cf77ce9aa93..1ca500f55096 100644
--- a/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
@@ -85,6 +85,16 @@ rst: reset-controller@e6160000 {
 			reg = <0 0xe6160000 0 0x4000>;
 		};
 
+		rwdt: watchdog@e6020000 {
+			compatible = "renesas,r8a779a0-wdt",
+				     "renesas,rcar-gen3-wdt";
+			reg = <0 0xe6020000 0 0x0c>;
+			clocks = <&cpg CPG_MOD 907>;
+			power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>;
+			resets = <&cpg 907>;
+			status = "disabled";
+		};
+
 		sysc: system-controller@e6180000 {
 			compatible = "renesas,r8a779a0-sysc";
 			reg = <0 0xe6180000 0 0x4000>;
-- 
2.29.2

Re: [PATCH 3/5] arm64: dts: renesas: r8a779a0: Add RWDT node

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 150 bytes --]


> +			reg = <0 0xe6020000 0 0x0c>;

I just understood that we sort by reg value and not by name. So, this
needs to be moved to another place then.



[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH 3/5] arm64: dts: renesas: r8a779a0: Add RWDT node

[Geert Uytterhoeven]

On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> From: Hoang Vo <hoang.vo.eb@renesas.com>
>
> Add a device node for the Watchdog Timer (WDT) controller on the
> R8A779A0 SoC.
>
> Signed-off-by: Hoang Vo <hoang.vo.eb@renesas.com>
> [wsa: rebased to mainline]
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v5.12 (with sort order fixed).

> --- a/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
> @@ -85,6 +85,16 @@ rst: reset-controller@e6160000 {
>                         reg = <0 0xe6160000 0 0x4000>;
>                 };
>
> +               rwdt: watchdog@e6020000 {
> +                       compatible = "renesas,r8a779a0-wdt",
> +                                    "renesas,rcar-gen3-wdt";
> +                       reg = <0 0xe6020000 0 0x0c>;
> +                       clocks = <&cpg CPG_MOD 907>;
> +                       power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>;
> +                       resets = <&cpg 907>;
> +                       status = "disabled";

No interrupts property? ;-)
As we don't have it described yet for the other R-Car Gen3 SoCs, I
suggest we do that in one batch...

> +               };
> +
>                 sysc: system-controller@e6180000 {
>                         compatible = "renesas,r8a779a0-sysc";
>                         reg = <0 0xe6180000 0 0x4000>;

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

Re: [PATCH 3/5] arm64: dts: renesas: r8a779a0: Add RWDT node

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 202 bytes --]


> No interrupts property? ;-)
> As we don't have it described yet for the other R-Car Gen3 SoCs, I
> suggest we do that in one batch...

Yes, I'll try to find some time for this as a seperate series.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[PATCH 4/5] arm64: dts: renesas: falcon: Enable watchdog timer

[Wolfram Sang]

From: Hoang Vo <hoang.vo.eb@renesas.com>

Enable the watchdog on the Falcon board.

Signed-off-by: Hoang Vo <hoang.vo.eb@renesas.com>
[wsa: rebased to mainline]
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts b/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
index 8eda70e5a82b..fb9978ea18f4 100644
--- a/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
+++ b/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
@@ -20,3 +20,8 @@ chosen {
 		stdout-path = "serial0:115200n8";
 	};
 };
+
+&rwdt {
+	timeout-sec = <60>;
+	status = "okay";
+};
-- 
2.29.2

Re: [PATCH 4/5] arm64: dts: renesas: falcon: Enable watchdog timer

[Geert Uytterhoeven]

On Fri, Dec 18, 2020 at 6:37 PM Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> From: Hoang Vo <hoang.vo.eb@renesas.com>
>
> Enable the watchdog on the Falcon board.
>
> Signed-off-by: Hoang Vo <hoang.vo.eb@renesas.com>
> [wsa: rebased to mainline]
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

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

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