| Message ID | 20221009230044.10961-1-prabhakar.mahadev-lad.rj@bp.renesas.com |
|---|---|
| Headers | show |
| Series | arm64: dts: renesas: rzg2l/rzg2lc/rzg2ul/rzv2l: Drop WDT2 | expand |
Hi Prabhakar, On Mon, Oct 10, 2022 at 1:01 AM Prabhakar <prabhakar.csengg@gmail.com> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.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@bp.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@12800c00 { > status = "disabled"; > }; > > - wdt2: watchdog@12800400 { > - 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@12801000 { > 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@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
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Hi All, This patch series aims to drop WDT CH2 from SoC DTSIs and enabling of it from board DTSs as WDT CH2 is specifically to check the operation of Cortex-M33 CPU. Cheers, Prabhakar Lad Prabhakar (5): arm64: dts: renesas: rzg2l-smarc-som: Drop enabling WDT2 arm64: dts: renesas: rzg2lc-smarc-som: Drop enabling WDT2 arm64: dts: renesas: r9a07g044: Drop WDT2 node arm64: dts: renesas: r9a07g054: Drop WDT2 node arm64: dts: renesas: r9a07g043: Drop WDT2 node arch/arm64/boot/dts/renesas/r9a07g043.dtsi | 15 --------------- arch/arm64/boot/dts/renesas/r9a07g044.dtsi | 15 --------------- arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 15 --------------- arch/arm64/boot/dts/renesas/rzg2l-smarc-som.dtsi | 5 ----- arch/arm64/boot/dts/renesas/rzg2lc-smarc-som.dtsi | 5 ----- 5 files changed, 55 deletions(-) base-commit: f621040b30c93b1a054c0d12b6e310eecbb1a58b