[PATCH 1/2] staging: media: bcm2048: radio-bcm2048.c - removed IRQF_DISABLED macro

2014-07-08 Thread Anil Belur
From: Anil Belur - this patch removes IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/stag

[PATCH 2/2] staging: nokia_h4p: nokia_core.c - removed IRQF_DISABLED macro

2014-07-08 Thread Anil Belur
From: Anil Belur - this patch removes the IRQF_DISABLED macro, as this is deprecated/noop. Signed-off-by: Anil Belur --- drivers/staging/nokia_h4p/nokia_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nokia_h4p/nokia_core.c b/drivers/staging/nokia_

Re: [PATCH 1/3] s5p-mfc: Fix selective sclk_mfc init

2014-07-08 Thread Arun Kumar K
Hi Jacek, On Tue, Jul 8, 2014 at 6:33 PM, Jacek Anaszewski wrote: > fc906b6d "Remove special clock usage in driver" removed > initialization of MFC special clock, arguing that there's > no need to do it explicitly, since it's one of MFC gate clock's > dependencies and gets enabled along with it.

cron job: media_tree daily build: OK

2014-07-08 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Wed Jul 9 04:00:21 CEST 2014 git branch: test git hash: 3c0d394ea7022bb9666d9df97a5776c4bcc3045c gcc versi

[PATCH 1/2] [media] V4L: Add camera pan/tilt speed controls

2014-07-08 Thread Vincent Palatin
The V4L2_CID_PAN_SPEED and V4L2_CID_TILT_SPEED controls allow to move the camera by setting its rotation speed around its axis. Signed-off-by: Vincent Palatin --- Documentation/DocBook/media/v4l/compat.xml | 10 ++ Documentation/DocBook/media/v4l/controls.xml | 21 +

[PATCH 2/2] V4L: uvcvideo: Add support for pan/tilt speed controls

2014-07-08 Thread Vincent Palatin
Map V4L2_CID_TILT_SPEED and V4L2_CID_PAN_SPEED to the standard UVC CT_PANTILT_RELATIVE_CONTROL terminal control request. Tested by plugging a Logitech ConferenceCam C3000e USB camera and controlling pan/tilt from the userspace using the VIDIOC_S_CTRL ioctl. Verified that it can pan and tilt at the

Re: [PATCH v2] V4L: uvcvideo: Add support for relative pan/tilt controls

2014-07-08 Thread Vincent Palatin
On Wed, Jun 25, 2014 at 2:54 AM, Laurent Pinchart wrote: > Hi Pawel, > > On Wednesday 25 June 2014 11:46:24 Pawel Osciak wrote: >> On Tue, Jun 17, 2014 at 11:45 PM, Vincent Palatin wrote: >> > Map V4L2_CID_TILT_RELATIVE and V4L2_CID_PAN_RELATIVE to the standard UVC >> > CT_PANTILT_RELATIVE_CONTROL

Re: fix PCTV 461e tuner I2C binding

2014-07-08 Thread Shuah Khan
On 07/08/2014 12:51 PM, Antti Palosaari wrote: On 07/08/2014 04:55 PM, Shuah Khan wrote: Moikka Antti, On 07/07/2014 09:45 PM, Antti Palosaari wrote: Moikka Shuah Why are we unregistering i2c devices and dvb in this resume path? Looks incorrect to me. I don't know. Original patch I send

Re: [PATCH v2 0/9] Updated fence patch series

2014-07-08 Thread Greg KH
On Wed, Jul 02, 2014 at 11:12:32AM +0200, Maarten Lankhorst wrote: > op 02-07-14 07:37, Greg KH schreef: > > On Tue, Jul 01, 2014 at 12:57:02PM +0200, Maarten Lankhorst wrote: > >> So after some more hacking I've moved dma-buf to its own subdirectory, > >> drivers/dma-buf and applied the fence patc

[PATCH] checkpatch: Warn on break after goto or return with same tab indentation

2014-07-08 Thread Joe Perches
Using break; after a goto or return is unnecessary so emit a warning when the break is at the same indent level. So this emits a warning on: switch (foo) { case 1: goto err; break; } but not on: switch (foo) { case 1:

Re: [PATCH 1/1] dvb-frontends: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
On Tue, 08 Jul 2014 21:35:58 +0300 Antti Palosaari wrote: > Moikka Fabian! > I have no reason to decline that patch (I will apply it) even it has > hardly meaning. But is there now some new tool which warns that kind of > issues? Hello Antti, Thanks :) AFAIK there's still no automatic

Re: fix PCTV 461e tuner I2C binding

2014-07-08 Thread Antti Palosaari
On 07/08/2014 04:55 PM, Shuah Khan wrote: Moikka Antti, On 07/07/2014 09:45 PM, Antti Palosaari wrote: Moikka Shuah Why are we unregistering i2c devices and dvb in this resume path? Looks incorrect to me. I don't know. Original patch I send was a bit different and tuner was removed only d

Re: [PATCH 1/1] dvb-frontends: remove unnecessary break after goto

2014-07-08 Thread Antti Palosaari
Moikka Fabian! I have no reason to decline that patch (I will apply it) even it has hardly meaning. But is there now some new tool which warns that kind of issues? regards Atnti On 07/08/2014 08:23 PM, Fabian Frederick wrote: Cc: Antti Palosaari Cc: Mauro Carvalho Chehab Cc: linux-media@v

[PATCH 1/1] xc5000: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/xc5000.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c index 2b3d514..f059ba2 100644 --- a/drivers/media/tuners/xc

[PATCH 1/1] xc4000: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/xc4000.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/xc4000.c b/drivers/media/tuners/xc4000.c index 2018bef..d2ee5df 100644 --- a/drivers/media/tuners/xc

Re: New v4l2 driver does not allow brightness/contrast control

2014-07-08 Thread Frank Schäfer
Am 07.07.2014 19:58, schrieb Rafael Coutinho: > I have a v4l2 video capture board that using kernel 2.6 with v4l2 > em28xx driver 3.0.36 allows me to control brightness, contrast etc... > However in kernel 3.2 with v4l2 em28xx driver version 3.2.0 it does not. > > This is what I get from the lates

Re: [PATCH v2 0/9] Updated fence patch series

2014-07-08 Thread Greg KH
On Tue, Jul 08, 2014 at 08:22:11PM +0530, Sumit Semwal wrote: > On 8 July 2014 20:09, Greg KH wrote: > > On Tue, Jul 08, 2014 at 03:44:27PM +0200, Daniel Vetter wrote: > >> On Mon, Jul 07, 2014 at 10:30:52AM -0700, Greg KH wrote: > >> > On Mon, Jul 07, 2014 at 03:23:17PM +0200, Daniel Vetter wrote

[PATCH 1/1] xc2028: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/tuner-xc2028.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/tuner-xc2028.c b/drivers/media/tuners/tuner-xc2028.c index 6ef93ee..565eeeb 100644 --- a/driv

[PATCH 1/1] dvb-frontends: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Antti Palosaari Cc: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/dvb-frontends/af9013.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/dvb-frontends/af9013.c b/drivers/media/dvb-frontends/af9013.c index fb504f1..e

[PATCH 1/1] r820t: remove unnecessary break after goto

2014-07-08 Thread Fabian Frederick
Cc: Mauro Carvalho Chehab Cc: Antti Palosaari Cc: Michael Krufky Cc: linux-media@vger.kernel.org Signed-off-by: Fabian Frederick --- drivers/media/tuners/r820t.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/tuners/r820t.c b/drivers/media/tuners/r820t.c index 96ccfeb..6906b1

[PATCH 6/8] videodev2.h: add V4L2_FIELD_HAS_T_OR_B macro

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil Add a macro to test if the field consists of a single top or bottom field. Signed-off-by: Hans Verkuil --- include/uapi/linux/videodev2.h | 4 1 file changed, 4 insertions(+) diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h index 168ff50..6d

[PATCH 8/8] vivi.txt: add a document describing the features of this driver.

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil The vivi driver emulates various types of video capture hardware and is ideal for testing applications. This document describes in detail all the features that this driver implements. Signed-off-by: Hans Verkuil --- Documentation/video4linux/vivi.txt | 460 ++

[PATCH 5/8] v4l2-dev: don't debug poll unless the debug level > 2

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil Some applications poll a lot, so prevent the poll message from flooding the log. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-dev.c b/drivers/media/v4l2-co

[PATCH 1/8] v4l2-ioctl: call g_selection before calling cropcap

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil If the vidioc_cropcap op is implemented by the driver then the v4l2 core will call that directly. If g_selection is available, then the core cropcap implementation uses g_selection to fill in the bounds and defrect and it sets the pixelaspect to 1x1. But if both are available

[PATCH 4/8] v4l2-ioctl: remove pointless INFO_FL_CLEAR.

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil The edid field is the last field of the struct, so there is nothing to clear. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c b/drivers/media/v4

[PATCH 3/8] v4l2-ioctl: clear reserved field of G/S_SELECTION.

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c b/drivers/media/v4l2-core/v4l2-ioctl.c index 0e90349..ede9b03 100644 --- a/drivers/media/v4l2-cor

[PATCH 2/8] v4l2-ioctl: clips, clipcount and bitmap should not be zeroed.

2014-07-08 Thread Hans Verkuil
From: Hans Verkuil Otherwise you cannot get the current clip and bitmap information from an overlay. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-

[PATCH 0/8] vivi, the next generation

2014-07-08 Thread Hans Verkuil
Hi all, This patch series replaces the current vivi driver by an almost completely rewritten version that provides a huge number of features. By default the driver has four inputs: Input 0 emulates a webcam, input 1 emulates a TV capture device, input 2 emulates an S-Video capture device and in

Re: [PATCH 3/4] media: rcar_vin: Fix race condition terminating stream

2014-07-08 Thread Sergei Shtylyov
Hello. On 07/08/2014 01:41 PM, Ian Molton wrote: This patch fixes a race condition whereby a frame being captured may generate an interrupt between requesting capture to halt and freeing buffers. This condition is exposed by the earlier patch that explicitly calls vb2_buffer_done() during

Re: [PATCH] [media] davinci-vpfe: Fix retcode check

2014-07-08 Thread Prabhakar Lad
Hi Andrey, Thanks for the patch! On Tue, Jul 8, 2014 at 3:58 PM, Andrey Utkin wrote: > 2014-07-08 17:32 GMT+03:00 Levente Kurusa : >> Hmm, while it is true that get_ipipe_mode returns an int, but >> the consequent call to regw_ip takes an u32 as its second >> argument. Did it cause a build warni

[PATCH 1/2] solo6x10: expose encoder quantization setting as V4L2 control

2014-07-08 Thread Andrey Utkin
solo6*10 boards have configurable quantization parameter which takes values from 0 to 31, inclusively. This change enables setting it with ioctl VIDIOC_S_CTRL with id V4L2_CID_MPEG_VIDEO_H264_MIN_QP. Signed-off-by: Andrey Utkin --- drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c | 5 + 1

[PATCH 2/2] solo6x10: update GOP size, QP immediately

2014-07-08 Thread Andrey Utkin
Previously, it was needed to reopen device to update GOP size and quantization parameter. Now we update device registers with new values immediately. Signed-off-by: Andrey Utkin --- drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers

Re: [PATCH 1/2] solo6x10: expose encoder quantization setting as V4L2 control

2014-07-08 Thread Dan Carpenter
On Tue, Jul 08, 2014 at 05:30:33PM +0300, Andrey Utkin wrote: > solo6*10 boards have configurable quantization parameter which takes > values from 0 to 31, inclusively. > > This change enables setting it with ioctl VIDIOC_S_CTRL with id > V4L2_CID_MPEG_VIDEO_H264_MIN_QP. Both of these two need si

Re: [PATCH 1/3] s5p-mfc: Fix selective sclk_mfc init

2014-07-08 Thread Sachin Kamat
Hi Jacek, On Tue, Jul 8, 2014 at 6:33 PM, Jacek Anaszewski wrote: > fc906b6d "Remove special clock usage in driver" removed > initialization of MFC special clock, arguing that there's > no need to do it explicitly, since it's one of MFC gate clock's > dependencies and gets enabled along with it.

DVB-T Freecom 14aa:0160 don't work

2014-07-08 Thread Jose
Sorry I'm mistake sending this bug, and is assigned to a bad mail. https://bugzilla.kernel.org/show_bug.cgi?id=79121 -- 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/majord

Re: [PATCH] [media] davinci-vpfe: Fix retcode check

2014-07-08 Thread Dan Carpenter
On Tue, Jul 08, 2014 at 04:32:57PM +0200, Levente Kurusa wrote: > 2014-07-08 16:08 GMT+02:00 Andrey Utkin : > > Use signed type to check correctly for negative error code. The issue > > was reported with static analyser: > > > > [linux-3.13/drivers/staging/media/davinci_vpfe/dm365_ipipe_hw.c:270]:

Re: [PATCH] [media] davinci-vpfe: Fix retcode check

2014-07-08 Thread Andrey Utkin
2014-07-08 17:32 GMT+03:00 Levente Kurusa : > Hmm, while it is true that get_ipipe_mode returns an int, but > the consequent call to regw_ip takes an u32 as its second > argument. Did it cause a build warning for you? (Can't really > check since I don't have ARM cross compilers close-by) > If not,

Re: [PATCH v2 0/9] Updated fence patch series

2014-07-08 Thread Sumit Semwal
On 8 July 2014 20:09, Greg KH wrote: > On Tue, Jul 08, 2014 at 03:44:27PM +0200, Daniel Vetter wrote: >> On Mon, Jul 07, 2014 at 10:30:52AM -0700, Greg KH wrote: >> > On Mon, Jul 07, 2014 at 03:23:17PM +0200, Daniel Vetter wrote: >> > > On Wed, Jul 2, 2014 at 7:37 AM, Greg KH >> > > wrote: >> >

Re: [PATCH v2 0/9] Updated fence patch series

2014-07-08 Thread Greg KH
On Tue, Jul 08, 2014 at 03:44:27PM +0200, Daniel Vetter wrote: > On Mon, Jul 07, 2014 at 10:30:52AM -0700, Greg KH wrote: > > On Mon, Jul 07, 2014 at 03:23:17PM +0200, Daniel Vetter wrote: > > > On Wed, Jul 2, 2014 at 7:37 AM, Greg KH > > > wrote: > > > >> Android can expose fences to userspace.

Re: [PATCH] [media] davinci-vpfe: Fix retcode check

2014-07-08 Thread Levente Kurusa
2014-07-08 16:08 GMT+02:00 Andrey Utkin : > Use signed type to check correctly for negative error code. The issue > was reported with static analyser: > > [linux-3.13/drivers/staging/media/davinci_vpfe/dm365_ipipe_hw.c:270]: > (style) A pointer can not be negative so it is either pointless or an >

[PATCH 2/2] solo6x10: update GOP size, QP immediately

2014-07-08 Thread Andrey Utkin
Previously, it was needed to reopen device to update GOP size and quantization parameter. Now we update device registers with new values immediately. --- drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/media/solo6x10/solo6x

[PATCH 1/2] solo6x10: expose encoder quantization setting as V4L2 control

2014-07-08 Thread Andrey Utkin
solo6*10 boards have configurable quantization parameter which takes values from 0 to 31, inclusively. This change enables setting it with ioctl VIDIOC_S_CTRL with id V4L2_CID_MPEG_VIDEO_H264_MIN_QP. --- drivers/staging/media/solo6x10/solo6x10-v4l2-enc.c | 5 + 1 file changed, 5 insertions(+)

[PATCH] [media] davinci-vpfe: Fix retcode check

2014-07-08 Thread Andrey Utkin
Use signed type to check correctly for negative error code. The issue was reported with static analyser: [linux-3.13/drivers/staging/media/davinci_vpfe/dm365_ipipe_hw.c:270]: (style) A pointer can not be negative so it is either pointless or an error to check if it is. Bugzilla: https://bugzilla.

Re: fix PCTV 461e tuner I2C binding

2014-07-08 Thread Shuah Khan
Moikka Antti, On 07/07/2014 09:45 PM, Antti Palosaari wrote: Moikka Shuah Why are we unregistering i2c devices and dvb in this resume path? Looks incorrect to me. I don't know. Original patch I send was a bit different and tuner was removed only during em28xx_dvb_fini() https://patchwork.

Re: [PATCH v2 0/9] Updated fence patch series

2014-07-08 Thread Daniel Vetter
On Mon, Jul 07, 2014 at 10:30:52AM -0700, Greg KH wrote: > On Mon, Jul 07, 2014 at 03:23:17PM +0200, Daniel Vetter wrote: > > On Wed, Jul 2, 2014 at 7:37 AM, Greg KH wrote: > > >> Android can expose fences to userspace. It's possible to make the new > > >> fence > > >> mechanism expose the same f

[PATCH 3/3] Documentation: devicetree: Document exynos3250 SoC related settings

2014-07-08 Thread Jacek Anaszewski
Signed-off-by: Jacek Anaszewski Signed-off-by: Kyungmin Park Cc: Rob Herring Cc: Pawel Moll Cc: Mark Rutland Cc: Ian Campbell Cc: Kumar Gala Cc: devicet...@vger.kernel.org --- .../devicetree/bindings/media/s5p-mfc.txt | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(

[PATCH 2/3] ARM: dts: exynos3250 add MFC codec device node

2014-07-08 Thread Jacek Anaszewski
Signed-off-by: Jacek Anaszewski Signed-off-by: Kyungmin Park Cc: Rob Herring Cc: Pawel Moll Cc: Mark Rutland Cc: Ian Campbell Cc: Kumar Gala Cc: devicet...@vger.kernel.org --- arch/arm/boot/dts/exynos3250.dtsi | 11 +++ 1 file changed, 11 insertions(+) diff --git a/arch/arm/boot/

[PATCH 1/3] s5p-mfc: Fix selective sclk_mfc init

2014-07-08 Thread Jacek Anaszewski
fc906b6d "Remove special clock usage in driver" removed initialization of MFC special clock, arguing that there's no need to do it explicitly, since it's one of MFC gate clock's dependencies and gets enabled along with it. However, there's no promise of keeping this hierarchy across Exynos SoC rele

[PATCH 0/3] Add support for Exynos3250 SoC to s5p-mfc driver

2014-07-08 Thread Jacek Anaszewski
This patch series adds support for MFC codec on Exynos3250 to the s5p-mfc driver. Thanks, Jacek Anaszewski Jacek Anaszewski (3): s5p-mfc: Fix selective sclk_mfc init ARM: dts: exynos3250 add MFC codec device node Documentation: devicetree: Document exynos3250 SoC related settings .../devi

RE: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME

2014-07-08 Thread Kamil Debski
Hi Hans, > From: Hans Verkuil [mailto:hverk...@xs4all.nl] > Sent: Tuesday, July 08, 2014 9:52 AM > To: Divneil Wadhawan; linux-media@vger.kernel.org; Kamil Debski; > pa...@osciak.com > Subject: Re: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME > > On 07/07/2014 09:27 AM, Divneil Wadhawa

[PATCH 2/2] libv4lcontrol: sync control strings/flags with the kernel

2014-07-08 Thread Hans Verkuil
Hans, I'd like your opinion on this. I really don't think the (sw) suffix serves any purpose and is just confusing to the end-user. If you think that it is important that apps/users know that a control is emulated, then I would propose adding a V4L2_CTRL_FLAG_EMULATED and setting it in libv4lcon

[PATCH 1/2] libv4lconvert: add support for extended controls

2014-07-08 Thread Hans Verkuil
libv4lconvert did not support the extended control API so qv4l2, which uses it, didn't work properly with libv4l2 since passing software emulated controls using the extended control API will fail as those controls are obviously not known to the driver. This patch adds support for this API. Signed

[PATCH] Fix typo in comments

2014-07-08 Thread Raimonds Cicans
Expression ((7bit i2c_addr << 1) & 0x01) can not be right because it is always 0 Signed-off-by: Raimonds Cicans --- drivers/media/usb/dvb-usb/dibusb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/dibusb.h b/drivers/media/usb/dvb-usb/dibusb.h ind

[PATCH 1/4] media: rcar_vin: Dont aggressively retire buffers

2014-07-08 Thread Ian Molton
rcar_vin_videobuf_release() is called once per buffer from the buf_cleanup hook. There is no need to look up the queue and free all buffers at this point. Signed-off-by: Ian Molton Signed-off-by: William Towle --- drivers/media/platform/soc_camera/rcar_vin.c | 12 +++- 1 file changed,

[PATCH 3/4] media: rcar_vin: Fix race condition terminating stream

2014-07-08 Thread Ian Molton
This patch fixes a race condition whereby a frame being captured may generate an interrupt between requesting capture to halt and freeing buffers. This condition is exposed by the earlier patch that explicitly calls vb2_buffer_done() during stop streaming. The solution is to wait for capture to

[PATCH 4/4] media: rcar_vin: Clean up rcar_vin_irq

2014-07-08 Thread Ian Molton
This patch makes the rcar_vin IRQ handler a little more readable. Removes an else clause, and simplifies the buffer handling. Signed-off-by: Ian Molton Reviewed-by: William Towle --- drivers/media/platform/soc_camera/rcar_vin.c | 24 ++-- 1 file changed, 14 insertions(+), 1

Resend: [PATCH 0/4] rcar_vin: fix soc_camera WARN_ON() issues.

2014-07-08 Thread Ian Molton
Resent to include the author and a couple of other interested parties :) This patch series provides fixes that allow the rcar_vin driver to function without triggering dozens of warnings from the videobuf2 and soc_camera layers. Patches 2/3 should probably be merged into a single, atomic change,

[PATCH 2/4] media: rcar_vin: Ensure all in-flight buffers are returned to error state before stopping.

2014-07-08 Thread Ian Molton
Videobuf2 complains about buffers that are still marked ACTIVE (in use by the driver) following a call to stop_streaming(). This patch returns all active buffers to state ERROR prior to stopping. Note: this introduces a (non fatal) race condition as the stream is not guaranteed to be stopped at

Re: vb2_reqbufs() is not allowing more than VIDEO_MAX_FRAME

2014-07-08 Thread Hans Verkuil
On 07/07/2014 09:27 AM, Divneil Wadhawan wrote: > Hi Hans, > > >> include/media/davinci/vpfe_capture.h > > It uses videobuf-dma-contig.h, so, I left it out. > > >> drivers/media/platform/vivi-core.c > > Cannot find this one. Checked with find, in case it changed location, but > couldn't. >