[PATCH 1/4] media: mx2_camera: Remove i.mx25 support.

2012-10-30 Thread Javier Martin
i.MX25 support has been broken for several releases now and nobody seems to care about it. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/soc_camera/mx2_camera.c | 405 ++-- 1 file changed, 100 insertions(+), 305 deletions(-) diff

[PATCH 2/4] media: mx2_camera: Add image size HW limits.

2012-10-30 Thread Javier Martin
The CSI has some constraints regarding image with. This patch makes sure those requirements are met in try_fmt(). Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/soc_camera/mx2_camera.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff

[PATCH 3/4] media: mx2_camera: Remove 'buf_cleanup' callback.

2012-10-30 Thread Javier Martin
All necessary tasks to end the streaming properly are already implemented in mx2_stop_streaming() and nothing remains to be done in this callback. Furthermore, it only included debug messages so it can be removed. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media

[PATCH 4/4] media: mx2_camera: Remove buffer states.

2012-10-30 Thread Javier Martin
After removing i.mx25 support and buf_cleanup() callback, buffer states are not used in the code any longer. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/soc_camera/mx2_camera.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers

Re: [PATCH 1/4] media: mx2_camera: Remove i.mx25 support.

2012-10-30 Thread javier Martin
Hi Guennadi, Fabio, On 30 October 2012 13:29, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Tue, 30 Oct 2012, Fabio Estevam wrote: Javier, On Tue, Oct 30, 2012 at 10:16 AM, Javier Martin javier.mar...@vista-silicon.com wrote: i.MX25 support has been broken for several releases

[PATCH v2 0/4] media: mx2_camera: Remove i.mx25 and clean up.

2012-10-30 Thread Javier Martin
Changes since v1: - Remove i.MX25 support in the Kconfig file too in patch 1. [PATCH v2 1/4] media: mx2_camera: Remove i.mx25 support. [PATCH v2 2/4] media: mx2_camera: Add image size HW limits. [PATCH v2 3/4] media: mx2_camera: Remove 'buf_cleanup' callback. [PATCH v2 4/4] media: mx2_camera:

[PATCH v2 1/4] media: mx2_camera: Remove i.mx25 support.

2012-10-30 Thread Javier Martin
i.MX25 support has been broken for several releases now and nobody seems to care about it. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/soc_camera/Kconfig |7 +- drivers/media/platform/soc_camera/mx2_camera.c | 405 ++-- 2

[PATCH v2 2/4] media: mx2_camera: Add image size HW limits.

2012-10-30 Thread Javier Martin
The CSI has some constraints regarding image with. This patch makes sure those requirements are met in try_fmt(). Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/soc_camera/mx2_camera.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff

[PATCH v2 3/4] media: mx2_camera: Remove 'buf_cleanup' callback.

2012-10-30 Thread Javier Martin
All necessary tasks to end the streaming properly are already implemented in mx2_stop_streaming() and nothing remains to be done in this callback. Furthermore, it only included debug messages so it can be removed. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media

[PATCH v2 4/4] media: mx2_camera: Remove buffer states.

2012-10-30 Thread Javier Martin
After removing i.mx25 support and buf_cleanup() callback, buffer states are not used in the code any longer. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/soc_camera/mx2_camera.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers

[PATCH] media: m2m-deinterlace: Do not set debugging flag to true.

2012-10-30 Thread Javier Martin
Default value should be 'debugging disabled'. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/m2m-deinterlace.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/m2m-deinterlace.c b/drivers/media/platform/m2m

Re: [PATCH 1/4] media: mx2_camera: Remove i.mx25 support.

2012-10-30 Thread javier Martin
On 30 October 2012 15:57, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Tue, 30 Oct 2012, javier Martin wrote: Hi Guennadi, Fabio, On 30 October 2012 13:29, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Tue, 30 Oct 2012, Fabio Estevam wrote: Javier, On Tue, Oct 30

[PATCH] media: ov7670: Allow 32x maximum gain for yuv422.

2012-10-30 Thread Javier Martin
4x gain ceiling is not enough to capture a decent image in conditions of total darkness and only a LED light source. Allow a maximum gain of 32x instead. This doesn't have any drawback since the image quality in 'normal' light conditions is the same. Signed-off-by: Javier Martin javier.mar

[PATCH] media: coda: Fix H.264 header alignment.

2012-10-29 Thread javier . martin
From: Javier Martin javier.mar...@vista-silicon.com Length of H.264 headers is variable and thus it might not be aligned for the coda to append the encoded frame. This causes the first frame to overwrite part of the H.264 PPS. In order to solve that, a filler NAL must be added between

[PATCH v2] media: coda: Fix H.264 header alignment.

2012-10-29 Thread javier . martin
From: Javier Martin javier.mar...@vista-silicon.com Length of H.264 headers is variable and thus it might not be aligned for the coda to append the encoded frame. This causes the first frame to overwrite part of the H.264 PPS. In order to solve that, a filler NAL must be added between

[PULL] video_visstrim staging/ov7670_for_v3.7

2012-10-29 Thread Javier Martin
callbacks. (2012-09-28 13:18:23 +0200) Javier Martin (9): media: ov7670: add support for ov7675. media: ov7670: make try_fmt() consistent with 'min_height' and 'min_width'. media: ov7670: calculate framerate properly

Re: [PATCH v2 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-10-26 Thread javier Martin
Hi Mauro, On 25 October 2012 18:18, Mauro Carvalho Chehab mche...@redhat.com wrote: Em Thu, 25 Oct 2012 15:24:49 +0200 javier Martin javier.mar...@vista-silicon.com escreveu: Hi Mauro, do you have any problems with this series? It didn't apply here, maybe due to its dependency from your

Re: [PATCH v3 0/4] ov7670: migrate this sensor and its users to ctrl framework.

2012-10-26 Thread javier Martin
Hi Mauro, On 25 October 2012 17:50, Mauro Carvalho Chehab mche...@redhat.com wrote: Hi Javier, Em Thu, 25 Oct 2012 15:27:08 +0200 javier Martin javier.mar...@vista-silicon.com escreveu: Hi Mauro, Jon, any more issues related with this series? Patch doesn't apply anymore: patching file

Re: [PATCH v2 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-10-25 Thread javier Martin
Hi Mauro, do you have any problems with this series? Regards. On 29 September 2012 21:20, Jonathan Corbet cor...@lwn.net wrote: On Thu, 27 Sep 2012 17:38:20 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: The following series includes all the changes discussed in [1] that don't

Re: [PATCH v3 0/4] ov7670: migrate this sensor and its users to ctrl framework.

2012-10-25 Thread javier Martin
Hi Mauro, Jon, any more issues related with this series? Regards. On 29 September 2012 21:25, Jonathan Corbet cor...@lwn.net wrote: On Fri, 28 Sep 2012 13:26:39 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: The following series migrate ov7670 sensor and current users to ctrl

Re: [PATCH v3 2/2] [media]: mx2_camera: Fix regression caused by clock conversion

2012-10-24 Thread javier Martin
. Does this mean, that this your patch is for i.MX25? But you're saying it's for i.MX27. Confused... I provide this mx27 clock in the first patch of the series: http://patchwork.linuxtv.org/patch/14915/ Yes, I made the same mistake. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C

Re: video: USB webcam fails since kernel 3.2

2012-10-10 Thread Martin-Éric Racine
2012/10/10 Jean-Francois Moine moin...@free.fr: On Tue, 9 Oct 2012 15:54:46 -0700 Jonathan Nieder jrnie...@gmail.com wrote: Hi, In June, Martin-Éric Racine wrote: Since recent kernels, this ASUS W5F's built-in webcam fails to be detected. Gstreamer-based applications (Cheese

Re: [PATCH] [media] coda: Do not use __cancel_delayed_work()

2012-10-10 Thread javier Martin
! -- tejun Thanks Fabio. 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 60 www.vista-silicon.com -- To unsubscribe from this list: send the line

Re: [PATCH v2 2/2] [media]: mx2_camera: Fix regression caused by clock conversion

2012-10-09 Thread javier Martin
-imx27.c;h=3b6b640eed247ea1b7848c7a7fa01801f0190cde;hb=b066f61482c7eac44e656499426a3c56d29c32ed#l226 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

Re: [PATCH v2 2/2] [media]: mx2_camera: Fix regression caused by clock conversion

2012-10-09 Thread javier Martin
823a mx2-camera mx2-camera.0: MX2 Camera (CSI) driver probed, clock frequency: 4433 Tested-by: Gaëtan Carlier gcem...@gmail.com Sorry I missed patch 1/2. Both patches are correct: Tested-by: Javier Martin javier.mar...@vista-silicon.com -- Javier Martin Vista Silicon S.L. CDTUC - FASE

Re: [PATCH 1/2] ARM: clk-imx27: Add missing clock for mx2-camera

2012-10-09 Thread javier Martin
-usb2-udc); -- 1.7.9.5 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: video: USB webcam fails since kernel 3.2

2012-10-09 Thread Martin-Éric Racine
2012/10/10 Jonathan Nieder jrnie...@gmail.com: Hi, In June, Martin-Éric Racine wrote: Since recent kernels, this ASUS W5F's built-in webcam fails to be detected. Gstreamer-based applications (Cheese, gstreamer-properties) immediately crash whenever trying to access the video device

Re: [PATCH 1/5] media: ov7670: add support for ov7675.

2012-10-08 Thread javier Martin
On 6 October 2012 17:19, Mauro Carvalho Chehab mche...@infradead.org wrote: Em Thu, 27 Sep 2012 08:58:33 +0200 javier Martin javier.mar...@vista-silicon.com escreveu: Hi Jonathan, thank you for your time. On 26 September 2012 18:40, Jonathan Corbet cor...@lwn.net wrote: This is going

Re: [PATCH 2/2] [media]: mx2_camera: Fix regression caused by clock conversion

2012-10-08 Thread javier Martin
. -- 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

[PATCH 1/2] em28xx: Better support for the Terratec Cinergy HTC USB XS. This intializes the card just like the windows driver does.

2012-10-04 Thread Martin Blumenstingl
Signed-off-by: Martin Blumenstingl martin.blumensti...@googlemail.com --- drivers/media/usb/em28xx/em28xx-cards.c | 13 - drivers/media/usb/em28xx/em28xx-dvb.c | 84 - drivers/media/usb/em28xx/em28xx.h | 1 + 3 files changed, 94 insertions(+), 4

[PATCH 2/2] drxk: Use the #define instead of hardcoded values.

2012-10-04 Thread Martin Blumenstingl
Signed-off-by: Martin Blumenstingl martin.blumensti...@googlemail.com --- drivers/media/dvb-frontends/drxk_hard.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb-frontends/drxk_hard.c b/drivers/media/dvb-frontends/drxk_hard.c index 8b4c6d5..894b6eca

Re: [PATCH 1/2] em28xx: Better support for the Terratec Cinergy HTC USB XS. This intializes the card just like the windows driver does.

2012-10-04 Thread Martin Blumenstingl
Stick HD back when I tested it). I tested with and without firmware - it was working fine in both cases. - the remote control should work fine now (also untested) Regards, Martin -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

Re: Current media_build doesn't succeed building on kernel 3.1.10

2012-10-01 Thread Martin Burnicki
Hi, Mauro Carvalho Chehab wrote: Em Sun, 30 Sep 2012 20:52:42 +0200 Martin Burnicki martin.burni...@burnicki.net escreveu: Hi all, is anybody out there who can help me with the media_build system? I'm trying to build the current modules on an openSUSE 12.1 system (kernel 3.1.10

Current media_build doesn't succeed building on kernel 3.1.10

2012-09-30 Thread Martin Burnicki
this might also work. BTW, if I understand the build environment correctly then there should be dayly test builds of the package for varions kernels. I'd expect those automated builds should also fail for kernels older than 3.4. Regards, Martin -- To unsubscribe from this list: send the line

[PATCH 0/3] ov7670: migrate this sensor and its users to ctrl framework.

2012-09-28 Thread Javier Martin
The following series migrate ov7670 sensor and current users to ctrl framework as discussed in [1]. This has been tested against mx2_camera soc-camera bridge, so tests or acks will be required from people using cam-core and via-camera out there. This will have to be applied on top of my previous

[PATCH 2/3] mcam-core: implement the control framework.

2012-09-28 Thread Javier Martin
Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/marvell-ccic/mcam-core.c | 54 --- drivers/media/platform/marvell-ccic/mcam-core.h |2 + 2 files changed, 11 insertions(+), 45 deletions

[PATCH 3/3] via-camera: implement the control framework.

2012-09-28 Thread Javier Martin
And added a missing kfree to clean up the via_camera struct. Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/via-camera.c | 60 --- 1 file changed, 14 insertions(+), 46

[PATCH 1/3] ov7670: use the control framework

2012-09-28 Thread Javier Martin
Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c | 295 +--- 1 file changed, 115 insertions(+), 180 deletions(-) diff --git a/drivers/media/i2c/ov7670.c b/drivers

Re: [PATCH 1/3] ov7670: use the control framework

2012-09-28 Thread javier Martin
Hi Hans, On 28 September 2012 10:23, Hans Verkuil hverk...@xs4all.nl wrote: On Fri September 28 2012 09:48:01 Javier Martin wrote: static const struct v4l2_subdev_core_ops ov7670_core_ops = { .g_chip_ident = ov7670_g_chip_ident, - .g_ctrl = ov7670_g_ctrl, - .s_ctrl

[PATCH v2 0/4] ov7670: migrate this sensor and its users to ctrl framework.

2012-09-28 Thread Javier Martin
The following series migrate ov7670 sensor and current users to ctrl framework as discussed in [1]. This has been tested against mx2_camera soc-camera bridge, so tests or acks will be required from people using cam-core and via-camera out there. This will have to be applied on top of my previous

[PATCH v2 1/4] ov7670: use the control framework

2012-09-28 Thread Javier Martin
Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v1: - Use v4l2_ctrl_auto_cluster() for auto_gain and auto_exp. --- drivers/media/i2c/ov7670.c | 310 1 file changed, 112

[PATCH v2 2/4] mcam-core: implement the control framework.

2012-09-28 Thread Javier Martin
Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/marvell-ccic/mcam-core.c | 54 --- drivers/media/platform/marvell-ccic/mcam-core.h |2 + 2 files changed, 11 insertions(+), 45 deletions

[PATCH v2 3/4] via-camera: implement the control framework.

2012-09-28 Thread Javier Martin
And added a missing kfree to clean up the via_camera struct. Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/via-camera.c | 60 --- 1 file changed, 14 insertions(+), 46

[PATCH v2 4/4] ov7670: remove legacy ctrl callbacks.

2012-09-28 Thread Javier Martin
via-camera and mcam-core were the only bridge drivers that used ov7670. Since now they have been moved to use the ctrl framework, the old legacy callbacks in the ov7670 can be removed. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c |7 --- 1

Re: [PATCH v2 1/4] ov7670: use the control framework

2012-09-28 Thread javier Martin
Hi Hans, On 28 September 2012 13:05, Hans Verkuil hverk...@xs4all.nl wrote: On Fri September 28 2012 12:50:55 Javier Martin wrote: Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v1: - Use v4l2_ctrl_auto_cluster

[PATCH v3 0/4] ov7670: migrate this sensor and its users to ctrl framework.

2012-09-28 Thread Javier Martin
The following series migrate ov7670 sensor and current users to ctrl framework as discussed in [1]. This has been tested against mx2_camera soc-camera bridge, so tests or acks will be required from people using cam-core and via-camera out there. This will have to be applied on top of my previous

[PATCH v3 1/4] ov7670: use the control framework

2012-09-28 Thread Javier Martin
Reviewed-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v2: - Do not use 'cur.val' to get gain value. --- drivers/media/i2c/ov7670.c | 310 1 file changed, 112 insertions

[PATCH v3 4/4] ov7670: remove legacy ctrl callbacks.

2012-09-28 Thread Javier Martin
via-camera and mcam-core were the only bridge drivers that used ov7670. Since now they have been moved to use the ctrl framework, the old legacy callbacks in the ov7670 can be removed. Reviewed-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Javier Martin javier.mar...@vista-silicon.com

[PATCH v3 3/4] via-camera: implement the control framework.

2012-09-28 Thread Javier Martin
And added a missing kfree to clean up the via_camera struct. Reviewed-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/via-camera.c | 60

[PATCH v3 2/4] mcam-core: implement the control framework.

2012-09-28 Thread Javier Martin
Reviewed-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Hans Verkuil hverk...@xs4all.nl Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/platform/marvell-ccic/mcam-core.c | 54 --- drivers/media/platform/marvell-ccic/mcam-core.h |2

Re: [PATCH 1/5] media: ov7670: add support for ov7675.

2012-09-27 Thread javier Martin
Hi Jonathan, thank you for your time. On 26 September 2012 18:40, Jonathan Corbet cor...@lwn.net wrote: This is going to have to be quick, sorry... On Wed, 26 Sep 2012 11:47:53 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: +static struct ov7670_win_size ov7670_win_sizes[2][4

Re: [PATCH 2/5] media: ov7670: make try_fmt() consistent with 'min_height' and 'min_width'.

2012-09-27 Thread javier Martin
On 26 September 2012 18:42, Jonathan Corbet cor...@lwn.net wrote: On Wed, 26 Sep 2012 11:47:54 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: 'min_height' and 'min_width' are variables that allow to specify the minimum resolution that the sensor will achieve. This patch make v4l2

Re: [PATCH 3/5] media: ov7670: calculate framerate properly for ov7675.

2012-09-27 Thread javier Martin
On 26 September 2012 18:50, Jonathan Corbet cor...@lwn.net wrote: On Wed, 26 Sep 2012 11:47:55 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: According to the datasheet ov7675 uses a formula to achieve the desired framerate that is different from the operations done

Re: [PATCH 4/5] media: ov7670: add possibility to bypass pll for ov7675.

2012-09-27 Thread javier Martin
On 26 September 2012 18:52, Jonathan Corbet cor...@lwn.net wrote: On Wed, 26 Sep 2012 11:47:56 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: Signed-off-by: Javier Martin javier.mar...@vista-silicon.com This one needs a changelog - what does bypassing the PLL do and why might

Re: [PATCH 5/5] media: ov7670: Add possibility to disable pixclk during hblank.

2012-09-27 Thread javier Martin
On 26 September 2012 18:52, Jonathan Corbet cor...@lwn.net wrote: On Wed, 26 Sep 2012 11:47:57 +0200 Javier Martin javier.mar...@vista-silicon.com wrote: Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c |8 include/media/ov7670.h

[PATCH v2 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-09-27 Thread Javier Martin
The following series includes all the changes discussed in [1] that don't affect either bridge drivers that use ov7670 or soc-camera framework For this reason they are considered non controversial and sent separately. At least 1 more series will follow in order to implement all features described

[PATCH v2 1/5] media: ov7670: add support for ov7675.

2012-09-27 Thread Javier Martin
ov7675 and ov7670 share the same registers but there is no way to distinguish them at runtime. However, they require different tweaks to achieve the desired resolution. For this reason this patch adds a new ov7675 entry to the ov7670_id table. Signed-off-by: Javier Martin javier.mar...@vista

[PATCH v2 3/5] media: ov7670: calculate framerate properly for ov7675.

2012-09-27 Thread Javier Martin
will be used for ov7675 only. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v1: - Create separate functions for frame rate control. --- drivers/media/i2c/ov7670.c | 136 ++-- 1 file changed, 118 insertions(+), 18 deletions

[PATCH v2 2/5] media: ov7670: make try_fmt() consistent with 'min_height' and 'min_width'.

2012-09-27 Thread Javier Martin
'min_height' and 'min_width' are variables that allow to specify the minimum resolution that the sensor will achieve. This patch make v4l2 fmt callbacks consider this parameters in order to return valid data to user space. Acked-by: Jonathan Corbet cor...@lwn.net Signed-off-by: Javier Martin

[PATCH v2 4/5] media: ov7670: add possibility to bypass pll for ov7675.

2012-09-27 Thread Javier Martin
-off-by: Javier Martin javier.mar...@vista-silicon.com --- Changes since v1: - Added changelog. --- drivers/media/i2c/ov7670.c | 28 ++-- include/media/ov7670.h |1 + 2 files changed, 27 insertions(+), 2 deletions(-) diff --git a/drivers/media/i2c/ov7670.c b

[PATCH v2 5/5] media: ov7670: Add possibility to disable pixclk during hblank.

2012-09-27 Thread Javier Martin
Some bridge drivers captures pixels during blanking periods if pixclk is enabled. In order to avoid capturing bogus data we need to disable pixclk in the sensor during those blanking periods. Acked-by: Jonathan Corbet cor...@lwn.net Signed-off-by: Javier Martin javier.mar...@vista-silicon.com

[PATCH 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-09-26 Thread Javier Martin
The following series includes all the changes discussed in [1] that don't affect either bridge drivers that use ov7670 or soc-camera framework For this reason they are considered non controversial and sent separately. At least 1 more series will follow in order to implement all features described

[PATCH 1/5] media: ov7670: add support for ov7675.

2012-09-26 Thread Javier Martin
ov7675 and ov7670 share the same registers but there is no way to distinguish them at runtime. However, they require different tweaks to achieve the desired resolution. For this reason this patch adds a new ov7675 entry to the ov7670_id table. Signed-off-by: Javier Martin javier.mar...@vista

[PATCH 2/5] media: ov7670: make try_fmt() consistent with 'min_height' and 'min_width'.

2012-09-26 Thread Javier Martin
'min_height' and 'min_width' are variables that allow to specify the minimum resolution that the sensor will achieve. This patch make v4l2 fmt callbacks consider this parameters in order to return valid data to user space. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers

[PATCH 4/5] media: ov7670: add possibility to bypass pll for ov7675.

2012-09-26 Thread Javier Martin
Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c | 24 ++-- include/media/ov7670.h |1 + 2 files changed, 23 insertions(+), 2 deletions(-) diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c index 175fbfc

[PATCH 3/5] media: ov7670: calculate framerate properly for ov7675.

2012-09-26 Thread Javier Martin
will be used for ov7675 only. Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c | 122 ++-- 1 file changed, 105 insertions(+), 17 deletions(-) diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c

[PATCH 5/5] media: ov7670: Add possibility to disable pixclk during hblank.

2012-09-26 Thread Javier Martin
Signed-off-by: Javier Martin javier.mar...@vista-silicon.com --- drivers/media/i2c/ov7670.c |8 include/media/ov7670.h |1 + 2 files changed, 9 insertions(+) diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c index 54fb535..f7e4341 100644 --- a/drivers

Re: [PATCH 0/5] media: ov7670: driver cleanup and support for ov7674.

2012-09-26 Thread javier Martin
On 26 September 2012 11:47, Javier Martin javier.mar...@vista-silicon.com wrote: The following series includes all the changes discussed in [1] that don't affect either bridge drivers that use ov7670 or soc-camera framework For this reason they are considered non controversial and sent

Re: [GIT PULL v2] Initial i.MX5/CODA7 support for the CODA driver

2012-09-20 Thread javier Martin
: http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/53627 The pull request is a way to tell Mauro this is ready to be merged in his linux-media tree and making things easier for him. Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de los

Re: [GIT PULL v2] Initial i.MX5/CODA7 support for the CODA driver

2012-09-20 Thread javier Martin
On 20 September 2012 10:03, Richard Zhao richard.z...@freescale.com wrote: On Thu, Sep 20, 2012 at 09:10:46AM +0200, javier Martin wrote: Hi Richard, On 20 September 2012 05:32, Richard Zhao richard.z...@freescale.com wrote: why is it a request-pull? After 5 version of Philipp's patches we

Re: [PATCH 27/34] media: mx2_camera: use managed functions to clean up code

2012-09-18 Thread javier Martin
Hi Shawn, On 18 September 2012 09:43, Shawn Guo shawn@linaro.org wrote: On Mon, Sep 17, 2012 at 03:36:07PM +0200, javier Martin wrote: This patch breaks the driver: Javier, Can you please apply the following change to see if it fixes the problem? Shawn @@ -1783,6 +1783,8 @@ static

Re: [PATCH 28/34] media: mx2_camera: remove mach/hardware.h inclusion

2012-09-18 Thread javier Martin
On 17 September 2012 15:59, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Mon, 17 Sep 2012, javier Martin wrote: Hi Shawn, On 17 September 2012 11:21, Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Mon, 17 Sep 2012, Shawn Guo wrote: It changes the driver to use

Re: [PATCH 27/34] media: mx2_camera: use managed functions to clean up code

2012-09-17 Thread javier Martin
+0x94/0x16c) from [c03c42f4] (kernel_init+0xe0/0x1ac) [c03c42f4] (kernel_init+0xe0/0x1ac) from [c0014900] (kernel_thread_exit+0x0/0x8) Code: bad PC value ---[ end trace 7f259a1ce2e10b1a ]--- Kernel panic - not syncing: Attempted to kill init! exitcode=0x000b -- Javier Martin Vista Silicon S.L

Re: [PATCH 26/34] media: mx2_camera: remove dead code in mx2_camera_add_device

2012-09-17 Thread javier Martin
Acked-by: Javier Martin javier.mar...@vista-silicon.com Acked-by: Guennadi Liakhovetski g.liakhovet...@gmx.de Thanks Guennadi --- drivers/media/video/mx2_camera.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/media/video/mx2_camera.c b/drivers/media

Re: [PATCH 28/34] media: mx2_camera: remove mach/hardware.h inclusion

2012-09-17 Thread javier Martin
, }, + .id_table = mx2_camera_devtype, .remove = __devexit_p(mx2_camera_remove), }; -- 1.7.9.5 I can't test this patch because it depends heavily on the previous one, which breaks the driver. Regards. -- Javier Martin Vista Silicon S.L. CDTUC - FASE C - Oficina S-345 Avda de

Re: Improving ov7670 sensor driver.

2012-09-14 Thread javier Martin
13-09-2012 06:48, javier Martin escreveu: Hi, our new i.MX27 based platform (Visstrim-SM20) uses an ov7675 sensor attached to the CSI interface. Apparently, this sensor is fully compatible with the old ov7670. For this reason, it seems rather sensible that they should share the same driver

Alignment problems: arm_memblock_steal() + dma_declare_coherent_memory()

2012-09-14 Thread javier Martin
requirements. 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

Re: [PATCH v5 12/13] media: coda: add byte size slice limit control

2012-09-13 Thread javier Martin
Hi Philipp, it now works properly. On 12 September 2012 17:02, Philipp Zabel p.za...@pengutronix.de wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- Changes since v4: - Fix menu_skip_mask for V4L2_CID_MPEG_VIDEO_MULTI_SLICE_MODE v4l2_ctrl. Tested-by: Javier Martin javier.mar

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

<    1   2   3   4   5   6   7   8   9   >