![]() | |||||||||||||
Messages in this thread |
On Sep 12 2007 12:59, Stephen Hemminger wrote: >> Other pr_*() macros are already defined in kernel.h, but pr_err() was defined >> multiple times in several other places>> >> Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com>>>pr_error seems better than pr_err>>Please add the full set:> pr_alert > pr_critical > pr_error > pr_warn > pr_notice I repeat to be totally against this. (1) You are trying to add pr_*(fmt, arg) as a lazy shorthand for printk(KERN_* fmt, arg...) and (2) do not seem to notice that pr_debug() changes its behavior depending on -DDEBUG, which pr_alert() etc. do not, so there is IMHO no point (that includes the already existing pr_info()). Why not just obsolete printk() altogether then? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2007-09-15 10:59 [from the cache] ©2003-2008 | |||||||||||||