From da5578bd85bc400e1d2c1eb6bea28c6777453b80 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?K=C3=A9vin=20Redon?= Date: Sun, 8 Jul 2018 16:32:20 +0200 Subject: [PATCH] sniffer: set LED pattern The red and green LEDs are now used as follows: - red on = power (firmware is running) - red blink = error detected (e.g. during ISO-7816 parsing) - green on = running (sniffer mode is running) - green blink = activity (e.g. USB message sent) Change-Id: Ib1b6ff87fb92d6ed3ecae4bc89386206aaab508e --- firmware/apps/trace/main.c | 9 +++++++-- firmware/libcommon/source/sniffer.c | 2 +- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/firmware/apps/trace/main.c b/firmware/apps/trace/main.c index 6b6d70be..f7eb15d5 100644 --- a/firmware/apps/trace/main.c +++ b/firmware/apps/trace/main.c @@ -142,10 +142,15 @@ extern int main(void) enum confNum last_simtrace_config = simtrace_config; unsigned int i = 0; - /* Configure LED output (red = on, green = activity */ + /* Configure LED output + * red on = power + * red blink = error + * green on = running + * green blink = activity + */ led_init(); led_blink(LED_RED, BLINK_ALWAYS_ON); - led_blink(LED_GREEN, BLINK_ALWAYS_OFF); + led_blink(LED_GREEN, BLINK_ALWAYS_ON); /* Enable watchdog for 2000 ms, with no window */ WDT_Enable(WDT, WDT_MR_WDRSTEN | WDT_MR_WDDBGHLT | WDT_MR_WDIDLEHLT | diff --git a/firmware/libcommon/source/sniffer.c b/firmware/libcommon/source/sniffer.c index 8262bec2..9202d8e9 100644 --- a/firmware/libcommon/source/sniffer.c +++ b/firmware/libcommon/source/sniffer.c @@ -302,7 +302,7 @@ static void usb_send_data(enum simtrace_msg_type_sniff type, const uint8_t* data } /* Show activity on LED */ - led_blink(LED_GREEN, BLINK_2O_F); + led_blink(LED_GREEN, BLINK_2F_O); /* Print message */ switch (type) {