[PATCH v2 1/2] ARM: mx27_3ds: Add VPU support

[PATCH v2 1/2] ARM: mx27_3ds: Add VPU support

[Fabio Estevam]

mx27 has a VPU (Video Processing Unit) block that allows doing H264/MPEG4
decoding and encoding in hardware.

Add support for it.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
Changes since v1:
- Do not allocate the coda buffers explicetely

 arch/arm/mach-imx/mach-mx27_3ds.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-imx/mach-mx27_3ds.c b/arch/arm/mach-imx/mach-mx27_3ds.c
index d0e547f..e37a114 100644
--- a/arch/arm/mach-imx/mach-mx27_3ds.c
+++ b/arch/arm/mach-imx/mach-mx27_3ds.c
@@ -531,6 +531,7 @@ static void __init mx27pdk_init(void)
 	imx27_add_imx_ssi(0, &mx27_3ds_ssi_pdata);
 
 	imx_add_platform_device("imx_mc13783", 0, NULL, 0, NULL, 0);
+	imx27_add_coda();
 }
 
 static void __init mx27pdk_timer_init(void)
-- 
1.7.9.5

[PATCH v2 2/2] ARM: imx_v4_v5_defconfig: Add VPU support

[Fabio Estevam]

Let the VPU driver be built by default.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/configs/imx_v4_v5_defconfig |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/configs/imx_v4_v5_defconfig b/arch/arm/configs/imx_v4_v5_defconfig
index f71302c..1190c18 100644
--- a/arch/arm/configs/imx_v4_v5_defconfig
+++ b/arch/arm/configs/imx_v4_v5_defconfig
@@ -126,6 +126,9 @@ CONFIG_V4L_PLATFORM_DRIVERS=y
 CONFIG_SOC_CAMERA=y
 CONFIG_SOC_CAMERA_OV2640=y
 CONFIG_VIDEO_MX2=y
+CONFIG_V4L_PLATFORM_DRIVERS=y
+CONFIG_V4L_MEM2MEM_DRIVERS=y
+CONFIG_VIDEO_CODA=y
 CONFIG_FB=y
 CONFIG_FB_IMX=y
 CONFIG_BACKLIGHT_LCD_SUPPORT=y
-- 
1.7.9.5

[PATCH v2 1/2] ARM: mx27_3ds: Add VPU support

[Sascha Hauer]

Hi Fabio,

On Wed, Nov 14, 2012 at 11:19:33AM -0200, Fabio Estevam wrote:
> mx27 has a VPU (Video Processing Unit) block that allows doing H264/MPEG4
> decoding and encoding in hardware.
> 
> Add support for it.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> Changes since v1:
> - Do not allocate the coda buffers explicetely
> 
>  arch/arm/mach-imx/mach-mx27_3ds.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/mach-imx/mach-mx27_3ds.c b/arch/arm/mach-imx/mach-mx27_3ds.c
> index d0e547f..e37a114 100644
> --- a/arch/arm/mach-imx/mach-mx27_3ds.c
> +++ b/arch/arm/mach-imx/mach-mx27_3ds.c
> @@ -531,6 +531,7 @@ static void __init mx27pdk_init(void)
>  	imx27_add_imx_ssi(0, &mx27_3ds_ssi_pdata);
>  
>  	imx_add_platform_device("imx_mc13783", 0, NULL, 0, NULL, 0);
> +	imx27_add_coda();

Why in the board? It's always present so I would register it in the SoC.

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-5555 |

[PATCH v2 1/2] ARM: mx27_3ds: Add VPU support

[Eric Bénard]

Hi Sascha,

Le Wed, 14 Nov 2012 15:34:00 +0100,
Sascha Hauer <s.hauer@pengutronix.de> a ?crit :
> On Wed, Nov 14, 2012 at 11:19:33AM -0200, Fabio Estevam wrote:
> > mx27 has a VPU (Video Processing Unit) block that allows doing H264/MPEG4
> > decoding and encoding in hardware.
> > 
> > Add support for it.
> > 
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > ---
> > +	imx27_add_coda();
> 
> Why in the board? It's always present so I would register it in the SoC.
> 
MCIMX27L* dosn't have the VPU. No idea how will behave the probe in
that case.

Eruc