On Mon, 2009-04-06 at 14:30 +0200, Wolfram Sang wrote: > On Mon, Apr 06, 2009 at 12:05:02PM +1000, Michael Ellerman wrote: > > diff --git a/arch/powerpc/kernel/irq.c b/arch/powerpc/kernel/irq.c > > index 5576147..8c1a496 100644 > > --- a/arch/powerpc/kernel/irq.c > > +++ b/arch/powerpc/kernel/irq.c > > @@ -672,11 +672,13 @@ unsigned int irq_create_mapping(struct irq_host *host, > > return NO_IRQ; > > } > > } > > - pr_debug("irq: -> obtained virq %d\n", virq); > > > > if (irq_setup_virq(host, virq, hwirq)) > > return NO_IRQ; > > > > + printk(KERN_DEBUG "irq: irq %lu on host %s mapped to virtual irq %u\n", > > + hwirq, host->of_node ? host->of_node->full_name : "null", virq); > > + > > Are you replacing the pr_debug because you want to drop the > DEBUG-dependency?
Yes. Several people have been confused by the value in /proc/interrupts not matching what they expected - so I want there to be something in dmesg by default that will hopefully make it clear to people what's going on. cheers -- Michael Ellerman OzLabs, IBM Australia Development Lab wwweb: http://michael.ellerman.id.au phone: +61 2 6212 1183 (tie line 70 21183) We do not inherit the earth from our ancestors, we borrow it from our children. - S.M.A.R.T Person
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev