Re: [PATCH -next] opp: use list_del_init instead of list_del/INIT_LIST_HEAD

From: Viresh Kumar
Date: Thu May 20 2021 - 00:03:42 EST


On 18-05-21, 12:49, Yang Yingliang wrote:
> Using list_del_init() instead of list_del() + INIT_LIST_HEAD()
> to simpify the code.
>
> Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx>
> ---
> drivers/opp/of.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/opp/of.c b/drivers/opp/of.c
> index c582a9ca397b..aa75a1caf08a 100644
> --- a/drivers/opp/of.c
> +++ b/drivers/opp/of.c
> @@ -433,8 +433,7 @@ static void lazy_link_required_opp_table(struct opp_table *new_table)
>
> /* All required opp-tables found, remove from lazy list */
> if (!lazy) {
> - list_del(&opp_table->lazy);
> - INIT_LIST_HEAD(&opp_table->lazy);
> + list_del_init(&opp_table->lazy);
>
> list_for_each_entry(opp, &opp_table->opp_list, node)
> _required_opps_available(opp, opp_table->required_opp_count);

Applied. Thanks.

--
viresh