laforge has submitted this change. ( https://gerrit.osmocom.org/c/simtrace2/+/30200 ) Change subject: firmware/sniffer: Enable interrupts for overrun/parity/frame errors ...................................................................... firmware/sniffer: Enable interrupts for overrun/parity/frame errors We so far didn't have interrupts enabled for those, and just caught them "by accident" if a byte was received or if a timeout happened. Let's explicitly enable those interrupts so we also catch those conditions by themselves. Change-Id: Ia27f537706b9a6252dd18175545c6f27a7d17d0e --- M firmware/libcommon/source/sniffer.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved Hoernchen: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/firmware/libcommon/source/sniffer.c b/firmware/libcommon/source/sniffer.c index 35de58f..948eee8 100644 --- a/firmware/libcommon/source/sniffer.c +++ b/firmware/libcommon/source/sniffer.c @@ -918,7 +918,7 @@ * Initialization routine *-----------------------------------------------------------------------------*/ -#define SNIFFER_IER (US_IER_RXRDY | US_IER_TIMEOUT) +#define SNIFFER_IER (US_IER_RXRDY | US_IER_TIMEOUT | US_IER_OVRE | US_IER_FRAME | US_IER_PARE) /* Called during USB enumeration after device is enumerated by host */ void Sniffer_configure(void) -- To view, visit https://gerrit.osmocom.org/c/simtrace2/+/30200 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-Change-Id: Ia27f537706b9a6252dd18175545c6f27a7d17d0e Gerrit-Change-Number: 30200 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: tsaitgaist <kredon(a)sysmocom.de> Gerrit-MessageType: merged