diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2019-11-20 18:39:27 +0100 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2019-11-20 19:35:08 +0100 |
| commit | 522fc939e079c1dfd9b395548c75b52b92fc8adf (patch) | |
| tree | 429533aa0413e2545afa74d10ceda35258042a6d /include/osmo-bts/gsm_data_shared.h | |
| parent | 595eb576fc370d54f1a137616b16a5748389a427 (diff) | |
power_control.c: Don't use announced MS Power level as input for loop calculations
Use instead the received MS Power currently in use by the MS matching
the measured signal. This way there's no need to wait for the MS to
reach the announced MS power level or add checks in case the MS doesn't
support that specific power level. Furthermore, more fine grained
announced power level value can be obtained faster due to more input
iterations not being dropped while waiting.
osmo-bts-trx specific algo was not following this approach and using
announced MS power instead because it's wowrking at a lower level and
henche was not using the transmitted MS Power level value by the MS as
input for the calculation.
The "if (diff < 2 && diff > -2))" condition is dropped since equal
signal strength may still result in a different MS power level announced
(the one currently used by the MS during tx of last SACCH block).
Related: OS#1851
Change-Id: I4494dc27a295a3dca1d3331d4ff712d486643e13
Diffstat (limited to 'include/osmo-bts/gsm_data_shared.h')
| -rw-r--r-- | include/osmo-bts/gsm_data_shared.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/osmo-bts/gsm_data_shared.h b/include/osmo-bts/gsm_data_shared.h index c19bb210..41998ad8 100644 --- a/include/osmo-bts/gsm_data_shared.h +++ b/include/osmo-bts/gsm_data_shared.h @@ -322,7 +322,6 @@ struct gsm_lchan { uint8_t current; uint8_t max; bool fixed; - int8_t last_received; /* last received MS Power in uplink L1 SACCH, -1 means not set */ } ms_power_ctrl; /* Power levels for BTS */ uint8_t bs_power; |