Commit 264a472d authored by Ajay Singh's avatar Ajay Singh Committed by Greg Kroah-Hartman

staging: wilc1000: remove unnecessary loop to traverse vif interfaces

Cleanup patch to avoid loop to traverse the interfaces instead make use
of vif received from net_device priv data.
Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 2f5e568b
...@@ -1634,19 +1634,14 @@ int wilc_init(struct net_device *dev, struct host_if_drv **hif_drv_handler) ...@@ -1634,19 +1634,14 @@ int wilc_init(struct net_device *dev, struct host_if_drv **hif_drv_handler)
struct host_if_drv *hif_drv; struct host_if_drv *hif_drv;
struct wilc_vif *vif = netdev_priv(dev); struct wilc_vif *vif = netdev_priv(dev);
struct wilc *wilc = vif->wilc; struct wilc *wilc = vif->wilc;
int i;
hif_drv = kzalloc(sizeof(*hif_drv), GFP_KERNEL); hif_drv = kzalloc(sizeof(*hif_drv), GFP_KERNEL);
if (!hif_drv) if (!hif_drv)
return -ENOMEM; return -ENOMEM;
*hif_drv_handler = hif_drv; *hif_drv_handler = hif_drv;
for (i = 0; i < wilc->vif_num; i++)
if (dev == wilc->vif[i]->ndev) {
wilc->vif[i]->hif_drv = hif_drv;
break;
}
vif->hif_drv = hif_drv;
vif->obtaining_ip = false; vif->obtaining_ip = false;
if (wilc->clients_count == 0) if (wilc->clients_count == 0)
......
...@@ -614,7 +614,6 @@ static int wilc_mac_open(struct net_device *ndev) ...@@ -614,7 +614,6 @@ static int wilc_mac_open(struct net_device *ndev)
struct wilc_priv *priv = wdev_priv(vif->ndev->ieee80211_ptr); struct wilc_priv *priv = wdev_priv(vif->ndev->ieee80211_ptr);
unsigned char mac_add[ETH_ALEN] = {0}; unsigned char mac_add[ETH_ALEN] = {0};
int ret = 0; int ret = 0;
int i = 0;
if (!wl || !wl->dev) { if (!wl || !wl->dev) {
netdev_err(ndev, "device not ready\n"); netdev_err(ndev, "device not ready\n");
...@@ -633,19 +632,14 @@ static int wilc_mac_open(struct net_device *ndev) ...@@ -633,19 +632,14 @@ static int wilc_mac_open(struct net_device *ndev)
return ret; return ret;
} }
for (i = 0; i < wl->vif_num; i++) { wilc_set_wfi_drv_handler(vif, wilc_get_vif_idx(vif), vif->iftype,
if (ndev == wl->vif[i]->ndev) { vif->idx);
wilc_set_wfi_drv_handler(vif, wilc_get_vif_idx(vif),
vif->iftype, vif->idx);
wilc_set_operation_mode(vif, vif->iftype); wilc_set_operation_mode(vif, vif->iftype);
break;
}
}
wilc_get_mac_address(vif, mac_add); wilc_get_mac_address(vif, mac_add);
netdev_dbg(ndev, "Mac address: %pM\n", mac_add); netdev_dbg(ndev, "Mac address: %pM\n", mac_add);
memcpy(wl->vif[i]->src_addr, mac_add, ETH_ALEN); memcpy(vif->src_addr, mac_add, ETH_ALEN);
memcpy(ndev->dev_addr, wl->vif[i]->src_addr, ETH_ALEN); memcpy(ndev->dev_addr, vif->src_addr, ETH_ALEN);
if (!is_valid_ether_addr(ndev->dev_addr)) { if (!is_valid_ether_addr(ndev->dev_addr)) {
netdev_err(ndev, "Wrong MAC address\n"); netdev_err(ndev, "Wrong MAC address\n");
......
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