| Message ID | 20210518044910.608878-1-yangyingliang@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | viresh kumar |
| Headers | show |
| Series | [-next] opp: use list_del_init instead of list_del/INIT_LIST_HEAD | expand |
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@huawei.com> > --- > 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.
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);
Using list_del_init() instead of list_del() + INIT_LIST_HEAD() to simpify the code. Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> --- drivers/opp/of.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)