* [PATCH v2] ARM: plat-mxc: audmux-v1: Remove unneeded ifdef's
@ 2012-02-17 0:15 Fabio Estevam
2012-02-18 16:01 ` Sergei Shtylyov
0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2012-02-17 0:15 UTC (permalink / raw)
To: linux-arm-kernel
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())
audmux_base = MX27_IO_ADDRESS(MX27_AUDMUX_BASE_ADDR);
else
-#endif
(void)0;
return 0;
--
1.7.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH v2] ARM: plat-mxc: audmux-v1: Remove unneeded ifdef's
2012-02-17 0:15 [PATCH v2] ARM: plat-mxc: audmux-v1: Remove unneeded ifdef's Fabio Estevam
@ 2012-02-18 16:01 ` Sergei Shtylyov
0 siblings, 0 replies; 2+ messages in thread
From: Sergei Shtylyov @ 2012-02-18 16:01 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-02-18 16:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-17 0:15 [PATCH v2] ARM: plat-mxc: audmux-v1: Remove unneeded ifdef's Fabio Estevam
2012-02-18 16:01 ` Sergei Shtylyov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).