On Tue, Sep 21, 2010 at 03:36:28PM +0530, Felix Fietkau wrote:
> On 2010-09-21 7:54 AM, Vasanthakumar Thiagarajan wrote:
> > Signed-off-by: Vasanthakumar Thiagarajan <vasanth@xxxxxxxxxxx>
> > ---
> > drivers/net/wireless/ath/ath9k/hw.h | 1 +
> > drivers/net/wireless/ath/ath9k/main.c | 18 +++++++++++++++++-
> > 2 files changed, 18 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/net/wireless/ath/ath9k/hw.h b/drivers/net/wireless/ath/ath9k/hw.h
> > index df47f79..c1b4962 100644
> > --- a/drivers/net/wireless/ath/ath9k/hw.h
> > +++ b/drivers/net/wireless/ath/ath9k/hw.h
> > @@ -645,6 +645,7 @@ struct ath_hw {
> > struct ath9k_hw_capabilities caps;
> > struct ath9k_channel channels[38];
> > struct ath9k_channel *curchan;
> > + struct ath9k_channel prev_paprd_chan;
> >
> > union {
> > struct ar5416_eeprom_def def;
> > diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c
> > index a133878..9150788 100644
> > --- a/drivers/net/wireless/ath/ath9k/main.c
> > +++ b/drivers/net/wireless/ath/ath9k/main.c
> > @@ -266,6 +266,20 @@ int ath_set_channel(struct ath_softc *sc, struct ieee80211_hw *hw,
> > return r;
> > }
> >
> > +static bool is_paprd_done(struct ath_hw *ah)
> > +{
> > + struct ath9k_channel *curchan, *paprd_chan;
> > +
> > + curchan = ah->curchan;
> > + paprd_chan = &ah->prev_paprd_chan;
> > +
> > + if ((paprd_chan->channel == curchan->channel) &&
> > + paprd_chan->chanmode == curchan->chanmode)
> > + return true;
> > +
> > + return false;
> > +}
> That seems like code duplication to me. The caldata already has the
> channel number and the channel flags. ath9k_hw_reset() clears the entire
> caldata whenever that changes. Because of that, ah->caldata->paprd_done
> should have already been set to zero automatically after the reset
> triggered by an operating channel change.
> Is that part not working, or why did you write this patch?
> Either way, we should not have a separate check just for paprd, it
> belongs to the other calibrations.
I don't want to do paprd again whenever coming back from off-channel
(like during background scanning).
Vasanth
--
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
