July 2026 - gerrit-log - lists.osmocom.org

[M] Change in osmo-ccid-firmware[master]: Implement vendor requests to set debug/break_on_panic
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42617?usp=email ) Change subject: Implement vendor requests to set debug/break_on_panic ...................................................................... Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42617?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: Ia2bffa05b96e3857610cb3ad2f9122fa75a90780 Gerrit-Change-Number: 42617 Gerrit-PatchSet: 4 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:50:35 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in osmo-ccid-firmware[master]: Rework default IRQ handler and osmo panic handler
by laforge 16 Jul '26

16 Jul '26

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42616?usp=email ) Change subject: Rework default IRQ handler and osmo panic handler ...................................................................... Rework default IRQ handler and osmo panic handler Don't break on panic by default. Using a variable allows to set it by SWD or later by a usb message. Use the same handler for osmo panics as well as unknown IRQ. Including faults (bus, hard). Change-Id: Ibcabe0c58ff14ebe76159470dd90fdef064febea --- M sysmoOCTSIM/gcc/gcc/startup_same54.c M sysmoOCTSIM/include/same54.h M sysmoOCTSIM/main.c 3 files changed, 17 insertions(+), 4 deletions(-) Approvals: Hoernchen: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/sysmoOCTSIM/gcc/gcc/startup_same54.c b/sysmoOCTSIM/gcc/gcc/startup_same54.c index 097a1c4..943c965 100644 --- a/sysmoOCTSIM/gcc/gcc/startup_same54.c +++ b/sysmoOCTSIM/gcc/gcc/startup_same54.c @@ -668,11 +668,18 @@ ; } +volatile bool break_on_panic = false; + /** * \brief Default interrupt handler for unused IRQs. */ void Dummy_Handler(void) { - while (1) { + if (break_on_panic) + __asm("BKPT #0"); /* wait for JTAG/SWD */ + else { + NVIC_SystemReset(); + while (1) + ; } } diff --git a/sysmoOCTSIM/include/same54.h b/sysmoOCTSIM/include/same54.h index 7da756e..c99d91f 100644 --- a/sysmoOCTSIM/include/same54.h +++ b/sysmoOCTSIM/include/same54.h @@ -47,4 +47,10 @@ #error Library does not support the specified device. #endif +#include <stdbool.h> + +/* Either reset or break on panic/unknown irq. Default is reset. */ +extern volatile bool break_on_panic; +void Dummy_Handler(void); + #endif /* _SAME54_ */ diff --git a/sysmoOCTSIM/main.c b/sysmoOCTSIM/main.c index 7873c86..02feeb6 100644 --- a/sysmoOCTSIM/main.c +++ b/sysmoOCTSIM/main.c @@ -38,9 +38,9 @@ #include "ccid_device.h" #include "usb_descriptors.h" -static void bdg_bkptpanic(const char *fmt, va_list args) +static void panic_handler(const char *fmt, va_list args) { - __asm("BKPT #0"); + Dummy_Handler(); } extern struct ccid_slot_ops iso_fsm_slot_ops; @@ -600,7 +600,7 @@ int main(void) { - osmo_set_panic_handler(&bdg_bkptpanic); + osmo_set_panic_handler(&panic_handler); #if 0 CoreDebug->DEMCR |= CoreDebug_DEMCR_TRCENA_Msk ; //| /* tracing*/ ////CoreDebug_DEMCR_MON_EN_Msk; /* mon interupt catcher */ -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42616?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: Ibcabe0c58ff14ebe76159470dd90fdef064febea Gerrit-Change-Number: 42616 Gerrit-PatchSet: 2 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[S] Change in osmo-ccid-firmware[master]: Rework default IRQ handler and osmo panic handler
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42616?usp=email ) Change subject: Rework default IRQ handler and osmo panic handler ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42616?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: Ibcabe0c58ff14ebe76159470dd90fdef064febea Gerrit-Change-Number: 42616 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:48:58 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in simtrace2[master]: firmware: Implement debug command via USB control vendor class
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/simtrace2/+/43005?usp=email ) Change subject: firmware: Implement debug command via USB control vendor class ...................................................................... Patch Set 8: Code-Review+1 Verified+1 -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/43005?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: I1f4d29335eb0d2feef376b8ecdfe57a0162914d6 Gerrit-Change-Number: 43005 Gerrit-PatchSet: 8 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:48:41 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in simtrace2[master]: firmware: Add talloc_report_buf() encode the allocation by bits
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/simtrace2/+/43004?usp=email ) Change subject: firmware: Add talloc_report_buf() encode the allocation by bits ...................................................................... Patch Set 6: Code-Review+1 (1 comment) File firmware/libcommon/source/pseudo_talloc.c: https://gerrit.osmocom.org/c/simtrace2/+/43004/comment/db64c02f_db343875?us… : PS6, Line 85: msgb_inuse_too_ig spelling? too_ig -> too_big ? -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/43004?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: I3a8ddbe4100f342db2c59ad1fb9aa0014efc62b8 Gerrit-Change-Number: 43004 Gerrit-PatchSet: 6 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:47:22 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes

1 0

[S] Change in simtrace2[master]: firmware: Drop CEMU_TX_DATA when sim is in-active
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: lynxis lazus. laforge has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/simtrace2/+/43003?usp=email ) Change subject: firmware: Drop CEMU_TX_DATA when sim is in-active ...................................................................... Patch Set 5: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/43003?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: I796e548225a742cddfde12bd6a60e4d2e5c04c8f Gerrit-Change-Number: 43003 Gerrit-PatchSet: 5 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:46:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in onomondo-eim[master]: esipa_asn1_handler_utils: clean up handling of EuiccPackageResult
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43016?usp=email ) Change subject: esipa_asn1_handler_utils: clean up handling of EuiccPackageResult ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43016?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Ic2f64c254f3e07ce8dfdf43af977decf3514627b Gerrit-Change-Number: 43016 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:45:52 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in onomondo-eim[master]: esipa_asn1_handler_utils: use eimTransactionId in EuiccPackageErrorUn...
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43015?usp=email ) Change subject: esipa_asn1_handler_utils: use eimTransactionId in EuiccPackageErrorUnsigned ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43015?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Ibfbc6debcd36d2aae1f4530a608db823caf3a394 Gerrit-Change-Number: 43015 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:45:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in onomondo-eim[master]: restop.py, tryme_*.sh: improve log output and handling
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43014?usp=email ) Change subject: restop.py, tryme_*.sh: improve log output and handling ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43014?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: Id257e7a541fc92386cecee5af3dad2077b62709c Gerrit-Change-Number: 43014 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:44:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in onomondo-eim[master]: esipa_asn1_handler: store IPAd stateChangeCause
by laforge 16 Jul '26

16 Jul '26

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43013?usp=email ) Change subject: esipa_asn1_handler: store IPAd stateChangeCause ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/43013?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: onomondo-eim Gerrit-Branch: master Gerrit-Change-Id: I3fe64687786d9184ae82cbb12f9d1292fdc945c8 Gerrit-Change-Number: 43013 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 16 Jul 2026 14:44:29 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2026

2025

2024

2023

2022

gerrit-log July 2026