RE: [PATCH] arm64: dts: renesas: rzg2ul-smarc-som: Drop enabling wdt2

From: Biju Das
Date: Mon Sep 19 2022 - 06:06:53 EST


Hi Prabhakar,

> -----Original Message-----
> From: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx>
> Sent: 19 September 2022 10:48
> To: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Cc: Magnus Damm <magnus.damm@xxxxxxxxx>; Rob Herring
> <robh+dt@xxxxxxxxxx>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@xxxxxxxxxx>; linux-renesas-
> soc@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Biju Das <biju.das.jz@xxxxxxxxxxxxxx>;
> Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
> Subject: Re: [PATCH] arm64: dts: renesas: rzg2ul-smarc-som: Drop
> enabling wdt2
>
> Hi Geert,
>
> Thank you for the review.
>
> On Sun, Sep 18, 2022 at 10:04 AM Geert Uytterhoeven <geert@linux-
> m68k.org> wrote:
> >
> > On Wed, Sep 14, 2022 at 3:43 PM Prabhakar
> <prabhakar.csengg@xxxxxxxxx> wrote:
> > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
> > >
> > > WDT CH2 is specifically to check the operation of Cortex-M33 CPU
> so
> > > don't enable WDT2 by default.
> > >
> For consistency I'll drop enabling WDT2 from the RZ/G2L{C} too.

Does remove unused clocks turn off WDT2 clock during late init??

If that is the case, we should remove entries from clock table as well?

Cheers,
Biju