Attention is currently required from: jolly, fixeria, dexter.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/33297
)
Change subject: ASCI: Add Notification CHannel (NCH) support
......................................................................
Patch Set 7:
(2 comments)
File include/osmo-bts/bts.h:
https://gerrit.osmocom.org/c/osmo-bts/+/33297/comment/9be843af_248d0537
PS7, Line 413: int is_nch_res, int is_ag_res);
sounds like these are bools?
File tests/agch/agch_test.c:
https://gerrit.osmocom.org/c/osmo-bts/+/33297/comment/b6c546a3_3205de66
PS7, Line 162: rc = bts_ccch_copy_msg(bts, out_buf, &g_time, 0, is_agch);
false
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bts/+/33297
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I82fdaba3faaced76267a99ae14a5458a1b41fdaa
Gerrit-Change-Number: 33297
Gerrit-PatchSet: 7
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 19 Jul 2023 14:56:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment