fixeria submitted this change.

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, but someone else must approve
  pespin: Looks good to me, but someone else must approve
  fixeria: Looks good to me, approved

scheduler: remove redundant OSMO_ASSERT() statements

It's guaranteed by the calling function bts_model_l1sap_down() that
the prim's operation is PRIM_OP_REQUEST and that the msg is present.

Change-Id: I6a01bba7b5eb337ae1552c442d74447565c52e25
---
M src/common/scheduler.c
1 file changed, 0 insertions(+), 6 deletions(-)

diff --git a/src/common/scheduler.c b/src/common/scheduler.c
index e6a664f..99ab597 100644
--- a/src/common/scheduler.c
+++ b/src/common/scheduler.c
@@ -854,9 +854,6 @@
 		"PH-DATA.req: chan_nr=0x%02x link_id=0x%02x\n",
 		l1sap->u.data.chan_nr, l1sap->u.data.link_id);
 
-	OSMO_ASSERT(l1sap->oph.operation == PRIM_OP_REQUEST);
-	OSMO_ASSERT(l1sap->oph.msg);
-
 	/* ignore empty frame */
 	if (!l1sap->oph.msg->l2h || msgb_l2len(l1sap->oph.msg) == 0) {
 		msgb_free(l1sap->oph.msg);
@@ -880,9 +877,6 @@
 	LOGL1S(DL1P, LOGL_DEBUG, l1ts, -1, l1sap->u.tch.fn,
 	       "TCH.req: chan_nr=0x%02x\n", l1sap->u.tch.chan_nr);
 
-	OSMO_ASSERT(l1sap->oph.operation == PRIM_OP_REQUEST);
-	OSMO_ASSERT(l1sap->oph.msg);
-
 	/* ignore empty frame */
 	if (!msgb_l2len(l1sap->oph.msg)) {
 		msgb_free(l1sap->oph.msg);

To view, visit change 27720. To unsubscribe, or for help writing mail filters, visit settings.