fixeria has submitted this change. (
https://gerrit.osmocom.org/c/osmocom-bb/+/29850 )
Change subject: trxcon: allow subsequent TRXCON_EV_FULL_POWER_SCAN_REQ
......................................................................
trxcon: allow subsequent TRXCON_EV_FULL_POWER_SCAN_REQ
The remote L1CTL peer may send another L1CTL_PM_REQ message right
after getting L1CTL_PM_CONF from us. Handle this properly.
Change-Id: I8e5fd778467567e8ca69ed420b9815073daa7e16
---
M src/host/trxcon/src/trxcon_fsm.c
1 file changed, 9 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
laforge: Looks good to me, approved
msuraev: Looks good to me, but someone else must approve
diff --git a/src/host/trxcon/src/trxcon_fsm.c b/src/host/trxcon/src/trxcon_fsm.c
index 4116291..59a29d9 100644
--- a/src/host/trxcon/src/trxcon_fsm.c
+++ b/src/host/trxcon/src/trxcon_fsm.c
@@ -175,6 +175,13 @@
l1ctl_tx_pm_conf(trxcon->l2if, res->band_arfcn, res->dbm,
res->last_result);
break;
}
+ case TRXCON_EV_FULL_POWER_SCAN_REQ:
+ {
+ const struct trxcon_param_full_power_scan_req *req = data;
+
+ trx_if_cmd_measure(trxcon->phyif, req->band_arfcn_start,
req->band_arfcn_stop);
+ break;
+ }
default:
OSMO_ASSERT(0);
}
@@ -479,7 +486,8 @@
[TRXCON_ST_FULL_POWER_SCAN] = {
.name = "FULL_POWER_SCAN",
.out_state_mask = S(TRXCON_ST_RESET),
- .in_event_mask = S(TRXCON_EV_FULL_POWER_SCAN_RES),
+ .in_event_mask = S(TRXCON_EV_FULL_POWER_SCAN_RES)
+ | S(TRXCON_EV_FULL_POWER_SCAN_REQ),
.action = &trxcon_st_full_power_scan_action,
},
[TRXCON_ST_FBSB_SEARCH] = {
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted
one.
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/29850
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I8e5fd778467567e8ca69ed420b9815073daa7e16
Gerrit-Change-Number: 29850
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: msuraev <msur...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
