laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Hoernchen: Looks good to me, but someone else must approve Jenkins Builder: Verified
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(-)

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 change 30200. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: simtrace2
Gerrit-Branch: master
Gerrit-Change-Id: Ia27f537706b9a6252dd18175545c6f27a7d17d0e
Gerrit-Change-Number: 30200
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: tsaitgaist <kredon@sysmocom.de>
Gerrit-MessageType: merged