Techwizz/kernel-fxtec-pro1x
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

prism54: set carrier flags correctly

Browse source 
> prism54 should set the carrier flags correctly when it thinks the
> link can be used.

Agreed, so sure, this is OK but I rather we turn the carrier on
or off *before* sending an event, like this.

Signed-off-by: Roy Marples <uberlord@gentoo.org>
Signed-off-by: Luis R. Rodriguez <mcgrof@winlab.rutgers.edu>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
Luis R. Rodriguez 2008-04-01 15:17:36 -04:00 • committed by John W. Linville
parent 8fe2b65a18
commit 7b463ced63
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
drivers/net/wireless/prism54/isl_ioctl.c
View file

@ -2081,6 +2081,7 @@ link_changed(struct net_device *ndev, u32 bitrate)
islpci_private *priv = netdev_priv(ndev); islpci_private *priv = netdev_priv(ndev);
if (bitrate) { if (bitrate) {
netif_carrier_on(ndev);
if (priv->iw_mode == IW_MODE_INFRA) { if (priv->iw_mode == IW_MODE_INFRA) {
union iwreq_data uwrq; union iwreq_data uwrq;
prism54_get_wap(ndev, NULL, (struct sockaddr *) &uwrq, prism54_get_wap(ndev, NULL, (struct sockaddr *) &uwrq,
@ -2089,8 +2090,10 @@ link_changed(struct net_device *ndev, u32 bitrate)
} else } else
send_simple_event(netdev_priv(ndev), send_simple_event(netdev_priv(ndev),
"Link established"); "Link established");
} else } else {
netif_carrier_off(ndev);
send_simple_event(netdev_priv(ndev), "Link lost"); send_simple_event(netdev_priv(ndev), "Link lost");
}
} }
/* Beacon/ProbeResp payload header */ /* Beacon/ProbeResp payload header */

4
drivers/net/wireless/prism54/islpci_dev.c
View file

@ -387,7 +387,9 @@ islpci_open(struct net_device *ndev)
} }
netif_start_queue(ndev); netif_start_queue(ndev);
/* netif_mark_up( ndev ); */
/* Turn off carrier unless we know we have associated */
netif_carrier_off(ndev);
return 0; return 0;
} }