diff options
| author |  Yury Norov <yury.norov@gmail.com>
| 2024-04-13 11:49:13 -0700 |
|---|---|---|
| committer |  Thomas Bogendoerfer <tsbogend@alpha.franken.de>
| 2024-04-15 10:29:36 +0200 |
| commit | 40e20fbccfb722f219ab8d3ff1edde99e4a7c46c (patch) | |
| tree | 9cc432b547962a9f0bf01be31efa70d297144ab4 /arch/mips | |
| parent | mips: dts: ralink: mt7621: reorder the attributes of the root node (diff) | |
| download | linux-40e20fbccfb722f219ab8d3ff1edde99e4a7c46c.tar.gz linux-40e20fbccfb722f219ab8d3ff1edde99e4a7c46c.tar.bz2 linux-40e20fbccfb722f219ab8d3ff1edde99e4a7c46c.zip | |
MIPS: SGI-IP27: micro-optimize arch_init_irq()
The function sets adjasted groups of bits in hub_irq_map by using
for-loops. There's a bitmap_set() function dedicated to do this.
Because [0, CPU_CALL_B_IRQ] and [NI_BRDCAST_ERR_A, MSC_PANIC_INTR]
ranges belong to the same machine word, bitmap_set() would boil down
to an inline wrapper in both cases, avoiding generating a loop, whth
the associate overhead.
Signed-off-by: Yury Norov <yury.norov@gmail.com>
Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Diffstat (limited to 'arch/mips')
| -rw-r--r-- | arch/mips/sgi-ip27/ip27-irq.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/arch/mips/sgi-ip27/ip27-irq.c b/arch/mips/sgi-ip27/ip27-irq.c index 8f5299b269e7..dcb14a234b1c 100644 --- a/arch/mips/sgi-ip27/ip27-irq.c +++ b/arch/mips/sgi-ip27/ip27-irq.c @@ -286,11 +286,8 @@ void __init arch_init_irq(void) * Mark these as reserved right away so they won't be used accidentally * later. */ - for (i = 0; i <= CPU_CALL_B_IRQ; i++) - set_bit(i, hub_irq_map); - - for (i = NI_BRDCAST_ERR_A; i <= MSC_PANIC_INTR; i++) - set_bit(i, hub_irq_map); + bitmap_set(hub_irq_map, 0, CPU_CALL_B_IRQ + 1); + bitmap_set(hub_irq_map, NI_BRDCAST_ERR_A, MSC_PANIC_INTR - NI_BRDCAST_ERR_A + 1); fn = irq_domain_alloc_named_fwnode("HUB"); WARN_ON(fn == NULL); |