Commit b8e1bddc authored by Tony Lindgren's avatar Tony Lindgren

ARM: OMAP2+: Use list_for_each_entry for hwmod slave_ports

We are just iterating over the slave_ports lists, so we can
use list_for_each_entry() no problem.
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
parent a1e31235
...@@ -220,29 +220,6 @@ static bool inited; ...@@ -220,29 +220,6 @@ static bool inited;
/* Private functions */ /* Private functions */
/**
* _fetch_next_ocp_if - return the next OCP interface in a list
* @p: ptr to a ptr to the list_head inside the ocp_if to return
* @i: pointer to the index of the element pointed to by @p in the list
*
* Return a pointer to the struct omap_hwmod_ocp_if record
* containing the struct list_head pointed to by @p, and increment
* @p such that a future call to this routine will return the next
* record.
*/
static struct omap_hwmod_ocp_if *_fetch_next_ocp_if(struct list_head **p,
int *i)
{
struct omap_hwmod_ocp_if *oi;
oi = list_entry(*p, struct omap_hwmod_ocp_if, node);
*p = (*p)->next;
*i = *i + 1;
return oi;
}
/** /**
* _update_sysc_cache - return the module OCP_SYSCONFIG register, keep copy * _update_sysc_cache - return the module OCP_SYSCONFIG register, keep copy
* @oh: struct omap_hwmod * * @oh: struct omap_hwmod *
...@@ -779,15 +756,10 @@ static int _init_main_clk(struct omap_hwmod *oh) ...@@ -779,15 +756,10 @@ static int _init_main_clk(struct omap_hwmod *oh)
static int _init_interface_clks(struct omap_hwmod *oh) static int _init_interface_clks(struct omap_hwmod *oh)
{ {
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p;
struct clk *c; struct clk *c;
int i = 0;
int ret = 0; int ret = 0;
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
if (!os->clk) if (!os->clk)
continue; continue;
...@@ -890,19 +862,13 @@ static void _disable_optional_clocks(struct omap_hwmod *oh) ...@@ -890,19 +862,13 @@ static void _disable_optional_clocks(struct omap_hwmod *oh)
static int _enable_clocks(struct omap_hwmod *oh) static int _enable_clocks(struct omap_hwmod *oh)
{ {
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p;
int i = 0;
pr_debug("omap_hwmod: %s: enabling clocks\n", oh->name); pr_debug("omap_hwmod: %s: enabling clocks\n", oh->name);
if (oh->_clk) if (oh->_clk)
clk_enable(oh->_clk); clk_enable(oh->_clk);
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
if (os->_clk && (os->flags & OCPIF_SWSUP_IDLE)) if (os->_clk && (os->flags & OCPIF_SWSUP_IDLE))
clk_enable(os->_clk); clk_enable(os->_clk);
} }
...@@ -924,19 +890,13 @@ static int _enable_clocks(struct omap_hwmod *oh) ...@@ -924,19 +890,13 @@ static int _enable_clocks(struct omap_hwmod *oh)
static int _disable_clocks(struct omap_hwmod *oh) static int _disable_clocks(struct omap_hwmod *oh)
{ {
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p;
int i = 0;
pr_debug("omap_hwmod: %s: disabling clocks\n", oh->name); pr_debug("omap_hwmod: %s: disabling clocks\n", oh->name);
if (oh->_clk) if (oh->_clk)
clk_disable(oh->_clk); clk_disable(oh->_clk);
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
if (os->_clk && (os->flags & OCPIF_SWSUP_IDLE)) if (os->_clk && (os->flags & OCPIF_SWSUP_IDLE))
clk_disable(os->_clk); clk_disable(os->_clk);
} }
...@@ -1175,16 +1135,11 @@ static int _get_sdma_req_by_name(struct omap_hwmod *oh, const char *name, ...@@ -1175,16 +1135,11 @@ static int _get_sdma_req_by_name(struct omap_hwmod *oh, const char *name,
static int _get_addr_space_by_name(struct omap_hwmod *oh, const char *name, static int _get_addr_space_by_name(struct omap_hwmod *oh, const char *name,
u32 *pa_start, u32 *pa_end) u32 *pa_start, u32 *pa_end)
{ {
int i, j; int j;
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p = NULL;
bool found = false; bool found = false;
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
i = 0;
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
if (!os->addr) if (!os->addr)
return -ENOENT; return -ENOENT;
...@@ -1224,18 +1179,13 @@ static int _get_addr_space_by_name(struct omap_hwmod *oh, const char *name, ...@@ -1224,18 +1179,13 @@ static int _get_addr_space_by_name(struct omap_hwmod *oh, const char *name,
static void __init _save_mpu_port_index(struct omap_hwmod *oh) static void __init _save_mpu_port_index(struct omap_hwmod *oh)
{ {
struct omap_hwmod_ocp_if *os = NULL; struct omap_hwmod_ocp_if *os = NULL;
struct list_head *p;
int i = 0;
if (!oh) if (!oh)
return; return;
oh->_int_flags |= _HWMOD_NO_MPU_PORT; oh->_int_flags |= _HWMOD_NO_MPU_PORT;
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
if (os->user & OCP_USER_MPU) { if (os->user & OCP_USER_MPU) {
oh->_mpu_port = os; oh->_mpu_port = os;
oh->_int_flags &= ~_HWMOD_NO_MPU_PORT; oh->_int_flags &= ~_HWMOD_NO_MPU_PORT;
...@@ -2436,15 +2386,11 @@ static int __init _init(struct omap_hwmod *oh, void *data) ...@@ -2436,15 +2386,11 @@ static int __init _init(struct omap_hwmod *oh, void *data)
static void __init _setup_iclk_autoidle(struct omap_hwmod *oh) static void __init _setup_iclk_autoidle(struct omap_hwmod *oh)
{ {
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p;
int i = 0;
if (oh->_state != _HWMOD_STATE_INITIALIZED) if (oh->_state != _HWMOD_STATE_INITIALIZED)
return; return;
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
if (!os->_clk) if (!os->_clk)
continue; continue;
...@@ -3288,15 +3234,11 @@ int omap_hwmod_count_resources(struct omap_hwmod *oh, unsigned long flags) ...@@ -3288,15 +3234,11 @@ int omap_hwmod_count_resources(struct omap_hwmod *oh, unsigned long flags)
ret += _count_sdma_reqs(oh); ret += _count_sdma_reqs(oh);
if (flags & IORESOURCE_MEM) { if (flags & IORESOURCE_MEM) {
int i = 0;
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p = oh->slave_ports.next;
while (i < oh->slaves_cnt) { list_for_each_entry(os, &oh->slave_ports, node)
os = _fetch_next_ocp_if(&p, &i);
ret += _count_ocp_if_addr_spaces(os); ret += _count_ocp_if_addr_spaces(os);
} }
}
return ret; return ret;
} }
...@@ -3314,7 +3256,6 @@ int omap_hwmod_count_resources(struct omap_hwmod *oh, unsigned long flags) ...@@ -3314,7 +3256,6 @@ int omap_hwmod_count_resources(struct omap_hwmod *oh, unsigned long flags)
int omap_hwmod_fill_resources(struct omap_hwmod *oh, struct resource *res) int omap_hwmod_fill_resources(struct omap_hwmod *oh, struct resource *res)
{ {
struct omap_hwmod_ocp_if *os; struct omap_hwmod_ocp_if *os;
struct list_head *p;
int i, j, mpu_irqs_cnt, sdma_reqs_cnt, addr_cnt; int i, j, mpu_irqs_cnt, sdma_reqs_cnt, addr_cnt;
int r = 0; int r = 0;
...@@ -3344,11 +3285,7 @@ int omap_hwmod_fill_resources(struct omap_hwmod *oh, struct resource *res) ...@@ -3344,11 +3285,7 @@ int omap_hwmod_fill_resources(struct omap_hwmod *oh, struct resource *res)
r++; r++;
} }
p = oh->slave_ports.next; list_for_each_entry(os, &oh->slave_ports, node) {
i = 0;
while (i < oh->slaves_cnt) {
os = _fetch_next_ocp_if(&p, &i);
addr_cnt = _count_ocp_if_addr_spaces(os); addr_cnt = _count_ocp_if_addr_spaces(os);
for (j = 0; j < addr_cnt; j++) { for (j = 0; j < addr_cnt; j++) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment