Attention is currently required from: laforge.
Hoernchen has posted comments on this change by Hoernchen. (
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/39445?usp=email )
Change subject: fix usb usb reset handling by delayed soft reset
......................................................................
Patch Set 1:
(2 comments)
File sysmoOCTSIM/hpl/core/hpl_init.c:
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/39445/comment/640bb66f_39…
:
PS1, Line 57: // _osc32kctrl_init_sources();
: // _oscctrl_init_sources();
: // _mclk_init();
: // #if _GCLK_INIT_1ST
: // _gclk_init_generators_by_fref(_GCLK_INIT_1ST);
: // #endif
: // _oscctrl_init_referenced_generators();
why are we just commenting this out rather than
removing it?
Because this serves as a reminder what must be done to run the fw
without a bl + allows conveniently enabling all that for debbuging by uncommenting it.
This is my change for me that allows me to semi-efficiently deal with debugging, it's
not supposed to look nice.
File sysmoOCTSIM/main.c:
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/39445/comment/3e6f8705_bb…
:
PS1, Line 579: if(delayed_usb_reset)
I'm wondering why we suddenly format
"if" as if it was a function. […]
the format issues happened during
cleanup and merging of a dozen different branches with code in seperate files that had
other formatting and was #included which prevented proper autoformat.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/39445?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: I0939930a42f3009abf7e670561a123963bbd3845
Gerrit-Change-Number: 39445
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Thu, 30 Jan 2025 13:14:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>