[PATCH 3/7] imx: use MXxy_IO_P2V macros to setup static mappings

[u.kleine-koenig@pengutronix.de (Uwe Kleine-König)]

Hello Dinh,

> >diff --git a/arch/arm/mach-mx5/mm.c b/arch/arm/mach-mx5/mm.c
> >index bc3f30d..01dff26 100644
> >--- a/arch/arm/mach-mx5/mm.c
> >+++ b/arch/arm/mach-mx5/mm.c
> >@@ -23,33 +23,12 @@
> > /*
> >  * Define the MX51 memory map.
> >  */
> >-static struct map_desc mxc_io_desc[] __initdata = {
> >-	{
> >-		.virtual = MX51_IRAM_BASE_ADDR_VIRT,
> >-		.pfn = __phys_to_pfn(MX51_IRAM_BASE_ADDR),
> >-		.length = MX51_IRAM_SIZE,
> >-		.type = MT_DEVICE
> >-	}, {
> >-		.virtual = MX51_DEBUG_BASE_ADDR_VIRT,
> >-		.pfn = __phys_to_pfn(MX51_DEBUG_BASE_ADDR),
> >-		.length = MX51_DEBUG_SIZE,
> >-		.type = MT_DEVICE
> >-	}, {
> >-		.virtual = MX51_AIPS1_BASE_ADDR_VIRT,
> >-		.pfn = __phys_to_pfn(MX51_AIPS1_BASE_ADDR),
> >-		.length = MX51_AIPS1_SIZE,
> >-		.type = MT_DEVICE
> >-	}, {
> >-		.virtual = MX51_SPBA0_BASE_ADDR_VIRT,
> >-		.pfn = __phys_to_pfn(MX51_SPBA0_BASE_ADDR),
> >-		.length = MX51_SPBA0_SIZE,
> >-		.type = MT_DEVICE
> >-	}, {
> >-		.virtual = MX51_AIPS2_BASE_ADDR_VIRT,
> >-		.pfn = __phys_to_pfn(MX51_AIPS2_BASE_ADDR),
> >-		.length = MX51_AIPS2_SIZE,
> >-		.type = MT_DEVICE
> >-	},
> >+static struct map_desc mx51_io_desc[] __initdata = {
> >+	imx_map_entry(MX51, IRAM, MT_DEVICE),
> 
> Do you still see a need a statically map IRAM? I added the iram
> allocator functions so that users of IRAM can dynamically allocate it.
No I don't, IMHO it shouldn't have been there in the first place.  Maybe
it even *must* go away to become reliable, don't know for sure though.
But this patch is not the right one to fix it.  So if you want to send a
patch that removes unneeded mappings that's fine.  (Preferably on
mxc91231 that has more mappings than all the other SoCs.)

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |