fixeria submitted this change.

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

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(-)

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 change 29850. To unsubscribe, or for help writing mail filters, visit settings.