Hi Biju, On Mon, Jun 10, 2024 at 2:00 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Also, I am planning to replace pm_runtime_enable(), pm_runtime_resume_and_get() > with devm_clk_enabled() to simplify the probe()/remove() as separate patch. > I guess it is ok to you?? I don't think that is a good idea, as it will cause breakage when extending PM Domain support on RZ/G2L (and RZ/G3S?). BTW, devm_pm_runtime_enable() does exist, devm_pm_runtime_resume_and_get() doesn't yet. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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