On Mon, 2008-03-31 at 14:28 -0400, John W. Linville wrote: > From: Roy Marples <uberlord@xxxxxxxxxx> > > prism54 should set the carrier flags correctly when it thinks the > link can be used. > > Signed-off-by: Roy Marples <uberlord@xxxxxxxxxx> > --- > I found this in my pile from a long time back. Anyone have any > comments on it? Other than the fact that the carrier flag on wireless devices is kind of useless, it's fine. I guess it would be somewhat useful if we could ensure that the carrier would be on _only_ when the card was successfully associated and authenticated to the requested AP, but until every driver was converted properly it would be useless because you couldn't depend on it. Dan > drivers/net/wireless/prism54/isl_ioctl.c | 5 ++++- > drivers/net/wireless/prism54/islpci_dev.c | 4 +++- > 2 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/prism54/isl_ioctl.c b/drivers/net/wireless/prism54/isl_ioctl.c > index 838d510..25d42ac 100644 > --- a/drivers/net/wireless/prism54/isl_ioctl.c > +++ b/drivers/net/wireless/prism54/isl_ioctl.c > @@ -2093,8 +2093,11 @@ link_changed(struct net_device *ndev, u32 bitrate) > } else > send_simple_event(netdev_priv(ndev), > "Link established"); > - } else > + netif_carrier_on(ndev); > + } else { > send_simple_event(netdev_priv(ndev), "Link lost"); > + netif_carrier_off(ndev); > + } > } > > /* Beacon/ProbeResp payload header */ > diff --git a/drivers/net/wireless/prism54/islpci_dev.c b/drivers/net/wireless/prism54/islpci_dev.c > index f057fd9..9c8c6f2 100644 > --- a/drivers/net/wireless/prism54/islpci_dev.c > +++ b/drivers/net/wireless/prism54/islpci_dev.c > @@ -386,7 +386,9 @@ islpci_open(struct net_device *ndev) > } > > netif_start_queue(ndev); > -/* netif_mark_up( ndev ); */ > + > + /* Turn off carrier unless we know we have associated */ > + netif_carrier_off(ndev); > > return 0; > } -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html