Attention is currently required from: laforge.

fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/36915?usp=email )

Change subject: trxcon/l1sched: refactor prim management in tx_tch[fh]_fn() ......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2:

I'm not sure it is an improvement, tbh. […]

Code duplication is the price I am ready to pay for cleaner / more predictable code flow. I was already several times that I overlooked / not considered some `goto` somewhere, and as a result we started hitting a segfault. The code path below the switch is no longer common since we added CSD specific channel coding, this is why we had to jump over it in several `case` statements.

All in all, I would still prefer to get this patch merged.