On Mon, Aug 17, 2009 at 09:33:06PM +0200, Gábor Stefanik wrote: > 2009/8/14 Gábor Stefanik <[email protected]>: > > Check the mode the hardware is in, not the mode we used the last time. > > > > Signed-off-by: Gábor Stefanik <[email protected]> > > --- > > Mark, please test if this fixes the TX power control WARN_ON you were > > seeing. > > > > drivers/net/wireless/b43/phy_lp.c | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/net/wireless/b43/phy_lp.c > > b/drivers/net/wireless/b43/phy_lp.c > > index 292ee51..76457f7 100644 > > --- a/drivers/net/wireless/b43/phy_lp.c > > +++ b/drivers/net/wireless/b43/phy_lp.c > > @@ -1015,9 +1015,9 @@ static void lpphy_set_tx_power_control(struct > > b43_wldev *dev, > > struct b43_phy_lp *lpphy = dev->phy.lp; > > enum b43_lpphy_txpctl_mode oldmode; > > > > - oldmode = lpphy->txpctl_mode; > > lpphy_read_tx_pctl_mode_from_hardware(dev); > > - if (lpphy->txpctl_mode == mode) > > + oldmode = lpphy->txpctl_mode; > > + if (oldmode == mode) > > return; > > lpphy->txpctl_mode = mode; > > > > -- > > 1.6.2.4 > > > > John, any news on this one? I can't see it in wireless testing.
Larry said "It does not fix it here. I'll take a look at the specs"... -- John W. Linville Someday the world will need a hero, and you [email protected] might be all we have. Be ready. _______________________________________________ Bcm43xx-dev mailing list [email protected] https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
