On Wed, 7 Oct 2009, Tony Lindgren wrote:
> >From f05ba4e3427bc0dc216f2fca32a9b1e8f0e38695 Mon Sep 17 00:00:00 2001
> From: Tony Lindgren <t...@atomide.com>
> Date: Wed, 7 Oct 2009 17:10:18 -0700
> Subject: [PATCH] omap: Use ioremap for omap_hwmod instead of OMAP2_IO_ADDRESS
>
> Otherwise we cannot get rid of OMAP2_IO_ADDRESS.
Acked-by: Paul Walmsley <p...@pwsan.com>
>
> Signed-off-by: Tony Lindgren <t...@atomide.com>
>
> diff --git a/arch/arm/mach-omap2/omap_hwmod.c
> b/arch/arm/mach-omap2/omap_hwmod.c
> index d2e0f1c..8ac8798 100644
> --- a/arch/arm/mach-omap2/omap_hwmod.c
> +++ b/arch/arm/mach-omap2/omap_hwmod.c
> @@ -496,6 +496,7 @@ static void __iomem *_find_mpu_rt_base(struct omap_hwmod
> *oh, u8 index)
> struct omap_hwmod_addr_space *mem;
> int i;
> int found = 0;
> + void __iomem *va_start;
>
> if (!oh || oh->slaves_cnt == 0)
> return NULL;
> @@ -509,16 +510,20 @@ static void __iomem *_find_mpu_rt_base(struct
> omap_hwmod *oh, u8 index)
> }
> }
>
> - /* XXX use ioremap() instead? */
> -
> - if (found)
> + if (found) {
> + va_start = ioremap(mem->pa_start, mem->pa_end - mem->pa_start);
> + if (!va_start) {
> + pr_err("omap_hwmod: %s: Could not ioremap\n", oh->name);
> + return NULL;
> + }
> pr_debug("omap_hwmod: %s: MPU register target at va %p\n",
> - oh->name, OMAP2_IO_ADDRESS(mem->pa_start));
> - else
> + oh->name, va_start);
> + } else {
> pr_debug("omap_hwmod: %s: no MPU register target found\n",
> oh->name);
> + }
>
> - return (found) ? OMAP2_IO_ADDRESS(mem->pa_start) : NULL;
> + return (found) ? va_start : NULL;
> }
>
> /**
> @@ -1148,6 +1153,7 @@ int omap_hwmod_unregister(struct omap_hwmod *oh)
> pr_debug("omap_hwmod: %s: unregistering\n", oh->name);
>
> mutex_lock(&omap_hwmod_mutex);
> + iounmap(oh->_rt_va);
> list_del(&oh->node);
> mutex_unlock(&omap_hwmod_mutex);
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
