Attention is currently required from: laforge, daniel.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31360 )
Change subject: Move include osmocom/gprs -> osmocom/gb, deprecate osmocom/gprs/
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> As I said already in the chat: this is not going to work. […]
This is not breaking API of old applications. It would break it if we at some point remove the dummy header left under gprs/.
While I'm in favour of keeping ABI/API compatibility, I disagree with the fact that we can ever break it. One of such cases is by properly preparing that break over time so that one makes sure all users are moved to a new API, by signalling to them that they should move to the newer API and give some time until dropping it (let's say 2 years).
If that's not the intention, why are we even marking functions as DEPRECATED in first place?
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31360
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I6e826775552766e34e4c06fe2390084596dfc286
Gerrit-Change-Number: 31360
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 10:15:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
Attention is currently required from: osmith, fixeria.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/31403 )
Change subject: Run struct_endianness.py
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
Patchset:
PS1:
The local copy of libosmocore (src/shared/libosmocore) should be kept unchanged. The idea is to eventually get rid of it and build the firmware with recent libosmocore.
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/31403
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I617c967c5f34c0be2cf6fd43ceb1af17897f2bf1
Gerrit-Change-Number: 31403
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 10:11:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
osmith has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/osmocom-bb/+/31402 )
Change subject: Cosmetic: fix encoding of ui-experiment/font.h
......................................................................
Cosmetic: fix encoding of ui-experiment/font.h
Remove characters 0x80 - 0xff from comments in this source file, as
they can't be parsed as UTF-8. This causes struct_endianness.py to
error on the file, git and gerrit can't display a proper diff and treat
it as binary file, etc.
Related: OS#5884
Change-Id: I0daebccf819595ee64fb4c9713a4f0ce219c58be
---
M src/target/ui-experiment/font.h
1 file changed, 15 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/02/31402/2
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/31402
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I0daebccf819595ee64fb4c9713a4f0ce219c58be
Gerrit-Change-Number: 31402
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-MessageType: newpatchset
osmith has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/osmocom-bb/+/31403 )
Change subject: Run struct_endianness.py
......................................................................
Run struct_endianness.py
Ensure there is no diff to prepare to run this in CI.
Related: OS#5884
Change-Id: I617c967c5f34c0be2cf6fd43ceb1af17897f2bf1
---
M src/host/gprsdecode/rlcmac.h
M src/host/layer23/include/osmocom/bb/common/sim.h
M src/shared/libosmocore/include/osmocom/gsm/protocol/gsm_03_41.h
M src/shared/libosmocore/include/osmocom/gsm/protocol/gsm_04_08.h
M src/shared/libosmocore/include/osmocom/gsm/protocol/gsm_04_12.h
M src/shared/libosmocore/include/osmocom/gsm/protocol/gsm_08_58.h
M src/shared/libosmocore/include/osmocom/gsm/protocol/gsm_44_318.h
7 files changed, 437 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/03/31403/2
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/31403
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I617c967c5f34c0be2cf6fd43ceb1af17897f2bf1
Gerrit-Change-Number: 31403
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-MessageType: newpatchset
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/31402 )
Change subject: Cosmetic: fix encoding of ui-experiment/font.h
......................................................................
Cosmetic: fix encoding of ui-experiment/font.h
Remove characters 0x80 - 0xff from this source file, as they can't be
parsed as UTF-8. This causes struct_endianness.py to error on the file,
git can't display a proper diff and treats it as binary file, etc.
Related: OS#5884
Change-Id: I0daebccf819595ee64fb4c9713a4f0ce219c58be
---
M src/target/ui-experiment/font.h
1 file changed, 14 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/02/31402/1
diff --git a/src/target/ui-experiment/font.h b/src/target/ui-experiment/font.h
index 5dc6eae..c75a0f8 100644
--- a/src/target/ui-experiment/font.h
+++ b/src/target/ui-experiment/font.h
Binary files differ
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/31402
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I0daebccf819595ee64fb4c9713a4f0ce219c58be
Gerrit-Change-Number: 31402
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange
Attention is currently required from: fixeria, dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/31341 )
Change subject: pcu_l1_if_phy: flexible phy access
......................................................................
Patch Set 8:
(1 comment)
Patchset:
PS8:
> There is one open question: How do we want to select the PHY. […]
what? I thought it was clear (at least to me) that the backend was to be selected using osmo-pcu VTY.
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/31341
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I8692d1bd5d137a17cf596ee2914722f419c9978d
Gerrit-Change-Number: 31341
Gerrit-PatchSet: 8
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 10:08:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment