Messages in this thread | | | From | David Brownell <> | Subject | Re: [patch/rfc 1/4] GPIO implementation framework | Date | Sat, 17 Nov 2007 09:36:24 -0800 |
| |
On Saturday 17 November 2007, Jean Delvare wrote: > On Tue, 13 Nov 2007 20:36:13 -0800, David Brownell wrote: > > On Tuesday 13 November 2007, eric miao wrote: > > > if (!requested) > > > - printk(KERN_DEBUG "GPIO-%d autorequested\n", > > > - chip->base + offset); > > > + pr_debug("GPIO-%d autorequested\n", gpio); > > > > Leave the printk in ... this is the sort of thing we want > > to see fixed, which becomes unlikely once you hide such > > diagnostics. And for that matter, what would be enabling > > the "-DDEBUG" that would trigger a pr_debug() message? > > The original code isn't correct either.
It's perfectly correct. That it's an idiom you don't seem to *like* but is distinct from correctness.
> Either this is a debug message > and indeed pr_debug() should be used, or it's not and KERN_DEBUG should > be replaced by a lower log level.
KERN_DEBUG is what says the message level is "debug". Both styles log messages at that priority level.
Which is distinct from saying that the message should vanish from non-debug builds ... that's what pr_debug and friends do, by relying implicitly on "-DDEBUG".
In this case, the original code was saying that the message should NOT just vanish. One reason the patch was incorrect was that even on its own terms, it was wrong ... since it used the "-DDEBUG" mechanism wrong, and prevented the message from *EVER* appearing.
- Dave - 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/
| |