[PATCH 3/3] clk: mediatek: support COMMON_CLK_MT8192 module build

Miles Chen miles.chen at mediatek.com
Fri Jan 14 00:35:10 PST 2022


Hi,

>Have you tried removing the modules? And then reloading them? I also see

Thanks for the comment.
I did not try it because we do not unload our clk modules.
After testing I understand that the clock driver does not unregister clks so 
we cannot remove then unload it.

I think I have to add some unregister logic to these drivers.

>that mtk_clk_simple_probe() doesn't actually unregister anything, it
>just frees some allocations. Please fix this before making these
>modules.

You meant that mtk_clk_simple_probe() has no unregister logic in error path?

Miles



More information about the Linux-mediatek mailing list