Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/33845 )
Change subject: layer23: modem: gmm: Update own TLLI copy though GMMRR-Assign.req hook ......................................................................
Patch Set 1:
(1 comment)
File src/host/layer23/src/modem/gmm.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/33845/comment/b530eec9_3aefd5b4 PS1, Line 143: OSMO_ASSERT(gmm_prim->oph.primitive == OSMO_GPRS_GMM_GMMRR_ASSIGN);
Would be cleaner to have this new code in a separate function.
I also thought about it initially but I'm not sure we are really worth is, since in principle this functions is not going to handle more code anyway, it's the only primitive to handle in this direction. So I ended up leaving it this way.