* [PATCH] Fix transmit power troubles on some Ralink RT30xx cards
@ 2013-04-14 10:54 minimumlaw
2013-04-14 13:32 ` Greg KH
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: minimumlaw @ 2013-04-14 10:54 UTC (permalink / raw)
To: linux-wireless; +Cc: stable, Alex A. Mihaylov
From: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
Some cards on Ralink RT30xx chipset not have correctly TX_MIXER_GAIN
value in them EEPROM/EFUSE. In this case, we must use default value,
but always used EEPROM/EFUSE value. As result we have tranmitt power
range from -10dBm to +6dBm instead 0dBm to +16dBm.
Correctly value in EEPROM/EFUSE is one or more for RT3070 and two or
more for other RT30xx chips.
Testen on Canyon CNP-WF518N1 usb Wi-Fi dongle and Jorjin WN8020 usb
embedded Wi-Fi module.
---
drivers/net/wireless/rt2x00/rt2800lib.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c
index a658b4b..92849e5 100644
--- a/drivers/net/wireless/rt2x00/rt2800lib.c
+++ b/drivers/net/wireless/rt2x00/rt2800lib.c
@@ -4436,6 +4436,8 @@ static int rt2800_init_rfcsr(struct rt2x00_dev *rt2x00dev)
if (!rt2x00_rt(rt2x00dev, RT5390) &&
!rt2x00_rt(rt2x00dev, RT5392)) {
+ u8 min_gain = rt2x00_rt(rt2x00dev, RT3070) ? 1 : 2;
+
rt2800_rfcsr_read(rt2x00dev, 17, &rfcsr);
rt2x00_set_field8(&rfcsr, RFCSR17_TX_LO1_EN, 0);
if (rt2x00_rt(rt2x00dev, RT3070) ||
@@ -4446,8 +4448,10 @@ static int rt2800_init_rfcsr(struct rt2x00_dev *rt2x00dev)
&rt2x00dev->cap_flags))
rt2x00_set_field8(&rfcsr, RFCSR17_R, 1);
}
- rt2x00_set_field8(&rfcsr, RFCSR17_TXMIXER_GAIN,
- drv_data->txmixer_gain_24g);
+ if (drv_data->txmixer_gain_24g >= min_gain) {
+ rt2x00_set_field8(&rfcsr, RFCSR17_TXMIXER_GAIN,
+ drv_data->txmixer_gain_24g);
+ }
rt2800_rfcsr_write(rt2x00dev, 17, rfcsr);
}
--
1.8.1.5
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] Fix transmit power troubles on some Ralink RT30xx cards
2013-04-14 10:54 [PATCH] Fix transmit power troubles on some Ralink RT30xx cards minimumlaw
@ 2013-04-14 13:32 ` Greg KH
2013-04-14 13:53 ` Stanislaw Gruszka
2013-04-15 17:08 ` Kalle Valo
2 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2013-04-14 13:32 UTC (permalink / raw)
To: minimumlaw; +Cc: linux-wireless, stable
On Sun, Apr 14, 2013 at 02:54:43PM +0400, minimumlaw@rambler.ru wrote:
> From: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
>
> Some cards on Ralink RT30xx chipset not have correctly TX_MIXER_GAIN
> value in them EEPROM/EFUSE. In this case, we must use default value,
> but always used EEPROM/EFUSE value. As result we have tranmitt power
> range from -10dBm to +6dBm instead 0dBm to +16dBm.
>
> Correctly value in EEPROM/EFUSE is one or more for RT3070 and two or
> more for other RT30xx chips.
>
> Testen on Canyon CNP-WF518N1 usb Wi-Fi dongle and Jorjin WN8020 usb
> embedded Wi-Fi module.
> ---
> drivers/net/wireless/rt2x00/rt2800lib.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
<formletter>
This is not the correct way to submit patches for inclusion in the
stable kernel tree. Please read Documentation/stable_kernel_rules.txt
for how to do this properly.
</formletter>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Fix transmit power troubles on some Ralink RT30xx cards
2013-04-14 10:54 [PATCH] Fix transmit power troubles on some Ralink RT30xx cards minimumlaw
2013-04-14 13:32 ` Greg KH
@ 2013-04-14 13:53 ` Stanislaw Gruszka
2013-04-14 13:58 ` Greg KH
2013-04-15 17:08 ` Kalle Valo
2 siblings, 1 reply; 5+ messages in thread
From: Stanislaw Gruszka @ 2013-04-14 13:53 UTC (permalink / raw)
To: minimumlaw; +Cc: linux-wireless, stable, users
On Sun, Apr 14, 2013 at 02:54:43PM +0400, minimumlaw@rambler.ru wrote:
> From: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
>
> Some cards on Ralink RT30xx chipset not have correctly TX_MIXER_GAIN
> value in them EEPROM/EFUSE. In this case, we must use default value,
> but always used EEPROM/EFUSE value. As result we have tranmitt power
> range from -10dBm to +6dBm instead 0dBm to +16dBm.
Thanks for great work on debugging this!
> Correctly value in EEPROM/EFUSE is one or more for RT3070 and two or
> more for other RT30xx chips.
>
> Testen on Canyon CNP-WF518N1 usb Wi-Fi dongle and Jorjin WN8020 usb
> embedded Wi-Fi module.
Signed-off-by: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
Cc: stable@vger.kernel.org
Acked-by: Stanislaw Gruszka <stf_xl@wp.pl>
> ---
> drivers/net/wireless/rt2x00/rt2800lib.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c
> index a658b4b..92849e5 100644
> --- a/drivers/net/wireless/rt2x00/rt2800lib.c
> +++ b/drivers/net/wireless/rt2x00/rt2800lib.c
> @@ -4436,6 +4436,8 @@ static int rt2800_init_rfcsr(struct rt2x00_dev *rt2x00dev)
>
> if (!rt2x00_rt(rt2x00dev, RT5390) &&
> !rt2x00_rt(rt2x00dev, RT5392)) {
> + u8 min_gain = rt2x00_rt(rt2x00dev, RT3070) ? 1 : 2;
> +
> rt2800_rfcsr_read(rt2x00dev, 17, &rfcsr);
> rt2x00_set_field8(&rfcsr, RFCSR17_TX_LO1_EN, 0);
> if (rt2x00_rt(rt2x00dev, RT3070) ||
> @@ -4446,8 +4448,10 @@ static int rt2800_init_rfcsr(struct rt2x00_dev *rt2x00dev)
> &rt2x00dev->cap_flags))
> rt2x00_set_field8(&rfcsr, RFCSR17_R, 1);
> }
> - rt2x00_set_field8(&rfcsr, RFCSR17_TXMIXER_GAIN,
> - drv_data->txmixer_gain_24g);
> + if (drv_data->txmixer_gain_24g >= min_gain) {
> + rt2x00_set_field8(&rfcsr, RFCSR17_TXMIXER_GAIN,
> + drv_data->txmixer_gain_24g);
> + }
> rt2800_rfcsr_write(rt2x00dev, 17, rfcsr);
> }
>
> --
> 1.8.1.5
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] Fix transmit power troubles on some Ralink RT30xx cards
2013-04-14 13:53 ` Stanislaw Gruszka
@ 2013-04-14 13:58 ` Greg KH
0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2013-04-14 13:58 UTC (permalink / raw)
To: Stanislaw Gruszka; +Cc: minimumlaw, linux-wireless, stable, users
On Sun, Apr 14, 2013 at 03:53:16PM +0200, Stanislaw Gruszka wrote:
> On Sun, Apr 14, 2013 at 02:54:43PM +0400, minimumlaw@rambler.ru wrote:
> > From: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
> >
> > Some cards on Ralink RT30xx chipset not have correctly TX_MIXER_GAIN
> > value in them EEPROM/EFUSE. In this case, we must use default value,
> > but always used EEPROM/EFUSE value. As result we have tranmitt power
> > range from -10dBm to +6dBm instead 0dBm to +16dBm.
>
> Thanks for great work on debugging this!
> > Correctly value in EEPROM/EFUSE is one or more for RT3070 and two or
> > more for other RT30xx chips.
> >
> > Testen on Canyon CNP-WF518N1 usb Wi-Fi dongle and Jorjin WN8020 usb
> > embedded Wi-Fi module.
>
> Signed-off-by: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
You can not just add someone else's Signed-off-by: to a patch, please go
read Documentation/SubmittingPatches for what it is (hint, a legal
statement). Never do that. Ever.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Fix transmit power troubles on some Ralink RT30xx cards
2013-04-14 10:54 [PATCH] Fix transmit power troubles on some Ralink RT30xx cards minimumlaw
2013-04-14 13:32 ` Greg KH
2013-04-14 13:53 ` Stanislaw Gruszka
@ 2013-04-15 17:08 ` Kalle Valo
2 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2013-04-15 17:08 UTC (permalink / raw)
To: minimumlaw; +Cc: linux-wireless, stable
minimumlaw@rambler.ru writes:
> From: "Alex A. Mihaylov" <minimumlaw@rambler.ru>
>
> Some cards on Ralink RT30xx chipset not have correctly TX_MIXER_GAIN
> value in them EEPROM/EFUSE. In this case, we must use default value,
> but always used EEPROM/EFUSE value. As result we have tranmitt power
> range from -10dBm to +6dBm instead 0dBm to +16dBm.
>
> Correctly value in EEPROM/EFUSE is one or more for RT3070 and two or
> more for other RT30xx chips.
>
> Testen on Canyon CNP-WF518N1 usb Wi-Fi dongle and Jorjin WN8020 usb
> embedded Wi-Fi module.
> ---
> drivers/net/wireless/rt2x00/rt2800lib.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
The subject should have "rt2x00: " prefix.
--
Kalle Valo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-04-15 17:08 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-14 10:54 [PATCH] Fix transmit power troubles on some Ralink RT30xx cards minimumlaw
2013-04-14 13:32 ` Greg KH
2013-04-14 13:53 ` Stanislaw Gruszka
2013-04-14 13:58 ` Greg KH
2013-04-15 17:08 ` Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).