laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/21500 )
Change subject: power_control: clarify units in 'struct lchan_power_ctrl_state'
......................................................................
power_control: clarify units in 'struct lchan_power_ctrl_state'
Change-Id: I934f2d1e1064eed7587bc5693ef981bf449a1b81
Related: SYS#4918
---
M include/osmo-bts/gsm_data.h
1 file changed, 3 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/include/osmo-bts/gsm_data.h b/include/osmo-bts/gsm_data.h
index 6af96fe..fdd5ba3 100644
--- a/include/osmo-bts/gsm_data.h
+++ b/include/osmo-bts/gsm_data.h
@@ -161,6 +161,9 @@
uint32_t fn;
};
+/* Depending on the context (MS or BS power control), fields 'current' and
'max'
+ * reflect either the MS power level (magic numbers), or BS Power reduction
level
+ * (attenuation, in dB). Field 'avg100_rxlev_dbm' is always in dBm. */
struct lchan_power_ctrl_state {
uint8_t current;
uint8_t max;
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/21500
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I934f2d1e1064eed7587bc5693ef981bf449a1b81
Gerrit-Change-Number: 21500
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
