On Mon, 7 Feb 2011, Thomas Gleixner wrote:
> On Sun, 6 Feb 2011, Mike Frysinger wrote:
>
> > this subject looks garbled, and the contents seem to be duplicated. i
> > guess this one can be ignored ?
>
> Yeah, sorry. Missed to zap the patch before sending it out.
But worse, I missed to add the following.
Thanks,
tglx
------>
Subject: blackfin: Use accessor functions in show_interrupts()
From: Thomas Gleixner <[email protected]>
Date: Mon, 07 Feb 2011 11:55:18 +0100
Signed-off-by: Thomas Gleixner <[email protected]>
---
arch/blackfin/kernel/irqchip.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
Index: linux-next/arch/blackfin/kernel/irqchip.c
===================================================================
--- linux-next.orig/arch/blackfin/kernel/irqchip.c
+++ linux-next/arch/blackfin/kernel/irqchip.c
@@ -39,21 +39,23 @@ int show_interrupts(struct seq_file *p,
unsigned long flags;
if (i < NR_IRQS) {
- raw_spin_lock_irqsave(&irq_desc[i].lock, flags);
- action = irq_desc[i].action;
+ struct irq_desc *desc = irq_to_desc(i);
+
+ raw_spin_lock_irqsave(&desc->lock, flags);
+ action = desc->action;
if (!action)
goto skip;
seq_printf(p, "%3d: ", i);
for_each_online_cpu(j)
seq_printf(p, "%10u ", kstat_irqs_cpu(i, j));
- seq_printf(p, " %8s", irq_desc[i].chip->name);
+ seq_printf(p, " %8s", get_irq_desc_chip(desc)->name);
seq_printf(p, " %s", action->name);
for (action = action->next; action; action = action->next)
seq_printf(p, " %s", action->name);
seq_putc(p, '\n');
skip:
- raw_spin_unlock_irqrestore(&irq_desc[i].lock, flags);
+ raw_spin_unlock_irqrestore(&desc->lock, flags);
} else if (i == NR_IRQS) {
seq_printf(p, "NMI: ");
for_each_online_cpu(j)
_______________________________________________
Uclinux-dist-devel mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/uclinux-dist-devel
