* [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
@ 2011-09-06 14:48 Rajkumar Manoharan
2011-09-13 17:32 ` John W. Linville
0 siblings, 1 reply; 9+ messages in thread
From: Rajkumar Manoharan @ 2011-09-06 14:48 UTC (permalink / raw)
To: linville; +Cc: linux-wireless, Rajkumar Manoharan, stable
Do the magnitude/phase coeff correction only if the outlier
is detected.
Cc: stable@kernel.org
Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
---
drivers/net/wireless/ath/ath9k/ar9003_calib.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/ar9003_calib.c b/drivers/net/wireless/ath/ath9k/ar9003_calib.c
index fa35a02..ee3a8a2 100644
--- a/drivers/net/wireless/ath/ath9k/ar9003_calib.c
+++ b/drivers/net/wireless/ath/ath9k/ar9003_calib.c
@@ -643,8 +643,9 @@ static void ar9003_hw_detect_outlier(int *mp_coeff, int nmeasurement,
outlier_idx = max_idx;
else
outlier_idx = min_idx;
+
+ mp_coeff[outlier_idx] = mp_avg;
}
- mp_coeff[outlier_idx] = mp_avg;
}
static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
--
1.7.6.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-06 14:48 [PATCH] ath9k_hw: Fix magnitude/phase coeff correction Rajkumar Manoharan
@ 2011-09-13 17:32 ` John W. Linville
2011-09-14 3:49 ` Rajkumar Manoharan
0 siblings, 1 reply; 9+ messages in thread
From: John W. Linville @ 2011-09-13 17:32 UTC (permalink / raw)
To: Rajkumar Manoharan; +Cc: linux-wireless, stable
On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
> Do the magnitude/phase coeff correction only if the outlier
> is detected.
>
> Cc: stable@kernel.org
> Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> ---
> drivers/net/wireless/ath/ath9k/ar9003_calib.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath9k/ar9003_calib.c b/drivers/net/wireless/ath/ath9k/ar9003_calib.c
> index fa35a02..ee3a8a2 100644
> --- a/drivers/net/wireless/ath/ath9k/ar9003_calib.c
> +++ b/drivers/net/wireless/ath/ath9k/ar9003_calib.c
> @@ -643,8 +643,9 @@ static void ar9003_hw_detect_outlier(int *mp_coeff, int nmeasurement,
> outlier_idx = max_idx;
> else
> outlier_idx = min_idx;
> +
> + mp_coeff[outlier_idx] = mp_avg;
> }
> - mp_coeff[outlier_idx] = mp_avg;
> }
>
> static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
What is the effect of the bug being fixed here? Is this fix intended for 3.1?
John
--
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] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-13 17:32 ` John W. Linville
@ 2011-09-14 3:49 ` Rajkumar Manoharan
2011-09-14 8:45 ` Kalle Valo
2011-09-14 13:27 ` John W. Linville
0 siblings, 2 replies; 9+ messages in thread
From: Rajkumar Manoharan @ 2011-09-14 3:49 UTC (permalink / raw)
To: John W. Linville; +Cc: linux-wireless, stable
On Tue, Sep 13, 2011 at 01:32:26PM -0400, John W. Linville wrote:
> On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
> > Do the magnitude/phase coeff correction only if the outlier
> > is detected.
> >
> > Cc: stable@kernel.org
> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> > ---
> >
> > static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
>
> What is the effect of the bug being fixed here? Is this fix intended for 3.1?
>
The first entry of phase/mag got changed to average if outlier is not detected.
This patch ensures to update appropriate index of magnitude/phase. This patch is
applicable from 3.0 onwards.
--
Rajkumar
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-14 3:49 ` Rajkumar Manoharan
@ 2011-09-14 8:45 ` Kalle Valo
2011-09-14 9:03 ` Rajkumar Manoharan
2011-09-14 13:27 ` John W. Linville
1 sibling, 1 reply; 9+ messages in thread
From: Kalle Valo @ 2011-09-14 8:45 UTC (permalink / raw)
To: Rajkumar Manoharan; +Cc: John W. Linville, linux-wireless, stable
Rajkumar Manoharan <rmanohar@qca.qualcomm.com> writes:
> On Tue, Sep 13, 2011 at 01:32:26PM -0400, John W. Linville wrote:
>> On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
>> > Do the magnitude/phase coeff correction only if the outlier
>> > is detected.
>> >
>> > Cc: stable@kernel.org
>> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
>> > ---
>> >
>> > static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
>>
>> What is the effect of the bug being fixed here? Is this fix intended for 3.1?
>
> The first entry of phase/mag got changed to average if outlier is
> not detected. This patch ensures to update appropriate index of
> magnitude/phase. This patch is applicable from 3.0 onwards.
Sorry, but this doesn't tell much. What's the user visible change
with this patch? Better sensitivity, less transmission errors or what?
--
Kalle Valo
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-14 8:45 ` Kalle Valo
@ 2011-09-14 9:03 ` Rajkumar Manoharan
2011-09-14 16:54 ` Rajkumar Manoharan
0 siblings, 1 reply; 9+ messages in thread
From: Rajkumar Manoharan @ 2011-09-14 9:03 UTC (permalink / raw)
To: Kalle Valo; +Cc: John W. Linville, linux-wireless, stable
On Wed, Sep 14, 2011 at 11:45:20AM +0300, Kalle Valo wrote:
> Rajkumar Manoharan <rmanohar@qca.qualcomm.com> writes:
>
> > On Tue, Sep 13, 2011 at 01:32:26PM -0400, John W. Linville wrote:
> >> On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
> >> > Do the magnitude/phase coeff correction only if the outlier
> >> > is detected.
> >> >
> >> > Cc: stable@kernel.org
> >> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> >> > ---
> >> >
> >> > static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
> >>
> >> What is the effect of the bug being fixed here? Is this fix intended for 3.1?
> >
> > The first entry of phase/mag got changed to average if outlier is
> > not detected. This patch ensures to update appropriate index of
> > magnitude/phase. This patch is applicable from 3.0 onwards.
>
> Sorry, but this doesn't tell much. What's the user visible change
> with this patch? Better sensitivity, less transmission errors or what?
>
It has minor impact because of this bug, an undesirable Tx IQ correction value
will be assigned to Tx gain settings wheen outlier is not happened. The impacts
on overall system throughput should be very minor.
--
Rajkumar
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-14 3:49 ` Rajkumar Manoharan
2011-09-14 8:45 ` Kalle Valo
@ 2011-09-14 13:27 ` John W. Linville
2011-09-14 13:47 ` Adrian Chadd
1 sibling, 1 reply; 9+ messages in thread
From: John W. Linville @ 2011-09-14 13:27 UTC (permalink / raw)
To: Rajkumar Manoharan; +Cc: linux-wireless, stable
On Wed, Sep 14, 2011 at 09:19:24AM +0530, Rajkumar Manoharan wrote:
> On Tue, Sep 13, 2011 at 01:32:26PM -0400, John W. Linville wrote:
> > On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
> > > Do the magnitude/phase coeff correction only if the outlier
> > > is detected.
> > >
> > > Cc: stable@kernel.org
> > > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> > > ---
> > >
> > > static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
> >
> > What is the effect of the bug being fixed here? Is this fix intended for 3.1?
> >
> The first entry of phase/mag got changed to average if outlier is not detected.
> This patch ensures to update appropriate index of magnitude/phase. This patch is
> applicable from 3.0 onwards.
OK, but what happens if this patch isn't applied? Is there a crash?
Does the moon explode? Does my beer leak through a crack in the side
of my mug?
John
--
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] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-14 13:27 ` John W. Linville
@ 2011-09-14 13:47 ` Adrian Chadd
0 siblings, 0 replies; 9+ messages in thread
From: Adrian Chadd @ 2011-09-14 13:47 UTC (permalink / raw)
To: John W. Linville; +Cc: Rajkumar Manoharan, linux-wireless, stable
.. and I wonder if there are any similar issues with AR9001/AR9002 series NICs.
I see some reasonably funny behaviour with calibration (under freebsd,
the calibration code is the same as ath9k) where the CRC error rates
for the high MCS rates fluctuate quite a bit.
I haven't bothered chasing it down as I've been busy chasing down TX
interrupt issues.
Adrian
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-14 9:03 ` Rajkumar Manoharan
@ 2011-09-14 16:54 ` Rajkumar Manoharan
2011-09-14 16:54 ` John W. Linville
0 siblings, 1 reply; 9+ messages in thread
From: Rajkumar Manoharan @ 2011-09-14 16:54 UTC (permalink / raw)
To: linville; +Cc: John W. Linville, linux-wireless, kvalo
On Wed, Sep 14, 2011 at 02:33:22PM +0530, Rajkumar Manoharan wrote:
> On Wed, Sep 14, 2011 at 11:45:20AM +0300, Kalle Valo wrote:
> > Rajkumar Manoharan <rmanohar@qca.qualcomm.com> writes:
> >
> > > On Tue, Sep 13, 2011 at 01:32:26PM -0400, John W. Linville wrote:
> > >> On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
> > >> > Do the magnitude/phase coeff correction only if the outlier
> > >> > is detected.
> > >> >
> > >> > Cc: stable@kernel.org
> > >> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> > >> > ---
> > >> >
> > >> > static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
> > >>
> > >> What is the effect of the bug being fixed here? Is this fix intended for 3.1?
> > >
> > > The first entry of phase/mag got changed to average if outlier is
> > > not detected. This patch ensures to update appropriate index of
> > > magnitude/phase. This patch is applicable from 3.0 onwards.
> >
> > Sorry, but this doesn't tell much. What's the user visible change
> > with this patch? Better sensitivity, less transmission errors or what?
> >
> It has minor impact because of this bug, an undesirable Tx IQ correction value
> will be assigned to Tx gain settings wheen outlier is not happened. The impacts
> on overall system throughput should be very minor.
>
John,
Not doing the calibrations in proper manner could impact performance in congested
network. Maybe i should update the impact in the commit log. Wanna send me v2
patch?
--
Rajkumar
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ath9k_hw: Fix magnitude/phase coeff correction
2011-09-14 16:54 ` Rajkumar Manoharan
@ 2011-09-14 16:54 ` John W. Linville
0 siblings, 0 replies; 9+ messages in thread
From: John W. Linville @ 2011-09-14 16:54 UTC (permalink / raw)
To: Rajkumar Manoharan; +Cc: linux-wireless, kvalo
On Wed, Sep 14, 2011 at 10:24:27PM +0530, Rajkumar Manoharan wrote:
> On Wed, Sep 14, 2011 at 02:33:22PM +0530, Rajkumar Manoharan wrote:
> > On Wed, Sep 14, 2011 at 11:45:20AM +0300, Kalle Valo wrote:
> > > Rajkumar Manoharan <rmanohar@qca.qualcomm.com> writes:
> > >
> > > > On Tue, Sep 13, 2011 at 01:32:26PM -0400, John W. Linville wrote:
> > > >> On Tue, Sep 06, 2011 at 08:18:50PM +0530, Rajkumar Manoharan wrote:
> > > >> > Do the magnitude/phase coeff correction only if the outlier
> > > >> > is detected.
> > > >> >
> > > >> > Cc: stable@kernel.org
> > > >> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> > > >> > ---
> > > >> >
> > > >> > static void ar9003_hw_tx_iqcal_load_avg_2_passes(struct ath_hw *ah,
> > > >>
> > > >> What is the effect of the bug being fixed here? Is this fix intended for 3.1?
> > > >
> > > > The first entry of phase/mag got changed to average if outlier is
> > > > not detected. This patch ensures to update appropriate index of
> > > > magnitude/phase. This patch is applicable from 3.0 onwards.
> > >
> > > Sorry, but this doesn't tell much. What's the user visible change
> > > with this patch? Better sensitivity, less transmission errors or what?
> > >
> > It has minor impact because of this bug, an undesirable Tx IQ correction value
> > will be assigned to Tx gain settings wheen outlier is not happened. The impacts
> > on overall system throughput should be very minor.
> >
> John,
>
> Not doing the calibrations in proper manner could impact performance in congested
> network. Maybe i should update the impact in the commit log. Wanna send me v2
> patch?
Please.
--
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] 9+ messages in thread
end of thread, other threads:[~2011-09-14 17:00 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-09-06 14:48 [PATCH] ath9k_hw: Fix magnitude/phase coeff correction Rajkumar Manoharan
2011-09-13 17:32 ` John W. Linville
2011-09-14 3:49 ` Rajkumar Manoharan
2011-09-14 8:45 ` Kalle Valo
2011-09-14 9:03 ` Rajkumar Manoharan
2011-09-14 16:54 ` Rajkumar Manoharan
2011-09-14 16:54 ` John W. Linville
2011-09-14 13:27 ` John W. Linville
2011-09-14 13:47 ` Adrian Chadd
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.