diff options
| author | Russell King <rmk+kernel@arm.linux.org.uk> | 2011-01-07 06:57:44 -0500 |
|---|---|---|
| committer | Tony Lindgren <tony@atomide.com> | 2011-01-27 18:33:47 -0500 |
| commit | 9f9605c2eda9679e6f63c605cbd9cbf6a9a7f3fa (patch) | |
| tree | 318072a09a23b66c553fd910f602447ef857d5dd | |
| parent | 6fb1b304255efc5c4c93874ac8c066272e257e28 (diff) | |
omap2+: Fix unused variable warning for omap_irq_base
Commit 5d190c40100793a6dfc37bf325677c10f3c80edf
(omap2+: Initialize omap_irq_base for entry-macro.S from
platform code) simplified the handling of omap_irq_base
for multi-omap builds. However, this patch also introduced
a build warning for !MULTI_OMAP2 builds:
arch/arm/mach-omap2/io.c: In function 'omap_irq_base_init':
arch/arm/mach-omap2/io.c:322: warning: unused variable 'omap_irq_base'
Fix this by removing the ifdef. Also simplify things further
by moving omap_irq_base out of entry-macro.S.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
[tony@atomide.com: updated comments]
Signed-off-by: Tony Lindgren <tony@atomide.com>
| -rw-r--r-- | arch/arm/mach-omap2/include/mach/entry-macro.S | 14 | ||||
| -rw-r--r-- | arch/arm/mach-omap2/io.c | 6 |
2 files changed, 2 insertions, 18 deletions
diff --git a/arch/arm/mach-omap2/include/mach/entry-macro.S b/arch/arm/mach-omap2/include/mach/entry-macro.S index befa321c4c13..81985a665cb3 100644 --- a/arch/arm/mach-omap2/include/mach/entry-macro.S +++ b/arch/arm/mach-omap2/include/mach/entry-macro.S | |||
| @@ -38,20 +38,6 @@ | |||
| 38 | */ | 38 | */ |
| 39 | 39 | ||
| 40 | #ifdef MULTI_OMAP2 | 40 | #ifdef MULTI_OMAP2 |
| 41 | |||
| 42 | /* | ||
| 43 | * We use __glue to avoid errors with multiple definitions of | ||
| 44 | * .globl omap_irq_base as it's included from entry-armv.S but not | ||
| 45 | * from entry-common.S. | ||
| 46 | */ | ||
| 47 | #ifdef __glue | ||
| 48 | .pushsection .data | ||
| 49 | .globl omap_irq_base | ||
| 50 | omap_irq_base: | ||
| 51 | .word 0 | ||
| 52 | .popsection | ||
| 53 | #endif | ||
| 54 | |||
| 55 | /* | 41 | /* |
| 56 | * Configure the interrupt base on the first interrupt. | 42 | * Configure the interrupt base on the first interrupt. |
| 57 | * See also omap_irq_base_init for setting omap_irq_base. | 43 | * See also omap_irq_base_init for setting omap_irq_base. |
diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c index e66687b0b9de..c2032041d26f 100644 --- a/arch/arm/mach-omap2/io.c +++ b/arch/arm/mach-omap2/io.c | |||
| @@ -314,14 +314,13 @@ static int _set_hwmod_postsetup_state(struct omap_hwmod *oh, void *data) | |||
| 314 | return omap_hwmod_set_postsetup_state(oh, *(u8 *)data); | 314 | return omap_hwmod_set_postsetup_state(oh, *(u8 *)data); |
| 315 | } | 315 | } |
| 316 | 316 | ||
| 317 | void __iomem *omap_irq_base; | ||
| 318 | |||
| 317 | /* | 319 | /* |
| 318 | * Initialize asm_irq_base for entry-macro.S | 320 | * Initialize asm_irq_base for entry-macro.S |
| 319 | */ | 321 | */ |
| 320 | static inline void omap_irq_base_init(void) | 322 | static inline void omap_irq_base_init(void) |
| 321 | { | 323 | { |
| 322 | extern void __iomem *omap_irq_base; | ||
| 323 | |||
| 324 | #ifdef MULTI_OMAP2 | ||
| 325 | if (cpu_is_omap24xx()) | 324 | if (cpu_is_omap24xx()) |
| 326 | omap_irq_base = OMAP2_L4_IO_ADDRESS(OMAP24XX_IC_BASE); | 325 | omap_irq_base = OMAP2_L4_IO_ADDRESS(OMAP24XX_IC_BASE); |
| 327 | else if (cpu_is_omap34xx()) | 326 | else if (cpu_is_omap34xx()) |
| @@ -330,7 +329,6 @@ static inline void omap_irq_base_init(void) | |||
| 330 | omap_irq_base = OMAP2_L4_IO_ADDRESS(OMAP44XX_GIC_CPU_BASE); | 329 | omap_irq_base = OMAP2_L4_IO_ADDRESS(OMAP44XX_GIC_CPU_BASE); |
| 331 | else | 330 | else |
| 332 | pr_err("Could not initialize omap_irq_base\n"); | 331 | pr_err("Could not initialize omap_irq_base\n"); |
| 333 | #endif | ||
| 334 | } | 332 | } |
| 335 | 333 | ||
| 336 | void __init omap2_init_common_infrastructure(void) | 334 | void __init omap2_init_common_infrastructure(void) |
