2017-05-21 2:58 GMT+09:00 Masahiro Yamada <yamada.masah...@socionext.com>:
> After long term efforts of fixing non-common clock implementations,
> clk_disable() is a no-op for a NULL pointer input, and this is now
> tree-wide consistent.
> All clock consumers can safely call clk_disable(_unprepare) without
> NULL pointer check.
> Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com>

Sorry, I retract this patch.

Krzysztof pointed out
cleanups only for clk_disable_unprepare() will lose the code symmetry.

NULL pointer checks for clk_prepare_enable() should be
removed to keep the code symmetrical.

This is possible for common-clock framework because
clk_prepare_enable() is also a no-op for a NULL clk input.
But it is not necessarily true for non-common clock implementations.

Best Regards
Masahiro Yamada
Freedreno mailing list

Reply via email to