Addressed review comments, in particular:

* Fixed wrapper functions for IAR
* Fixed and renamed profile to minimal-printf.json
* Moved minimal-printf under platform
* Removed minimal-printf/mbed_lib.json
* Modified exporter template to work with partial profile
* Prevented optimization of printf to avoid compiler function substitution

Author: evedon

features/minimal-printf/TESTS/minimal-printf/compliance/README.md renamed to TESTS/mbed_platform/minimal-printf/compliance/README.md

@@ -8,4 +8,4 @@ You can use the following command to run tests:
 
 `mbed test -m K64F -t GCC_ARM -n *printf* -v -c`
 
-Do not use --profile mprintf so minimal-printf is not compared with itself.
+Do not use --profile minimal-printf so minimal-printf is not compared with itself.

features/minimal-printf/TESTS/minimal-printf/compliance/main.cpp renamed to TESTS/mbed_platform/minimal-printf/compliance/main.cpp

@@ -17,7 +17,7 @@
 #ifdef TARGET_LIKE_MBED
 #include "mbed.h"
 #endif
-#include "mbed_printf.h"
+#include "platform/minimal-printf/mbed_printf.h"
 
 #include "utest/utest.h"
 #include "unity/unity.h"
@@ -721,7 +721,7 @@ static control_t test_snprintf_x(const size_t call_count)
     return CaseNext;
 }
 
-#if MBED_CONF_MINIMAL_PRINTF_ENABLE_FLOATING_POINT
+#if MBED_CONF_PLATFORM_MINIMAL_PRINTF_ENABLE_FLOATING_POINT
 static control_t test_printf_f(const size_t call_count)
 {
     int result_baseline;
@@ -891,7 +891,7 @@ static control_t test_snprintf_buffer_overflow_llx(const size_t call_count)
 
 utest::v1::status_t greentea_setup(const size_t number_of_cases)
 {
-    GREENTEA_SETUP(30*60, "default_auto");
+    GREENTEA_SETUP(30 * 60, "default_auto");
     return greentea_test_setup_handler(number_of_cases);
 }
 
@@ -902,7 +902,7 @@ Case cases[] = {
     Case("snprintf %u", test_snprintf_u),
     Case("printf %x", test_printf_x),
     Case("snprintf %x", test_snprintf_x),
-#if MBED_CONF_MINIMAL_PRINTF_ENABLE_FLOATING_POINT
+#if MBED_CONF_PLATFORM_MINIMAL_PRINTF_ENABLE_FLOATING_POINT
     Case("printf %f", test_printf_f),
     Case("snprintf %f", test_snprintf_f),
 #endif

features/minimal-printf/mbed_printf.c renamed to TESTS/mbed_platform/minimal-printf/compliance/mbed_printf.c

@@ -18,6 +18,7 @@
 
 #include <limits.h>
 
+
 int mbed_printf(const char *format, ...)
 {
     va_list arguments;
@@ -28,7 +29,7 @@ int mbed_printf(const char *format, ...)
     return result;
 }
 
-int mbed_snprintf(char* buffer, size_t length, const char* format, ...)
+int mbed_snprintf(char *buffer, size_t length, const char *format, ...)
 {
     va_list arguments;
     va_start(arguments, format);
@@ -38,18 +39,18 @@ int mbed_snprintf(char* buffer, size_t length, const char* format, ...)
     return result;
 }
 
-int mbed_vprintf(const char* format, va_list arguments)
+int mbed_vprintf(const char *format, va_list arguments)
 {
     return mbed_minimal_formatted_string(NULL, LONG_MAX, format, arguments, NULL);
 }
 
-int mbed_vsnprintf(char* buffer, size_t length, const char* format, va_list arguments)
+int mbed_vsnprintf(char *buffer, size_t length, const char *format, va_list arguments)
 {
     return mbed_minimal_formatted_string(buffer, length, format, arguments, NULL);
 }
 
-#if MBED_CONF_MINIMAL_PRINTF_ENABLE_FILE_STREAM
-int mbed_fprintf(FILE* stream, const char *format, ...)
+#if MBED_CONF_PLATFORM_MINIMAL_PRINTF_ENABLE_FILE_STREAM
+int mbed_fprintf(FILE *stream, const char *format, ...)
 {
     va_list arguments;
     va_start(arguments, format);
@@ -59,7 +60,7 @@ int mbed_fprintf(FILE* stream, const char *format, ...)
     return result;
 }
 
-int mbed_vfprintf(FILE* stream, const char* format, va_list arguments)
+int mbed_vfprintf(FILE *stream, const char *format, va_list arguments)
 {
     return mbed_minimal_formatted_string(NULL, LONG_MAX, format, arguments, stream);
 }

features/minimal-printf/mbed_printf.h renamed to TESTS/mbed_platform/minimal-printf/compliance/mbed_printf.h

@@ -36,36 +36,36 @@ int mbed_printf(const char *format, ...);
  *
  * Prints directly to buffer without using malloc.
  */
-int mbed_snprintf(char* buffer, size_t length, const char* format, ...);
+int mbed_snprintf(char *buffer, size_t length, const char *format, ...);
 
 /**
  * Minimal printf
  *
  * Prints directly to stdio/UART without using malloc.
  */
-int mbed_vprintf(const char* format, va_list arguments);
+int mbed_vprintf(const char *format, va_list arguments);
 
 /**
  * Minimal snprintf
  *
  * Prints directly to buffer without using malloc.
  */
-int mbed_vsnprintf(char* buffer, size_t length, const char* format, va_list arguments);
+int mbed_vsnprintf(char *buffer, size_t length, const char *format, va_list arguments);
 
-#if MBED_CONF_MINIMAL_PRINTF_ENABLE_FILE_STREAM
+#if MBED_CONF_PLATFORM_MINIMAL_PRINTF_ENABLE_FILE_STREAM
 /**
  * Minimal fprintf
  *
  * Prints directly to file stream without using malloc.
  */
-int mbed_fprintf(FILE* stream, const char *format, ...);
+int mbed_fprintf(FILE *stream, const char *format, ...);
 
 /**
  * Minimal vfprintf
  *
  * Prints directly to file stream without using malloc.
  */
-int mbed_vfprintf(FILE* stream, const char* format, va_list arguments);
+int mbed_vfprintf(FILE *stream, const char *format, va_list arguments);
 #endif
 
 #ifdef __cplusplus

features/minimal-printf/TESTS/minimal-printf/compliance/test_app.json renamed to TESTS/mbed_platform/minimal-printf/compliance/test_app.json

@@ -1,7 +1,7 @@
 {
     "target_overrides": {
         "*": {
-            "minimal-printf.enable-file-stream": 1
+            "platform.minimal-printf-enable-file-stream": 1
         }
     }
 }

features/minimal-printf/README.md

@@ -1,4 +1,4 @@
 {
   "name": "bare-metal",
-  "requires": ["platform", "drivers", "minimal-printf"]
+  "requires": ["platform", "drivers"]
 }

features/minimal-printf/mbed_lib.json

@@ -127,6 +127,26 @@
         "use-mpu": {
             "help": "Use the MPU if available to fault execution from RAM and writes to ROM. Can be disabled to reduce image size.",
             "value": true
+        },
+        "minimal-printf-console-output": {
+            "help": "Console output when using mprintf profile. Options: UART, SWO",
+            "value": "UART"
+        },
+        "minimal-printf-enable-64-bit": {
+            "help": "Enable printing 64 bit integers when using mprintf profile",
+            "value": true
+        },
+        "minimal-printf-enable-file-stream": {
+            "help": "Enable printing to a FILE stream when using mprintf profile",
+            "value": true
+        },
+        "minimal-printf-enable-floating-point": {
+            "help": "Enable floating point printing when using mprintf profile",
+            "value": true
+        },
+        "minimal-printf-set-floating-point-max-decimals": {
+            "help": "Maximum number of decimals to be printed",
+            "value": 6
         }
     },
     "target_overrides": {