* Re: [PATCH v3] clk: hisilicon: refine hi3620_mmc_clk_init() and fix memory leak issues
[not found] ` <ef1eac81-c4f9-ca4d-f056-3cdbddcaad73@web.de>
@ 2020-11-19 12:01 ` Dongjiu Geng
[not found] ` <393a7eca-e6a0-ddf8-e0aa-ca9121acf4f3@web.de>
0 siblings, 1 reply; 2+ messages in thread
From: Dongjiu Geng @ 2020-11-19 12:01 UTC (permalink / raw)
To: Markus Elfring, linux-clk
Cc: linux-kernel, kernel-janitors, Michael Turquette, Stephen Boyd,
Julia Lawall
On 2020/11/19 17:07, Markus Elfring wrote:
>> Refine hi3620_mmc_clk_init() to use of_clk_add_hw_provider()
>> instead of of_clk_add_provider(), …
>
> …
>> +++ b/drivers/clk/hisilicon/clk-hi3620.c
> …
>> @@ -439,17 +440,22 @@ static struct clk *hisi_register_clk_mmc(struct hisi_mmc_clock *mmc_clk,
> …
>> + err = clk_hw_register(NULL, hw);
>> +
>> + if (err) {
>
> I suggest to omit a blank line here.
>
>
> …
>> +++ b/drivers/clk/hisilicon/clk.c
>> @@ -65,25 +65,26 @@ struct hisi_clock_data *hisi_clk_init(struct device_node *np,
> …
>> of_clk_add_provider(np, of_clk_src_onecell_get, &clk_data->clk_data);
>> return clk_data;
>> -err_data:
>> +free_clk_data:
>
> How do you think about to adjust also such a function call for this function implementation?
>
> Will further collateral evolution become interesting?
> https://elixir.bootlin.com/linux/v5.10-rc4/C/ident/of_clk_add_provider
Thanks for the review.
How about we adjust such a function call in another series patches? I suggest do it in another series.
For this patch, how about we firstly merge it after I fix the above comments(omit a blank line)?
>
> Regards,
> Markus
> .
>
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [v3] clk: hisilicon: refine hi3620_mmc_clk_init() and fix memory leak issues
[not found] ` <393a7eca-e6a0-ddf8-e0aa-ca9121acf4f3@web.de>
@ 2020-11-20 3:08 ` Dongjiu Geng
0 siblings, 0 replies; 2+ messages in thread
From: Dongjiu Geng @ 2020-11-20 3:08 UTC (permalink / raw)
To: Markus Elfring, linux-clk
Cc: linux-kernel, kernel-janitors, Michael Turquette, Stephen Boyd,
Julia Lawall
On 2020/11/19 22:40, Markus Elfring wrote:
>> How about we adjust such a function call in another series patches?
>
> You can try to offer desirable changes also in a corresponding patch series as usual.
sure, ok
>
> Regards,
> Markus
> .
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-11-20 3:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20201112192214.48926-1-gengdongjiu@huawei.com>
[not found] ` <ef1eac81-c4f9-ca4d-f056-3cdbddcaad73@web.de>
2020-11-19 12:01 ` [PATCH v3] clk: hisilicon: refine hi3620_mmc_clk_init() and fix memory leak issues Dongjiu Geng
[not found] ` <393a7eca-e6a0-ddf8-e0aa-ca9121acf4f3@web.de>
2020-11-20 3:08 ` [v3] " Dongjiu Geng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).