Changed minimal-printf to call fputc so that it does not bypass the retargetting code

Author: evedon

platform/source/minimal-printf/mbed_printf_implementation.c

@@ -37,39 +37,9 @@ static char mbed_stdio_out_prev = 0;
 #endif
 
 #if CONSOLE_OUTPUT == CONSOLE_OUTPUT_UART
-#if DEVICE_SERIAL
-/*
- Serial initialization and new line replacement is a direct copy from mbed_retarget.cpp
- If the static modifier were to be removed, this part of the code would not be necessary.
-*/
-#include "hal/serial_api.h"
-
-static serial_t stdio_uart = { 0 };
-
-/* module variable for keeping track of initialization */
-static bool not_initialized = true;
-
-static void init_serial()
-{
-    if (not_initialized) {
-        not_initialized = false;
-
-        serial_init(&stdio_uart, STDIO_UART_TX, STDIO_UART_RX);
-#if MBED_CONF_PLATFORM_STDIO_BAUD_RATE
-        serial_baud(&stdio_uart, MBED_CONF_PLATFORM_STDIO_BAUD_RATE);
-#endif
-    }
-}
-
-#define MBED_INITIALIZE_PRINT(x) { init_serial(); }
-#define MBED_PRINT_CHARACTER(x) { serial_putc(&stdio_uart, x); }
-
-#else
 
 #define MBED_INITIALIZE_PRINT(x)
-#define MBED_PRINT_CHARACTER(x)
-
-#endif // if DEVICE_SERIAL
+#define MBED_PRINT_CHARACTER(x) { fputc(x, stdout); }
 
 #elif CONSOLE_OUTPUT == CONSOLE_OUTPUT_SWO
 

platform/source/minimal-printf/mbed_printf_wrapper.c

@@ -64,18 +64,6 @@
 #warning "This compiler is not yet supported."
 #endif
 
-// Prevent optimization of printf() by the ARMCC or IAR compiler.
-// This is done to prevent optimization which can cause printf() to be
-// substituted with a different function (e.g. puts()) and cause
-// the output to be missing some strings.
-// Note: Optimization prevention for other supported compilers is done
-//       via compiler optional command line arguments.
-#if defined(__CC_ARM) /* ARMC5 */
-#pragma push
-#pragma O0
-#elif defined(__ICCARM__) /* IAR */
-#pragma optimize=none
-#endif
 int SUB_PRINTF(const char *format, ...)
 {
     va_list arguments;

tools/profiles/extensions/minimal-printf.json

@@ -1,12 +1,12 @@
 {
     "GCC_ARM": {
-       "common": ["-DMBED_MINIMAL_PRINTF", "-fno-builtin-printf"],
+       "common": ["-DMBED_MINIMAL_PRINTF"],
         "ld": ["-Wl,--wrap,printf", "-Wl,--wrap,sprintf", "-Wl,--wrap,snprintf",
                "-Wl,--wrap,vprintf", "-Wl,--wrap,vsprintf", "-Wl,--wrap,vsnprintf",
                "-Wl,--wrap,fprintf", "-Wl,--wrap,vfprintf"]
     },
     "ARMC6": {
-        "common": ["-DMBED_MINIMAL_PRINTF", "-fno-builtin-printf"]
+        "common": ["-DMBED_MINIMAL_PRINTF"]
     },
     "ARM": {
         "common": ["-DMBED_MINIMAL_PRINTF"]