New generic transmit power handling

In order to support transmit power reduction by thermal management
as well as the variety of new internal / external PA configurations
of BTSs, we need a slightly more complex system.

Also, as at high power a single dB can be quite a big difference,
we are now doing all computations in milli-dB(m), i.e. 1/10000 bel.

Ramping is now used both for up and down ramping, as that is useful in
cases where you want to gracefully shut down a cell by shrinking its
radius, gradually handing over subscribers to neighboring cells.

Furthermore, this code is becoming part of the 'common' codebase, as it
is not really specific to how sysmobts is working.

The user can specify a single aggregate value for external system
gain/attenuation.  Let's say you have 1dB loss of antenna cable, so you
can put that as 'user-gain -1' into the config, which means that a
'transmit power of 20dBm' will be compensatet for that and the TRX is
instructed to output 21dBm to compensate the cable loss.  Similarly,
external PAs can be described by a positive user-gain.

One of the next steps will be to communicate those values and the
nominal power capability of the specific BTS to the BSC, so the BSC will
automatically show correct signal levels in the VTY and log files.

The code includes provisions for future extensions regarding
* an external and an internal PA with calibration tables
* a thermal attenuation setting to be controlled by the site manager

1 files changed, 9 insertions, 12 deletions

diff --git a/src/osmo-bts-sysmo/oml.c b/src/osmo-bts-sysmo/oml.c
index e5f9260b..207cae89 100644
--- a/src/osmo-bts-sysmo/oml.c
+++ b/src/osmo-bts-sysmo/oml.c
@@ -272,7 +272,7 @@ static int trx_init_compl_cb(struct gsm_bts_trx *trx, struct msgb *l1_msg)
 #endif
 
 	/* Begin to ramp up the power */
-	sysmobts_pa_maybe_step(trx);
+	power_ramp_start(trx, get_p_target_mdBm(trx, 0), 0);
 
 	return opstart_compl(&trx->mo, l1_msg);
 }
@@ -327,9 +327,7 @@ static int trx_init(struct gsm_bts_trx *trx)
 	dev_par->u8NbTsc = trx->bts->bsic & 7;
 	dev_par->fRxPowerLevel = fl1h->ul_power_target;
 
-	/* initialize the power */
-	sysmobts_pa_pwr_init(trx);
-	dev_par->fTxPowerLevel = trx->pa.current_power;
+	dev_par->fTxPowerLevel = 0.0;
 	LOGP(DL1C, LOGL_NOTICE, "Init TRX (ARFCN %u, TSC %u, RxPower % 2f dBm, "
 		"TxPower % 2.2f dBm\n", dev_par->u16Arfcn, dev_par->u8NbTsc,
 		dev_par->fRxPowerLevel, dev_par->fTxPowerLevel);
@@ -1074,12 +1072,8 @@ static int chmod_txpower_compl_cb(struct gsm_bts_trx *trx, struct msgb *l1_msg)
 	LOGPC(DL1C, LOGL_INFO, "setTxPower %f dBm\n",
 		cc->cfgParams.setTxPowerLevel.fTxPowerLevel);
 
-	trx->pa.current_power = cc->cfgParams.setTxPowerLevel.fTxPowerLevel;
 	msgb_free(l1_msg);
 
-	/* Schedule the next step up */
-	sysmobts_pa_maybe_step(trx);
-
 	return 0;
 }
 
@@ -1553,10 +1547,8 @@ int bts_model_apply_oml(struct gsm_bts *bts, struct msgb *msg,
 		struct femtol1_hdl *fl1h = trx_femtol1_hdl(trx);
 
 		/* Did we go through MphInit yet? If yes fire and forget */
-		if (fl1h->hLayer1) {
-			sysmobts_pa_pwr_init(trx);
-			l1if_set_txpower(fl1h, (float) trx->pa.current_power);
-		}
+		if (fl1h->hLayer1)
+			power_ramp_start(trx, get_p_target_mdBm(trx, 0), 0);
 	}
 
 	/* FIXME: we actaully need to send a ACK or NACK for the OML message */
@@ -1722,3 +1714,8 @@ int bts_model_trx_deact_rf(struct gsm_bts_trx *trx)
 
 	return l1if_activate_rf(fl1, 0);
 }
+
+int bts_model_change_power(struct gsm_bts_trx *trx, int p_trxout_mdBm)
+{
+	return l1if_set_txpower(trx_femtol1_hdl(trx), ((float) p_trxout_mdBm)/1000.0);
+}