[PATCH MC Next Gen v3 1/6] sound/usb: Fix media_stream_delete() to remove intf devnode

2015-11-10 Thread Shuah Khan
media_stream_delete() doesn't remove intf_devnode. Fix it to call media_devnode_remove() to remove the intf_devnode. Signed-off-by: Shuah Khan --- sound/usb/media.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/usb/media.c b/sound/usb/media.c index

[PATCH MC Next Gen v3 3/6] media: au0828 create link between ALSA Mixer and decoder

2015-11-10 Thread Shuah Khan
Change au0828_create_media_graph() to create pad link between MEDIA_ENT_F_AUDIO_MIXER entity and decoder's AU8522_PAD_AUDIO_OUT. With mixer entity now linked to decoder, change to link MEDIA_ENT_F_AUDIO_CAPTURE to mixer's source pad. Signed-off-by: Shuah Khan ---

[PATCH MC Next Gen v3 4/6] sound/usb: Fix media_stream_init() and media_stream_delete() error paths

2015-11-10 Thread Shuah Khan
Fix media_stream_init() to free memory in error paths. Fix media_stream_delete() to remove unnecessary set media_dev pointer to null before free mctl. Signed-off-by: Shuah Khan --- sound/usb/media.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff

[PATCH MC Next Gen v3 0/6] Add mixer and control interface media entities

2015-11-10 Thread Shuah Khan
This patch v3 series adds the following: A fix to media_stream_delete() to remove intf devnode. Add support for creating MEDIA_ENT_F_AUDIO_MIXER entity for each mixer and a MEDIA_INTF_T_ALSA_CONTROL control interface entity that links to mixer entities. MEDIA_INTF_T_ALSA_CONTROL entity

[PATCH MC Next Gen v3 5/6] media: Fix compile warning when CONFIG_MEDIA_CONTROLLER_DVB is disabled

2015-11-10 Thread Shuah Khan
Fix the following compile warning when CONFIG_MEDIA_CONTROLLER_DVB is disabled: drivers/media/dvb-core/dvb_frontend.c: In function 'dvb_frontend_open': drivers/media/dvb-core/dvb_frontend.c:2526:1: warning: label 'err2' defined but not used [-Wunused-label] err2: Signed-off-by: Shuah Khan

[PATCH MC Next Gen v3 2/6] sound/usb: Create media mixer function and control interface entities

2015-11-10 Thread Shuah Khan
Add support for creating MEDIA_ENT_F_AUDIO_MIXER entity for each mixer and a MEDIA_INTF_T_ALSA_CONTROL control interface entity that links to mixer entities. MEDIA_INTF_T_ALSA_CONTROL entity corresponds to the control device for the card. Signed-off-by: Shuah Khan ---

[PATCH MC Next Gen v3 6/6] media: au0828-core fix ignoring return value errors

2015-11-10 Thread Shuah Khan
Fix the ignoring return value errors found during kbuild bot compile test. drivers/media/usb/au0828/au0828-core.c:298:5: warning: ignoring return value of 'media_create_pad_link', declared with attribute warn_unused_result [-Wunused-result] drivers/media/usb/au0828/au0828-core.c:356:2: warning:

[PATCH 1/1] omap3isp: preview: Mark output buffer done first

2015-11-10 Thread Sakari Ailus
The sequence number counter is incremented on each output buffer, and that incremented value is used as the sequence number of that buffer. The input buffer sequence numbering is based just on reading the same counter. If the input buffer is marked done first, its sequence number ends up being

[patch 1/2] [media] av7110: don't allow negative volumes

2015-11-10 Thread Dan Carpenter
The issue here is that we there is a static checker warning because we have a user controlled volume setting and we cap the upper bound but we allow negative numbers. Negative volumes don't make sense, so let's make these variables unsigned. Signed-off-by: Dan Carpenter

Re: [PATCH 04/19] v4l: omap3isp: fix handling platform_get_irq result

2015-11-10 Thread Andrzej Hajda
On 11/10/2015 09:53 AM, Laurent Pinchart wrote: > Hi Andrzej, > > On Tuesday 10 November 2015 07:48:54 Andrzej Hajda wrote: >> On 11/09/2015 09:16 PM, Laurent Pinchart wrote: >>> On Thursday 24 September 2015 16:00:12 Andrzej Hajda wrote: The function can return negative value. The

[PATCH] v4l: omap_vout: Don't free buffers if they haven't been allocated

2015-11-10 Thread Laurent Pinchart
The VRFB buffers are freed when the device is closed even if they haven't been allocated by a call to VIDIOC_REQBUFS, resulting in a crash. Fix it by not trying to free buffers that are not allocated. Signed-off-by: Laurent Pinchart ---

Re: [PATCH 04/19] v4l: omap3isp: fix handling platform_get_irq result

2015-11-10 Thread Laurent Pinchart
Hi Andrzej, On Tuesday 10 November 2015 07:48:54 Andrzej Hajda wrote: > On 11/09/2015 09:16 PM, Laurent Pinchart wrote: > > On Thursday 24 September 2015 16:00:12 Andrzej Hajda wrote: > >> The function can return negative value. > >> > >> The problem has been detected using proposed semantic

Re: PCIe capture driver

2015-11-10 Thread Hans Verkuil
On 11/11/2015 07:04 AM, Ran Shalit wrote: > On Wed, Oct 28, 2015 at 12:50 AM, Hans Verkuil wrote: >> >> >> On 10/27/2015 22:56, Ran Shalit wrote: >>> On Tue, Oct 27, 2015 at 12:21 AM, Hans Verkuil wrote: On 10/27/2015 02:04, Ran Shalit

Re: PCIe capture driver

2015-11-10 Thread Ran Shalit
On Wed, Oct 28, 2015 at 12:50 AM, Hans Verkuil wrote: > > > On 10/27/2015 22:56, Ran Shalit wrote: >> On Tue, Oct 27, 2015 at 12:21 AM, Hans Verkuil wrote: >>> >>> >>> On 10/27/2015 02:04, Ran Shalit wrote: On Mon, Oct 26, 2015 at 1:46 PM, Steven Toth

[patch 2/2] [media] av7110: potential divide by zero

2015-11-10 Thread Dan Carpenter
"len" comes from dvb_video_ioctl() and there is a possibility that it is zero. We do a divide by len later in the function so that's not ok. Signed-off-by: Dan Carpenter diff --git a/drivers/media/pci/ttpci/av7110_av.c b/drivers/media/pci/ttpci/av7110_av.c index

sh_vou.c:undefined reference to `vb2_dma_contig_cleanup_ctx'

2015-11-10 Thread kbuild test robot
Hi Hans, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 3934bbc0446e4cae977a6efe82ce993d37434e53 commit: 57af3ad59d953f300a1fcb143e72d024aff73550 [media] sh-vou: convert to vb2 date: 4 months ago config:

[PATCH] media: adv7180: increase delay after reset to 5ms

2015-11-10 Thread Ulrich Hecht
Initialization of the ADV7180 chip fails on the Renesas R8A7790-based Lager board about 50% of the time. This patch resolves the issue by increasing the minimum delay after reset from 2 ms to 5 ms, following the recommendation in the ADV7180 datasheet: "Executing a software reset takes

cron job: media_tree daily build: ERRORS

2015-11-10 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 Nov 11 04:00:28 CET 2015 git branch: test git hash: 79f5b6ae960d380c829fb67d5dadcd1d025d2775 gcc