lkml.org 
[lkml]   [2022]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/3] clk: mediatek: support COMMON_CLK_MT8192 module build
From
Date
Quoting Miles Chen (2022-01-14 00:35:10)
> 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.

Yes

>
> >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?
>

Yes

\
 
 \ /
  Last update: 2022-01-14 23:20    [W:0.037 / U:1.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site