Attention is currently required from: msuraev, dexter.
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/31362 )
Change subject: pcu_l1_if: get rid of l1if checks ......................................................................
Patch Set 4: Code-Review-1
(1 comment)
Patchset:
PS4: sorry, i think this patch is going the wrong way.
If a PHY is being set up with phy L1 callbacks, then its l1 context object being NULL is not plausible. I mean, the phy implementation should not check in every single function that its context pointer is non-NULL. It has been set up, that is why its callbacks are being called.
It also does not seem very plausible that there is a TRX set up on a NULL PHY ... if these NULL checks should happen at all, they seem more fit in the places where they were before this patch .. or they can be dropped completely, AFAICT.