[PATCH v2 0/2] OMAP2xxx EAC build fix and sparse cleanup

[PATCH v2 0/2] OMAP2xxx EAC build fix and sparse cleanup

[Paul Walmsley]

This series fixes a build bug in the EAC driver after the recent IO address
changes, and resolves some sparse warnings.  Updated to use ioremap().

Compile-tested only.

Signed-off-by: Paul Walmsley <paul@pwsan.com>

---

 arch/arm/mach-omap2/devices.c |    4 ++--
 sound/arm/omap/eac.c          |    6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

[PATCH v2 1/2] OMAP2xxx EAC: update per recent IO address changes

[Paul Walmsley]

Commit f228a725b975832ac5771ab2fc86d06bd694cdb3 breaks the build for this
driver; fix by using ioremap().

Signed-off-by: Paul Walmsley <paul@pwsan.com>
---
 arch/arm/mach-omap2/devices.c |    4 ++--
 sound/arm/omap/eac.c          |    2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c
index bd49a0f..c7de03e 100644
--- a/arch/arm/mach-omap2/devices.c
+++ b/arch/arm/mach-omap2/devices.c
@@ -270,12 +270,12 @@ static inline void omap_init_mcspi(void) {}
 
 #ifdef CONFIG_SND_OMAP24XX_EAC
 
-#define OMAP2_EAC_BASE			0x48090000
+#define OMAP2_EAC_BASE			(L4_24XX_BASE + 0x90000)
 
 static struct resource omap2_eac_resources[] = {
 	{
 		.start		= OMAP2_EAC_BASE,
-		.end		= OMAP2_EAC_BASE + 0x109,
+		.end		= OMAP2_EAC_BASE + 0xfff,
 		.flags		= IORESOURCE_MEM,
 	},
 };
diff --git a/sound/arm/omap/eac.c b/sound/arm/omap/eac.c
index 9fe8d74..9752ece 100644
--- a/sound/arm/omap/eac.c
+++ b/sound/arm/omap/eac.c
@@ -707,7 +707,7 @@ static int __devinit eac_probe(struct platform_device *pdev)
 		err = -ENODEV;
 		goto err1;
 	}
-	eac->base = (void __iomem *)io_p2v(res->start);
+	eac->base = ioremap(res->start, (res->end - res->start) + 1);
 	eac->pdata = pdata;
 
 	/* pre-initialize EAC hw */

[PATCH v2 2/2] OMAP2xxx EAC: fix sparse warnings

[Paul Walmsley]

This patch resolves the following two sparse warnings:

sound/arm/omap/eac.c:788:12: warning: symbol 'eac_init' was not declared. Should it be static?
sound/arm/omap/eac.c:793:13: warning: symbol 'eac_exit' was not declared. Should it be static?

Signed-off-by: Paul Walmsley <paul@pwsan.com>
---
 sound/arm/omap/eac.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/arm/omap/eac.c b/sound/arm/omap/eac.c
index 9752ece..35ec8d6 100644
--- a/sound/arm/omap/eac.c
+++ b/sound/arm/omap/eac.c
@@ -785,12 +785,12 @@ static struct platform_driver eac_driver = {
 	.remove		= eac_remove,
 };
 
-int __init eac_init(void)
+static int __init eac_init(void)
 {
 	return platform_driver_register(&eac_driver);
 }
 
-void __exit eac_exit(void)
+static void __exit eac_exit(void)
 {
 	platform_driver_unregister(&eac_driver);
 }

Re: [PATCH v2 0/2] OMAP2xxx EAC build fix and sparse cleanup

[Jarkko Nikula]

On Thu, 04 Sep 2008 23:50:48 -0600
"ext Paul Walmsley" <paul@pwsan.com> wrote:

> This series fixes a build bug in the EAC driver after the recent IO
> address changes, and resolves some sparse warnings.  Updated to use
> ioremap().
> 
> Compile-tested only.
> 
> Signed-off-by: Paul Walmsley <paul@pwsan.com>
> 
Thanks Paul. I tested these and plays fine (OS2007).

Acked-by: Jarkko Nikula <jarkko.nikula@nokia.com>

Re: [PATCH v2 1/2] OMAP2xxx EAC: update per recent IO address changes

[Russell King - ARM Linux]

On Thu, Sep 04, 2008 at 11:51:15PM -0600, Paul Walmsley wrote:
> @@ -707,7 +707,7 @@ static int __devinit eac_probe(struct platform_device *pdev)
>  		err = -ENODEV;
>  		goto err1;
>  	}
> -	eac->base = (void __iomem *)io_p2v(res->start);
> +	eac->base = ioremap(res->start, (res->end - res->start) + 1);

The parens around the subtraction are just noise; please remove.