<p>tsaitgaist has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/simtrace2/+/15649">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">make LED definitions board specific<br><br>the LEDs (2 of them) were connected to the same pins on all<br>boards, up to the octsim-tester.<br>to be able to have board specific LEDs the definitions have moved<br>from common to the each board.<br>at the same time I added a bit of documentation what the LEDs are<br>used for.<br><br>Change-Id: I3226a9187a8d0b657ccf5dcd8f3586b2578f96d2<br>---<br>M firmware/libboard/common/include/board_common.h<br>M firmware/libboard/owhw/include/board.h<br>M firmware/libboard/qmod/include/board.h<br>M firmware/libboard/sam3p256/include/board.h<br>M firmware/libboard/simtrace/include/board.h<br>5 files changed, 70 insertions(+), 13 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/simtrace2 refs/changes/49/15649/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/firmware/libboard/common/include/board_common.h b/firmware/libboard/common/include/board_common.h</span><br><span>index ded4762..dd21e4b 100644</span><br><span>--- a/firmware/libboard/common/include/board_common.h</span><br><span>+++ b/firmware/libboard/common/include/board_common.h</span><br><span>@@ -56,16 +56,12 @@</span><br><span> /** Core definition */</span><br><span> #define cortexm3</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-#define PIO_LED_RED     PIO_PA17</span><br><span style="color: hsl(0, 100%, 40%);">-#define PIO_LED_GREEN   PIO_PA18</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-#define PIN_LED_RED     {PIO_LED_RED, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(0, 100%, 40%);">-#define PIN_LED_GREEN   {PIO_LED_GREEN, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(0, 100%, 40%);">-#define PINS_LEDS       PIN_LED_RED, PIN_LED_GREEN </span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-#define LED_NUM_RED     0</span><br><span style="color: hsl(0, 100%, 40%);">-#define LED_NUM_GREEN   1</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(120, 100%, 40%);">+/* LEDs are used to indicate the status</span><br><span style="color: hsl(120, 100%, 40%);">+ * the LED definition is board specific</span><br><span style="color: hsl(120, 100%, 40%);">+ * most boards have two LEDs, one green and one red</span><br><span style="color: hsl(120, 100%, 40%);">+ * the red LED indicates of the main firmware is ready (on) or if there is an error (blinking)</span><br><span style="color: hsl(120, 100%, 40%);">+ * the green LED indicates if the firmware is idling (on) or if there is activity (blinking)</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span> /** USART0 pin RX */</span><br><span> #define PIN_USART0_RXD    {PIO_PA9A_URXD0, PIOA, ID_PIOA, PIO_PERIPH_A, PIO_DEFAULT}</span><br><span> /** USART0 pin TX */</span><br><span>diff --git a/firmware/libboard/owhw/include/board.h b/firmware/libboard/owhw/include/board.h</span><br><span>index 8c0052d..44472d0 100644</span><br><span>--- a/firmware/libboard/owhw/include/board.h</span><br><span>+++ b/firmware/libboard/owhw/include/board.h</span><br><span>@@ -31,6 +31,21 @@</span><br><span> /** desired main clock frequency (in Hz, based on BOARD_MAINOSC) */</span><br><span> #define BOARD_MCK 58982400 // 18.432 * 16 / 5</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to red LED */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_RED     PIO_PA17</span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to green LED */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_GREEN   PIO_PA18</span><br><span style="color: hsl(120, 100%, 40%);">+/** red LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_RED     {PIO_LED_RED, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** green LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_GREEN   {PIO_LED_GREEN, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** LEDs pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PINS_LEDS       PIN_LED_RED, PIN_LED_GREEN </span><br><span style="color: hsl(120, 100%, 40%);">+/** index for red LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_RED     0</span><br><span style="color: hsl(120, 100%, 40%);">+/** index for green LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_GREEN   1</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* USIM 2 interface (USART) */</span><br><span> #define PIN_USIM2_CLK         {PIO_PA2, PIOA, ID_PIOA, PIO_PERIPH_B, PIO_DEFAULT}</span><br><span> #define PIN_USIM2_IO             {PIO_PA6, PIOA, ID_PIOA, PIO_PERIPH_A, PIO_DEFAULT}</span><br><span>diff --git a/firmware/libboard/qmod/include/board.h b/firmware/libboard/qmod/include/board.h</span><br><span>index a738daa..5165434 100644</span><br><span>--- a/firmware/libboard/qmod/include/board.h</span><br><span>+++ b/firmware/libboard/qmod/include/board.h</span><br><span>@@ -20,9 +20,6 @@</span><br><span> #include "board_common.h"</span><br><span> #include "simtrace_usb.h"</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-#define LED_USIM1 LED_GREEN</span><br><span style="color: hsl(0, 100%, 40%);">-#define LED_USIM2      LED_RED</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span> /** Name of the board */</span><br><span> #define BOARD_NAME "QMOD"</span><br><span> /** Board definition */</span><br><span>@@ -33,6 +30,25 @@</span><br><span> /** desired main clock frequency (in Hz, based on BOARD_MAINOSC) */</span><br><span> #define BOARD_MCK 58000000 // 18.432 * 29 / 6</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to red LED */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_RED     PIO_PA17</span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to green LED */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_GREEN   PIO_PA18</span><br><span style="color: hsl(120, 100%, 40%);">+/** red LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_RED     {PIO_LED_RED, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** green LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_GREEN   {PIO_LED_GREEN, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** LEDs pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PINS_LEDS       PIN_LED_RED, PIN_LED_GREEN </span><br><span style="color: hsl(120, 100%, 40%);">+/** index for red LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_RED     0</span><br><span style="color: hsl(120, 100%, 40%);">+/** index for green LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_GREEN   1</span><br><span style="color: hsl(120, 100%, 40%);">+/** the green LED is actually red and used as indication for USIM1 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_USIM1     LED_GREEN</span><br><span style="color: hsl(120, 100%, 40%);">+/** the green LED is actually red and used as indication for USIM2 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_USIM2     LED_RED</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* USIM 2 interface (USART) */</span><br><span> #define PIN_USIM2_CLK            {PIO_PA2, PIOA, ID_PIOA, PIO_PERIPH_B, PIO_DEFAULT}</span><br><span> #define PIN_USIM2_IO             {PIO_PA6, PIOA, ID_PIOA, PIO_PERIPH_A, PIO_DEFAULT}</span><br><span>diff --git a/firmware/libboard/sam3p256/include/board.h b/firmware/libboard/sam3p256/include/board.h</span><br><span>index bd2ab49..58e8a24 100644</span><br><span>--- a/firmware/libboard/sam3p256/include/board.h</span><br><span>+++ b/firmware/libboard/sam3p256/include/board.h</span><br><span>@@ -30,6 +30,21 @@</span><br><span> /** desired main clock frequency (in Hz, based on BOARD_MAINOSC) */</span><br><span> #define BOARD_MCK 58000000</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to yellow LED2 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_RED     PIO_PA17</span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to green LED1 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_GREEN   PIO_PA18</span><br><span style="color: hsl(120, 100%, 40%);">+/** red LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_RED     {PIO_LED_RED, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** green LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_GREEN   {PIO_LED_GREEN, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** LEDs pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PINS_LEDS       PIN_LED_RED, PIN_LED_GREEN </span><br><span style="color: hsl(120, 100%, 40%);">+/** index for red LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_RED     0</span><br><span style="color: hsl(120, 100%, 40%);">+/** index for green LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_GREEN   1</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /** Pin configuration **/</span><br><span> /* Button to force bootloader start (shorted to ground when pressed */</span><br><span> #define PIN_BOOTLOADER_SW      {PIO_PA20, PIOA, ID_PIOA, PIO_INPUT, PIO_DEFAULT}</span><br><span>diff --git a/firmware/libboard/simtrace/include/board.h b/firmware/libboard/simtrace/include/board.h</span><br><span>index 7703451..c109bea 100644</span><br><span>--- a/firmware/libboard/simtrace/include/board.h</span><br><span>+++ b/firmware/libboard/simtrace/include/board.h</span><br><span>@@ -31,6 +31,21 @@</span><br><span> /** desired main clock frequency (in Hz, based on BOARD_MAINOSC) */</span><br><span> #define BOARD_MCK 58982400 // 18.432 * 16 / 5</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to red LED */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_RED     PIO_PA17</span><br><span style="color: hsl(120, 100%, 40%);">+/** MCU pin connected to green LED */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIO_LED_GREEN   PIO_PA18</span><br><span style="color: hsl(120, 100%, 40%);">+/** red LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_RED     {PIO_LED_RED, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** green LED pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PIN_LED_GREEN   {PIO_LED_GREEN, PIOA, ID_PIOA, PIO_OUTPUT_1, PIO_DEFAULT}</span><br><span style="color: hsl(120, 100%, 40%);">+/** LEDs pin definition */</span><br><span style="color: hsl(120, 100%, 40%);">+#define PINS_LEDS       PIN_LED_RED, PIN_LED_GREEN </span><br><span style="color: hsl(120, 100%, 40%);">+/** index for red LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_RED     0</span><br><span style="color: hsl(120, 100%, 40%);">+/** index for green LED in LEDs pin definition array */</span><br><span style="color: hsl(120, 100%, 40%);">+#define LED_NUM_GREEN   1</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /** Pin configuration **/</span><br><span> /* Button to force bootloader start (shorted to ground when pressed */</span><br><span> #define PIN_BOOTLOADER_SW      {PIO_PA31, PIOA, ID_PIOA, PIO_INPUT, PIO_PULLUP}</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/simtrace2/+/15649">change 15649</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/simtrace2/+/15649"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: simtrace2 </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I3226a9187a8d0b657ccf5dcd8f3586b2578f96d2 </div>
<div style="display:none"> Gerrit-Change-Number: 15649 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: tsaitgaist <kredon@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>