Re: [PATCH] OMAP2: fix cpu_is_omap24* optimizations

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

 



Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> writes:

> Currently if omap2420 is defined but not omap2430, cpu_is_omap2430()
> is still defined as a macro, instead of #define'd to zero.  This
> results in conditional cpu_is_omap2430() code still being compiled,
> and leads to possible compile/link errors.  In particular for hwmod
> init calls in io.c
>
> To fix, add extra #ifdefs to CPU check macros to ensure that the
> is_omap* macros are zero for each OMAP2 if they are not configured
> into the kernel.
>
> Signed-off-by: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
> ---

I forgot to mention that this applies on top of omap-fixes branch.

Kevin

>  arch/arm/plat-omap/include/plat/cpu.h |    8 ++++++--
>  1 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/plat-omap/include/plat/cpu.h b/arch/arm/plat-omap/include/plat/cpu.h
> index ed8786c..7514174 100644
> --- a/arch/arm/plat-omap/include/plat/cpu.h
> +++ b/arch/arm/plat-omap/include/plat/cpu.h
> @@ -167,10 +167,14 @@ IS_OMAP_SUBCLASS(443x, 0x443)
>  #if defined(MULTI_OMAP2)
>  # if defined(CONFIG_ARCH_OMAP2)
>  #  undef  cpu_is_omap24xx
> -#  undef  cpu_is_omap242x
> -#  undef  cpu_is_omap243x
>  #  define cpu_is_omap24xx()		is_omap24xx()
> +# endif
> +# if defined (CONFIG_ARCH_OMAP2420)
> +#  undef  cpu_is_omap242x
>  #  define cpu_is_omap242x()		is_omap242x()
> +# endif
> +# if defined (CONFIG_ARCH_OMAP2430)
> +#  undef  cpu_is_omap243x
>  #  define cpu_is_omap243x()		is_omap243x()
>  # endif
>  # if defined(CONFIG_ARCH_OMAP3)
> -- 
> 1.6.6
--
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