On Sat, 28 Oct 2017, I wrote:
Any particular reason why you didn't use pr_debug() here? I'm guessing
it's because this is not a known pointer value?
It's because the call to psc_debug_dump() is already conditional on
#ifdef DEBUG_PSC.
Having the printk conditional on both DEBUG and DEBUG_PSC would be
annoying. And I didn't want an unconditional call to psc_debug_dump()
because I think PSC_DEBUG could become more useful given that PSC
support is woefully incomplete.
Perhaps PSC_DEBUG should be scrapped in favour of DEBUG. Presently
DEBUG_PSC is set and I think that's useful as long as those drivers are
incomplete. So we would end up with this:
#define DEBUG
#include ...
...
static void psc_debug_dump(void)
{
...
pr_debug(...);
...
}
void __init psc_init(void)
{
...
#if DEBUG
psc_debug_dump()
#endif
...
}
In this version, the "#define DEBUG" at the top of the file has obscure
side effects (not just in printk.h) considering all of the headers that
get included, and their includes, and so on. I still prefer the patch that
I sent.
--
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
