Re: [PATCH 1/2] ARM: i.MX27: Add platform support for IRAM.

2012-11-22 Thread javier Martin
On 16 November 2012 13:48, Sascha Hauer s.ha...@pengutronix.de wrote:
 On Mon, Nov 05, 2012 at 04:59:44PM +0100, Javier Martin wrote:
 Add support for IRAM to i.MX27 non-DT platforms using
 iram_init() function.

 Signed-off-by: Javier Martin javier.mar...@vista-silicon.com
 ---
  arch/arm/mach-imx/mm-imx27.c |3 +++
  1 file changed, 3 insertions(+)

 diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c
 index e7e24af..fd2416d 100644
 --- a/arch/arm/mach-imx/mm-imx27.c
 +++ b/arch/arm/mach-imx/mm-imx27.c
 @@ -27,6 +27,7 @@
  #include asm/pgtable.h
  #include asm/mach/map.h
  #include mach/iomux-v1.h
 +#include mach/iram.h

  /* MX27 memory map definition */
  static struct map_desc imx27_io_desc[] __initdata = {
 @@ -94,4 +95,6 @@ void __init imx27_soc_init(void)
   /* imx27 has the imx21 type audmux */
   platform_device_register_simple(imx21-audmux, 0, imx27_audmux_res,
   ARRAY_SIZE(imx27_audmux_res));
 + /* imx27 has an iram of 46080 bytes size */
 + iram_init(MX27_IRAM_BASE_ADDR, 46080);

 For this rather Philipps sram allocater patches should be used. This
 would also solve the problem that mach/iram.h cannot be accessed anymore
 in current -next. Fabio already sent a patch addressing this, but I
 think we should go for a proper fix rather than just moving iram.h
 to include/linux/

Fine, I'll take a look at Philipps' patches.

Regards.

-- 
Javier Martin
Vista Silicon S.L.
CDTUC - FASE C - Oficina S-345
Avda de los Castros s/n
39005- Santander. Cantabria. Spain
+34 942 25 32 60
www.vista-silicon.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/2] ARM: i.MX27: Add platform support for IRAM.

2012-11-16 Thread Sascha Hauer
On Mon, Nov 05, 2012 at 04:59:44PM +0100, Javier Martin wrote:
 Add support for IRAM to i.MX27 non-DT platforms using
 iram_init() function.
 
 Signed-off-by: Javier Martin javier.mar...@vista-silicon.com
 ---
  arch/arm/mach-imx/mm-imx27.c |3 +++
  1 file changed, 3 insertions(+)
 
 diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c
 index e7e24af..fd2416d 100644
 --- a/arch/arm/mach-imx/mm-imx27.c
 +++ b/arch/arm/mach-imx/mm-imx27.c
 @@ -27,6 +27,7 @@
  #include asm/pgtable.h
  #include asm/mach/map.h
  #include mach/iomux-v1.h
 +#include mach/iram.h
  
  /* MX27 memory map definition */
  static struct map_desc imx27_io_desc[] __initdata = {
 @@ -94,4 +95,6 @@ void __init imx27_soc_init(void)
   /* imx27 has the imx21 type audmux */
   platform_device_register_simple(imx21-audmux, 0, imx27_audmux_res,
   ARRAY_SIZE(imx27_audmux_res));
 + /* imx27 has an iram of 46080 bytes size */
 + iram_init(MX27_IRAM_BASE_ADDR, 46080);

For this rather Philipps sram allocater patches should be used. This
would also solve the problem that mach/iram.h cannot be accessed anymore
in current -next. Fabio already sent a patch addressing this, but I
think we should go for a proper fix rather than just moving iram.h
to include/linux/

Sascha

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/2] ARM: i.MX27: Add platform support for IRAM.

2012-11-06 Thread Guennadi Liakhovetski
Hi Javier

On Mon, 5 Nov 2012, Javier Martin wrote:

 Add support for IRAM to i.MX27 non-DT platforms using
 iram_init() function.

I'm not sure this belongs in a camera driver. Can IRAM not be used for 
anything else? I'll check the i.MX27 datasheet when I'm back home after 
the conference, so far this seems a bit odd.

Thanks
Guennadi

 
 Signed-off-by: Javier Martin javier.mar...@vista-silicon.com
 ---
  arch/arm/mach-imx/mm-imx27.c |3 +++
  1 file changed, 3 insertions(+)
 
 diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c
 index e7e24af..fd2416d 100644
 --- a/arch/arm/mach-imx/mm-imx27.c
 +++ b/arch/arm/mach-imx/mm-imx27.c
 @@ -27,6 +27,7 @@
  #include asm/pgtable.h
  #include asm/mach/map.h
  #include mach/iomux-v1.h
 +#include mach/iram.h
  
  /* MX27 memory map definition */
  static struct map_desc imx27_io_desc[] __initdata = {
 @@ -94,4 +95,6 @@ void __init imx27_soc_init(void)
   /* imx27 has the imx21 type audmux */
   platform_device_register_simple(imx21-audmux, 0, imx27_audmux_res,
   ARRAY_SIZE(imx27_audmux_res));
 + /* imx27 has an iram of 46080 bytes size */
 + iram_init(MX27_IRAM_BASE_ADDR, 46080);
  }
 -- 
 1.7.9.5
 
 --
 To unsubscribe from this list: send the line unsubscribe linux-media in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
 

---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/2] ARM: i.MX27: Add platform support for IRAM.

2012-11-06 Thread Sascha Hauer
On Tue, Nov 06, 2012 at 12:37:35PM +0100, Guennadi Liakhovetski wrote:
 Hi Javier
 
 On Mon, 5 Nov 2012, Javier Martin wrote:
 
  Add support for IRAM to i.MX27 non-DT platforms using
  iram_init() function.
 
 I'm not sure this belongs in a camera driver. Can IRAM not be used for 
 anything else? I'll check the i.MX27 datasheet when I'm back home after 
 the conference, so far this seems a bit odd.

This patch just adds the sram pool to the system in i.MX27 code, the
patch is not camera specific.

Sascha

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 1/2] ARM: i.MX27: Add platform support for IRAM.

2012-11-05 Thread Javier Martin
Add support for IRAM to i.MX27 non-DT platforms using
iram_init() function.

Signed-off-by: Javier Martin javier.mar...@vista-silicon.com
---
 arch/arm/mach-imx/mm-imx27.c |3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/mach-imx/mm-imx27.c b/arch/arm/mach-imx/mm-imx27.c
index e7e24af..fd2416d 100644
--- a/arch/arm/mach-imx/mm-imx27.c
+++ b/arch/arm/mach-imx/mm-imx27.c
@@ -27,6 +27,7 @@
 #include asm/pgtable.h
 #include asm/mach/map.h
 #include mach/iomux-v1.h
+#include mach/iram.h
 
 /* MX27 memory map definition */
 static struct map_desc imx27_io_desc[] __initdata = {
@@ -94,4 +95,6 @@ void __init imx27_soc_init(void)
/* imx27 has the imx21 type audmux */
platform_device_register_simple(imx21-audmux, 0, imx27_audmux_res,
ARRAY_SIZE(imx27_audmux_res));
+   /* imx27 has an iram of 46080 bytes size */
+   iram_init(MX27_IRAM_BASE_ADDR, 46080);
 }
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html