Re: [PATCH v5 13/13] media: coda: set up buffers to be sized as negotiated with s_fmt

2012-09-13 Thread javier Martin
--- Acked-by: Javier Martin javier.mar...@vista-silicon.com drivers/media/platform/coda.c | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index 0235f4e..0e0b4fe 100644 --- a/drivers/media/platform

Re: [PATCH v5 0/13] Initial i.MX5/CODA7 support for the CODA driver

2012-09-13 Thread javier Martin
+--- drivers/media/platform/coda.h | 30 ++- 3 files changed, 337 insertions(+), 118 deletions(-) -- 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

Re: [PATCH v5 0/13] Initial i.MX5/CODA7 support for the CODA driver

2012-09-13 Thread javier Martin
://patchwork.kernel.org/patch/1367011/ https://patchwork.kernel.org/patch/1363331/ https://patchwork.kernel.org/patch/1352551/ 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

Improving ov7670 sensor driver.

2012-09-13 Thread javier Martin
trivial. The reason of this e-mail is to discuss whether you find these solution suitable or not and, more important, whether you think the suggested changes could break existing drivers. Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005

Re: Improving ov7670 sensor driver.

2012-09-13 Thread javier Martin
Hi Hans, thank you for your response. On 13 September 2012 12:07, Hans Verkuil hverk...@xs4all.nl wrote: On Thu 13 September 2012 11:48:17 javier Martin wrote: Hi, our new i.MX27 based platform (Visstrim-SM20) uses an ov7675 sensor attached to the CSI interface. Apparently, this sensor

Re: Improving ov7670 sensor driver.

2012-09-13 Thread javier Martin
On 13 September 2012 13:00, Hans Verkuil hverk...@xs4all.nl wrote: On Thu 13 September 2012 12:47:53 javier Martin wrote: Hi Hans, thank you for your response. On 13 September 2012 12:07, Hans Verkuil hverk...@xs4all.nl wrote: On Thu 13 September 2012 11:48:17 javier Martin wrote: Hi

Re: [PATCH v5 0/13] Initial i.MX5/CODA7 support for the CODA driver

2012-09-13 Thread javier Martin
Hi Philipp, On 13 September 2012 13:24, Philipp Zabel p.za...@pengutronix.de wrote: Am Donnerstag, den 13.09.2012, 09:51 +0200 schrieb javier Martin: If you want to speed up the process and you have you could send a pull request to Mauro. Should I include the four patches below in the pull

Re: [PATCH v4 01/16] media: coda: firmware loading for 64-bit AXI bus width

2012-09-11 Thread javier Martin
); return; -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- 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

Re: [PATCH v4 02/16] media: coda: add i.MX53 / CODA7541 platform support

2012-09-11 Thread javier Martin
[] = { { .compatible = fsl,imx27-vpu, .data = coda_platform_ids[CODA_IMX27] }, + { .compatible = fsl,imx53-vpu, .data = coda_devdata[CODA_IMX53] }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(of, coda_dt_ids); -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- Javier

Re: [PATCH v4 03/16] media: coda: fix IRAM/AXI handling for i.MX53

2012-09-11 Thread javier Martin
+#define CODA7_CMD_SET_FRAME_AXI_OVL_ADDR 0x1a0 +#define CODA7_CMD_SET_FRAME_SOURCE_BUF_STRIDE 0x1a8 /* Encoder Header */ #define CODA_CMD_ENC_HEADER_CODE 0x180 -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- Javier Martin Vista Silicon S.L. CDTUC - FASE C

Re: [PATCH v4 04/16] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-09-11 Thread javier Martin
Reviewed-by: Javier Martin javier.mar...@vista-silicon.com Tested-by: Javier Martin javier.mar...@vista-silicon.com -- 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

Re: [PATCH v4 05/16] media: coda: ignore coda busy status in coda_job_ready

2012-09-11 Thread javier Martin
-v4l2_dev, -not ready: coda is still busy.\n); - return 0; - } - v4l2_dbg(1, coda_debug, ctx-dev-v4l2_dev, job ready\n); return 1; -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- Javier

Re: [PATCH v4 06/16] media: coda: keep track of active instances

2012-09-11 Thread javier Martin
) } spin_lock_init(dev-irqlock); + INIT_LIST_HEAD(dev-instances); dev-plat_dev = pdev; dev-clk_per = devm_clk_get(pdev-dev, per); -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- Javier Martin Vista Silicon S.L. CDTUC - FASE C

Re: [PATCH v4 07/16] media: coda: stop all queues in case of lockup

2012-09-11 Thread javier Martin
; dev-clk_per = devm_clk_get(pdev-dev, per); -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- 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

Re: [PATCH v4 08/16] media: coda: enable user pointer support

2012-09-11 Thread javier Martin
; dst_vq-buf_struct_size = sizeof(struct v4l2_m2m_buffer); dst_vq-ops = coda_qops; -- 1.7.10.4 Acked-by: Javier Martin javier.mar...@vista-silicon.com -- 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

Re: [PATCH v4 09/16] media: coda: wait for picture run completion in start/stop_streaming

2012-09-11 Thread javier Martin
); INIT_LIST_HEAD(dev-instances); INIT_DELAYED_WORK(dev-timeout, coda_timeout); + init_completion(dev-done); + complete(dev-done); dev-plat_dev = pdev; dev-clk_per = devm_clk_get(pdev-dev, per); -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista

Re: [PATCH v4 10/16] media: coda: fix sizeimage setting in try_fmt

2012-09-11 Thread javier Martin
, Setting format for type %d, wxh: %dx%d, fmt: %d\n, -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- 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

Re: [PATCH v4 11/16] media: coda: add horizontal / vertical flipping support

2012-09-11 Thread javier Martin
#define CODA_CMD_ENC_PIC_BB_START 0x198 #define CODA_CMD_ENC_PIC_BB_SIZE 0x19c -- 1.7.10.4 Tested-by: Javier Martin javier.mar...@vista-silicon.com -- 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

Re: [PATCH v4 12/16] media: coda: add byte size slice limit control

2012-09-11 Thread javier Martin
, V4L2_CID_MPEG_VIDEO_HEADER_MODE, V4L2_MPEG_VIDEO_HEADER_MODE_JOINED_WITH_1ST_FRAME, -- 1.7.10.4 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

Re: [PATCH v4 16/16] media: coda: support 1024 px height on CODA7, set max frame size to 1080p

2012-09-11 Thread javier Martin
-- 1.7.10.4 -- 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

Re: [PATCH v3] media: mx2_camera: Don't modify non volatile parameters in try_fmt.

2012-09-05 Thread javier Martin
On 5 September 2012 10:29, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Wed, 5 Sep 2012, Guennadi Liakhovetski wrote: Hi Javier On Mon, 3 Sep 2012, javier Martin wrote: Hi, Guennadi,did you pick this one? Wanted to do so, but I've applied this your patch with only

Re: [PATCH v3 03/16] media: coda: fix IRAM/AXI handling for i.MX53

2012-09-03 Thread javier Martin
+#define CODA7_CMD_SET_FRAME_AXI_OVL_ADDR 0x1a0 +#define CODA7_CMD_SET_FRAME_SOURCE_BUF_STRIDE 0x1a8 /* Encoder Header */ #define CODA_CMD_ENC_HEADER_CODE 0x180 -- 1.7.10.4 Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005

Re: [PATCH v3 04/16] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-09-03 Thread javier Martin
) - v4l2_m2m_buf_done(src_buf, VB2_BUF_STATE_DONE); - else - ctx-reference = tmp_buf; - + v4l2_m2m_buf_done(src_buf, VB2_BUF_STATE_DONE); v4l2_m2m_buf_done(dst_buf, VB2_BUF_STATE_DONE); ctx-gopcounter--; -- 1.7.10.4 -- Javier Martin Vista

Re: [PATCH v3 07/16] media: coda: stop all queues in case of lockup

2012-09-03 Thread javier Martin
= devm_clk_get(pdev-dev, per); -- 1.7.10.4 -- 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

Re: [PATCH v3 07/16] media: coda: stop all queues in case of lockup

2012-09-03 Thread javier Martin
-dev, per); -- 1.7.10.4 -- 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

Re: [PATCH v3] media: mx2_camera: Don't modify non volatile parameters in try_fmt.

2012-09-03 Thread javier Martin
Hi, Guennadi,did you pick this one? Regards. On 20 August 2012 11:49, Javier Martin javier.mar...@vista-silicon.com wrote: Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v2: - Add Signed-off-by line. --- drivers/media/video/mx2_camera.c |5 +++-- 1

Re: [PATCH 04/12] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-08-28 Thread javier Martin
; - + v4l2_m2m_buf_done(src_buf, VB2_BUF_STATE_DONE); v4l2_m2m_buf_done(dst_buf, VB2_BUF_STATE_DONE); ctx-gopcounter--; -- 1.7.10.4 Otherwise looks good. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005- Santander. Cantabria. Spain

Re: [PATCH 04/12] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-08-28 Thread javier Martin
On 28 August 2012 09:50, javier Martin javier.mar...@vista-silicon.com wrote: Hi Phillip, I see you are trying a cleaner approach for reference and reconstructed frames. On 24 August 2012 18:17, Philipp Zabel p.za...@pengutronix.de wrote: Some codecs running on CODA need internal

Re: [PATCH 0/12] Initial i.MX5/CODA7 support for the CODA driver

2012-08-28 Thread javier Martin
++ drivers/media/video/coda.h | 21 +--- 6 files changed, 305 insertions(+), 103 deletions(-) -- Javier Martin Vista

Re: [PATCH 03/12] coda: fix IRAM/AXI handling for i.MX53

2012-08-28 Thread javier Martin
On 28 August 2012 10:24, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, thank you for the comments, Am Montag, den 27.08.2012, 10:59 +0200 schrieb javier Martin: Hi Philipp, thank you for your patch. Please, find some comments below. On 24 August 2012 18:17, Philipp Zabel p.za

Re: [PATCH 04/12] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-08-28 Thread javier Martin
On 28 August 2012 10:40, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, Am Dienstag, den 28.08.2012, 09:50 +0200 schrieb javier Martin: Hi Phillip, I see you are trying a cleaner approach for reference and reconstructed frames. -#define CODA_OUTPUT_BUFS 4 -#define

Re: [PATCH 0/12] Initial i.MX5/CODA7 support for the CODA driver

2012-08-28 Thread javier Martin
[i + 1]); fwrite(data, sizeof(data), 1, f); } fclose(f); } On 28 August 2012 12:17, Gaƫtan Carlier gcem...@gmail.com wrote: Hi Javier, On 08/28/2012 10:10 AM, javier Martin wrote: Hi Philipp, in order to give you my ACK I need that patch 3 gets fixed and patches 3-10 are resent so

TerraTec Cinergy T PCIe Dual bug

2012-08-28 Thread Javier Marcet
and the debug1c has debug=1. I'd appreciate any help. -- Javier Marcet jmar...@gmail.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 03/12] coda: fix IRAM/AXI handling for i.MX53

2012-08-27 Thread javier Martin
*/ #define CODA_CMD_ENC_HEADER_CODE 0x180 -- 1.7.10.4 Regards. [1] http://lxr.free-electrons.com/source/arch/arm/plat-mxc/iram_alloc.c#L58 -- 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

Re: [PATCH 05/10] coda: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-24 Thread javier Martin
On 23 August 2012 15:08, Ezequiel Garcia elezegar...@gmail.com wrote: queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Javier Martin javier.mar...@vista

Re: [PATCH 07/10] mem2mem-emmaprp: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-24 Thread javier Martin
On 23 August 2012 15:08, Ezequiel Garcia elezegar...@gmail.com wrote: queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Javier Martin javier.mar...@vista

Re: [PATCH] coda: Add V4L2_CAP_VIDEO_M2M capability flag

2012-08-23 Thread javier Martin
: Javier Martin javier.mar...@vista-silicon.com Signed-off-by: Sylwester Nawrocki sylvester.nawro...@gmail.com --- drivers/media/platform/coda.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index

Re: [PATCH] m2m-deinterlace: Add V4L2_CAP_VIDEO_M2M capability flag

2012-08-23 Thread javier Martin
: Javier Martin javier.mar...@vista-silicon.com Signed-off-by: Sylwester Nawrocki sylvester.nawro...@gmail.com --- drivers/media/platform/m2m-deinterlace.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/m2m-deinterlace.c b/drivers/media

Re: [PATCH] media: coda: remove duplicated call of fh_to_ctx in vidioc_s_fmt_vid_out

2012-08-21 Thread javier Martin
); + ret = vidioc_s_fmt(ctx, f); if (ret) ctx-colorspace = f-fmt.pix.colorspace; -- 1.7.9.5 Good catch. Acked-by: Javier Martin javier.mar...@vista-silicon.com -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los Castros s/n 39005- Santander

Re: [v7] media: coda: Add driver for Coda video codec.

2012-08-20 Thread javier Martin
On 20 August 2012 05:03, Richard Zhao richard.z...@freescale.com wrote: Hi Javier, Did the patch get picked? I didn't see it on git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media.git It got merged 6 days ago in 'for_v3.7' branch of the linux-media tree: http://git.linuxtv.org

Re: [PATCH] media: mx2_camera: Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags.

2012-08-20 Thread javier Martin
Hi, On 30 July 2012 17:33, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Javier On Mon, 30 Jul 2012, javier Martin wrote: Hi, thank you for yor ACKs. On 20 July 2012 13:31, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Thu, 12 Jul 2012, Javier Martin wrote

[PATCH v3] media: mx2_camera: Don't modify non volatile parameters in try_fmt.

2012-08-20 Thread Javier Martin
Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v2: - Add Signed-off-by line. --- drivers/media/video/mx2_camera.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c index

Re: [v7] media: coda: Add driver for Coda video codec.

2012-08-03 Thread javier Martin
suggestions so that adding support for different chips later is as straightforward as possible. -- 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

Vacations.

2012-08-03 Thread javier Martin
. [PATCH 2/4] media: mx2_camera: Mark i.MX25 support as BROKEN. [PATCH 3/4] Schedule removal of i.MX25 support in mx2_camera.c [PATCH v2] media: mx2_camera: Fix clock handling for i.MX27. For Sascha: [PATCH v2] Visstrim SM10: Use mo_version to decide board video mode. Regards. -- Javier Martin

Re: [PULL] video_visstrim for 3.6

2012-08-01 Thread javier Martin
On 31 July 2012 15:05, Sascha Hauer s.ha...@pengutronix.de wrote: On Tue, Jul 31, 2012 at 08:13:59AM +0200, javier Martin wrote: On 31 July 2012 00:42, Mauro Carvalho Chehab mche...@redhat.com wrote: Em 26-07-2012 06:36, Javier Martin escreveu: Hi Mauro, this pull request is composed

[PATCH v2] media: mx2_camera: Fix clock handling for i.MX27.

2012-08-01 Thread Javier Martin
This driver wasn't converted to the new clock framework (e038ed50a4a767add205094c035b6943e7b30140). Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/video/mx2_camera.c | 47 +- 1 file changed, 31 insertions(+), 16 deletions

Re: [PULL] video_visstrim for 3.6

2012-07-31 Thread javier Martin
On 31 July 2012 00:42, Mauro Carvalho Chehab mche...@redhat.com wrote: Em 26-07-2012 06:36, Javier Martin escreveu: Hi Mauro, this pull request is composed of two series that provide support for two mem2mem devices: - 'm2m-deinterlace' video deinterlacer - 'coda video codec' I've included

Re: [PULL] video_visstrim for 3.6

2012-07-31 Thread javier Martin
we finally missed that deadline. 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

[PATCH] media: i.MX27: Fix mx2_emmaprp mem2mem driver clocks.

2012-07-30 Thread Javier Martin
This driver wasn't converted to the new clock framework (e038ed50a4a767add205094c035b6943e7b30140). Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- This is broken in current stable 3.5 too. So it should be applied to both stable and 3.6. --- drivers/media/video/mx2_emmaprp.c

Re: [PATCH] media: mx2_camera: Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags.

2012-07-30 Thread javier Martin
Hi, thank you for yor ACKs. On 20 July 2012 13:31, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Thu, 12 Jul 2012, Javier Martin wrote: These flags are not used any longer and can be safely removed since the following patch: http://www.spinics.net/lists/linux-media/msg50165.html

[PULL] video_visstrim for 3.6

2012-07-26 Thread Javier Martin
) Javier Martin (5): i.MX: coda: Add platform support for coda in i.MX27. media: coda: Add driver for Coda video codec. Visstrim M10: Add support for Coda. media: Add mem2mem deinterlacing driver. i.MX27

[PATCH 0/4] Fix clocks for i.MX27 in mx2_camera.c

2012-07-26 Thread Javier Martin
The following series follows the steps purposed by Guennadi (http://www.spinics.net/lists/arm-kernel/msg18.html) in order to fix clock handlings for i.MX27 in mx2_camera.c [PATCH 1/4] i.MX27: Fix emma-prp and csi clocks. [PATCH 2/4] media: mx2_camera: Mark i.MX25 support as BROKEN. [PATCH

[PATCH 1/4] i.MX27: Fix emma-prp and csi clocks.

2012-07-26 Thread Javier Martin
Naming of emma-prp related clocks for the i.MX27 is not correct. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- arch/arm/mach-imx/clk-imx27.c |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/arch/arm/mach-imx/clk-imx27.c b/arch/arm/mach-imx/clk

[PATCH 2/4] media: mx2_camera: Mark i.MX25 support as BROKEN.

2012-07-26 Thread Javier Martin
i.MX25 support is known to have been broken for almost a year. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/video/Kconfig |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/video/Kconfig b/drivers/media/video/Kconfig index

[PATCH 4/4] media: mx2_camera: Fix clock handling for i.MX27.

2012-07-26 Thread Javier Martin
This driver wasn't converted to the new clock framework (e038ed50a4a767add205094c035b6943e7b30140). Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/video/mx2_camera.c | 67 ++ 1 file changed, 39 insertions(+), 28 deletions

[PATCH 3/4] Schedule removal of i.MX25 support in mx2_camera.c

2012-07-26 Thread Javier Martin
Support for i.MX25 in mx2_camera.c has been broken for a year. Furthermore, i.MX25 video capture HW doesn't have much in common with i.MX27. A separate driver would be desirable. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Documentation/feature-removal-schedule.txt |9

[PATCH 1/2 v3] media: Add mem2mem deinterlacing driver.

2012-07-24 Thread Javier Martin
effect in the image but this can be accepted for some low quality applications. Currently only YUV420 and YUYV formats are supported but it can be extended later. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v2: - Passes all v4l2-compliance tests. - Field

Re: [PATCH v6] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
Hi Hans, On 21 July 2012 11:50, Hans Verkuil hverk...@xs4all.nl wrote: On Fri July 20 2012 13:08:35 Javier Martin wrote: Coda is a range of video codecs from ChipsMedia that support H.264, H.263, MPEG4 and other video standards. Currently only support for the codadx6 included in the i.MX27

Re: [PATCH] [v3] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-23 Thread javier Martin
On 20 July 2012 15:43, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Fri, 20 Jul 2012, javier Martin wrote: On 20 July 2012 13:19, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Javier Thanks for the patch On Mon, 9 Jul 2012, javier Martin wrote: On 9 July 2012 10

Re: [PATCH v6] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
On 23 July 2012 10:20, Hans Verkuil hverk...@xs4all.nl wrote: On Mon July 23 2012 10:02:04 javier Martin wrote: Hi Hans, On 21 July 2012 11:50, Hans Verkuil hverk...@xs4all.nl wrote: On Fri July 20 2012 13:08:35 Javier Martin wrote: Coda is a range of video codecs from ChipsMedia

Fwd: [PATCH v6] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
-- Forwarded message -- From: javier Martin javier.mar...@vista-silicon.com Date: 23 July 2012 10:39 Subject: Re: [PATCH v6] media: coda: Add driver for Coda video codec. To: Hans Verkuil hverk...@xs4all.nl On 23 July 2012 10:36, Hans Verkuil hverk...@xs4all.nl wrote: On Mon

Re: [PATCH v6] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
: fail: v4l2-test-buffers.cpp(111): doioctl(node-node2, VIDIOC_REQBUFS, bufs) != EBUSY test VIDIOC_REQBUFS/CREATE_BUFS: FAIL test read/write: OK Total: 34 Succeeded: 26 Failed: 8 Warnings: 0 -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de

Re: [PATCH v6] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
On 23 July 2012 11:45, javier Martin javier.mar...@vista-silicon.com wrote: Sorry, I had a problem with my buildroot environment. This is the v4l2-compliance output with the most recent version: # v4l2-compliance -d /dev/video2 Driver Info: Driver name : coda Card type

Re: [PATCH v6] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
On 23 July 2012 12:14, Hans Verkuil hverk...@xs4all.nl wrote: On Mon July 23 2012 12:00:30 javier Martin wrote: On 23 July 2012 11:45, javier Martin javier.mar...@vista-silicon.com wrote: Sorry, I had a problem with my buildroot environment. This is the v4l2-compliance output with the most

Re: [PATCH v7] media: coda: Add driver for Coda video codec.

2012-07-23 Thread javier Martin
On 23 July 2012 13:38, Hans Verkuil hverk...@xs4all.nl wrote: On Mon July 23 2012 13:31:01 Javier Martin wrote: Coda is a range of video codecs from ChipsMedia that support H.264, H.263, MPEG4 and other video standards. Currently only support for the codadx6 included in the i.MX27 SoC

Re: [PATCH] media: mx2_camera: Add YUYV output format.

2012-07-20 Thread javier Martin
On 12 July 2012 10:56, Javier Martin javier.mar...@vista-silicon.com wrote: Add explicit conversions from UYVY and YUYV to YUYV so that csicr1 configuration can be set properly for each format. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/video/mx2_camera.c

Re: [PATCH] [v3] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-20 Thread javier Martin
On 20 July 2012 13:19, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Javier Thanks for the patch On Mon, 9 Jul 2012, javier Martin wrote: On 9 July 2012 10:07, Sascha Hauer s.ha...@pengutronix.de wrote: On Mon, Jul 09, 2012 at 09:46:03AM +0200, javier Martin wrote: On 9 July

Re: [PATCH v3] media: coda: Add driver for Coda video codec.

2012-07-18 Thread javier Martin
Hi Pilipp, On 16 July 2012 19:21, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, I'm trying to use this driver (on i.MX53) with GStreamer, with two buffers on the encoded bitstream side. I'm getting negative bytesused values calculated in coda_isr(). The CODA_REG_BIT_WR_PTR address

Re: [PATCH v3] media: coda: Add driver for Coda video codec.

2012-07-18 Thread javier Martin
On 18 July 2012 10:35, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, Am Mittwoch, den 18.07.2012, 09:12 +0200 schrieb javier Martin: [...] I see there is a comment about the expected register setting not working for CODA_REG_BIT_STREAM_CTRL in start_streaming(). Could

Re: [PATCH v3] media: coda: Add driver for Coda video codec.

2012-07-18 Thread javier Martin
On 18 July 2012 11:22, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, Am Mittwoch, den 18.07.2012, 11:01 +0200 schrieb javier Martin: On 18 July 2012 10:35, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, Am Mittwoch, den 18.07.2012, 09:12 +0200 schrieb javier Martin

Re: [PATCH v3] media: coda: Add driver for Coda video codec.

2012-07-18 Thread javier Martin
On 18 July 2012 14:12, Philipp Zabel p.za...@pengutronix.de wrote: Am Mittwoch, den 18.07.2012, 11:26 +0200 schrieb javier Martin: On 18 July 2012 11:22, Philipp Zabel p.za...@pengutronix.de wrote: Hi Javier, Am Mittwoch, den 18.07.2012, 11:01 +0200 schrieb javier Martin: On 18 July 2012

Re: [PATCH v4] media: coda: Add driver for Coda video codec.

2012-07-18 Thread javier Martin
Hi Hans, thank you for your review. On 18 July 2012 13:00, Hans Verkuil hverk...@xs4all.nl wrote: Hi Javier! On Wed 18 July 2012 12:21:35 Javier Martin wrote: Coda is a range of video codecs from ChipsMedia that support H.264, H.263, MPEG4 and other video standards. Currently only support

[PATCH] media: mx2_camera: Add YUYV output format.

2012-07-12 Thread Javier Martin
Add explicit conversions from UYVY and YUYV to YUYV so that csicr1 configuration can be set properly for each format. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/video/mx2_camera.c | 40 ++ 1 file changed, 40 insertions

[PATCH] media: mx2_camera: Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags.

2012-07-12 Thread Javier Martin
These flags are not used any longer and can be safely removed since the following patch: http://www.spinics.net/lists/linux-media/msg50165.html Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- arch/arm/plat-mxc/include/mach/mx2_cam.h |2 -- 1 file changed, 2 deletions

[PATCH 1/2 v2] media: Add mem2mem deinterlacing driver.

2012-07-12 Thread Javier Martin
effect in the image but this can be accepted for some low quality applications. Currently YUV420 and YUYV formats are supported but can be extended later. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v1: - Added support for V4L2_FIELD_SEQ_BT

[PATCH 2/2 v2] i.MX27: Visstrim_M10: Add support for deinterlacing driver.

2012-07-12 Thread Javier Martin
Visstrim_M10 have a tvp5150 whose video output must be deinterlaced. The new mem2mem deinterlacing driver is very useful for that purpose. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v1: - Removed commented out code. --- arch/arm/mach-imx/mach

[PATCH v2] media: mx2_camera: Don't modify non volatile parameters in try_fmt.

2012-07-11 Thread Javier Martin
--- drivers/media/video/mx2_camera.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c index d5355de..eda98fc 100644 --- a/drivers/media/video/mx2_camera.c +++ b/drivers/media/video/mx2_camera.c @@

[PATCH v5] media: mx2_camera: Fix mbus format handling

2012-07-11 Thread Javier Martin
Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags so that the driver can negotiate with the attached sensor whether the mbus format needs convertion from UYUV to YUYV or not. --- drivers/media/video/mx2_camera.c | 28 +++- 1 file changed, 23 insertions(+), 5

[PATCH 0/2] Add video deinterlacing support.

2012-07-11 Thread Javier Martin
[PATCH 1/2] media: Add mem2mem deinterlacing driver. [PATCH 2/2] i.MX27: Visstrim_M10: Add support for deinterlacing -- 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

Re: [PATCH 1/2] media: Add mem2mem deinterlacing driver.

2012-07-11 Thread javier Martin
Hi Hans, thank you for your comments. On 11 July 2012 11:45, Hans Verkuil hverk...@xs4all.nl wrote: Hi Javier! Thanks for the patch. On Wed 11 July 2012 10:55:03 Javier Martin wrote: Some video decoders such as tvp5150 provide separate video fields (V4L2_FIELD_SEQ_TB). This driver uses

Re: [PATCH v5] media: mx2_camera: Fix mbus format handling

2012-07-11 Thread javier Martin
Hi, On 11 July 2012 12:08, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Javier, Thanks for the patch. On Wednesday 11 July 2012 09:56:49 Javier Martin wrote: Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags so that the driver can negotiate with the attached sensor

[PATCH v6] media: mx2_camera: Fix mbus format handling

2012-07-11 Thread Javier Martin
Do not use MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags. The driver must negotiate with the attached sensor whether the mbus format is UYUV or YUYV and set CSICR1 configuration accordingly. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/video/mx2_camera.c

[PATCH] media: mx2_camera: Don't modify non volatile parameters in try_fmt.

2012-07-10 Thread Javier Martin
--- drivers/media/video/mx2_camera.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/video/mx2_camera.c b/drivers/media/video/mx2_camera.c index d5355de..4a96989 100644 --- a/drivers/media/video/mx2_camera.c +++ b/drivers/media/video/mx2_camera.c @@ -1370,6

Re: [PATCH v3][for_v3.5] media: mx2_camera: Fix mbus format handling

2012-07-09 Thread javier Martin
On 6 July 2012 14:55, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Javier Thanks for the patch, and sorry for delay. I was away first 10 days of June and still haven't come round to cleaning up my todo list since then... On Fri, 1 Jun 2012, Javier Martin wrote: Remove

Re: [PATCH v3][for_v3.5] media: mx2_camera: Fix mbus format handling

2012-07-09 Thread javier Martin
Javier Thanks for the patch, and sorry for delay. I was away first 10 days of June and still haven't come round to cleaning up my todo list since then... On Fri, 1 Jun 2012, Javier Martin wrote: [snip] @@ -1024,14 +1039,28 @@ static int mx2_camera_set_bus_param(struct

[PATCH v4] media: mx2_camera: Fix mbus format handling

2012-07-09 Thread Javier Martin
Remove MX2_CAMERA_SWAP16 and MX2_CAMERA_PACK_DIR_MSB flags so that the driver can negotiate with the attached sensor whether the mbus format needs convertion from UYUV to YUYV or not. --- Changes since v3: Do not provide pass-through for UYVY8. --- arch/arm/plat-mxc/include/mach/mx2_cam.h |2

Re: [PATCH] [v3] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-09 Thread javier Martin
On 9 July 2012 09:28, Sascha Hauer s.ha...@pengutronix.de wrote: On Fri, Jul 06, 2012 at 12:56:02PM +0200, Javier Martin wrote: This driver wasn't converted to the new clock changes (clk_prepare_enable/clk_disable_unprepare). Also naming of emma-prp related clocks for the i.MX27

Re: [PATCH] [v3] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-09 Thread javier Martin
On 9 July 2012 09:43, Sascha Hauer s.ha...@pengutronix.de wrote: On Mon, Jul 09, 2012 at 09:37:25AM +0200, javier Martin wrote: On 9 July 2012 09:28, Sascha Hauer s.ha...@pengutronix.de wrote: On Fri, Jul 06, 2012 at 12:56:02PM +0200, Javier Martin wrote: This driver wasn't converted

Re: [PATCH 2/3] media: coda: Add driver for Coda video codec.

2012-07-09 Thread javier Martin
On 9 July 2012 10:19, Philipp Zabel p.za...@pengutronix.de wrote: Am Montag, den 09.07.2012, 10:07 +0200 schrieb javier Martin: [...] +static int vidioc_s_parm(struct file *file, void *priv, struct v4l2_streamparm *a) +{ + struct coda_ctx *ctx = fh_to_ctx(priv

[PATCH v2 3/3] Visstrim M10: Add support for Coda.

2012-07-09 Thread Javier Martin
Support the codadx6 that is included in the i.MX27 SoC. --- Changes since v1: - Use 'arm_memblock_steal'. --- arch/arm/mach-imx/mach-imx27_visstrim_m10.c | 24 1 file changed, 20 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-imx/mach-imx27_visstrim_m10.c

media: i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-06 Thread Javier Martin
This driver wasn't converted to the new clock changes (clk_prepare_enable/clk_disable_unprepare). Also naming of emma-prp related clocks for the i.MX27 was not correct. Signed-of-by: Javier Martin javier.mar...@vista-silicon.com --- diff --git a/arch/arm/mach-imx/clk-imx27.c b/arch/arm/mach-imx

Re: media: i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-06 Thread javier Martin
On 6 July 2012 08:43, Sascha Hauer s.ha...@pengutronix.de wrote: Hi Javier, On Fri, Jul 06, 2012 at 08:31:49AM +0200, Javier Martin wrote: This driver wasn't converted to the new clock changes (clk_prepare_enable/clk_disable_unprepare). Also naming of emma-prp related clocks for the i.MX27

[PATCH] [v2] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-06 Thread Javier Martin
This driver wasn't converted to the new clock changes (clk_prepare_enable/clk_disable_unprepare). Also naming of emma-prp related clocks for the i.MX27 was not correct. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- arch/arm/mach-imx/clk-imx27.c|8 --- drivers/media

Re: [PATCH] [v2] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-06 Thread javier Martin
On 6 July 2012 09:34, Sascha Hauer s.ha...@pengutronix.de wrote: On Fri, Jul 06, 2012 at 09:13:11AM +0200, Javier Martin wrote: This driver wasn't converted to the new clock changes (clk_prepare_enable/clk_disable_unprepare). Also naming of emma-prp related clocks for the i.MX27

[PATCH] [v3] i.MX27: Fix emma-prp clocks in mx2_camera.c

2012-07-06 Thread Javier Martin
This driver wasn't converted to the new clock changes (clk_prepare_enable/clk_disable_unprepare). Also naming of emma-prp related clocks for the i.MX27 was not correct. --- Enable clocks only for i.MX27. --- arch/arm/mach-imx/clk-imx27.c|8 +++-- drivers/media/video/mx2_camera.c | 67

Re: [PATCH v3][for_v3.5] media: mx2_camera: Fix mbus format handling

2012-07-06 Thread javier Martin
Hi, can this patch be applied please? It solves a BUG for 3.5. Guennadi, Fabio, could you give me an ack for this? 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

Re: [PATCH v3][for_v3.5] media: mx2_camera: Fix mbus format handling

2012-07-06 Thread javier Martin
On 6 July 2012 13:00, javier Martin javier.mar...@vista-silicon.com wrote: Hi, can this patch be applied please? It solves a BUG for 3.5. Guennadi, Fabio, could you give me an ack for this? Regards. But it should be applied after this one to preserve bisectability: http

Re: [PATCH v3][for_v3.5] media: mx2_camera: Fix mbus format handling

2012-07-06 Thread javier Martin
Hi Guennadi, On 6 July 2012 13:09, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Fri, 6 Jul 2012, javier Martin wrote: Hi, can this patch be applied please? It solves a BUG for 3.5. Guennadi, Fabio, could you give me an ack for this? Sorry? This patch has been applied and proven

Re: [PATCH v3][for_v3.5] media: mx2_camera: Fix mbus format handling

2012-07-06 Thread javier Martin
Hi Guennadi, On 6 July 2012 13:39, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Fri, 6 Jul 2012, javier Martin wrote: Hi Guennadi, On 6 July 2012 13:09, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Fri, 6 Jul 2012, javier Martin wrote: Hi, can this patch be applied

[PATCH 0/3] Add support for 'Coda' video codec IP.

2012-07-06 Thread Javier Martin
This patch provides support for the codadx6 that is inside the i.MX27. Currently only H.264 and MPEG4 Part-2 encoding operations are supported. This is a refined version of the RFC sent two weeks ago [1]. The following tasks have been addressed since then: - Get rid of 'runtime' structure. -

<    1   2   3   4   5   6   7   8   9   >