Attention is currently required from: jolly, fixeria, dexter. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/33448 )
Change subject: ASCI: Add Notification/FACCH support ...................................................................... Patch Set 6: (5 comments) File src/common/rsl.c: https://gerrit.osmocom.org/c/osmo-bts/+/33448/comment/07b0d121_fc069430 PS6, Line 816: * the call is active. Most MSs in dedicated mode may not be able to receive the NCH instead. s/instead/otherwise/ https://gerrit.osmocom.org/c/osmo-bts/+/33448/comment/5a1624d2_00e3e061 PS6, Line 820: uint8_t chan_desc_len, int count) unsigned count https://gerrit.osmocom.org/c/osmo-bts/+/33448/comment/199095e1_45f66374 PS6, Line 826: int tn, ln, n; all these can be unsigned https://gerrit.osmocom.org/c/osmo-bts/+/33448/comment/89be0c0a_69679fa7 PS6, Line 837: if (lchan->type != GSM_LCHAN_SDCCH isn't there an lchan_is_dedicated() or alike? https://gerrit.osmocom.org/c/osmo-bts/+/33448/comment/8c66d0cb_5e6b495a PS6, Line 845: msgb_tl16v_put(msg, RSL_IE_L3_INFO, sizeof(notif), (uint8_t *) ¬if); can you explain a bit more all this loop with the extra call after it? -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/33448 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I3ed14fa54a907891e492a7ada8e745a2c56cd46d Gerrit-Change-Number: 33448 Gerrit-PatchSet: 6 Gerrit-Owner: jolly <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <[email protected]> Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: osmith <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: jolly <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Attention: dexter <[email protected]> Gerrit-Comment-Date: Wed, 19 Jul 2023 14:53:15 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
