Merge pull request #14457 from ARMmbed/uart-stdio-alias

Update UART pin names & add MBED_CONF_TARGET_STDIO_UART overrides

Author: adbridge

TEST_APPS/device/nanostack_mac_tester/main.cpp

@@ -44,7 +44,7 @@
 #endif
 
 extern mac_api_s *mac_interface;
-UnbufferedSerial pc(USBTX, USBRX);
+UnbufferedSerial pc(CONSOLE_TX, CONSOLE_RX);
 osThreadId_t main_thread;
 static CircularBuffer<uint8_t, RX_BUFFER_SIZE> rx_buffer;
 static uint8_t ns_heap[HEAP_FOR_MAC_TESTER_SIZE];

drivers/docs/serial/serial.md

@@ -232,7 +232,7 @@ Other alternatives to printing to stdout are by calling `puts` or `write`.
 puts("Hello");
 ```
 ```C
-BufferedSerial pc(USBTX, USBRX);
+BufferedSerial pc(CONSOLE_TX, CONSOLE_RX);
 pc.write("Hello", 6);
 ```
 

drivers/tests/TESTS/mbed_drivers/buffered_serial/main.cpp

@@ -49,7 +49,7 @@ using namespace utest::v1;
 
 
 static BufferedSerial buffered_serial_obj(
-    USBTX, USBRX, MBED_CONF_PLATFORM_STDIO_BAUD_RATE
+    CONSOLE_TX, CONSOLE_RX, MBED_CONF_PLATFORM_STDIO_BAUD_RATE
 );
 
 

drivers/tests/TESTS/mbed_drivers/unbuffered_serial/main.cpp

@@ -47,7 +47,7 @@ using namespace utest::v1;
 
 
 static UnbufferedSerial unbuffered_serial_obj(
-    USBTX, USBRX, MBED_CONF_PLATFORM_STDIO_BAUD_RATE
+    CONSOLE_TX, CONSOLE_RX, MBED_CONF_PLATFORM_STDIO_BAUD_RATE
 );
 
 static ssize_t unbuffered_serial_read(void *buffer, ssize_t length)

features/frameworks/greentea-client/source/greentea_test_env.cpp

@@ -284,7 +284,7 @@ static void greentea_write_int(const int val)
  * \brief Encapsulate and send key-value message from DUT to host
  *
  *        This function uses underlying functions to write directly
- *        to the serial port, (USBTX). This allows KVs to be used
+ *        to the serial port, (CONSOLE_TX). This allows KVs to be used
  *        from within interrupt context.
  *
  * \param key Message key (message/event name)
@@ -305,7 +305,7 @@ extern "C" void greentea_send_kv(const char *key, const char *val) {
  * \brief Encapsulate and send key-value message from DUT to host
  *
  *        This function uses underlying functions to write directly
- *        to the serial port, (USBTX). This allows KVs to be used
+ *        to the serial port, (CONSOLE_TX). This allows KVs to be used
  *        from within interrupt context.
  *        Last value is an integer to avoid integer to string conversion
  *        made by the user.
@@ -328,7 +328,7 @@ void greentea_send_kv(const char *key, const int val) {
  * \brief Encapsulate and send key-value-value message from DUT to host
  *
  *        This function uses underlying functions to write directly
- *        to the serial port, (USBTX). This allows KVs to be used
+ *        to the serial port, (CONSOLE_TX). This allows KVs to be used
  *        from within interrupt context.
  *        Last value is an integer to avoid integer to string conversion
  *        made by the user.
@@ -355,7 +355,7 @@ void greentea_send_kv(const char *key, const char *val, const int result) {
  * \brief Encapsulate and send key-value-value-value message from DUT to host
  *
  *        This function uses underlying functions to write directly
- *        to the serial port, (USBTX). This allows KVs to be used
+ *        to the serial port, (CONSOLE_TX). This allows KVs to be used
  *        from within interrupt context.
  *        Last 2 values are integers to avoid integer to string conversion
  *        made by the user.
@@ -389,7 +389,7 @@ void greentea_send_kv(const char *key, const char *val, const int passes, const
  * \brief Encapsulate and send key-value-value message from DUT to host
  *
  *        This function uses underlying functions to write directly
- *        to the serial port, (USBTX). This allows key-value-value to be used
+ *        to the serial port, (CONSOLE_TX). This allows key-value-value to be used
  *        from within interrupt context.
  *        Both values are integers to avoid integer to string conversion
  *        made by the user.

hal/docs/0002-pinmap-extension.md

@@ -74,7 +74,7 @@ Some boards have pins which cannot be tested without causing problems elsewhere.
 MBED_WEAK const PinList *pinmap_restricted_pins()
 {
     static const PinName pins[] = {
-        USBTX, USBRX
+        CONSOLE_TX, CONSOLE_RX
     };
     static const PinList pin_list = {
         sizeof(pins) / sizeof(pins[0]),

hal/include/hal/PinNameAliases.h

@@ -21,13 +21,8 @@
 #define MBED_PIN_NAME_ALIASES_H
 
 /* Aliases for legacy reasons. To be removed in the next Mbed OS version */
-#if defined (CONSOLE_TX) && (CONSOLE_RX)
 #define USBTX CONSOLE_TX
 #define USBRX CONSOLE_RX
-#else
-#define CONSOLE_TX USBTX
-#define CONSOLE_RX USBRX
-#endif
 
 #if defined (TARGET_FF_ARDUINO) || (TARGET_FF_ARDUINO_UNO)
 

hal/include/hal/pinmap.h

@@ -143,7 +143,7 @@ bool pinmap_list_has_peripheral(const PeripheralList *list, int peripheral);
  *
  * The restricted pin list is used to indicate to testing
  * that a pin should be skipped due to some caveat about it.
- * For example, using USBRX and USBTX during tests will interfere
+ * For example, using CONSOLE_RX and CONSOLE_TX during tests will interfere
  * with the test runner and should be avoided.
  *
  * Targets should override the weak implementation of this

targets/TARGET_ARM_FM/TARGET_FVP_MPS2/PeripheralNames.h

@@ -75,15 +75,23 @@ typedef enum {
     PWM_11
 } PWMName;
 
-#define STDIO_UART_TX     USBTX
-#define STDIO_UART_RX     USBRX
+#if defined(MBED_CONF_TARGET_STDIO_UART_TX)
+#define STDIO_UART_TX MBED_CONF_TARGET_STDIO_UART_TX
+#else
+#define STDIO_UART_TX CONSOLE_TX
+#endif
+#if defined(MBED_CONF_TARGET_STDIO_UART_RX)
+#define STDIO_UART_RX MBED_CONF_TARGET_STDIO_UART_RX
+#else
+#define STDIO_UART_RX CONSOLE_RX
+#endif
 #define STDIO_UART        UART_0
 
-#define MBED_UART0        USBTX, USBRX
+#define MBED_UART0        CONSOLE_TX, CONSOLE_RX
 #define MBED_UART1        XB_TX, XB_RX
 #define MBED_UART2        SH0_TX, SH0_RX
 #define MBED_UART3        SH1_TX, SH1_RX
-#define MBED_UARTUSB      USBTX, USBRX
+#define MBED_UARTUSB      CONSOLE_TX, CONSOLE_RX
 
 #ifdef __cplusplus
 }

targets/TARGET_ARM_FM/TARGET_FVP_MPS2/PinNames.h

@@ -156,8 +156,8 @@ typedef enum {
     ADC_SSEL = 653,
 
     //MPS2 Uart
-    USBTX  = 400,
-    USBRX  = 401,
+    CONSOLE_TX  = 400,
+    CONSOLE_RX  = 401,
     XB_TX = 402,
     XB_RX = 403,
     UART_TX2 = 404,