diff options
| author | Wolfgang Denk <wd@denx.de> | 2011-10-04 21:19:19 +0200 |
|---|---|---|
| committer | Wolfgang Denk <wd@denx.de> | 2011-10-04 21:19:19 +0200 |
| commit | f092f15d39d1a223b240eb83dc3f4c64a02a8ea6 (patch) | |
| tree | 331b256fb756d9c199456003c3f07804bcbe8896 /include/common.h | |
| parent | 7b8ffea2ac44097ed1c99ba70b8c6a4cf12ba0b4 (diff) | |
Revert "GCC4.6: Convert various empty macros to inline functions"
This reverts commit 60ce53cf9f408d9ad721f8e7a87d6a564e6d5bac.
The commit causes build breakage for a number of boards. This results
from the fact that now the arguments of debug() actually get
referenced (even if there is hope that the compiler will optimize
away the debug() call). The obvious fix to that probem (change the
code to always declare the referenced variables and data structures)
increases the code size, and was this rejected. So it was decided to
revert this commit until a better solution is found.
Diffstat (limited to 'include/common.h')
| -rw-r--r-- | include/common.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/common.h b/include/common.h index aeb2d8436b6..d244bd40b5b 100644 --- a/include/common.h +++ b/include/common.h @@ -120,8 +120,8 @@ typedef volatile unsigned char vu_char; #define debug(fmt,args...) printf (fmt ,##args) #define debugX(level,fmt,args...) if (DEBUG>=level) printf(fmt,##args); #else -static inline void debug(const char *fmt, ...) {} -static inline void debugX(int level, const char *fmt, ...) {} +#define debug(fmt,args...) +#define debugX(level,fmt,args...) #endif /* DEBUG */ #ifdef DEBUG |