pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40729?usp=email )
Change subject: ngap: Fix deviation in ASN1 from TS 38.413 version 17.5.0
......................................................................
ngap: Fix deviation in ASN1 from TS 38.413 version 17.5.0
I originally copied the ASN1 files from https://forge.etsi.org/rep/int/5g-core/ngap.git
directory ttcn/LibNGAP/lib/asn1, commit fa43d01e202379a969e991a9454d9c6c6930b5b5.
Apparently one file contained a deviation from the 3GPP content and it
was also fixed in ngap.git in 47e7239dd80f2f12a4cbcdd7977ca0a32356f3be.
I found out this problem when trying to use this ASN1 file to generate
new libfftranscode, which then failed to properly decode
DownlinkNASTransport due to the deviation.
Change-Id: I7d127998ea162a392f3381ebb1d198ad3c783a56
---
M library/ngap/NGAP_PDU_Descriptions.asn
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/29/40729/1
diff --git a/library/ngap/NGAP_PDU_Descriptions.asn b/library/ngap/NGAP_PDU_Descriptions.asn
index d8dbd98..dcf4704 100644
--- a/library/ngap/NGAP_PDU_Descriptions.asn
+++ b/library/ngap/NGAP_PDU_Descriptions.asn
@@ -266,7 +266,7 @@
InitiatingMessage ::= SEQUENCE {
procedureCode NGAP-ELEMENTARY-PROCEDURE.&procedureCode ({NGAP-ELEMENTARY-PROCEDURES}),
criticality NGAP-ELEMENTARY-PROCEDURE.&criticality ({NGAP-ELEMENTARY-PROCEDURES}{@procedureCode}),
- value NGAP-ELEMENTARY-PROCEDURE.&InitiatingMessage ({NGAP-ELEMENTARY-PROCEDURES}{@procedureCode}) OPTIONAL
+ value NGAP-ELEMENTARY-PROCEDURE.&InitiatingMessage ({NGAP-ELEMENTARY-PROCEDURES}{@procedureCode})
}
SuccessfulOutcome ::= SEQUENCE {
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40729?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I7d127998ea162a392f3381ebb1d198ad3c783a56
Gerrit-Change-Number: 40729
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/40722?usp=email )
Change subject: timers: fix them
......................................................................
timers: fix them
Since they are barely used at all they silently broke during the usb fixing attempts, which in turn broke ATR timeouts for cards that don't play well with 5V.
Change-Id: I0a9ba7a9097fe09614beceeae7541bd7b82b9087
---
M sysmoOCTSIM/main.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/sysmoOCTSIM/main.c b/sysmoOCTSIM/main.c
index b0c0229..7d81853 100644
--- a/sysmoOCTSIM/main.c
+++ b/sysmoOCTSIM/main.c
@@ -543,7 +543,7 @@
void reset_all_stuff_irq(void)
{
- SysTick->CTRL = 0; // no clock ticks for osmo timers
+ SysTick->CTRL &= ~SysTick_CTRL_ENABLE_Msk; // no clock ticks for osmo timers
Sercom *const sercom_modules[] = SERCOM_INSTS;
for (uint32_t i = 0; i < SERCOM_INST_NUM; i++) {
@@ -605,7 +605,7 @@
#endif
}
- SysTick->CTRL = 1;
+ SysTick->CTRL |= SysTick_CTRL_ENABLE_Msk;
// while (!ccid_df_is_enabled())
// ;
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/40722?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I0a9ba7a9097fe09614beceeae7541bd7b82b9087
Gerrit-Change-Number: 40722
Gerrit-PatchSet: 3
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Attention is currently required from: Hoernchen.
laforge has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/40722?usp=email )
Change subject: timers: fix them
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
File sysmoOCTSIM/main.c:
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/40722/comment/fda5aa0b_50… :
PS2, Line 546: SysTick->CTRL &= ~SysTick_CTRL_ENABLE_Msk; // no clock ticks for osmo timers
> It temporarily disables the systick to ensure there are no clock ticks for the timers when doing the […]
ah, nevermind. How could I miss the negation...
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/40722?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I0a9ba7a9097fe09614beceeae7541bd7b82b9087
Gerrit-Change-Number: 40722
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 22 Jul 2025 08:22:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Hoernchen <ewild(a)sysmocom.de>
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
osmith has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40728?usp=email )
Change subject: rebar.lock: update enftables commit to latest
......................................................................
rebar.lock: update enftables commit to latest
Update the enftables dependency to the latest commit "Add missing
include of string.h", which is right after the one currently listed in
rebar.lock. This should fix the build failing for ubuntu 24.10 in the
Osmocom OBS:
[ 281s] /usr/src/packages/BUILD/_checkouts/enftables/c_src/enftables.c:68:28: error: implicit declaration of function 'strlen' [-Wimplicit-function-declaration]
[ 281s] 68 | bin.size = strlen(obuf);
[ 281s] | ^~~~~~
[ 281s] /usr/src/packages/BUILD/_checkouts/enftables/c_src/enftables.c:16:1: note: include '<string.h>' or provide a declaration of 'strlen'
Related: https://gitea.osmocom.org/erlang/enftables/commit/8b9b4acd624b9d0d3a58d725f…
Change-Id: Iffd691bc5010d913728a9d683946e39f80c5655b
---
M rebar.lock
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, approved
diff --git a/rebar.lock b/rebar.lock
index c5a90df..0c6730a 100644
--- a/rebar.lock
+++ b/rebar.lock
@@ -2,7 +2,7 @@
[{<<"cut">>,{pkg,<<"cut">>,<<"1.0.3">>},1},
{<<"enftables">>,
{git,"https://gitea.osmocom.org/erlang/enftables.git",
- {ref,"be23c8ebac948aa1eea450b368af437d12a7173a"}},
+ {ref,"8b9b4acd624b9d0d3a58d725f11afa96d31fe5d7"}},
0},
{<<"exometer_core">>,
{git,"https://github.com/Feuerlabs/exometer_core.git",
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40728?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Iffd691bc5010d913728a9d683946e39f80c5655b
Gerrit-Change-Number: 40728
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40728?usp=email )
Change subject: rebar.lock: update enftables commit to latest
......................................................................
rebar.lock: update enftables commit to latest
Update the enftables dependency to the latest commit "Add missing
include of string.h", which is right after the one currently listed in
rebar.lock. This should fix the build failing for ubuntu 24.10 in the
Osmocom OBS:
[ 281s] /usr/src/packages/BUILD/_checkouts/enftables/c_src/enftables.c:68:28: error: implicit declaration of function 'strlen' [-Wimplicit-function-declaration]
[ 281s] 68 | bin.size = strlen(obuf);
[ 281s] | ^~~~~~
[ 281s] /usr/src/packages/BUILD/_checkouts/enftables/c_src/enftables.c:16:1: note: include '<string.h>' or provide a declaration of 'strlen'
Related: https://gitea.osmocom.org/erlang/enftables/commit/8b9b4acd624b9d0d3a58d725f…
Change-Id: Iffd691bc5010d913728a9d683946e39f80c5655b
---
M rebar.lock
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-s1gw refs/changes/28/40728/1
diff --git a/rebar.lock b/rebar.lock
index c5a90df..0c6730a 100644
--- a/rebar.lock
+++ b/rebar.lock
@@ -2,7 +2,7 @@
[{<<"cut">>,{pkg,<<"cut">>,<<"1.0.3">>},1},
{<<"enftables">>,
{git,"https://gitea.osmocom.org/erlang/enftables.git",
- {ref,"be23c8ebac948aa1eea450b368af437d12a7173a"}},
+ {ref,"8b9b4acd624b9d0d3a58d725f11afa96d31fe5d7"}},
0},
{<<"exometer_core">>,
{git,"https://github.com/Feuerlabs/exometer_core.git",
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40728?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Iffd691bc5010d913728a9d683946e39f80c5655b
Gerrit-Change-Number: 40728
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>