Removed unused port functionality port_puts().

git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@800 35acf78f-673a-0410-8e92-d51de3d6d3f4

demos/Win32-MinGW/chcore.c

@@ -93,14 +93,6 @@ void ChkIntSources(void) {
   }
 }
 
-/**
- * Prints a message on the system console.
- * @param msg pointer to the message
- */
-__attribute__((fastcall))
-void port_puts(char *msg) {
-}
-
 /**
  * Performs a context switch between two threads.
  * @param otp the thread to be switched out

demos/Win32-MinGW/chcore.h

@@ -192,7 +192,6 @@ struct context {
 #ifdef __cplusplus
 extern "C" {
 #endif
-  __attribute__((fastcall)) void port_puts(char *msg);
   __attribute__((fastcall)) void port_switch(Thread *otp, Thread *ntp);
   __attribute__((fastcall)) void port_halt(void);
   void InitCore(void);

ports/ARM7/chcore.c

@@ -26,16 +26,6 @@
 
 #include <ch.h>
 
-/**
- * Prints a message on the system console.
- * @param msg pointer to the message
- */
-/** @cond never */
-__attribute__((weak))
-/** @endcond */
-void port_puts(char *msg) {
-}
-
 /**
  * Halts the system.
  */

ports/ARM7/chcore.h

@@ -310,7 +310,6 @@ struct context {
 #ifdef __cplusplus
 extern "C" {
 #endif
-  void port_puts(char *msg);
   void port_halt(void);
 #ifdef THUMB
   void _port_switch_thumb(Thread *otp, Thread *ntp);

ports/ARMCM3/chcore.c

@@ -27,19 +27,6 @@
 #include <ch.h>
 #include <nvic.h>
 
-/**
- * The default implementation of this function is void so no messages are
- * actually printed.
- * @note The function is declared as a weak symbol, it is possible to redefine
- *       it in your application code.
- * @param msg pointer to the message string
- */
-/** @cond never */
-__attribute__((weak))
-/** @endcond */
-void port_puts(char *msg) {
-}
-
 /**
  * Halts the system.
  * @note The function is declared as a weak symbol, it is possible to redefine

ports/ARMCM3/chcore.h

@@ -302,7 +302,6 @@ struct context {
 #ifdef __cplusplus
 extern "C" {
 #endif
-  void port_puts(char *msg);
   void port_halt(void);
   void threadstart(void);
 #ifdef __cplusplus

ports/AVR/chcore.c

@@ -26,19 +26,6 @@
 
 #include <ch.h>
 
-/**
- * The default implementation of this function is void so no messages are
- * actually printed.
- * @note The function is declared as a weak symbol, it is possible to redefine
- *       it in your application code.
- * @param msg pointer to the message string
- */
-/** @cond never */
-__attribute__((weak))
-/** @endcond */
-void port_puts(char *msg) {
-}
-
 /**
  * Performs a context switch between two threads.
  * @param otp the thread to be switched out

ports/AVR/chcore.h

@@ -250,7 +250,6 @@ asm ("" : : : "r18", "r19", "r20", "r21", "r22", "r23", "r24",          \
 #ifdef __cplusplus
 extern "C" {
 #endif
-  void port_puts(char *msg);
   void port_switch(Thread *otp, Thread *ntp);
   void port_halt(void);
   void threadstart(void);

ports/MSP430/chcore.c

@@ -26,19 +26,6 @@
 
 #include <ch.h>
 
-/**
- * The default implementation of this function is void so no messages are
- * actually printed.
- * @note The function is declared as a weak symbol, it is possible to redefine
- *       it in your application code.
- * @param msg pointer to the message string
- */
-/** @cond never */
-__attribute__((weak))
-/** @endcond */
-void port_puts(char *msg) {
-}
-
 /**
  * Performs a context switch between two threads.
  * @param otp the thread to be switched out

ports/MSP430/chcore.h

@@ -221,7 +221,6 @@ struct context {
 #ifdef __cplusplus
 extern "C" {
 #endif
-  void port_puts(char *msg);
   void port_switch(Thread *otp, Thread *ntp);
   void port_halt(void);
   void threadstart(void);

readme.txt

@@ -98,6 +98,8 @@ Win32-MinGW            - ChibiOS/RT simulator and demo into a WIN32 process,
   option can be overriden from outside.
 - CHANGE: Changed the chSemFastWaitS() macro in chSemFastWaitI() and
   chSemGetCounter() in chSemGetCounterI().
+- CHANGE: Removed the port_puts() function from the port templates. It was not
+  implemented on all ports.
 - Improvements to the test suite, added a new level of indirection that allows
   to make tests depend on the configuration options without have to put #ifs
   into the test main module.

src/templates/chcore.c

@@ -130,12 +130,4 @@ void port_halt(void) {
 void port_switch(Thread *otp, Thread *ntp) {
 }
 
-/**
- * @brief Prints a message on the system console.
- *
- * @param msg pointer to the message
- */
-void port_puts(char *msg) {
-}
-
 /** @} */

src/templates/chcore.h

@@ -145,7 +145,6 @@ extern "C" {
   void port_wait_for_interrupt(void);
   void port_halt(void);
   void port_switch(Thread *otp, Thread *ntp);
-  void port_puts(char *msg);
 #ifdef __cplusplus
 }
 #endif

todo.txt

@@ -15,12 +15,11 @@ After 1.0.0:
 * Multiple debug switches.
   * Split asserts from parameters checks.
   * Add checks to all APIs.
-  X Stack checks option.
+  * Stack checks option.
   * Threads profiling option.
 * Idle loop hook macro.
 * Switch the configuration options to TRUE/FALSE rather than def/undef.
 X Remove port_puts() from all the ports.
-- Look into the wrong mutex assert.
 
 After 1.2.0:
 - Threads Pools manager in the library.