Use list_for_each_entry() instead of list_for_each() to simplify the code. Signed-off-by: Geliang Tang <geliangt...@163.com> --- net/dsa/dsa.c | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c index 208d1b2..2ab4a19 100644 --- a/net/dsa/dsa.c +++ b/net/dsa/dsa.c @@ -53,19 +53,14 @@ EXPORT_SYMBOL_GPL(unregister_switch_driver); static struct dsa_switch_driver * dsa_switch_probe(struct device *host_dev, int sw_addr, char **_name) { - struct dsa_switch_driver *ret; - struct list_head *list; + struct dsa_switch_driver *ret, *drv; char *name; ret = NULL; name = NULL; mutex_lock(&dsa_switch_drivers_mutex); - list_for_each(list, &dsa_switch_drivers) { - struct dsa_switch_driver *drv; - - drv = list_entry(list, struct dsa_switch_driver, list); - + list_for_each_entry(drv, &dsa_switch_drivers, list) { name = drv->probe(host_dev, sw_addr); if (name != NULL) { ret = drv; -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html