Attention is currently required from: osmith.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31620 )
Change subject: rsl: put values for Channel Mode into enums
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File include/osmocom/gsm/protocol/gsm_08_58.h:
https://gerrit.osmocom.org/c/libosmocore/+/31620/comment/782642ba_c56319e7
PS1, Line 544: RSL_CMOD_CSD_T_14400 = 0x18,
see my related comment in the TTCN3 patch: might make sense to standardize on the XXkY notation, so somehing kike 32k, 29k, 14k4, 9k6, 4k8, 2k4, 1k2, _600, 1200_74 ?
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31620
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I25bfd02aa1428a35492b20376a31635a442e545f
Gerrit-Change-Number: 31620
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 02 Mar 2023 14:24:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: jolly, tnt.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/31500 )
Change subject: Send raw data with e1d as it arrives from application
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
I think the question is why would we allow this? Yes, it may work with e1d, but it will likely not work with other drivers (DAHDI). In that case, I think libosmo-abis should make sure that code will use the API in a way that will work with all drivers, and not just with one.
Is this really enabling an actual valid use case, or is it just some kind of "let's permit it, because we can"? Not a rhethorical question, I want to understand!
--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/31500
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I3e3ef623a02bffe75b168dc42514455b4df1007f
Gerrit-Change-Number: 31500
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: tnt <tnt(a)246tNt.com>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: tnt <tnt(a)246tNt.com>
Gerrit-Comment-Date: Thu, 02 Mar 2023 14:20:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: dexter.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/31636 )
Change subject: mgcp_e1: rename e1_send to e1_send_ts_frame
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
Patchset:
PS1:
e1_ts_send_frame would be more logical, IMHO, as we typically tend to specify the "subject" (e1inp_ts) and then the action. But in any case, the new name is an improvement, and its just a static function, so no need to be super picky about it.
--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/31636
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I5fba7993860a63149a13d943edbfabc4008680ef
Gerrit-Change-Number: 31636
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 02 Mar 2023 14:17:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: dexter.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31619 )
Change subject: pcu_sock: drop usage of PCUIF flag PCU_IF_FLAG_DT
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/31619
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I0dc20e351deb14906b2edffc39499bad9659cc35
Gerrit-Change-Number: 31619
Gerrit-PatchSet: 3
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-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 02 Mar 2023 14:15:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment