On Mon, Apr 4, 2011 at 7:06 PM, Joe Perches <j...@perches.com> wrote: > Add pr_fmt. > Removed local TPACPI_<level> #defines, convert to pr_<level>. > Neaten dbg_<foo> macros. > Added a few missing newlines to logging messages. > Added static inline str_supported for !CONFIG_THINKPAD_ACPI_DEBUG vdbg_printk > defect reported by Sedat Dilek <sedat.di...@googlemail.com>. > > Signed-off-by: Joe Perches <j...@perches.com> > --- > drivers/platform/x86/thinkpad_acpi.c | 472 > ++++++++++++++-------------------- > 1 files changed, 188 insertions(+), 284 deletions(-) > > diff --git a/drivers/platform/x86/thinkpad_acpi.c > b/drivers/platform/x86/thinkpad_acpi.c [...] > #ifdef CONFIG_THINKPAD_ACPI_DEBUG > #define vdbg_printk dbg_printk > static const char *str_supported(int is_supported); > #else > -#define vdbg_printk(a_dbg_level, format, arg...) \ > - do { } while (0) > +static inline const char *str_supported(int is_supported) { return ""; } > +#define vdbg_printk(a_dbg_level, format, arg...) \ > + no_printk(format, ##arg) > #endif
Should be... + do { } while (0) ...not... + no_printk(format, ##arg) - Sedat - ------------------------------------------------------------------------------ Xperia(TM) PLAY It's a major breakthrough. An authentic gaming smartphone on the nation's most reliable network. And it wants your games. http://p.sf.net/sfu/verizon-sfdev _______________________________________________ ibm-acpi-devel mailing list ibm-acpi-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel