Commit d1a5b6fb authored by Franky Lin's avatar Franky Lin Committed by John W. Linville

brcm80211: fmac: remove state from brcmf_if in fullmac

The usage of state decrease readability. Optimize the code flow to
get rid of it
Reviewed-by: default avatarRoland Vossen <rvossen@broadcom.com>
Reviewed-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent e1b83586
...@@ -122,7 +122,6 @@ ...@@ -122,7 +122,6 @@
/* For supporting multiple interfaces */ /* For supporting multiple interfaces */
#define BRCMF_MAX_IFS 16 #define BRCMF_MAX_IFS 16
#define BRCMF_DEL_IF -0xe
#define DOT11_BSSTYPE_ANY 2 #define DOT11_BSSTYPE_ANY 2
#define DOT11_MAX_DEFAULT_KEYS 4 #define DOT11_MAX_DEFAULT_KEYS 4
......
...@@ -58,7 +58,6 @@ struct brcmf_if { ...@@ -58,7 +58,6 @@ struct brcmf_if {
struct net_device *ndev; struct net_device *ndev;
struct net_device_stats stats; struct net_device_stats stats;
int idx; /* iface idx in dongle */ int idx; /* iface idx in dongle */
int state; /* interface state */
u8 mac_addr[ETH_ALEN]; /* assigned MAC address */ u8 mac_addr[ETH_ALEN]; /* assigned MAC address */
}; };
...@@ -456,12 +455,10 @@ void brcmf_rx_frame(struct brcmf_pub *drvr, int ifidx, struct sk_buff *skb, ...@@ -456,12 +455,10 @@ void brcmf_rx_frame(struct brcmf_pub *drvr, int ifidx, struct sk_buff *skb,
skb_mac_header(skb), skb_mac_header(skb),
&event, &data); &event, &data);
if (drvr_priv->iflist[ifidx] && if (drvr_priv->iflist[ifidx]) {
!drvr_priv->iflist[ifidx]->state)
ifp = drvr_priv->iflist[ifidx]; ifp = drvr_priv->iflist[ifidx];
if (ifp->ndev)
ifp->ndev->last_rx = jiffies; ifp->ndev->last_rx = jiffies;
}
drvr->dstats.rx_bytes += skb->len; drvr->dstats.rx_bytes += skb->len;
drvr->rx_packets++; /* Local count */ drvr->rx_packets++; /* Local count */
...@@ -896,7 +893,6 @@ brcmf_add_if(struct brcmf_info *drvr_priv, int ifidx, char *name, u8 *mac_addr) ...@@ -896,7 +893,6 @@ brcmf_add_if(struct brcmf_info *drvr_priv, int ifidx, char *name, u8 *mac_addr)
ifp->ndev = ndev; ifp->ndev = ndev;
ifp->info = drvr_priv; ifp->info = drvr_priv;
drvr_priv->iflist[ifidx] = ifp; drvr_priv->iflist[ifidx] = ifp;
ifp->state = BRCMF_E_IF_ADD;
ifp->idx = ifidx; ifp->idx = ifidx;
if (mac_addr != NULL) if (mac_addr != NULL)
memcpy(&ifp->mac_addr, mac_addr, ETH_ALEN); memcpy(&ifp->mac_addr, mac_addr, ETH_ALEN);
...@@ -910,7 +906,6 @@ brcmf_add_if(struct brcmf_info *drvr_priv, int ifidx, char *name, u8 *mac_addr) ...@@ -910,7 +906,6 @@ brcmf_add_if(struct brcmf_info *drvr_priv, int ifidx, char *name, u8 *mac_addr)
brcmf_dbg(TRACE, " ==== pid:%x, net_device for if:%s created ===\n", brcmf_dbg(TRACE, " ==== pid:%x, net_device for if:%s created ===\n",
current->pid, ifp->ndev->name); current->pid, ifp->ndev->name);
ifp->state = 0;
return 0; return 0;
} }
...@@ -926,7 +921,6 @@ void brcmf_del_if(struct brcmf_info *drvr_priv, int ifidx) ...@@ -926,7 +921,6 @@ void brcmf_del_if(struct brcmf_info *drvr_priv, int ifidx)
brcmf_dbg(ERROR, "Null interface\n"); brcmf_dbg(ERROR, "Null interface\n");
return; return;
} }
ifp->state = BRCMF_E_IF_DEL;
if (ifp->ndev) { if (ifp->ndev) {
if (ifidx == 0) { if (ifidx == 0) {
if (ifp->ndev->netdev_ops == &brcmf_netdev_ops_pri) { if (ifp->ndev->netdev_ops == &brcmf_netdev_ops_pri) {
......
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