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

Miles Chen miles.chen at mediatek.com
Mon Jan 17 01:18:04 PST 2022


>> 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
>> we cannot remove then unload it.
>> I think I have to add some unregister logic to these drivers.
>
>Yes

I will do this later.

>
>> >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 pa=
>
>Yes

I post a patch to fix this first:
https://lore.kernel.org/lkml/20220117090030.13064-1-miles.chen@mediatek.com/T/#u

After this, I think that I can use mtk_clk_simple_probe() in other mediatek clk
drivers by submitting some clean up patches.

Miles



More information about the Linux-mediatek mailing list