Attention is currently required from: msuraev, dexter.

Patch set 4:Code-Review -1

View Change

1 comment:

• Patchset:

  • Patch Set #4:

    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.

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