Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34139 )
Change subject: trxcon: properly handle PDCH slotmask in UL/DL TBF CFG.Req ...................................................................... Patch Set 2: (2 comments) File include/l1gprs.h: https://gerrit.osmocom.org/c/osmocom-bb/+/34139/comment/34a6e4f7_d792758a PS2, Line 37: void *priv > The `priv` is already accessible via `gprs->priv`, this argument can be > removed. I really prefer passing the priv pointer as a param. This is a totally usual way of providing callbacks, and it provides the user with immedatie concern that the priv pointer is there, no need to go check if it and how it can be accessed. https://gerrit.osmocom.org/c/osmocom-bb/+/34139/comment/4f7e2ff3_34991bd9 PS2, Line 48: changed_cb > It's currently way too generic: "something has changed, figure out what > exactly yourself by checking […] I'll look into this. -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34139 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I4c2ff25217fba0b6b4704f023071b86ed9afb55c Gerrit-Change-Number: 34139 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Comment-Date: Fri, 11 Aug 2023 16:06:16 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <[email protected]> Gerrit-MessageType: comment
