diff options
author | KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> | 2008-12-25 22:29:48 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-12-26 03:48:18 -0500 |
commit | 18eefedfe8ad33e8fc7614c13359e29a9fab4644 (patch) | |
tree | e0b7c801c27a3933137e7e4056a3bca3c3244744 /arch/x86/kernel/io_apic.c | |
parent | 26ddd8d5cac8a563953d5febe8c6e40909f7bce1 (diff) |
irq: simplify for_each_irq_desc() usage
Impact: cleanup
all for_each_irq_desc() usage point have !desc check.
then its check can move into for_each_irq_desc() macro.
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Acked-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86/kernel/io_apic.c')
-rw-r--r-- | arch/x86/kernel/io_apic.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/arch/x86/kernel/io_apic.c b/arch/x86/kernel/io_apic.c index a74887b416cc..2fe543f58ac8 100644 --- a/arch/x86/kernel/io_apic.c +++ b/arch/x86/kernel/io_apic.c | |||
@@ -1345,8 +1345,6 @@ void __setup_vector_irq(int cpu) | |||
1345 | 1345 | ||
1346 | /* Mark the inuse vectors */ | 1346 | /* Mark the inuse vectors */ |
1347 | for_each_irq_desc(irq, desc) { | 1347 | for_each_irq_desc(irq, desc) { |
1348 | if (!desc) | ||
1349 | continue; | ||
1350 | cfg = desc->chip_data; | 1348 | cfg = desc->chip_data; |
1351 | if (!cpu_isset(cpu, cfg->domain)) | 1349 | if (!cpu_isset(cpu, cfg->domain)) |
1352 | continue; | 1350 | continue; |
@@ -1730,8 +1728,6 @@ __apicdebuginit(void) print_IO_APIC(void) | |||
1730 | for_each_irq_desc(irq, desc) { | 1728 | for_each_irq_desc(irq, desc) { |
1731 | struct irq_pin_list *entry; | 1729 | struct irq_pin_list *entry; |
1732 | 1730 | ||
1733 | if (!desc) | ||
1734 | continue; | ||
1735 | cfg = desc->chip_data; | 1731 | cfg = desc->chip_data; |
1736 | entry = cfg->irq_2_pin; | 1732 | entry = cfg->irq_2_pin; |
1737 | if (!entry) | 1733 | if (!entry) |
@@ -2378,9 +2374,6 @@ static void ir_irq_migration(struct work_struct *work) | |||
2378 | struct irq_desc *desc; | 2374 | struct irq_desc *desc; |
2379 | 2375 | ||
2380 | for_each_irq_desc(irq, desc) { | 2376 | for_each_irq_desc(irq, desc) { |
2381 | if (!desc) | ||
2382 | continue; | ||
2383 | |||
2384 | if (desc->status & IRQ_MOVE_PENDING) { | 2377 | if (desc->status & IRQ_MOVE_PENDING) { |
2385 | unsigned long flags; | 2378 | unsigned long flags; |
2386 | 2379 | ||
@@ -2671,9 +2664,6 @@ static inline void init_IO_APIC_traps(void) | |||
2671 | * 0x80, because int 0x80 is hm, kind of importantish. ;) | 2664 | * 0x80, because int 0x80 is hm, kind of importantish. ;) |
2672 | */ | 2665 | */ |
2673 | for_each_irq_desc(irq, desc) { | 2666 | for_each_irq_desc(irq, desc) { |
2674 | if (!desc) | ||
2675 | continue; | ||
2676 | |||
2677 | cfg = desc->chip_data; | 2667 | cfg = desc->chip_data; |
2678 | if (IO_APIC_IRQ(irq) && cfg && !cfg->vector) { | 2668 | if (IO_APIC_IRQ(irq) && cfg && !cfg->vector) { |
2679 | /* | 2669 | /* |