keith has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/29649
)
Change subject: osmo-bts-trx: respond to tx-attenuation config in real time.
......................................................................
Patch Set 3:
(1 comment)
File src/osmo-bts-trx/trx_vty.c:
https://gerrit.osmocom.org/c/osmo-bts/+/29649/comment/c09570d3_704a2817
PS3, Line 411: if (pinst->trx && pinst->trx->mo.nm_state.operational ==
NM_OPSTATE_ENABLED) {
Actually, from adding some printf locally, I see checking for pinst->trx was enough to
not trigger the power ramp on initially reading the config file. I suppose the check for
opstate does no harm.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bts/+/29649
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I1fa5e8130202fb509593db4132863b762b0f40b7
Gerrit-Change-Number: 29649
Gerrit-PatchSet: 3
Gerrit-Owner: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 06 Oct 2022 16:52:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment