2007-02-02 21:42:58

by [email protected]

Subject: d80211, redundant casts to iw_handler_def

These two casts are redundant.

Signed-off-by: Jon Smirl <[email protected]>

--
diff --git a/net/d80211/ieee80211.c b/net/d80211/ieee80211.c
index 7353ed3..260ce9c 100644
--- a/net/d80211/ieee80211.c
+++ b/net/d80211/ieee80211.c
@@ -4313,8 +4313,7 @@ void ieee80211_if_setup(struct net_device *dev)
{
ether_setup(dev);
dev->hard_start_xmit = ieee80211_subif_start_xmit;
- dev->wireless_handlers =
- (struct iw_handler_def *) &ieee80211_iw_handler_def;
+ dev->wireless_handlers = &ieee80211_iw_handler_def;
dev->do_ioctl = ieee80211_ioctl;
dev->set_mac_address = ieee80211_set_mac_address;
dev->set_multicast_list = ieee80211_set_multicast_list;
@@ -4472,8 +4471,7 @@ struct ieee80211_hw *ieee80211_alloc_hw(size_t
priv_data_len,
sta_info_init(local);

mdev->hard_start_xmit = ieee80211_master_start_xmit;
- mdev->wireless_handlers =
- (struct iw_handler_def *) &ieee80211_iw_master_handler_def;
+ mdev->wireless_handlers = &ieee80211_iw_master_handler_def;
mdev->do_ioctl = ieee80211_ioctl;
mdev->change_mtu = ieee80211_change_mtu;
mdev->tx_timeout = ieee80211_tx_timeout;

2007-02-05 21:58:25

by Jiri Benc

Subject: Re: d80211, redundant casts to iw_handler_def

On Fri, 2 Feb 2007 16:42:57 -0500, Jon Smirl wrote:
> These two casts are redundant.

Applied to my tree, thanks!

Jiri

--
Jiri Benc
SUSE Labs