ieee802154: hwsim: remove not used ifup list
Subsystems (2)
PATCH 1/1 ieee802154: hwsim: remove not used ifup list
This list is a leftover from fakelb driver which had always a full mesh topology. Idea was to remember all phy's which are currently used by the subsystem and deliver everything out. The hwsim driver works differently each phy has a list of other phy's to deliver frames which allows a own mesh topology.
--- drivers/net/ieee802154/mac802154_hwsim.c | 8 -------- 1 file changed, 8 deletions(-)
drivers/net/ieee802154/mac802154_hwsim.c +0 -8
@@ -37,8 +37,6 @@ MODULE_LICENSE("GPL");static LIST_HEAD (hwsim_phys);static DEFINE_MUTEX (hwsim_phys_lock); -static LIST_HEAD (hwsim_ifup_phys); -static struct platform_device *mac802154hwsim_dev;/* MAC802154_HWSIM netlink family */
@@ -85,7 +83,6 @@ struct hwsim_phy {struct list_head edges;struct list_head list; -struct list_head list_ifup; }; static int hwsim_add_one(struct genl_info *info, struct device *dev,
@@ -159,9 +156,6 @@ static int hwsim_hw_start(struct ieee802154_hw *hw)struct hwsim_phy *phy = hw->priv; phy->suspended = false; -list_add_rcu (&phy->list_ifup, &hwsim_ifup_phys); -synchronize_rcu (); -return 0 ; }
@@ -170,8 +164,6 @@ static void hwsim_hw_stop(struct ieee802154_hw *hw)struct hwsim_phy *phy = hw->priv; phy->suspended = true; -list_del_rcu (&phy->list_ifup); -synchronize_rcu (); } static int
Signature · 1 line
2.11.0
Stefan Schmidt
maintainerHello.
Quoted 44 lines · Alexander Aring
This list is a leftover from fakelb driver which had always a full mesh topology. Idea was to remember all phy's which are currently used by the subsystem and deliver everything out. The hwsim driver works differently each phy has a list of other phy's to deliver frames which allows a own mesh topology.
Signed-off-by Alexander Aring <[email protected]>--- drivers/net/ieee802154/mac802154_hwsim.c | 8 -------- 1 file changed, 8 deletions(-)drivers/net/ieee802154/mac802154_hwsim.c +0
-6@@ -37,8 +37,6 @@ MODULE_LICENSE("GPL");static LIST_HEAD (hwsim_phys);static DEFINE_MUTEX (hwsim_phys_lock); -static LIST_HEAD (hwsim_ifup_phys); -static struct platform_device *mac802154hwsim_dev;/* MAC802154_HWSIM netlink family */ @@ -85,7 +83,6 @@ struct hwsim_phy {struct list_head edges;struct list_head list; -struct list_head list_ifup; }; static int hwsim_add_one(struct genl_info *info, struct device *dev,@@ -159,9 +156,6 @@ static int hwsim_hw_start(struct ieee802154_hw *hw)struct hwsim_phy *phy = hw->priv; phy->suspended = false; -list_add_rcu (&phy->list_ifup, &hwsim_ifup_phys); -synchronize_rcu (); -return 0 ; }
drivers/net/ieee802154/mac802154_hwsim.c static void hwsim_hw_stop(struct ieee802154_hw *hw) · line 169
Quoted 8 lines
drivers/net/ieee802154/mac802154_hwsim.c +0
-2@@ -170,8 +164,6 @@ static void hwsim_hw_stop(struct ieee802154_hw *hw)struct hwsim_phy *phy = hw->priv; phy->suspended = true; -list_del_rcu (&phy->list_ifup); -synchronize_rcu (); } static int
This patch has been applied to the wpan-next tree and will be part of the next pull request to net-next. Thanks!
regards
Stefan Schmidt