Messages in this thread | | | Subject | Re: [PATCH] mediatek/clk-mt8173-apmixedsys: use devm_of_iomap to avoid resource leak in clk_mt8173_apmixed_probe | From | Stephen Boyd <> | Date | Thu, 11 May 2023 15:37:45 -0700 |
| |
Quoting Hao Ge (2023-05-10 18:26:54) > On Wed, 10 May 2023 14:18:08 -0700 > Stephen Boyd <sboyd@kernel.org> wrote: > > > Quoting Hao Ge (2023-05-08 19:12:14) > > > Use devm_platform_ioremap_resource to take the place of of_iomap for > > > avoid that we don't called iounmap when return some error or remove > > > device. > > > > > > Fixes: 4c02c9af3cb9 ("clk: mediatek: mt8173: Break down clock > > > drivers and allow module build") Signed-off-by: Hao Ge > > > <gehao@kylinos.cn> --- > > > > Is this v3? > Hi Stephen > > yes,Perhaps you are too busy,I have not received a response > in the future, so you are unsure about the status of V2. > So I adjusted the code and title make changes smaller and > sent V3 for it.It is precisely because of these that I did > not continue the original mail thread. >
I'm waiting for the usual mediatek clk driver folks to review the patch.
| |