On Wednesday 21 February 2007 17:37, Michael Buesch wrote:
> On Wednesday 21 February 2007 17:02, Ivo van Doorn wrote:
> > Let broadcom use the antenna_tx field for hardware configuration.
> > I did not check if the device is also capable of RX antenna selection,
> > but current implementation only suggested support for the TX antenna.
> >
> > Signed-off-by: Ivo van Doorn <IvDoorn@xxxxxxxxx>
> >
> > ---
> >
> > diff --git a/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c b/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c
> > index cdce227..580ea33 100644
> > --- a/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c
> > +++ b/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c
> > @@ -2609,7 +2609,7 @@ static int bcm43xx_dev_config(struct ieee80211_hw *hw,
> > int antenna = -1;
> > int err = 0;
> >
> > - switch (conf->antenna_sel) {
> > + switch (conf->antenna_sel_tx) {
>
> Nope, this is conf->antenna_sel_rx
> ;)
On further inspection not even that. :P
The outcome of that switch is stored in the antenna variable, which is used with
bcm43xx_mgmtframe_txantenna(dev, antenna);
bcm43xx_set_rx_antenna(dev, antenna);
So I guess that needs to be fixed to create a switch statement for
the RX antenna and one for the TX antenna?
Ivo
-
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
