linux-kernel - Re: [PATCH 4/5] arm64: dts: renesas: r9a07g054: Drop WDT2 node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdWcvfWVid+bFFENFDXe9eG32J=OxzSp95+myk=ADdao=w@mail.gmail.com>
Date:   Tue, 11 Oct 2022 10:19:28 +0200
From:   Geert Uytterhoeven <geert@...ux-m68k.org>
To:     Prabhakar <prabhakar.csengg@...il.com>
Cc:     Magnus Damm <magnus.damm@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        linux-renesas-soc@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Biju Das <biju.das.jz@...renesas.com>,
        Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Subject: Re: [PATCH 4/5] arm64: dts: renesas: r9a07g054: Drop WDT2 node

Hi Prabhakar,

On Mon, Oct 10, 2022 at 1:01 AM Prabhakar <prabhakar.csengg@...il.com> wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
>
> WDT CH2 is specifically to check the operation of Cortex-M33 CPU and if
> used from CA55 CPU would result in an unexpected behaviour. Hence drop
> WDT2 node from RZ/V2L SoC DTSI.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/r9a07g054.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r9a07g054.dtsi
> @@ -1000,21 +1000,6 @@ wdt1: watchdog@...00c00 {
>                         status = "disabled";
>                 };
>
> -               wdt2: watchdog@...00400 {
> -                       compatible = "renesas,r9a07g054-wdt",
> -                                    "renesas,rzg2l-wdt";
> -                       reg = <0 0x12800400 0 0x400>;
> -                       clocks = <&cpg CPG_MOD R9A07G054_WDT2_PCLK>,
> -                                <&cpg CPG_MOD R9A07G054_WDT2_CLK>;
> -                       clock-names = "pclk", "oscclk";
> -                       interrupts = <GIC_SPI 53 IRQ_TYPE_LEVEL_HIGH>,
> -                                    <GIC_SPI 54 IRQ_TYPE_LEVEL_HIGH>;
> -                       interrupt-names = "wdt", "perrout";
> -                       resets = <&cpg R9A07G054_WDT2_PRESETN>;
> -                       power-domains = <&cpg>;
> -                       status = "disabled";
> -               };
> -
>                 ostm0: timer@...01000 {
>                         compatible = "renesas,r9a07g054-ostm",
>                                      "renesas,ostm";

As this is hardware description, and the node is disabled by default,
we could keep it.  However, as it is to be used by the CM33, its
interrupts property should point to the CM33 NVIC instead of the
CA55 GIC.  So let's drop it for now...

Reviewed-by: Geert Uytterhoeven <geert+renesas@...der.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@...ux-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