Skip to content

Commit

Permalink
x86: ioapic: Use setup function in setup_IO_APIC_irq_extra()
Browse files Browse the repository at this point in the history
Another version of the same thing. Only set the pin programmed, when
the setup function succeeds.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
  • Loading branch information
Thomas Gleixner committed Feb 23, 2011
1 parent 2d57e37 commit da1ad9d
Showing 1 changed file with 6 additions and 10 deletions.
16 changes: 6 additions & 10 deletions arch/x86/kernel/apic/io_apic.c
Original file line number Diff line number Diff line change
Expand Up @@ -1447,7 +1447,7 @@ static void __init setup_IO_APIC_irqs(void)
void setup_IO_APIC_irq_extra(u32 gsi)
{
int apic_id = 0, pin, idx, irq, node = cpu_to_node(0);
struct irq_cfg *cfg;
struct io_apic_irq_attr attr;

/*
* Convert 'gsi' to 'ioapic.pin'.
Expand All @@ -1467,21 +1467,17 @@ void setup_IO_APIC_irq_extra(u32 gsi)
if (apic_id == 0 || irq < NR_IRQS_LEGACY)
return;

cfg = alloc_irq_and_cfg_at(irq, node);
if (!cfg)
return;

add_pin_to_irq_node(cfg, node, apic_id, pin);

if (test_bit(pin, mp_ioapic_routing[apic_id].pin_programmed)) {
pr_debug("Pin %d-%d already programmed\n",
mp_ioapics[apic_id].apicid, pin);
return;
}
set_bit(pin, mp_ioapic_routing[apic_id].pin_programmed);

setup_ioapic_irq(apic_id, pin, irq, cfg,
irq_trigger(idx), irq_polarity(idx));
set_io_apic_irq_attr(&attr, apic_id, pin, irq_trigger(idx),
irq_polarity(idx));

if (!io_apic_setup_irq_pin(irq, node, &attr))
set_bit(pin, mp_ioapic_routing[apic_id].pin_programmed);
}

/*
Expand Down

0 comments on commit da1ad9d

Please sign in to comment.