Commit 87dd2d76 authored by Stanislaw Gruszka's avatar Stanislaw Gruszka Committed by John W. Linville

rt2800: calculate tx power temperature compensation on selected chips

Currently implemented temperature compensation is only valid on some of
supported chips. Other chips do not need temperature compensation or
need different way to do this (not yet implemented in the rt2800
driver). Trying to do run rt2800_get_gain_calibration_delta() when this
is not appropriate on particular chip gives bogus result of TX power
and can make connection unstable.

This is follow up to commit 8c8d2017
"rt2800: fix RT5390 & RT3290 TX power settings regression". On that
commit we avoid setting BBP_R1 register, but the real problem is wrong
temperature compensation calculation.
Reported-and-tested-by: default avatarRonald Wahl <ronald.wahl@raritan.com>
Debugged-by: default avatarRonald Wahl <ronald.wahl@raritan.com>
Cc: Mike Romberg <mike-romberg@comcast.net>
Signed-off-by: default avatarStanislaw Gruszka <sgruszka@redhat.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 8e09b7d2
...@@ -4119,7 +4119,20 @@ static void rt2800_config_txpower_rt28xx(struct rt2x00_dev *rt2x00dev, ...@@ -4119,7 +4119,20 @@ static void rt2800_config_txpower_rt28xx(struct rt2x00_dev *rt2x00dev,
* expected. We adjust it, based on TSSI reference and boundaries values * expected. We adjust it, based on TSSI reference and boundaries values
* provided in EEPROM. * provided in EEPROM.
*/ */
switch (rt2x00dev->chip.rt) {
case RT2860:
case RT2872:
case RT2883:
case RT3070:
case RT3071:
case RT3090:
case RT3572:
delta += rt2800_get_gain_calibration_delta(rt2x00dev); delta += rt2800_get_gain_calibration_delta(rt2x00dev);
break;
default:
/* TODO: temperature compensation code for other chips. */
break;
}
/* /*
* Decrease power according to user settings, on devices with unknown * Decrease power according to user settings, on devices with unknown
...@@ -4136,13 +4149,7 @@ static void rt2800_config_txpower_rt28xx(struct rt2x00_dev *rt2x00dev, ...@@ -4136,13 +4149,7 @@ static void rt2800_config_txpower_rt28xx(struct rt2x00_dev *rt2x00dev,
* TODO: we do not use +6 dBm option to do not increase power beyond * TODO: we do not use +6 dBm option to do not increase power beyond
* regulatory limit, however this could be utilized for devices with * regulatory limit, however this could be utilized for devices with
* CAPABILITY_POWER_LIMIT. * CAPABILITY_POWER_LIMIT.
*
* TODO: add different temperature compensation code for RT3290 & RT5390
* to allow to use BBP_R1 for those chips.
*/ */
if (!rt2x00_rt(rt2x00dev, RT3290) &&
!rt2x00_rt(rt2x00dev, RT5390)) {
rt2800_bbp_read(rt2x00dev, 1, &r1);
if (delta <= -12) { if (delta <= -12) {
power_ctrl = 2; power_ctrl = 2;
delta += 12; delta += 12;
...@@ -4152,9 +4159,9 @@ static void rt2800_config_txpower_rt28xx(struct rt2x00_dev *rt2x00dev, ...@@ -4152,9 +4159,9 @@ static void rt2800_config_txpower_rt28xx(struct rt2x00_dev *rt2x00dev,
} else { } else {
power_ctrl = 0; power_ctrl = 0;
} }
rt2800_bbp_read(rt2x00dev, 1, &r1);
rt2x00_set_field8(&r1, BBP1_TX_POWER_CTRL, power_ctrl); rt2x00_set_field8(&r1, BBP1_TX_POWER_CTRL, power_ctrl);
rt2800_bbp_write(rt2x00dev, 1, r1); rt2800_bbp_write(rt2x00dev, 1, r1);
}
offset = TX_PWR_CFG_0; offset = TX_PWR_CFG_0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment