diff options
| author | Jeremy Fitzhardinge | 2009-06-08 12:29:26 +0200 |
|---|---|---|
| committer | Jeremy Fitzhardinge | 2009-07-14 22:32:50 +0200 |
| commit | 535b64291a9d1ff8bc54642494a5fce27e1e1170 (patch) | |
| tree | f4fc8d28fb64ff5c4a847cc89c97512611ee1c75 /arch/x86/kernel/apic | |
| parent | x86/ioapic.c: simplify add_pin_to_irq_node() (diff) | |
| download | kernel-qcow2-linux-535b64291a9d1ff8bc54642494a5fce27e1e1170.tar.gz kernel-qcow2-linux-535b64291a9d1ff8bc54642494a5fce27e1e1170.tar.xz kernel-qcow2-linux-535b64291a9d1ff8bc54642494a5fce27e1e1170.zip | |
x86/ioapic.c: convert replace_pin_at_irq_node to conventional for() loop
Use a conventional for() loop in replace_pin_at_irq_node().
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Diffstat (limited to 'arch/x86/kernel/apic')
| -rw-r--r-- | arch/x86/kernel/apic/io_apic.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c index d9e8f19088d4..9386976b675b 100644 --- a/arch/x86/kernel/apic/io_apic.c +++ b/arch/x86/kernel/apic/io_apic.c @@ -515,10 +515,10 @@ static void __init replace_pin_at_irq_node(struct irq_cfg *cfg, int node, int oldapic, int oldpin, int newapic, int newpin) { - struct irq_pin_list *entry = cfg->irq_2_pin; + struct irq_pin_list *entry; int replaced = 0; - while (entry) { + for (entry = cfg->irq_2_pin; entry != NULL; entry = entry->next) { if (entry->apic == oldapic && entry->pin == oldpin) { entry->apic = newapic; entry->pin = newpin; @@ -526,7 +526,6 @@ static void __init replace_pin_at_irq_node(struct irq_cfg *cfg, int node, /* every one is different, right? */ break; } - entry = entry->next; } /* why? call replace before add? */ |