mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-12-16 23:45:31 +08:00
ARM: OMAP2+: hwmod: drop unnecessary list initialization
ml->node and sl->node are currently initialized by means of INIT_LIST_HEAD(). That initialiation is followed by a list_add() call. Looking at what both these functions do we will have: ml->node.next = &ml->node; ml->node.prev = &ml->node; oi->master->master_ports.next.prev = &ml->node; ml->node.next = &oi->master->master_ports.next; ml->node.prev = &oi->master->master_ports; oi->master->master_ports.next = &ml->node; from this, it's clear that both INIT_LIST_HEAD() calls are unnecessary and can be safely removed. Signed-off-by: Felipe Balbi <balbi@ti.com> Signed-off-by: Paul Walmsley <paul@pwsan.com>
This commit is contained in:
parent
2e1a7b014f
commit
79005fbd3e
@ -2832,12 +2832,10 @@ static int __init _add_link(struct omap_hwmod_ocp_if *oi)
|
|||||||
_alloc_links(&ml, &sl);
|
_alloc_links(&ml, &sl);
|
||||||
|
|
||||||
ml->ocp_if = oi;
|
ml->ocp_if = oi;
|
||||||
INIT_LIST_HEAD(&ml->node);
|
|
||||||
list_add(&ml->node, &oi->master->master_ports);
|
list_add(&ml->node, &oi->master->master_ports);
|
||||||
oi->master->masters_cnt++;
|
oi->master->masters_cnt++;
|
||||||
|
|
||||||
sl->ocp_if = oi;
|
sl->ocp_if = oi;
|
||||||
INIT_LIST_HEAD(&sl->node);
|
|
||||||
list_add(&sl->node, &oi->slave->slave_ports);
|
list_add(&sl->node, &oi->slave->slave_ports);
|
||||||
oi->slave->slaves_cnt++;
|
oi->slave->slaves_cnt++;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user