Thread (4 messages) flat view 4 messages, 4 authors, 2007-09-15

Re: [PATCH] Move the definition of pr_err() into kernel.h

From: Jan Engelhardt <hidden>
Date: 2007-09-15 08:57:20
Also in: linux-omap, lkml

On Sep 12 2007 12:59, Stephen Hemminger wrote:
quoted
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 <redacted>
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?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help