Re: [PATCH 1/1] ARM: OMAP: Add conditional compilation for board specific init functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Looks like you're missing one more section for OMAP3. Will reply with
a complete patch I was about to submit.

@@ -382,11 +387,14 @@ void __init omap2430_init_early(void)
        omap_hwmod_init_postsetup();
        omap2430_clk_init();
 }
+#endif

 /*
  * Currently only board-omap3beagle.c should call this because of the
  * same machine_id for 34xx and 36xx beagle.. Will get fixed with DT.
  */
+#if defined(CONFIG_ARCH_OMAP3)
+
 void __init omap3_init_early(void)
 {
        omap2_set_globals_3xxx();
@@ -398,6 +406,7 @@ void __init omap3_init_early(void)
        omap_hwmod_init_postsetup();
        omap3xxx_clk_init();
 }
+#endif

 void __init omap3430_init_early(void)
 {


On Fri, Oct 14, 2011 at 5:00 PM, Javier Martinez Canillas
<martinez.javier@xxxxxxxxx> wrote:
>
> The definition for board specific initialization functions:
> - omap2_set_globals_xxxx
> - omapxxxx_powerdomains_init
> - omapxxxx_clockdomains_init
> - omapxxxx_hwmod_init
>
> is conditionally compiled. So we also have to conditionally compile
> the call to these functions or the linker will find undefined symbols.
>
> Signed-off-by: Javier Martinez Canillas <martinez.javier@xxxxxxxxx>
> ---
>  arch/arm/mach-omap2/io.c |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
> index aa96538..d2d045b 100644
> --- a/arch/arm/mach-omap2/io.c
> +++ b/arch/arm/mach-omap2/io.c
> @@ -359,6 +359,7 @@ static void __init omap_hwmod_init_postsetup(void)
>        omap_pm_if_early_init();
>  }
>
> +#ifdef CONFIG_SOC_OMAP2420
>  void __init omap2420_init_early(void)
>  {
>        omap2_set_globals_242x();
> @@ -370,7 +371,9 @@ void __init omap2420_init_early(void)
>        omap_hwmod_init_postsetup();
>        omap2420_clk_init();
>  }
> +#endif
>
> +#ifdef CONFIG_SOC_OMAP2430
>  void __init omap2430_init_early(void)
>  {
>        omap2_set_globals_243x();
> @@ -382,6 +385,7 @@ void __init omap2430_init_early(void)
>        omap_hwmod_init_postsetup();
>        omap2430_clk_init();
>  }
> +#endif
>
>  /*
>  * Currently only board-omap3beagle.c should call this because of the
> @@ -431,6 +435,7 @@ void __init ti816x_init_early(void)
>        omap3xxx_clk_init();
>  }
>
> +#ifdef CONFIG_ARCH_OMAP4
>  void __init omap4430_init_early(void)
>  {
>        omap2_set_globals_443x();
> @@ -442,6 +447,7 @@ void __init omap4430_init_early(void)
>        omap_hwmod_init_postsetup();
>        omap4xxx_clk_init();
>  }
> +#endif
>
>  void __init omap_sdrc_init(struct omap_sdrc_params *sdrc_cs0,
>                                      struct omap_sdrc_params *sdrc_cs1)
> --
> 1.7.4.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux