- gerrit-log - lists.osmocom.org

[M] Change in osmo-bts[master]: RTP: make ortp optional at compile time
by falconia 25 Feb '26

25 Feb '26

Attention is currently required from: pespin. falconia has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-bts/+/42197?usp=email ) Change subject: RTP: make ortp optional at compile time ...................................................................... Patch Set 2: (3 comments) Patchset: PS2: > I see these 2 ORTP references, but they are probably wrong and can be dropped since they should come […] I will make a follow-up patch removing these bogons. And yes, they are bogons because osmo-bts configure has no pkg-config calls for ortp, only libosmo-abis configure does. File contrib/ber/Makefile.am: https://gerrit.osmocom.org/c/osmo-bts/+/42197/comment/6cde183d_86b99258?usp… : PS2, Line 1: if ENABLE_ORTP > See how it's done for instance in osmo-bsc/src/Makefile. […] Will do. File src/common/bts.c: https://gerrit.osmocom.org/c/osmo-bts/+/42197/comment/3d7653e4_cb5794d4?usp… : PS2, Line 406: { > you don't really need {} afaict, we are enforcing -std=gnu11 during build in configure. […] But I prefer to have these braces here, as they keep the local var contained in this block, not accessible outside of it. Initially I just put `#ifdef HAVE_ORTP` around the two original executing-code lines, but then the result was a compiler warning about unused automatic variable when building with `--disable-ortp`. If we remove the braces while keeping the local var declaration inside the `#ifdef`, then subsequent code in this function either sees or not sees this local var depending on ortp build config - not good IMO. -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42197?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Ib171bd42a65117457319befee2615e0c36c8d9e1 Gerrit-Change-Number: 42197 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <falcon(a)freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Wed, 25 Feb 2026 16:08:43 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in osmo-bts[master]: RTP: add vty option for ortp vs twrtp selection
by pespin 25 Feb '26

25 Feb '26

Attention is currently required from: falconia, fixeria, laforge. pespin has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-bts/+/42167?usp=email ) Change subject: RTP: add vty option for ortp vs twrtp selection ...................................................................... Patch Set 6: (2 comments) File src/common/vty.c: https://gerrit.osmocom.org/c/osmo-bts/+/42167/comment/e18fdc95_a5596f02?usp… : PS5, Line 434: if (bts->use_twrtp != g_use_twrtp_default) { > Can this comment be marked as resolved now that the code has been changed to the way you asked for? In general if I'm giving a +1 assume the issue is resolved on my side unless obvious exceptions 😊 https://gerrit.osmocom.org/c/osmo-bts/+/42167/comment/87f52515_b02dccd3?usp… : PS5, Line 1529: vty_out(vty, > Can this comment be marked as resolved now that the code has been changed to the way you asked for? Done -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42167?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Iff4e3a399250c16ba8fe4cb12e4e22f4c6b346ec Gerrit-Change-Number: 42167 Gerrit-PatchSet: 6 Gerrit-Owner: falconia <falcon(a)freecalypso.org> 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: falconia <falcon(a)freecalypso.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Wed, 25 Feb 2026 16:01:00 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: falconia <falcon(a)freecalypso.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

[S] Change in libosmocore[master]: core: fix config.h
by pespin 25 Feb '26

25 Feb '26

Attention is currently required from: Hoernchen, laforge. pespin has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/libosmocore/+/42205?usp=email ) Change subject: core: fix config.h ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: Once the 1st patch is reworked and this is merged, I guess it makes sense to make a libosmocore patch release with these fixes. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42205?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ic2cf52a3b60f43a2f5d3fe01c41a41f6fd9a8000 Gerrit-Change-Number: 42205 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 25 Feb 2026 15:59:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[S] Change in libosmocore[master]: core: fix config.h
by pespin 25 Feb '26

25 Feb '26

Attention is currently required from: Hoernchen, laforge. pespin has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/libosmocore/+/42205?usp=email ) Change subject: core: fix config.h ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42205?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ic2cf52a3b60f43a2f5d3fe01c41a41f6fd9a8000 Gerrit-Change-Number: 42205 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 25 Feb 2026 15:59:14 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in osmo-bts[master]: RTP: add vty option for ortp vs twrtp selection
by falconia 25 Feb '26

25 Feb '26

Attention is currently required from: fixeria, laforge, pespin. falconia has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/osmo-bts/+/42167?usp=email ) Change subject: RTP: add vty option for ortp vs twrtp selection ...................................................................... Patch Set 6: (2 comments) File src/common/vty.c: https://gerrit.osmocom.org/c/osmo-bts/+/42167/comment/b11b31ba_e9c6b294?usp… : PS5, Line 434: if (bts->use_twrtp != g_use_twrtp_default) { > I reworked the patch into the manner you asked for - is it good now? Can this comment be marked as resolved now that the code has been changed to the way you asked for? https://gerrit.osmocom.org/c/osmo-bts/+/42167/comment/e92bf0a9_66ae8b77?usp… : PS5, Line 1529: vty_out(vty, > I changed this show command to display only the current setting, and display it in the exact format […] Can this comment be marked as resolved now that the code has been changed to the way you asked for? -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42167?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Iff4e3a399250c16ba8fe4cb12e4e22f4c6b346ec Gerrit-Change-Number: 42167 Gerrit-PatchSet: 6 Gerrit-Owner: falconia <falcon(a)freecalypso.org> 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: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Wed, 25 Feb 2026 15:57:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: falconia <falcon(a)freecalypso.org> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in libosmocore[master]: fix --disable-log-macros
by pespin 25 Feb '26

25 Feb '26

Attention is currently required from: Hoernchen, laforge. pespin has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/libosmocore/+/42203?usp=email ) Change subject: fix --disable-log-macros ...................................................................... Patch Set 1: Code-Review-1 (1 comment) File configure.ac: https://gerrit.osmocom.org/c/libosmocore/+/42203/comment/f08175be_a31be9e6?… : PS1, Line 511: if test x"$log_macros" == x"yes" The problem is not the $enableval above, which I think is fine. The problem is actually the "if" here. We should check for x"$log_macros" != x"yes", or for x"$log_macros" == x"no" here. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42203?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I809ab2f61e72428ba21061055296eb83d6d710ab Gerrit-Change-Number: 42203 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 25 Feb 2026 15:57:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[XS] Change in osmo-trx[master]: checkpatch.conf: lint more code
by pespin 25 Feb '26

25 Feb '26

Attention is currently required from: Hoernchen, laforge, osmith. pespin has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-trx/+/42204?usp=email ) Change subject: checkpatch.conf: lint more code ...................................................................... Patch Set 2: (1 comment) File .checkpatch.conf: https://gerrit.osmocom.org/c/osmo-trx/+/42204/comment/307ab527_605b66e3?usp… : PS2, Line 4: --ignore INDENTED_LABEL Can we apply these ignores only to c++ files? We still have multiple .c files which would welcome all those. -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42204?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I0df5b6f2f0bf1469a80a1f5859809c30f523f683 Gerrit-Change-Number: 42204 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Wed, 25 Feb 2026 15:51:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

1 0

[XS] Change in libosmocore[master]: fix --disable-log-macros
by Hoernchen 25 Feb '26

25 Feb '26

Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/libosmocore/+/42203?usp=email ) Change subject: fix --disable-log-macros ...................................................................... Set Ready For Review -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42203?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I809ab2f61e72428ba21061055296eb83d6d710ab Gerrit-Change-Number: 42203 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Comment-Date: Wed, 25 Feb 2026 15:24:05 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No

1 0

[S] Change in libosmocore[master]: core: fix config.h
by Hoernchen 25 Feb '26

25 Feb '26

Hoernchen has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/42205?usp=email ) Change subject: core: fix config.h ...................................................................... core: fix config.h While exploring why logging is disabled but still shows up in the binaries I kind of wondered how all the config.h HAVE_XX checks we have everwhere work. Apparently they do not work at all, due to missing or misplaced include of config.h. This affects tons of other checks as well. Change-Id: Ic2cf52a3b60f43a2f5d3fe01c41a41f6fd9a8000 --- M src/core/application.c M src/core/backtrace.c M src/core/base64.c M src/core/bitcomp.c M src/core/bits.c M src/core/bitvec.c M src/core/context.c M src/core/conv_acc_generic.c M src/core/counter.c M src/core/crc16.c M src/core/crcXXgen.c.tpl M src/core/fsm.c M src/core/isdnhdlc.c M src/core/logging_emscripten.c M src/core/logging_systemd.c M src/core/loggingrb.c M src/core/mnl.c M src/core/msgb.c M src/core/msgfile.c M src/core/panic.c M src/core/prbs.c M src/core/prim.c M src/core/rate_ctr.c M src/core/rbtree.c M src/core/select.c M src/core/serial.c M src/core/signal.c M src/core/soft_uart.c M src/core/stat_item.c M src/core/strrb.c M src/core/tdef.c M src/core/timer.c M src/core/timer_gettimeofday.c M src/core/use_count.c M src/core/utils.c M src/core/write_queue.c 36 files changed, 36 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/05/42205/1 diff --git a/src/core/application.c b/src/core/application.c index f7e5816..bb89adb 100644 --- a/src/core/application.c +++ b/src/core/application.c @@ -68,6 +68,7 @@ * <http://sysmocom.de/> */ +#include "config.h" #include <osmocom/core/application.h> #include <osmocom/core/logging.h> diff --git a/src/core/backtrace.c b/src/core/backtrace.c index 60bd238..5930cb7 100644 --- a/src/core/backtrace.c +++ b/src/core/backtrace.c @@ -22,9 +22,9 @@ #include <stdio.h> #include <stdlib.h> +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/logging.h> -#include "config.h" #ifdef HAVE_EXECINFO_H #include <execinfo.h> diff --git a/src/core/base64.c b/src/core/base64.c index 0c161ce..0be4e8c 100644 --- a/src/core/base64.c +++ b/src/core/base64.c @@ -16,6 +16,7 @@ * GNU General Public License for more details. */ +#include "config.h" #include <osmocom/core/base64.h> #include <stdint.h> diff --git a/src/core/bitcomp.c b/src/core/bitcomp.c index 5fb2cba..193c149 100644 --- a/src/core/bitcomp.c +++ b/src/core/bitcomp.c @@ -29,6 +29,7 @@ #include <errno.h> #include <string.h> +#include "config.h" #include <osmocom/core/bitvec.h> #include <osmocom/core/bitcomp.h> diff --git a/src/core/bits.c b/src/core/bits.c index 3da7d9b..8616d32 100644 --- a/src/core/bits.c +++ b/src/core/bits.c @@ -20,6 +20,7 @@ #include <stdint.h> +#include "config.h" #include <osmocom/core/bits.h> /*! \addtogroup bits diff --git a/src/core/bitvec.c b/src/core/bitvec.c index ac702b9..5e1d2c8 100644 --- a/src/core/bitvec.c +++ b/src/core/bitvec.c @@ -38,6 +38,7 @@ #include <stdio.h> #include <stdbool.h> +#include "config.h" #include <osmocom/core/bits.h> #include <osmocom/core/bitvec.h> #include <osmocom/core/panic.h> diff --git a/src/core/context.c b/src/core/context.c index a0b3a55..b194b9e 100644 --- a/src/core/context.c +++ b/src/core/context.c @@ -18,6 +18,7 @@ */ #include <string.h> #include <errno.h> +#include "config.h" #include <osmocom/core/talloc.h> #include <osmocom/core/utils.h> diff --git a/src/core/conv_acc_generic.c b/src/core/conv_acc_generic.c index 2257e6a..8589e2f 100644 --- a/src/core/conv_acc_generic.c +++ b/src/core/conv_acc_generic.c @@ -22,6 +22,7 @@ #include <stdlib.h> #include <stdint.h> #include <string.h> +#include "config.h" /* Add-Compare-Select (ACS-Butterfly) * Compute 4 accumulated path metrics and 4 path selections. Note that path diff --git a/src/core/counter.c b/src/core/counter.c index dace15f..62b7d70 100644 --- a/src/core/counter.c +++ b/src/core/counter.c @@ -21,6 +21,7 @@ #include <string.h> +#include "config.h" #include <osmocom/core/linuxlist.h> #include <osmocom/core/talloc.h> #include <osmocom/core/counter.h> diff --git a/src/core/crc16.c b/src/core/crc16.c index 29dace2..bbded42 100644 --- a/src/core/crc16.c +++ b/src/core/crc16.c @@ -12,6 +12,7 @@ * SPDX-License-Identifier: GPL-2.0 */ +#include "config.h" #include <osmocom/core/crc16.h> /*! CRC table for the CRC-16. The poly is 0x8005 (x^16 + x^15 + x^2 + 1) */ diff --git a/src/core/crcXXgen.c.tpl b/src/core/crcXXgen.c.tpl index 154291c..75de2ad 100644 --- a/src/core/crcXXgen.c.tpl +++ b/src/core/crcXXgen.c.tpl @@ -25,7 +25,7 @@ * \file crcXXgen.c.tpl */ #include <stdint.h> - +#include "config.h" #include <osmocom/core/bits.h> #include <osmocom/core/crcXXgen.h> diff --git a/src/core/fsm.c b/src/core/fsm.c index a4f1bcb..54cea99 100644 --- a/src/core/fsm.c +++ b/src/core/fsm.c @@ -21,6 +21,7 @@ #include <string.h> #include <inttypes.h> +#include "config.h" #include <osmocom/core/fsm.h> #include <osmocom/core/talloc.h> #include <osmocom/core/logging.h> diff --git a/src/core/isdnhdlc.c b/src/core/isdnhdlc.c index 4ced5af..af9c941 100644 --- a/src/core/isdnhdlc.c +++ b/src/core/isdnhdlc.c @@ -22,6 +22,7 @@ #include <string.h> +#include "config.h" #include <osmocom/core/crc16.h> #include <osmocom/core/bits.h> #include <osmocom/core/isdnhdlc.h> diff --git a/src/core/logging_emscripten.c b/src/core/logging_emscripten.c index a186fd2..76af90c 100644 --- a/src/core/logging_emscripten.c +++ b/src/core/logging_emscripten.c @@ -27,6 +27,7 @@ #include <stdarg.h> #include <stdio.h> +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/logging.h> #include <osmocom/core/logging_internal.h> diff --git a/src/core/logging_systemd.c b/src/core/logging_systemd.c index 2e86feb..7ce8aa5 100644 --- a/src/core/logging_systemd.c +++ b/src/core/logging_systemd.c @@ -28,6 +28,7 @@ #include <systemd/sd-journal.h> +#include "config.h" #include <osmocom/core/talloc.h> #include <osmocom/core/utils.h> #include <osmocom/core/logging.h> diff --git a/src/core/loggingrb.c b/src/core/loggingrb.c index 2bf7b66..1cb2cde 100644 --- a/src/core/loggingrb.c +++ b/src/core/loggingrb.c @@ -29,6 +29,7 @@ * * \file loggingrb.c */ +#include "config.h" #include <osmocom/core/strrb.h> #include <osmocom/core/logging.h> #include <osmocom/core/loggingrb.h> diff --git a/src/core/mnl.c b/src/core/mnl.c index d148e1b..d789015 100644 --- a/src/core/mnl.c +++ b/src/core/mnl.c @@ -23,6 +23,7 @@ * GNU General Public License for more details. */ +#include "config.h" #include <osmocom/core/select.h> #include <osmocom/core/talloc.h> #include <osmocom/core/logging.h> diff --git a/src/core/msgb.c b/src/core/msgb.c index 6f081bb..6e672a1 100644 --- a/src/core/msgb.c +++ b/src/core/msgb.c @@ -56,6 +56,7 @@ #include <stdarg.h> #include <errno.h> +#include "config.h" #include <osmocom/core/msgb.h> #include <osmocom/core/talloc.h> #include <osmocom/core/logging.h> diff --git a/src/core/msgfile.c b/src/core/msgfile.c index abb4e7c..4a9592b 100644 --- a/src/core/msgfile.c +++ b/src/core/msgfile.c @@ -21,6 +21,7 @@ #define _WITH_GETLINE +#include "config.h" #include <osmocom/core/msgfile.h> #include <osmocom/core/talloc.h> diff --git a/src/core/panic.c b/src/core/panic.c index bbf6d08..b22af59 100644 --- a/src/core/panic.c +++ b/src/core/panic.c @@ -24,10 +24,10 @@ * \file panic.c */ #include <unistd.h> +#include "config.h" #include <osmocom/core/panic.h> #include <osmocom/core/backtrace.h> -#include "config.h" static osmo_panic_handler_t osmo_panic_handler = (void*)0; diff --git a/src/core/prbs.c b/src/core/prbs.c index 8fa04bb..31797ab 100644 --- a/src/core/prbs.c +++ b/src/core/prbs.c @@ -7,6 +7,7 @@ #include <stdint.h> #include <string.h> +#include "config.h" #include <osmocom/core/bits.h> #include <osmocom/core/prbs.h> diff --git a/src/core/prim.c b/src/core/prim.c index 3c8a7f1..2cd00e6 100644 --- a/src/core/prim.c +++ b/src/core/prim.c @@ -8,6 +8,7 @@ * @{ * \file prim.c */ +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/prim.h> diff --git a/src/core/rate_ctr.c b/src/core/rate_ctr.c index 44e2658..a90ad88 100644 --- a/src/core/rate_ctr.c +++ b/src/core/rate_ctr.c @@ -60,6 +60,7 @@ #include <unistd.h> #include <inttypes.h> +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/linuxlist.h> #include <osmocom/core/talloc.h> diff --git a/src/core/rbtree.c b/src/core/rbtree.c index f4dc219..d651210 100644 --- a/src/core/rbtree.c +++ b/src/core/rbtree.c @@ -18,6 +18,7 @@ linux/lib/rbtree.c */ +#include "config.h" #include <osmocom/core/linuxrbtree.h> static void __rb_rotate_left(struct rb_node *node, struct rb_root *root) diff --git a/src/core/select.c b/src/core/select.c index a577c5b..726eb5b 100644 --- a/src/core/select.c +++ b/src/core/select.c @@ -27,6 +27,7 @@ #include <stdbool.h> #include <errno.h> +#include "config.h" #include <osmocom/core/select.h> #include <osmocom/core/linuxlist.h> #include <osmocom/core/timer.h> @@ -36,7 +37,6 @@ #include <osmocom/core/stat_item.h> #include <osmocom/core/stats_tcp.h> -#include "config.h" #if defined(HAVE_SYS_SELECT_H) && defined(HAVE_POLL_H) #include <sys/select.h> diff --git a/src/core/serial.c b/src/core/serial.c index 117c049..91ee0e0 100644 --- a/src/core/serial.c +++ b/src/core/serial.c @@ -36,6 +36,7 @@ #include <linux/serial.h> #endif +#include "config.h" #include <osmocom/core/serial.h> diff --git a/src/core/signal.c b/src/core/signal.c index ba1555a..111513a 100644 --- a/src/core/signal.c +++ b/src/core/signal.c @@ -18,6 +18,7 @@ * */ +#include "config.h" #include <osmocom/core/signal.h> #include <osmocom/core/talloc.h> #include <osmocom/core/linuxlist.h> diff --git a/src/core/soft_uart.c b/src/core/soft_uart.c index f969ab7..259099e 100644 --- a/src/core/soft_uart.c +++ b/src/core/soft_uart.c @@ -24,6 +24,7 @@ #include <stdint.h> #include <errno.h> +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/timer.h> #include <osmocom/core/soft_uart.h> diff --git a/src/core/stat_item.c b/src/core/stat_item.c index 804972b..3b2d1bf 100644 --- a/src/core/stat_item.c +++ b/src/core/stat_item.c @@ -154,6 +154,7 @@ #include <stdint.h> #include <string.h> +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/linuxlist.h> #include <osmocom/core/logging.h> diff --git a/src/core/strrb.c b/src/core/strrb.c index c5a5ed6..d162d3a 100644 --- a/src/core/strrb.c +++ b/src/core/strrb.c @@ -41,6 +41,7 @@ #include <string.h> #include <string.h> +#include "config.h" #include <osmocom/core/strrb.h> #include <osmocom/core/talloc.h> diff --git a/src/core/tdef.c b/src/core/tdef.c index f0c0f2e..c565639 100644 --- a/src/core/tdef.c +++ b/src/core/tdef.c @@ -27,6 +27,7 @@ #include <limits.h> #include <errno.h> +#include "config.h" #include <osmocom/core/fsm.h> #include <osmocom/core/tdef.h> diff --git a/src/core/timer.c b/src/core/timer.c index 067bd87..c9cd841 100644 --- a/src/core/timer.c +++ b/src/core/timer.c @@ -31,6 +31,7 @@ #include <assert.h> #include <string.h> #include <limits.h> +#include "config.h" #include <osmocom/core/timer.h> #include <osmocom/core/timer_compat.h> #include <osmocom/core/linuxlist.h> diff --git a/src/core/timer_gettimeofday.c b/src/core/timer_gettimeofday.c index e0212b5..bad7053 100644 --- a/src/core/timer_gettimeofday.c +++ b/src/core/timer_gettimeofday.c @@ -39,6 +39,7 @@ #include <stdbool.h> #include <sys/time.h> +#include "config.h" #include <osmocom/core/timer_compat.h> bool osmo_gettimeofday_override = false; diff --git a/src/core/use_count.c b/src/core/use_count.c index 9714403..e99f5e3 100644 --- a/src/core/use_count.c +++ b/src/core/use_count.c @@ -25,6 +25,7 @@ #include <inttypes.h> #include <string.h> +#include "config.h" #include <osmocom/core/linuxlist.h> #include <osmocom/core/utils.h> #include <osmocom/core/use_count.h> diff --git a/src/core/utils.c b/src/core/utils.c index 5757b52..2ae68f1 100644 --- a/src/core/utils.c +++ b/src/core/utils.c @@ -36,6 +36,7 @@ #include <inttypes.h> #include <limits.h> +#include "config.h" #include <osmocom/core/utils.h> #include <osmocom/core/bit64gen.h> @@ -469,7 +470,6 @@ __attribute__((weak, alias("osmo_hexdump_nospc"))); #endif -#include "config.h" #ifdef HAVE_CTYPE_H #include <ctype.h> /*! Convert an entire string to lower case diff --git a/src/core/write_queue.c b/src/core/write_queue.c index 8fb73a6..cea875f 100644 --- a/src/core/write_queue.c +++ b/src/core/write_queue.c @@ -19,6 +19,7 @@ */ #include <errno.h> +#include "config.h" #include <osmocom/core/write_queue.h> #include <osmocom/core/logging.h> -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/42205?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ic2cf52a3b60f43a2f5d3fe01c41a41f6fd9a8000 Gerrit-Change-Number: 42205 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>

1 0

[L] Change in osmo-trx[master]: feat(usdr): add USDR backend support (osmo-trx-usdr)
by osmith 25 Feb '26

25 Feb '26

Attention is currently required from: Hoernchen, Timur Davydov, laforge. osmith has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email ) Change subject: feat(usdr): add USDR backend support (osmo-trx-usdr) ...................................................................... Patch Set 3: (1 comment) Patchset: PS3: > The linter does not understand c++ and is mostly useless, so the thing to use here is clang-format w […] please take a look at the patch. with that config, it works well even with the c++ code. -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42198?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: I5d1d25921514954c4929ae6e7352168b3ceb05df Gerrit-Change-Number: 42198 Gerrit-PatchSet: 3 Gerrit-Owner: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: Hoernchen <ewild(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-CC: osmith <osmith(a)sysmocom.de> Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: Timur Davydov <dtv.comp(a)gmail.com> Gerrit-Comment-Date: Wed, 25 Feb 2026 15:21:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: osmith <osmith(a)sysmocom.de> Comment-In-Reply-To: Hoernchen <ewild(a)sysmocom.de> Comment-In-Reply-To: laforge <laforge(a)osmocom.org>

1 0

2026

2025

2024

2023

2022

gerrit-log