diff options
author | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2009-06-08 06:29:26 -0400 |
---|---|---|
committer | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2009-07-14 16:32:50 -0400 |
commit | 535b64291a9d1ff8bc54642494a5fce27e1e1170 (patch) | |
tree | f4fc8d28fb64ff5c4a847cc89c97512611ee1c75 /arch/x86 | |
parent | 875e68ec32fc5495f3edf987aaae1c52306184b7 (diff) |
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')
-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, | |||
515 | int oldapic, int oldpin, | 515 | int oldapic, int oldpin, |
516 | int newapic, int newpin) | 516 | int newapic, int newpin) |
517 | { | 517 | { |
518 | struct irq_pin_list *entry = cfg->irq_2_pin; | 518 | struct irq_pin_list *entry; |
519 | int replaced = 0; | 519 | int replaced = 0; |
520 | 520 | ||
521 | while (entry) { | 521 | for (entry = cfg->irq_2_pin; entry != NULL; entry = entry->next) { |
522 | if (entry->apic == oldapic && entry->pin == oldpin) { | 522 | if (entry->apic == oldapic && entry->pin == oldpin) { |
523 | entry->apic = newapic; | 523 | entry->apic = newapic; |
524 | entry->pin = newpin; | 524 | entry->pin = newpin; |
@@ -526,7 +526,6 @@ static void __init replace_pin_at_irq_node(struct irq_cfg *cfg, int node, | |||
526 | /* every one is different, right? */ | 526 | /* every one is different, right? */ |
527 | break; | 527 | break; |
528 | } | 528 | } |
529 | entry = entry->next; | ||
530 | } | 529 | } |
531 | 530 | ||
532 | /* why? call replace before add? */ | 531 | /* why? call replace before add? */ |