Commit c37235cc authored by Moritz Fischer's avatar Moritz Fischer Committed by Greg Kroah-Hartman

fpga: bridge: Replace open-coded list_for_each + list_entry

Replaces open-coded list_for_each() + list_entry() with macro
list_for_each_entry()
Signed-off-by: default avatarMoritz Fischer <mdf@kernel.org>
Cc: linux-fpga@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Acked-by: default avatarAlan Tull <atull@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 21f8ba2e
...@@ -146,11 +146,9 @@ EXPORT_SYMBOL_GPL(fpga_bridge_put); ...@@ -146,11 +146,9 @@ EXPORT_SYMBOL_GPL(fpga_bridge_put);
int fpga_bridges_enable(struct list_head *bridge_list) int fpga_bridges_enable(struct list_head *bridge_list)
{ {
struct fpga_bridge *bridge; struct fpga_bridge *bridge;
struct list_head *node;
int ret; int ret;
list_for_each(node, bridge_list) { list_for_each_entry(bridge, bridge_list, node) {
bridge = list_entry(node, struct fpga_bridge, node);
ret = fpga_bridge_enable(bridge); ret = fpga_bridge_enable(bridge);
if (ret) if (ret)
return ret; return ret;
...@@ -172,11 +170,9 @@ EXPORT_SYMBOL_GPL(fpga_bridges_enable); ...@@ -172,11 +170,9 @@ EXPORT_SYMBOL_GPL(fpga_bridges_enable);
int fpga_bridges_disable(struct list_head *bridge_list) int fpga_bridges_disable(struct list_head *bridge_list)
{ {
struct fpga_bridge *bridge; struct fpga_bridge *bridge;
struct list_head *node;
int ret; int ret;
list_for_each(node, bridge_list) { list_for_each_entry(bridge, bridge_list, node) {
bridge = list_entry(node, struct fpga_bridge, node);
ret = fpga_bridge_disable(bridge); ret = fpga_bridge_disable(bridge);
if (ret) if (ret)
return ret; return ret;
...@@ -196,13 +192,10 @@ EXPORT_SYMBOL_GPL(fpga_bridges_disable); ...@@ -196,13 +192,10 @@ EXPORT_SYMBOL_GPL(fpga_bridges_disable);
*/ */
void fpga_bridges_put(struct list_head *bridge_list) void fpga_bridges_put(struct list_head *bridge_list)
{ {
struct fpga_bridge *bridge; struct fpga_bridge *bridge, *next;
struct list_head *node, *next;
unsigned long flags; unsigned long flags;
list_for_each_safe(node, next, bridge_list) { list_for_each_entry_safe(bridge, next, bridge_list, node) {
bridge = list_entry(node, struct fpga_bridge, node);
fpga_bridge_put(bridge); fpga_bridge_put(bridge);
spin_lock_irqsave(&bridge_list_lock, flags); spin_lock_irqsave(&bridge_list_lock, flags);
......
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