fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/33230 )
Change subject: rlcmac: l1ctl_prim: add 'start_rfn' field to cfg_{ul,dl}_tbf_req
......................................................................
Set Ready For Review
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/33230
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I27a7a896fe3839fa4f9b8cd9500c4ab7867bbaa0
Gerrit-Change-Number: 33230
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Wed, 09 Aug 2023 18:31:26 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/34124 )
Change subject: tlv: Introduce API msgb_tv32_push()
......................................................................
Patch Set 1:
(1 comment)
File include/osmocom/gsm/tlv.h:
https://gerrit.osmocom.org/c/libosmocore/+/34124/comment/22ffc255_35eba5bc
PS1, Line 464: 5
> `1 + sizeof(val)`
yeah I just wanted to keep it inline with msgb_tv16_push
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/34124
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I68d5693a18d55ce8d0647359044157d7e5bfae50
Gerrit-Change-Number: 34124
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 09 Aug 2023 18:30:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34115 )
Change subject: l1ctl: Fill ph_data_param fn field
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmocom-bb/+/34115/comment/60aea3de_86119f98
PS1, Line 7: Fill ph_data_param fn field
> Should have been noted in the commit message that the current LAPDm implementation makes no use of t […]
that will change soon hopefully, already WIP :D
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34115
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I3317296b2bda1088a2af81ba9b135714f8393afd
Gerrit-Change-Number: 34115
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Wed, 09 Aug 2023 18:29:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment