Attention is currently required from: jolly.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33606 )
Change subject: ASCI: Do not wait for RLL establishment
......................................................................
Patch Set 3:
(1 comment)
File src/osmo-bsc/lchan_fsm.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/33606/comment/b863ce29_c0d1a961
PS3, Line 1046: lchan->activate.info.activ_for == ACTIVATE_FOR_VGCS_CHANNEL) &&
this indentation is still wrong. It should be one space further right.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33606
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I96390924736029b92e54590157e38093be749dd9
Gerrit-Change-Number: 33606
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Mon, 10 Jul 2023 12:02:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: jolly.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33605 )
Change subject: ASCI: Do not release channel, if SAPI 0 is released
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33605
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ief1e1894362c4917f6e0092268690f68c8193750
Gerrit-Change-Number: 33605
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Mon, 10 Jul 2023 12:01:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
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 <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Mon, 10 Jul 2023 11:57:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: jolly <andreas(a)eversberg.eu>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: dexter.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/33522 )
Change subject: pcu_sock: use PCUIF version 11 (direct TLLI)
......................................................................
Patch Set 2: Code-Review+2
(2 comments)
Commit Message:
https://gerrit.osmocom.org/c/osmo-bts/+/33522/comment/37314a3f_376cb7a3
PS2, Line 10: OsmoPCU is still compatible to version 10
> At the moment OsmoPCU has support for v11 but still accepts v10 PCUIF. […]
Ah, sorry for the noise. I misread and thought this sentence is about osmo-bts.
https://gerrit.osmocom.org/c/osmo-bts/+/33522/comment/aaed856a_a224391e
PS2, Line 23: Depends
Related?
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/33522
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I25816ac12e63cc6b641eb414e6bc7eaa9c85fc25
Gerrit-Change-Number: 33522
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 10 Jul 2023 11:57:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: pespin.
jolly has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33596 )
Change subject: ASCI: Add new rate counters to support VGCS/VBS messages
......................................................................
Patch Set 1:
(1 comment)
File src/osmo-bsc/osmo_bsc_msc.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/33596/comment/f202c398_fd86913d
PS1, Line 149: [MSC_CTR_BSSMAP_TX_DT1_VGCS_VBS_SETUP_ACK] = {
> Same, format change.
Format cleaned in another patch.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33596
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I18e4ca3599e480de2d0f64cc1b6f4bb6ce8020d4
Gerrit-Change-Number: 33596
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 10 Jul 2023 11:52:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment