Re: [PATCH v7 2/9] watchdog: rzg2l_wdt: Make the driver depend on PM

From: Geert Uytterhoeven
Date: Thu Feb 15 2024 - 09:34:42 EST


On Thu, Feb 15, 2024 at 3:08 PM Claudiu <claudiu.beznea@xxxxxxxxx> wrote:
> From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx>
>
> The rzg2l_wdt watchdog driver cannot work w/o CONFIG_PM=y (e.g. the
> clocks are enabled though pm_runtime_* specific APIs). To avoid building
> a driver that doesn't work make explicit the dependency on CONFIG_PM.
>
> Suggested-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx>
> ---
>
> Changes in v7:
> - updated the dependency to PM || COMPILE_TEST to be able to
> compile-test the driver when compiling for a
> !(ARCH_RZG2L || ARCH_R9A09G011) platform and CONFIG_PM is disabled

Although this doesn't make any practical difference:
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68korg

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