On Thu, Aug 22, 2024 at 03:28:40PM +0200, Richard GENOUD wrote: > Le 22/08/2024 à 05:39, Lei Liu a écrit : > > The devm_clk_get_enabled() helpers: > > - call devm_clk_get() > > - call clk_prepare_enable() and register what is needed in order to > > call clk_disable_unprepare() when needed, as a managed resource. > > > > This simplifies the code and avoids calls to clk_disable_unprepare(). ... > > * The peripheral clock can now safely be disabled till the port > > * is used > > */ > > - clk_disable_unprepare(atmel_port->clk); > > - > Why removing this ? > This is not an error path. Good point, I wouldn't apply this patch as well as a few others in this series due to this reason. Instead it might make sense to add a comment on top of devm_clk_get() to explain why _enabled() variant is *not* used. -- With Best Regards, Andy Shevchenko
