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

[PATCH v3 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(). 
Thanks to Jarkko Nikula <jarkko.nikula@nokia.com> for testing and Russell
King <linux@arm.linux.org.uk> for review.

Compile-tested only.

- Paul

---


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

[PATCH v3 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>
Tested-by: Jarkko Nikula <jarkko.nikula@nokia.com>

---
 arch/arm/mach-omap2/devices.c |    4 ++--
 sound/arm/omap/eac.c          |    4 +++-
 2 files changed, 5 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..47a95e2 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 */
@@ -771,6 +771,8 @@ static int __devexit eac_remove(struct platform_device *pdev)
 	eac_disable_clocks(eac);
 	eac_put_clocks(eac);
 
+	iounmap(eac->base);
+
 	platform_set_drvdata(pdev, NULL);
 
 	return 0;

[PATCH v3 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 47a95e2..667051e 100644
--- a/sound/arm/omap/eac.c
+++ b/sound/arm/omap/eac.c
@@ -787,12 +787,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 v3 0/2] OMAP2xxx EAC build fix and sparse cleanup

[Tony Lindgren]

* Paul Walmsley <paul@pwsan.com> [080905 01:29]:
> 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(). 
> Thanks to Jarkko Nikula <jarkko.nikula@nokia.com> for testing and Russell
> King <linux@arm.linux.org.uk> for review.
> 
> Compile-tested only.

Pushing both today.

Tony