trx_sched_ul_burst(): get rid of the 'switch' statement

Both TRXC_RACH and TRXC_PTCCH are handled in the rx_rach_fn(),
so we can eliminate the need of having a 'switch' statement in
the general (perfrmance critical) code path.

Change-Id: I66d8785a63215af37a77e258039549e4e6292e49

src/common/scheduler.c

@@ -1546,16 +1546,9 @@ int trx_sched_ul_burst(struct l1sched_ts *l1ts, struct trx_ul_burst_ind *bi)
 
 	/* handle NOPE indications */
 	if (bi->flags & TRX_BI_F_NOPE_IND) {
-		switch (bi->chan) {
-		case TRXC_PTCCH:
-		case TRXC_RACH:
-			/* For some logical channel types NOPE.ind is valueless. */
-			return 0;
-		default:
-			/* NOTE: Uplink burst handler must check bi->burst_len before
-			 * accessing bi->burst to avoid uninitialized memory access. */
-			return func(l1ts, bi);
-		}
+		/* NOTE: Uplink burst handler must check bi->burst_len before
+		 * accessing bi->burst to avoid uninitialized memory access. */
+		return func(l1ts, bi);
 	}
 
 	/* decrypt */

src/osmo-bts-trx/sched_lchan_rach.c

@@ -112,6 +112,10 @@ int rx_rach_fn(struct l1sched_ts *l1ts, const struct trx_ul_burst_ind *bi)
 	uint8_t ra;
 	int rc;
 
+	/* Ignore NOPE indications, they're of no use here */
+	if (bi->flags & TRX_BI_F_NOPE_IND)
+		return 0;
+
 	/* TSC (Training Sequence Code) is an optional parameter of the UL burst
 	 * indication. We need this information in order to decide whether an
 	 * Access Burst is 11-bit encoded or not (see OS#1854). If this information