* [PATCH v2 1/8] ath5k: add missing checks for rfgain probe @ 2011-07-05 3:11 Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 2/8] ath5k: apply the synth voltage tweak only on AR5112 rev 2 Felix Fietkau 2011-07-05 4:46 ` [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Nick Kossifidis 0 siblings, 2 replies; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm rfgain probe is only necessary for OFDM operation on AR5111 and AR5112. Signed-off-by: Felix Fietkau <nbd@openwrt.org> --- drivers/net/wireless/ath/ath5k/phy.c | 10 ++++++---- 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c index 5544191..d0bfcad 100644 --- a/drivers/net/wireless/ath/ath5k/phy.c +++ b/drivers/net/wireless/ath/ath5k/phy.c @@ -1604,11 +1604,13 @@ int ath5k_hw_phy_calibrate(struct ath5k_hw *ah, int ret; if (ah->ah_radio == AR5K_RF5110) - ret = ath5k_hw_rf5110_calibrate(ah, channel); - else { - ret = ath5k_hw_rf511x_iq_calibrate(ah); + return ath5k_hw_rf5110_calibrate(ah, channel); + + ret = ath5k_hw_rf511x_iq_calibrate(ah); + + if ((ah->ah_radio == AR5K_RF5112 || ah->ah_radio == AR5K_RF5112) && + (channel->hw_value & CHANNEL_OFDM)) ath5k_hw_request_rfgain_probe(ah); - } return ret; } -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 2/8] ath5k: apply the synth voltage tweak only on AR5112 rev 2 2011-07-05 3:11 [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 3/8] ath5k: fix reference clock frequency for spur mitigation on AR2413 Felix Fietkau 2011-07-05 4:46 ` [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Nick Kossifidis 1 sibling, 1 reply; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm Might fix some stability issues on newer chips Signed-off-by: Felix Fietkau <nbd@openwrt.org> Acked-by: Nick Kossifidis <mickflemm@gmail.com> --- drivers/net/wireless/ath/ath5k/phy.c | 19 +++++++++++-------- 1 files changed, 11 insertions(+), 8 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c index d0bfcad..a28ef9e 100644 --- a/drivers/net/wireless/ath/ath5k/phy.c +++ b/drivers/net/wireless/ath/ath5k/phy.c @@ -970,17 +970,20 @@ static int ath5k_hw_rfregs_init(struct ath5k_hw *ah, } /* Lower synth voltage on Rev 2 */ - ath5k_hw_rfb_op(ah, rf_regs, 2, - AR5K_RF_HIGH_VC_CP, true); + if (ah->ah_radio == AR5K_RF5112 && + (ah->ah_radio_5ghz_revision & AR5K_SREV_REV) > 0) { + ath5k_hw_rfb_op(ah, rf_regs, 2, + AR5K_RF_HIGH_VC_CP, true); - ath5k_hw_rfb_op(ah, rf_regs, 2, - AR5K_RF_MID_VC_CP, true); + ath5k_hw_rfb_op(ah, rf_regs, 2, + AR5K_RF_MID_VC_CP, true); - ath5k_hw_rfb_op(ah, rf_regs, 2, - AR5K_RF_LOW_VC_CP, true); + ath5k_hw_rfb_op(ah, rf_regs, 2, + AR5K_RF_LOW_VC_CP, true); - ath5k_hw_rfb_op(ah, rf_regs, 2, - AR5K_RF_PUSH_UP, true); + ath5k_hw_rfb_op(ah, rf_regs, 2, + AR5K_RF_PUSH_UP, true); + } /* Decrease power consumption on 5213+ BaseBand */ if (ah->ah_phy_revision >= AR5K_SREV_PHY_5212A) { -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 3/8] ath5k: fix reference clock frequency for spur mitigation on AR2413 2011-07-05 3:11 ` [PATCH v2 2/8] ath5k: apply the synth voltage tweak only on AR5112 rev 2 Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 4/8] ath5k: initialize common->clockrate Felix Fietkau 0 siblings, 1 reply; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm AR2413 uses the same reference clock as AR5413 Signed-off-by: Felix Fietkau <nbd@openwrt.org> --- drivers/net/wireless/ath/ath5k/phy.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c index a28ef9e..b6085d0 100644 --- a/drivers/net/wireless/ath/ath5k/phy.c +++ b/drivers/net/wireless/ath/ath5k/phy.c @@ -105,6 +105,7 @@ bool ath5k_hw_chan_has_spur_noise(struct ath5k_hw *ah, if ((ah->ah_radio == AR5K_RF5112) || (ah->ah_radio == AR5K_RF5413) || + (ah->ah_radio == AR5K_RF2413) || (ah->ah_mac_version == (AR5K_SREV_AR2417 >> 4))) refclk_freq = 40; else -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 4/8] ath5k: initialize common->clockrate 2011-07-05 3:11 ` [PATCH v2 3/8] ath5k: fix reference clock frequency for spur mitigation on AR2413 Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 5/8] ath5k: delay full calibration after reset Felix Fietkau 0 siblings, 1 reply; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm Fixes a division by zero when setting distance before activating the device for the first time. Signed-off-by: Felix Fietkau <nbd@openwrt.org> Acked-by: Nick Kossifidis <mickflemm@gmail.com> --- drivers/net/wireless/ath/ath5k/base.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c index a6623dd..a413aa7 100644 --- a/drivers/net/wireless/ath/ath5k/base.c +++ b/drivers/net/wireless/ath/ath5k/base.c @@ -2422,6 +2422,7 @@ ath5k_init_softc(struct ath5k_softc *sc, const struct ath_bus_ops *bus_ops) common->ah = sc->ah; common->hw = hw; common->priv = sc; + common->clockrate = 40; /* * Cache line size is used to size and align various -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 5/8] ath5k: delay full calibration after reset 2011-07-05 3:11 ` [PATCH v2 4/8] ath5k: initialize common->clockrate Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 6/8] ath5k: fix reference clock usec duration setting restore Felix Fietkau 0 siblings, 1 reply; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm During scans the full calibration usually does not make much sense, PAPD probing and IQ calibration should be deferred until there is enough time to complete them. Adding 100 ms to the initial full calibration delay should be enough to do this. Signed-off-by: Felix Fietkau <nbd@openwrt.org> Acked-by: Nick Kossifidis <mickflemm@gmail.com> --- drivers/net/wireless/ath/ath5k/base.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c index a413aa7..efec14f 100644 --- a/drivers/net/wireless/ath/ath5k/base.c +++ b/drivers/net/wireless/ath/ath5k/base.c @@ -2728,7 +2728,7 @@ ath5k_reset(struct ath5k_softc *sc, struct ieee80211_channel *chan, ath5k_ani_init(ah, ani_mode); - ah->ah_cal_next_full = jiffies; + ah->ah_cal_next_full = jiffies + msecs_to_jiffies(100); ah->ah_cal_next_ani = jiffies; ah->ah_cal_next_nf = jiffies; ewma_init(&ah->ah_beacon_rssi_avg, 1024, 8); -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 6/8] ath5k: fix reference clock usec duration setting restore 2011-07-05 3:11 ` [PATCH v2 5/8] ath5k: delay full calibration after reset Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Felix Fietkau 0 siblings, 1 reply; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm enabling the sleep clock alters the AR5K_USEC_32 field, but disabling it didn't restore it. Signed-off-by: Felix Fietkau <nbd@openwrt.org> Acked-by: Nick Kossifidis <mickflemm@gmail.com> --- drivers/net/wireless/ath/ath5k/reset.c | 12 +++++++++++- 1 files changed, 11 insertions(+), 1 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/reset.c b/drivers/net/wireless/ath/ath5k/reset.c index 1676a3e..efcc4df 100644 --- a/drivers/net/wireless/ath/ath5k/reset.c +++ b/drivers/net/wireless/ath/ath5k/reset.c @@ -142,6 +142,7 @@ static void ath5k_hw_init_core_clock(struct ath5k_hw *ah) /* Set 32MHz USEC counter */ if ((ah->ah_radio == AR5K_RF5112) || + (ah->ah_radio == AR5K_RF2413) || (ah->ah_radio == AR5K_RF5413) || (ah->ah_radio == AR5K_RF2316) || (ah->ah_radio == AR5K_RF2317)) @@ -233,7 +234,7 @@ static void ath5k_hw_init_core_clock(struct ath5k_hw *ah) static void ath5k_hw_set_sleep_clock(struct ath5k_hw *ah, bool enable) { struct ath5k_eeprom_info *ee = &ah->ah_capabilities.cap_eeprom; - u32 scal, spending; + u32 scal, spending, sclock; /* Only set 32KHz settings if we have an external * 32KHz crystal present */ @@ -317,6 +318,15 @@ static void ath5k_hw_set_sleep_clock(struct ath5k_hw *ah, bool enable) /* Set up tsf increment on each cycle */ AR5K_REG_WRITE_BITS(ah, AR5K_TSF_PARM, AR5K_TSF_PARM_INC, 1); + + if ((ah->ah_radio == AR5K_RF5112) || + (ah->ah_radio == AR5K_RF5413) || + (ah->ah_radio == AR5K_RF2316) || + (ah->ah_radio == AR5K_RF2317)) + sclock = 40 - 1; + else + sclock = 32 - 1; + AR5K_REG_WRITE_BITS(ah, AR5K_USEC_5211, AR5K_USEC_32, sclock); } } -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default 2011-07-05 3:11 ` [PATCH v2 6/8] ath5k: fix reference clock usec duration setting restore Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 8/8] ath5k: do not call ieee80211_stop_queue for queues not managed by mac80211 Felix Fietkau 2011-07-11 21:26 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Nick Kossifidis 0 siblings, 2 replies; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm While 32 KHz sleep clock might provide some power saving benefits, it is also a major source of stability issues, on OpenWrt it produced some reproducible data bus errors on register accesses on several different MIPS platforms. All the Atheros drivers that I can find do not enable this feature, so it makes sense to leave it disabled in ath5k as well. Signed-off-by: Felix Fietkau <nbd@openwrt.org> --- drivers/net/wireless/ath/ath5k/ath5k.h | 2 ++ drivers/net/wireless/ath/ath5k/debug.c | 3 +++ drivers/net/wireless/ath/ath5k/reset.c | 11 ++++++++--- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/ath5k.h b/drivers/net/wireless/ath/ath5k/ath5k.h index c4c02d5..6ea4d5d 100644 --- a/drivers/net/wireless/ath/ath5k/ath5k.h +++ b/drivers/net/wireless/ath/ath5k/ath5k.h @@ -1066,6 +1066,8 @@ struct ath5k_hw { u8 ah_retry_long; u8 ah_retry_short; + u32 ah_use_32khz_clock; + u8 ah_coverage_class; bool ah_ack_bitrate_high; u8 ah_bwmode; diff --git a/drivers/net/wireless/ath/ath5k/debug.c b/drivers/net/wireless/ath/ath5k/debug.c index 0bf7313..c0cebb7 100644 --- a/drivers/net/wireless/ath/ath5k/debug.c +++ b/drivers/net/wireless/ath/ath5k/debug.c @@ -918,6 +918,9 @@ ath5k_debug_init_device(struct ath5k_softc *sc) debugfs_create_file("queue", S_IWUSR | S_IRUSR, phydir, sc, &fops_queue); + + debugfs_create_bool("32khz_clock", S_IWUSR | S_IRUSR, phydir, + &sc->ah->ah_use_32khz_clock); } /* functions used in other places */ diff --git a/drivers/net/wireless/ath/ath5k/reset.c b/drivers/net/wireless/ath/ath5k/reset.c index efcc4df..d94db31 100644 --- a/drivers/net/wireless/ath/ath5k/reset.c +++ b/drivers/net/wireless/ath/ath5k/reset.c @@ -1287,11 +1287,16 @@ int ath5k_hw_reset(struct ath5k_hw *ah, enum nl80211_iftype op_mode, ath5k_hw_dma_init(ah); - /* Enable 32KHz clock function for AR5212+ chips + /* + * Enable 32KHz clock function for AR5212+ chips * Set clocks to 32KHz operation and use an * external 32KHz crystal when sleeping if one - * exists */ - if (ah->ah_version == AR5K_AR5212 && + * exists. + * Disabled by default because it is also disabled in + * other drivers and it is known to cause stability + * issues on some devices + */ + if (ah->ah_use_32khz_clock && ah->ah_version == AR5K_AR5212 && op_mode != NL80211_IFTYPE_AP) ath5k_hw_set_sleep_clock(ah, true); -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 8/8] ath5k: do not call ieee80211_stop_queue for queues not managed by mac80211 2011-07-05 3:11 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Felix Fietkau @ 2011-07-05 3:11 ` Felix Fietkau 2011-07-11 21:26 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Nick Kossifidis 1 sibling, 0 replies; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 3:11 UTC (permalink / raw) To: linux-wireless; +Cc: linville, ath5k-devel, mickflemm Instead of using ieee80211_stop_queue, check the configured tx queue limit before calling ieee80211_get_buffered_bc. Signed-off-by: Felix Fietkau <nbd@openwrt.org> --- drivers/net/wireless/ath/ath5k/base.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c index efec14f..d889f33 100644 --- a/drivers/net/wireless/ath/ath5k/base.c +++ b/drivers/net/wireless/ath/ath5k/base.c @@ -1555,7 +1555,8 @@ ath5k_tx_queue(struct ieee80211_hw *hw, struct sk_buff *skb, goto drop_packet; } - if (txq->txq_len >= txq->txq_max) + if (txq->txq_len >= txq->txq_max && + txq->qnum <= AR5K_TX_QUEUE_ID_DATA_MAX) ieee80211_stop_queue(hw, txq->qnum); spin_lock_irqsave(&sc->txbuflock, flags); @@ -1931,6 +1932,10 @@ ath5k_beacon_send(struct ath5k_softc *sc) skb = ieee80211_get_buffered_bc(sc->hw, vif); while (skb) { ath5k_tx_queue(sc->hw, skb, sc->cabq); + + if (sc->cabq->txq_len >= sc->cabq->txq_max) + break; + skb = ieee80211_get_buffered_bc(sc->hw, vif); } -- 1.7.3.2 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default 2011-07-05 3:11 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 8/8] ath5k: do not call ieee80211_stop_queue for queues not managed by mac80211 Felix Fietkau @ 2011-07-11 21:26 ` Nick Kossifidis 1 sibling, 0 replies; 12+ messages in thread From: Nick Kossifidis @ 2011-07-11 21:26 UTC (permalink / raw) To: Felix Fietkau; +Cc: linux-wireless, linville, ath5k-devel 2011/7/5 Felix Fietkau <nbd@openwrt.org>: > While 32 KHz sleep clock might provide some power saving benefits, > it is also a major source of stability issues, on OpenWrt it produced > some reproducible data bus errors on register accesses on several > different MIPS platforms. > > All the Atheros drivers that I can find do not enable this feature, > so it makes sense to leave it disabled in ath5k as well. > > Signed-off-by: Felix Fietkau <nbd@openwrt.org> > --- > drivers/net/wireless/ath/ath5k/ath5k.h | 2 ++ > drivers/net/wireless/ath/ath5k/debug.c | 3 +++ > drivers/net/wireless/ath/ath5k/reset.c | 11 ++++++++--- > 3 files changed, 13 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath5k/ath5k.h b/drivers/net/wireless/ath/ath5k/ath5k.h > index c4c02d5..6ea4d5d 100644 > --- a/drivers/net/wireless/ath/ath5k/ath5k.h > +++ b/drivers/net/wireless/ath/ath5k/ath5k.h > @@ -1066,6 +1066,8 @@ struct ath5k_hw { > u8 ah_retry_long; > u8 ah_retry_short; > > + u32 ah_use_32khz_clock; > + > u8 ah_coverage_class; > bool ah_ack_bitrate_high; > u8 ah_bwmode; > diff --git a/drivers/net/wireless/ath/ath5k/debug.c b/drivers/net/wireless/ath/ath5k/debug.c > index 0bf7313..c0cebb7 100644 > --- a/drivers/net/wireless/ath/ath5k/debug.c > +++ b/drivers/net/wireless/ath/ath5k/debug.c > @@ -918,6 +918,9 @@ ath5k_debug_init_device(struct ath5k_softc *sc) > > debugfs_create_file("queue", S_IWUSR | S_IRUSR, phydir, sc, > &fops_queue); > + > + debugfs_create_bool("32khz_clock", S_IWUSR | S_IRUSR, phydir, > + &sc->ah->ah_use_32khz_clock); > } > > /* functions used in other places */ > diff --git a/drivers/net/wireless/ath/ath5k/reset.c b/drivers/net/wireless/ath/ath5k/reset.c > index efcc4df..d94db31 100644 > --- a/drivers/net/wireless/ath/ath5k/reset.c > +++ b/drivers/net/wireless/ath/ath5k/reset.c > @@ -1287,11 +1287,16 @@ int ath5k_hw_reset(struct ath5k_hw *ah, enum nl80211_iftype op_mode, > ath5k_hw_dma_init(ah); > > > - /* Enable 32KHz clock function for AR5212+ chips > + /* > + * Enable 32KHz clock function for AR5212+ chips > * Set clocks to 32KHz operation and use an > * external 32KHz crystal when sleeping if one > - * exists */ > - if (ah->ah_version == AR5K_AR5212 && > + * exists. > + * Disabled by default because it is also disabled in > + * other drivers and it is known to cause stability > + * issues on some devices > + */ > + if (ah->ah_use_32khz_clock && ah->ah_version == AR5K_AR5212 && > op_mode != NL80211_IFTYPE_AP) > ath5k_hw_set_sleep_clock(ah, true); > Acked-by: Nick Kossifidis <mickflemm@gmail.com> -- GPG ID: 0xD21DB2DB As you read this post global entropy rises. Have Fun ;-) Nick ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/8] ath5k: add missing checks for rfgain probe 2011-07-05 3:11 [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 2/8] ath5k: apply the synth voltage tweak only on AR5112 rev 2 Felix Fietkau @ 2011-07-05 4:46 ` Nick Kossifidis 2011-07-05 4:48 ` Felix Fietkau 1 sibling, 1 reply; 12+ messages in thread From: Nick Kossifidis @ 2011-07-05 4:46 UTC (permalink / raw) To: Felix Fietkau; +Cc: linux-wireless, linville, ath5k-devel 2011/7/5 Felix Fietkau <nbd@openwrt.org>: > rfgain probe is only necessary for OFDM operation on AR5111 and AR5112. > > Signed-off-by: Felix Fietkau <nbd@openwrt.org> > --- > drivers/net/wireless/ath/ath5k/phy.c | 10 ++++++---- > 1 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c > index 5544191..d0bfcad 100644 > --- a/drivers/net/wireless/ath/ath5k/phy.c > +++ b/drivers/net/wireless/ath/ath5k/phy.c > @@ -1604,11 +1604,13 @@ int ath5k_hw_phy_calibrate(struct ath5k_hw *ah, > int ret; > > if (ah->ah_radio == AR5K_RF5110) > - ret = ath5k_hw_rf5110_calibrate(ah, channel); > - else { > - ret = ath5k_hw_rf511x_iq_calibrate(ah); > + return ath5k_hw_rf5110_calibrate(ah, channel); > + > + ret = ath5k_hw_rf511x_iq_calibrate(ah); > + > + if ((ah->ah_radio == AR5K_RF5112 || ah->ah_radio == AR5K_RF5112) && You probably mean ah->ah_radio == AR5K_RF5111 || ah->ah_radio == AR5K_RF5112 > + (channel->hw_value & CHANNEL_OFDM)) > ath5k_hw_request_rfgain_probe(ah); > - } > > return ret; > } > -- > 1.7.3.2 > > -- GPG ID: 0xD21DB2DB As you read this post global entropy rises. Have Fun ;-) Nick ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/8] ath5k: add missing checks for rfgain probe 2011-07-05 4:46 ` [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Nick Kossifidis @ 2011-07-05 4:48 ` Felix Fietkau 2011-07-11 18:37 ` John W. Linville 0 siblings, 1 reply; 12+ messages in thread From: Felix Fietkau @ 2011-07-05 4:48 UTC (permalink / raw) To: Nick Kossifidis; +Cc: linux-wireless, linville, ath5k-devel On 2011-07-05 11:46 AM, Nick Kossifidis wrote: > 2011/7/5 Felix Fietkau<nbd@openwrt.org>: >> rfgain probe is only necessary for OFDM operation on AR5111 and AR5112. >> >> Signed-off-by: Felix Fietkau<nbd@openwrt.org> >> --- >> drivers/net/wireless/ath/ath5k/phy.c | 10 ++++++---- >> 1 files changed, 6 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c >> index 5544191..d0bfcad 100644 >> --- a/drivers/net/wireless/ath/ath5k/phy.c >> +++ b/drivers/net/wireless/ath/ath5k/phy.c >> @@ -1604,11 +1604,13 @@ int ath5k_hw_phy_calibrate(struct ath5k_hw *ah, >> int ret; >> >> if (ah->ah_radio == AR5K_RF5110) >> - ret = ath5k_hw_rf5110_calibrate(ah, channel); >> - else { >> - ret = ath5k_hw_rf511x_iq_calibrate(ah); >> + return ath5k_hw_rf5110_calibrate(ah, channel); >> + >> + ret = ath5k_hw_rf511x_iq_calibrate(ah); >> + >> + if ((ah->ah_radio == AR5K_RF5112 || ah->ah_radio == AR5K_RF5112)&& > > You probably mean > ah->ah_radio == AR5K_RF5111 || ah->ah_radio == AR5K_RF5112 You're right. I need to be more careful with copy&paste ;) Will send a v3 once you've looked over the other patches. - Felix ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/8] ath5k: add missing checks for rfgain probe 2011-07-05 4:48 ` Felix Fietkau @ 2011-07-11 18:37 ` John W. Linville 0 siblings, 0 replies; 12+ messages in thread From: John W. Linville @ 2011-07-11 18:37 UTC (permalink / raw) To: Felix Fietkau; +Cc: Nick Kossifidis, linux-wireless, ath5k-devel On Tue, Jul 05, 2011 at 11:48:08AM +0700, Felix Fietkau wrote: > On 2011-07-05 11:46 AM, Nick Kossifidis wrote: > >2011/7/5 Felix Fietkau<nbd@openwrt.org>: > >> rfgain probe is only necessary for OFDM operation on AR5111 and AR5112. > >> > >> Signed-off-by: Felix Fietkau<nbd@openwrt.org> > >> --- > >> drivers/net/wireless/ath/ath5k/phy.c | 10 ++++++---- > >> 1 files changed, 6 insertions(+), 4 deletions(-) > >> > >> diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c > >> index 5544191..d0bfcad 100644 > >> --- a/drivers/net/wireless/ath/ath5k/phy.c > >> +++ b/drivers/net/wireless/ath/ath5k/phy.c > >> @@ -1604,11 +1604,13 @@ int ath5k_hw_phy_calibrate(struct ath5k_hw *ah, > >> int ret; > >> > >> if (ah->ah_radio == AR5K_RF5110) > >> - ret = ath5k_hw_rf5110_calibrate(ah, channel); > >> - else { > >> - ret = ath5k_hw_rf511x_iq_calibrate(ah); > >> + return ath5k_hw_rf5110_calibrate(ah, channel); > >> + > >> + ret = ath5k_hw_rf511x_iq_calibrate(ah); > >> + > >> + if ((ah->ah_radio == AR5K_RF5112 || ah->ah_radio == AR5K_RF5112)&& > > > >You probably mean > >ah->ah_radio == AR5K_RF5111 || ah->ah_radio == AR5K_RF5112 > You're right. I need to be more careful with copy&paste ;) > Will send a v3 once you've looked over the other patches. > > - Felix Ping? Time is running short... -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready. ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2011-07-11 21:26 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2011-07-05 3:11 [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 2/8] ath5k: apply the synth voltage tweak only on AR5112 rev 2 Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 3/8] ath5k: fix reference clock frequency for spur mitigation on AR2413 Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 4/8] ath5k: initialize common->clockrate Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 5/8] ath5k: delay full calibration after reset Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 6/8] ath5k: fix reference clock usec duration setting restore Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Felix Fietkau 2011-07-05 3:11 ` [PATCH v2 8/8] ath5k: do not call ieee80211_stop_queue for queues not managed by mac80211 Felix Fietkau 2011-07-11 21:26 ` [PATCH v2 7/8] ath5k: disable 32KHz sleep clock operation by default Nick Kossifidis 2011-07-05 4:46 ` [PATCH v2 1/8] ath5k: add missing checks for rfgain probe Nick Kossifidis 2011-07-05 4:48 ` Felix Fietkau 2011-07-11 18:37 ` John W. Linville
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.