Hi Tony,

> -----Original Message-----
> From: linux-arm-kernel-boun...@lists.infradead.org
[mailto:linux-arm-kernel-boun...@lists.infradead.org] On Behalf
> Of Tony Lindgren
> Sent: Wednesday, December 08, 2010 7:19 AM
> To: Nicolas Pitre
> Cc: linux-omap@vger.kernel.org; linux-arm-ker...@lists.infradead.org
> Subject: [PATCH 3/3] omap2+: Initialize omap_irq_base for
entry-macro.Sfrom platform code
>
> This way we can use the generic omap SoC detection code instead.
>
> Signed-off-by: Tony Lindgren <t...@atomide.com>
>
> diff --git a/arch/arm/mach-omap2/include/mach/entry-macro.S
b/arch/arm/mach-omap2/include/mach/entry-macro.S
> index 06e64e1..6032941 100644
> --- a/arch/arm/mach-omap2/include/mach/entry-macro.S
> +++ b/arch/arm/mach-omap2/include/mach/entry-macro.S
> @@ -38,41 +38,27 @@
>   */
>
>  #ifdef MULTI_OMAP2
> +
> +/*
> + * We use __glue to avoid errors with multiple definitions of
> + * .globl omap_irq_base as it's included from entry-armv.S but not
> + * from entry-common.S.
> + */
> +#ifdef __glue
>               .pushsection .data
> -omap_irq_base:       .word   0
> +             .globl  omap_irq_base
> +omap_irq_base:
> +             .word   0
>               .popsection
> +#endif
>
> -             /* Configure the interrupt base on the first interrupt */
> +             /*
> +              * Configure the interrupt base on the first interrupt.
> +              * See also omap_irq_base_init for setting omap_irq_base.
> +              */
>               .macro  get_irqnr_preamble, base, tmp
> -9:
>               ldr     \base, =omap_irq_base   @ irq base address
>               ldr     \base, [\base, #0]      @ irq base value
> -             cmp     \base, #0               @ already configured?
> -             bne     9997f                   @ nothing to do
> -
> -             mrc     p15, 0, \tmp, c0, c0, 0 @ get processor revision
> -             and     \tmp, \tmp, #0x000f0000 @ only check architecture
> -             cmp     \tmp, #0x00070000       @ is v6?
> -             beq     2400f                   @ found v6 so it's
omap24xx
> -             mrc     p15, 0, \tmp, c0, c0, 0 @ get processor revision
> -             and     \tmp, \tmp, #0x000000f0 @ check cortex 8 or 9
> -             cmp     \tmp, #0x00000080       @ cortex A-8?
> -             beq     3400f                   @ found A-8 so it's
omap34xx
> -             cmp     \tmp, #0x00000090       @ cortex A-9?
> -             beq     4400f                   @ found A-9 so it's
omap44xx
> -2400:                ldr     \base, =OMAP2_IRQ_BASE
> -             ldr     \tmp, =omap_irq_base
> -             str     \base, [\tmp, #0]
> -             b       9b
> -3400:                ldr     \base, =OMAP3_IRQ_BASE
> -             ldr     \tmp, =omap_irq_base
> -             str     \base, [\tmp, #0]
> -             b       9b
> -4400:                ldr     \base, =OMAP4_IRQ_BASE
> -             ldr     \tmp, =omap_irq_base
> -             str     \base, [\tmp, #0]
> -             b       9b
> -9997:
>               .endm
>
>               /* Check the pending interrupts. Note that base already
set */
> diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
> index 40562dd..3d18349 100644
> --- a/arch/arm/mach-omap2/io.c
> +++ b/arch/arm/mach-omap2/io.c
> @@ -46,6 +46,7 @@
>  #include "clockdomains.h"
>
>  #include <plat/omap_hwmod.h>
> +#include <plat/multi.h>
>
>  /*
>   * The machine specific code may provide the extra mapping besides the
> @@ -311,6 +312,25 @@ static int __init _omap2_init_reprogram_sdrc(void)
>       return v;
>  }
>
> +/*
> + * Initialize asm_irq_base for entry-macro.S
> + */
> +static inline void omap_irq_base_init(void)
> +{
> +     extern void __iomem *omap_irq_base;
> +
> +#ifdef MULTI_OMAP2
> +     if (cpu_is_omap242x())

Looks like this should be a cpu_is_omap24xx(). The
current master is broken on a 2430sdp and a git-bisect
pointed me to this patch.

Regards,
Rajendra

> +             omap_irq_base = OMAP2_L4_IO_ADDRESS(OMAP24XX_IC_BASE);
> +     else if (cpu_is_omap34xx())
> +             omap_irq_base = OMAP2_L4_IO_ADDRESS(OMAP34XX_IC_BASE);
> +     else if (cpu_is_omap44xx())
> +             omap_irq_base =
OMAP2_L4_IO_ADDRESS(OMAP44XX_GIC_CPU_BASE);
> +     else
> +             pr_err("Could not initialize omap_irq_base\n");
> +#endif
> +}
> +
>  void __init omap2_init_common_hw(struct omap_sdrc_params *sdrc_cs0,
>                                struct omap_sdrc_params *sdrc_cs1)
>  {
> @@ -352,4 +372,6 @@ void __init omap2_init_common_hw(struct
omap_sdrc_params *sdrc_cs0,
>               _omap2_init_reprogram_sdrc();
>       }
>       gpmc_init();
> +
> +     omap_irq_base_init();
>  }
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-ker...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to