Re: [PATCH] cleanup legacy console_print function

From: Linus Torvalds
Date: Mon Sep 14 2009 - 10:51:29 EST




On Fri, 11 Sep 2009, Anirban Sinha wrote:
>
> - console_print(text); \
> - console_print("\n"); \
> + printk(KERN_EMERG "%s", text); \
> + printk(KERN_EMERG "\n"); \

Just clean it up to be

printk(KERN_EMERG "%s\n", text);

instead while at it.

> #define TRACE_CHR(chr) \
> { \
> if(dtlk_trace) \
> - console_print(chr); \
> + printk(KERN_EMERG "%c", chr); \

That can't be right. The translation is to use "%s", not "%c".

Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/