Attention is currently required from: jolly. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33603 )
Change subject: ASCI: Make bssmap_handle_ass_req_ct_speech() publically available. ...................................................................... Patch Set 3: (1 comment) File src/osmo-bsc/osmo_bsc_bssap.c: https://gerrit.osmocom.org/c/osmo-bsc/+/33603/comment/35604072_8a692ecd PS2, Line 744: static int select_codecs(struct assignment_request *req, const struct gsm0808_channel_type *ct, > bssmap_handle_ass_req_ct_speech() is called by vgcs_fsm.c. […] Can you point me to where is this function newly used? (I guess it's some other patch?) Also, the commit message is misleading. You are not simply making the function public, you are changing the logic and params also imho. Finally, to make the code paths easier, maybe avoid passing NULL in the only caller so far, and pass conn_get_bts() as a param there. -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33603 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: Ifc1e315d5282f01f8d1bd600d62476c2ae74eca9 Gerrit-Change-Number: 33603 Gerrit-PatchSet: 3 Gerrit-Owner: jolly <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <[email protected]> Gerrit-Attention: jolly <[email protected]> Gerrit-Comment-Date: Mon, 10 Jul 2023 11:57:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: jolly <[email protected]> Comment-In-Reply-To: pespin <[email protected]> Gerrit-MessageType: comment
