Message ID | 1329437737-8378-1-git-send-email-festevam@gmail.com |
---|---|
State | New |
Headers | show |
Hello. On 17-02-2012 4:15, Fabio Estevam wrote: > As we are able to build a single kernel that can run on mx21 and mx27, > there is no need for the ifdef's anymore inside audmux-v1.c. > Signed-off-by: Fabio Estevam<fabio.estevam@freescale.com> > --- > Changes since v1: > - Replace mx25 with mx27 in the commit message > arch/arm/plat-mxc/audmux-v1.c | 5 +---- > 1 files changed, 1 insertions(+), 4 deletions(-) > diff --git a/arch/arm/plat-mxc/audmux-v1.c b/arch/arm/plat-mxc/audmux-v1.c > index 1180bef..14a3c04 100644 > --- a/arch/arm/plat-mxc/audmux-v1.c > +++ b/arch/arm/plat-mxc/audmux-v1.c > @@ -46,16 +46,13 @@ EXPORT_SYMBOL_GPL(mxc_audmux_v1_configure_port); > > static int mxc_audmux_v1_init(void) > { > -#ifdef CONFIG_MACH_MX21 > if (cpu_is_mx21()) > audmux_base = MX21_IO_ADDRESS(MX21_AUDMUX_BASE_ADDR); > else > -#endif > -#ifdef CONFIG_MACH_MX27 > + > if (cpu_is_mx27()) Same comment about *else if*. WBR, Sergei
diff --git a/arch/arm/plat-mxc/audmux-v1.c b/arch/arm/plat-mxc/audmux-v1.c index 1180bef..14a3c04 100644 --- a/arch/arm/plat-mxc/audmux-v1.c +++ b/arch/arm/plat-mxc/audmux-v1.c @@ -46,16 +46,13 @@ EXPORT_SYMBOL_GPL(mxc_audmux_v1_configure_port); static int mxc_audmux_v1_init(void) { -#ifdef CONFIG_MACH_MX21 if (cpu_is_mx21()) audmux_base = MX21_IO_ADDRESS(MX21_AUDMUX_BASE_ADDR); else -#endif -#ifdef CONFIG_MACH_MX27 + if (cpu_is_mx27()) audmux_base = MX27_IO_ADDRESS(MX27_AUDMUX_BASE_ADDR); else -#endif (void)0; return 0;
As we are able to build a single kernel that can run on mx21 and mx27, there is no need for the ifdef's anymore inside audmux-v1.c. Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com> --- Changes since v1: - Replace mx25 with mx27 in the commit message arch/arm/plat-mxc/audmux-v1.c | 5 +---- 1 files changed, 1 insertions(+), 4 deletions(-)