diff options
author | Moritz Fischer <mdf@kernel.org> | 2017-03-10 15:47:11 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-17 02:10:48 -0400 |
commit | c37235cce31d5e03b73692f31e994964f18e2dc2 (patch) | |
tree | 4c7304e86b6d6cb0e1e3f63717a0c3cdbee8f156 /drivers/fpga/fpga-bridge.c | |
parent | 21f8ba2ef378d906374a26f7abc05fb92219e9c6 (diff) |
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: Moritz Fischer <mdf@kernel.org>
Cc: linux-fpga@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Acked-by: Alan Tull <atull@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/fpga/fpga-bridge.c')
-rw-r--r-- | drivers/fpga/fpga-bridge.c | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/drivers/fpga/fpga-bridge.c b/drivers/fpga/fpga-bridge.c index e04a899d1a90..9651aa56244a 100644 --- a/drivers/fpga/fpga-bridge.c +++ b/drivers/fpga/fpga-bridge.c | |||
@@ -146,11 +146,9 @@ EXPORT_SYMBOL_GPL(fpga_bridge_put); | |||
146 | int fpga_bridges_enable(struct list_head *bridge_list) | 146 | int fpga_bridges_enable(struct list_head *bridge_list) |
147 | { | 147 | { |
148 | struct fpga_bridge *bridge; | 148 | struct fpga_bridge *bridge; |
149 | struct list_head *node; | ||
150 | int ret; | 149 | int ret; |
151 | 150 | ||
152 | list_for_each(node, bridge_list) { | 151 | list_for_each_entry(bridge, bridge_list, node) { |
153 | bridge = list_entry(node, struct fpga_bridge, node); | ||
154 | ret = fpga_bridge_enable(bridge); | 152 | ret = fpga_bridge_enable(bridge); |
155 | if (ret) | 153 | if (ret) |
156 | return ret; | 154 | return ret; |
@@ -172,11 +170,9 @@ EXPORT_SYMBOL_GPL(fpga_bridges_enable); | |||
172 | int fpga_bridges_disable(struct list_head *bridge_list) | 170 | int fpga_bridges_disable(struct list_head *bridge_list) |
173 | { | 171 | { |
174 | struct fpga_bridge *bridge; | 172 | struct fpga_bridge *bridge; |
175 | struct list_head *node; | ||
176 | int ret; | 173 | int ret; |
177 | 174 | ||
178 | list_for_each(node, bridge_list) { | 175 | list_for_each_entry(bridge, bridge_list, node) { |
179 | bridge = list_entry(node, struct fpga_bridge, node); | ||
180 | ret = fpga_bridge_disable(bridge); | 176 | ret = fpga_bridge_disable(bridge); |
181 | if (ret) | 177 | if (ret) |
182 | return ret; | 178 | return ret; |
@@ -196,13 +192,10 @@ EXPORT_SYMBOL_GPL(fpga_bridges_disable); | |||
196 | */ | 192 | */ |
197 | void fpga_bridges_put(struct list_head *bridge_list) | 193 | void fpga_bridges_put(struct list_head *bridge_list) |
198 | { | 194 | { |
199 | struct fpga_bridge *bridge; | 195 | struct fpga_bridge *bridge, *next; |
200 | struct list_head *node, *next; | ||
201 | unsigned long flags; | 196 | unsigned long flags; |
202 | 197 | ||
203 | list_for_each_safe(node, next, bridge_list) { | 198 | list_for_each_entry_safe(bridge, next, bridge_list, node) { |
204 | bridge = list_entry(node, struct fpga_bridge, node); | ||
205 | |||
206 | fpga_bridge_put(bridge); | 199 | fpga_bridge_put(bridge); |
207 | 200 | ||
208 | spin_lock_irqsave(&bridge_list_lock, flags); | 201 | spin_lock_irqsave(&bridge_list_lock, flags); |