cron job: media_tree daily build: WARNINGS

2015-07-20 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: Tue Jul 21 04:00:21 CEST 2015 git branch: test git hash: 6727d4fce95586e60922bdaf57b8a0eb99482557 gcc versi

[PATCH v5 1/3] V4L2: platform: Add Renesas R-Car JPEG codec driver

2015-07-20 Thread Mikhail Ulyanov
Here's the driver for the Renesas R-Car JPEG processing unit. The driver is implemented within the V4L2 framework as a memory-to-memory device. It presents two video nodes to userspace, one for the encoding part, and one for the decoding part. It was found that the only working mode for encoding

[PATCH v3 2/3] devicetree: bindings: Document Renesas R-Car JPEG Processing Unit

2015-07-20 Thread Mikhail Ulyanov
Add Renesas R-Car JPEG processing unit driver device tree bindings documentation. Signed-off-by: Mikhail Ulyanov --- Changes since v2: - remove generic "renesas,jpu-gen2" descriptor Changes since v1: - Fix typos .../devicetree/bindings/media/renesas,jpu.txt | 24 ++

[PATCH 3/3] MAINTAINERS: V4L2: PLATFORM: Add entry for Renesas JPEG Processing Unit driver

2015-07-20 Thread Mikhail Ulyanov
Update RENESAS JPU driver maintainer in MAINTAINERS file. Signed-off-by: Mikhail Ulyanov --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index b65b22b..da57ec1 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5381,6 +5381,12 @@ S: Maintaine

[PATCH 0/3] R-Car JPEG Processing Unit

2015-07-20 Thread Mikhail Ulyanov
This series of patches contains a driver for the JPEG codec integrated peripheral found in the Renesas R-Car SoCs and associated DT documentation. This series of patches is against the 'master' branch of linuxtv.org/media_tree.git v4l2-compliance -s Driver Info: Driver name : rcar_jpu

[ezequielg:tw68_ids_upstream_v2 82/109] warning: (VIDEO_COBALT) selects VIDEO_ADV7604 which has unmet direct dependencies (MEDIA_SUPPORT && ..))

2015-07-20 Thread kbuild test robot
tree: git://git.infradead.org/users/ezequielg/linux tw68_ids_upstream_v2 head: 182300e6004285b7cbc26d76bf2fffe3b3bed212 commit: 2f8e75d2762496bb2fcea7fa437a3339d2a6d9d4 [82/109] [media] adv7604/cobalt: Allow compile test if !GPIOLIB config: x86_64-randconfig-n0-07210714 (attached as .config) r

[PATCH 7/7] [PATCH FIXES] Revert "[media] rc: rc-ir-raw: Add scancode encoder callback"

2015-07-20 Thread David Härdeman
This reverts commit 9869da5bacc5c9b865a183bd36c04be76cdd325d. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/rc-co

[PATCH 6/7] [PATCH FIXES] Revert "[media] rc: rc-ir-raw: Add Manchester encoder (phase encoder) helper"

2015-07-20 Thread David Härdeman
This reverts commit 1d971d927efa2e10194c96ed0475b6d6054342d8. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/rc-co

[PATCH 4/7] [PATCH FIXES] Revert "[media] rc: ir-rc6-decoder: Add encode capability"

2015-07-20 Thread David Härdeman
This reverts commit cf257e288ad3a134d4bb809c542a3ae6c87ddfa3. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/ir-rc

[PATCH 3/7] [PATCH FIXES] Revert "[media] rc: rc-core: Add support for encode_wakeup drivers"

2015-07-20 Thread David Härdeman
This reverts commit 0d830b2d1295fee82546d57185da5a6604f11ae2. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/rc-co

[PATCH 5/7] [PATCH FIXES] Revert "[media] rc: ir-rc5-decoder: Add encode capability"

2015-07-20 Thread David Härdeman
This reverts commit a0466f15b4654cf1ac9e387d7c1a401eff494b4f. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/ir-rc

[PATCH 1/7] [PATCH FIXES] Revert "[media] rc: nuvoton-cir: Add support for writing wakeup samples via sysfs filter callback"

2015-07-20 Thread David Härdeman
This reverts commit da7ee60b03bd66bb10974d7444aa444de6391312. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/nuvot

[PATCH 0/7] rc-core: Revert encoding patchset

2015-07-20 Thread David Härdeman
The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. The issues were discussed in this thread: https://www.mail-archive.com/linux-media@vger.kernel.org/msg86998.html And Ant

[PATCH 2/7] [PATCH FIXES] Revert "[media] rc: rc-loopback: Add loopback of filter scancodes"

2015-07-20 Thread David Härdeman
This reverts commit 2e4ebde269236da2a41183522127715b6d9d80ce. The current code is not mature enough, the API should allow a single protocol to be specified. Also, the current code contains heuristics that will depend on module load order. Signed-off-by: David Härdeman --- drivers/media/rc/rc-lo

Re: [PATCH 6/7] media: au0828 change to use Managed Media Controller API

2015-07-20 Thread Shuah Khan
On 07/20/2015 01:01 PM, Dan Carpenter wrote: > On Mon, Jul 20, 2015 at 09:55:50AM -0600, Shuah Khan wrote: #ifdef CONFIG_MEDIA_CONTROLLER - if (dev->media_dev) { - media_device_unregister(dev->media_dev); - kfree(dev->media_dev); - dev->media_de

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Steven Toth
>> We can agree or disagree about whether a part should be tri-stated in >> init/sleep() and under what circumstances, but why bother when someone >> has gone to the trouble of declaring a perfectly good tr-state >> interface in dvb-core, taht automatically asserts and de-asserts any >> dvb_fronten

Re: [PATCH 6/7] media: au0828 change to use Managed Media Controller API

2015-07-20 Thread Dan Carpenter
On Mon, Jul 20, 2015 at 09:55:50AM -0600, Shuah Khan wrote: > >> #ifdef CONFIG_MEDIA_CONTROLLER > >> - if (dev->media_dev) { > >> - media_device_unregister(dev->media_dev); > >> - kfree(dev->media_dev); > >> - dev->media_dev = NULL; > >> + if (dev->media_dev && > >> +

[PATCH] tw68: Move PCI vendor and device IDs to pci_ids.h

2015-07-20 Thread Ezequiel Garcia
This commits moves the Intersil/Techwell PCI vendor ID, and the device IDs for the TW68 PCI video capture cards. This will allow to support future Intersil/Techwell devices without duplicating the IDs. Signed-off-by: Ezequiel Garcia --- drivers/media/pci/tw68/tw68-core.c | 15 --- d

Re: [RESEND PATCH v2 1/2] x86/mm/pat, drivers/infiniband/ipath: replace WARN() with pr_warn()

2015-07-20 Thread Doug Ledford
> On Jul 17, 2015, at 5:07 PM, Luis R. Rodriguez > wrote: > > From: "Luis R. Rodriguez" > > WARN() may confuse users, fix that. ipath_init_one() is part the > device's probe so this would only be triggered if a corresponding > device was found. > > Signed-off-by: Luis R. Rodriguez Acked-by

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Antti Palosaari
On 07/20/2015 08:14 PM, Steven Toth wrote: On Mon, Jul 20, 2015 at 12:54 PM, Antti Palosaari wrote: On 07/20/2015 07:45 PM, Devin Heitmueller wrote: Look at the em28xx driver and you will probably see why it does not work as expected. For my eyes, according to em28xx driver, it looks like tha

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Steven Toth
On Mon, Jul 20, 2015 at 12:54 PM, Antti Palosaari wrote: > On 07/20/2015 07:45 PM, Devin Heitmueller wrote: >>> >>> Look at the em28xx driver and you will probably see why it does not work >>> as >>> expected. For my eyes, according to em28xx driver, it looks like that bus >>> control is aimed for

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Antti Palosaari
On 07/20/2015 07:45 PM, Devin Heitmueller wrote: Look at the em28xx driver and you will probably see why it does not work as expected. For my eyes, according to em28xx driver, it looks like that bus control is aimed for bridge driver. You or em28xx is wrong. Neither are wrong. In some cases th

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Devin Heitmueller
> Look at the em28xx driver and you will probably see why it does not work as > expected. For my eyes, according to em28xx driver, it looks like that bus > control is aimed for bridge driver. You or em28xx is wrong. Neither are wrong. In some cases the call needs to be intercepted by the frontend

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Antti Palosaari
On 07/20/2015 06:00 PM, Steven Toth wrote: On Mon, Jul 20, 2015 at 10:30 AM, Antti Palosaari wrote: On 07/19/2015 01:21 AM, Steven Toth wrote: http://git.linuxtv.org/cgit.cgi/stoth/hvr1275.git/log/?h=hvr-1275 Patches above are available for test. Antti, note the change to SI2168 to add supp

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Steven Toth
On Mon, Jul 20, 2015 at 12:06 PM, Tycho Lürsen wrote: > Hi Steven, > I was not aware of the fact that your patch depends on dvb-core as in > 4.2-RC2 (and up, I guess) > I tested against 3.18.18 and 4.1.2. That might explain the failures. > Anyhow, as soon as Antti and you are on the same page rega

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Tycho Lürsen
Hi Steven, I was not aware of the fact that your patch depends on dvb-core as in 4.2-RC2 (and up, I guess) I tested against 3.18.18 and 4.1.2. That might explain the failures. Anyhow, as soon as Antti and you are on the same page regarding this patch, I'll test again against a 4.2-RC>1 Regards

Re: [PATCH 6/7] media: au0828 change to use Managed Media Controller API

2015-07-20 Thread Shuah Khan
On 07/20/2015 02:42 AM, Dan Carpenter wrote: > Sorry for this nit-pick and some time after you sent this patch. It's > not a redo the patch" complaint, it's something that could be fixed > later. > > On Tue, Jul 14, 2015 at 06:34:05PM -0600, Shuah Khan wrote: >> @@ -131,10 +132,12 @@ static void

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Steven Toth
On Mon, Jul 20, 2015 at 10:30 AM, Antti Palosaari wrote: > On 07/19/2015 01:21 AM, Steven Toth wrote: >> >> http://git.linuxtv.org/cgit.cgi/stoth/hvr1275.git/log/?h=hvr-1275 >> >> Patches above are available for test. >> >> Antti, note the change to SI2168 to add support for enabling and >> disabl

Re: [RFC v3 02/19] media/v4l2-core: add new ioctl VIDIOC_G_DEF_EXT_CTRLS

2015-07-20 Thread Hans Verkuil
On 07/20/2015 03:52 PM, Ricardo Ribalda Delgado wrote: > Hello > > I have no preference over the two implementations, but I see an issue > with this suggestion. > > > What happens to out out tree drivers, or drivers that don't support > this functionality? > > With the ioctl, the user receives

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Antti Palosaari
On 07/19/2015 01:21 AM, Steven Toth wrote: http://git.linuxtv.org/cgit.cgi/stoth/hvr1275.git/log/?h=hvr-1275 Patches above are available for test. Antti, note the change to SI2168 to add support for enabling and disabling the SI2168 transport bus dynamically. I've tested with a combo card, swi

Re: [PATCH 7/7] sound/usb: Update ALSA driver to use Managed Media Controller API

2015-07-20 Thread Shuah Khan
On 07/20/2015 03:00 AM, Takashi Iwai wrote: > On Mon, 20 Jul 2015 10:47:46 +0200, > Dan Carpenter wrote: >> >> On Tue, Jul 14, 2015 at 06:34:06PM -0600, Shuah Khan wrote: >>> + ret = media_entity_setup_link(link, flags); >>> + if (ret) { >>> + dev_err(mctl->med

Re: [RFC v3 02/19] media/v4l2-core: add new ioctl VIDIOC_G_DEF_EXT_CTRLS

2015-07-20 Thread Ricardo Ribalda Delgado
Hello I have no preference over the two implementations, but I see an issue with this suggestion. What happens to out out tree drivers, or drivers that don't support this functionality? With the ioctl, the user receives a -ENOTTY. So he knows there is something wrong with the driver. With this

Re: [RFC v3 02/19] media/v4l2-core: add new ioctl VIDIOC_G_DEF_EXT_CTRLS

2015-07-20 Thread Hans Verkuil
On 06/12/2015 06:46 PM, Ricardo Ribalda Delgado wrote: > This ioctl returns the default value of one or more extended controls. > It has the same interface as VIDIOC_EXT_CTRLS. > > It is needed due to the fact that QUERYCTRL was not enough to > provide the initial value of pointer type controls.

V4L2 Xilinx driver state

2015-07-20 Thread Franck Jullien
Hi, I'm very new to video input devices. I have a new project where I need to get a SDI video stream to a CPU via a FPGA. I'll use a Xilinx device and Xilinx video IP cores. This is what I (think) need: ++ +-+ ++ | CUSTO

[PATCH 4/5] zoran: convert to the control framework and to v4l2_fh

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Switch this driver to the control framework and to v4l2_fh for handling control events. Signed-off-by: Hans Verkuil --- drivers/media/pci/zoran/zoran.h| 4 ++ drivers/media/pci/zoran/zoran_card.c | 5 +++ drivers/media/pci/zoran/zoran_driver.c | 69 +++---

[PATCH 5/5] bt819/saa7110/vpx3220: remove legacy control ops

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The zoran driver has now been converted to the control framework which means that these three subdevice drivers no longer need to support the legacy core control ops since the last bridge driver that needed that has now been converted. Signed-off-by: Hans Verkuil --- drivers

[PATCH 3/5] zoran: use standard core lock

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Use the standard core lock to take care of serializing ioctl calls and to serialize file operations. Signed-off-by: Hans Verkuil --- drivers/media/pci/zoran/zoran.h| 3 +- drivers/media/pci/zoran/zoran_card.c | 6 +- drivers/media/pci/zoran/zoran_driver.c | 249

[PATCH 0/5] zoran: convert to the control framework

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Convert the zoran driver to the control framework. As a result of that it is now also possible to drop the legacy control ops from three subdev drivers that were only used by the zoran driver. Regards, Hans Hans Verkuil (5): zoran: remove unnecessary memset zoran

[PATCH 1/5] zoran: remove unnecessary memset

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil There is no need to zero the v4l2_capability struct, the v4l2 core has done that already. Signed-off-by: Hans Verkuil --- drivers/media/pci/zoran/zoran_driver.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/pci/zoran/zoran_driver.c b/drivers/media/pci/zor

[PATCH 2/5] zoran: remove unused read/write functions

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The zoran_read/write functions always return an error. Just remove them. Signed-off-by: Hans Verkuil --- drivers/media/pci/zoran/zoran_driver.c | 25 - 1 file changed, 25 deletions(-) diff --git a/drivers/media/pci/zoran/zoran_driver.c b/drivers/med

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Steven Toth
On Sun, Jul 19, 2015 at 3:34 AM, Tycho Lürsen wrote: > Hi Steven, > > Tested your si2186 patch with my DVBSky T982 and TBS 6285 cards using > European DVB-C > Since MythTV can't handle multistandard frontends (yet), I've disabled > DVB-T/T2 like this (I always do that): > > sed -i 's/SYS_DVBT, SYS

[PATCH 6/6] fsl-viu: drop format names

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The names of the pixelformats is set by the core. So there no longer is any need for drivers to fill it in. Signed-off-by: Hans Verkuil --- drivers/media/platform/fsl-viu.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/media/platform/fs

[PATCH 5/6] fsl-viu: small fixes.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil - Fix an off-by-one index check in vidioc_enum_fmt() - Fill in the pix.sizeimage field in vidioc_try_fmt_cap() - Fix an off-by-one index check in vidioc_s_input() Signed-off-by: Hans Verkuil --- drivers/media/platform/fsl-viu.c | 5 +++-- 1 file changed, 3 insertions(+), 2 d

[PATCH 4/6] fsl-viu: add control event support.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Convert the driver to use v4l2_fh in order to support control events. Signed-off-by: Hans Verkuil --- drivers/media/platform/fsl-viu.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/drivers/media/platform/fsl-viu.c b/drivers/media/

[PATCH 2/6] fsl-viu: fill in bus_info in vidioc_querycap.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The bus_info field was never filled. Signed-off-by: Hans Verkuil --- drivers/media/platform/fsl-viu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/fsl-viu.c b/drivers/media/platform/fsl-viu.c index f0ec551..ab8012c 100644 --- a/drivers/media/pla

[PATCH 3/6] fsl-viu: fill in colorspace, always set field to interlaced.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil - fill in the missing colorspace value. - don't reject incorrect field values, always replace with a valid value. Signed-off-by: Hans Verkuil --- drivers/media/platform/fsl-viu.c | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/drivers/medi

[PATCH 1/6] fsl-viu: convert to the control framework.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Interestingly enough, the existing control handling code basically did nothing. At least the new code will inherit the controls from the saa7115 driver. Signed-off-by: Hans Verkuil --- drivers/media/platform/fsl-viu.c | 110 +-- 1 file cha

[PATCH 0/6] fsl-viu: convert to the control framework

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Convert this driver to the control framework and do a few more fixes to make v4l2-compliance happy. Most of these patches have been posted before over two years ago but I never made a pull request for it. This is the original patch series: http://comments.gmane.org/gmane.lin

[PATCH 11/12] usbvision: fix standards for S-Video/Composite inputs.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The standards supported by S-Video and Composite inputs are not limited by PAL, so make it more generic. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/usb/

[PATCH 10/12] usbvision: fix DMA from stack warnings.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil In various places the stack was used to provide buffers for USB data, but this should be allocated memory. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-core.c | 18 ++ drivers/media/usb/usbvision/usbvision-i2c.c | 2 +- 2 files chan

[PATCH 09/12] usbvision: fix locking error

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil If remove_pending is non-zero, then the v4l2_lock is never unlocked. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/usbvision/usbvision-video.c b/drivers/med

[PATCH 08/12] usbvision: set field and colorspace.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Set the colorspace and field in vidioc_try_fmt_vid_cap(). Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/usb/usbvision/usbvision-video.c b/drivers/media/usb/usbvision/usbvisio

[PATCH 07/12] usbvision: frequency fixes.

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil - setup initial radio and tv frequencies. - set/get the correct frequency (radio vs tv). - disable tuner/freq ioctls if there is no tuner. - fix some tuner index checks. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 44 +-

[PATCH 12/12] usbvision: move init code to probe()

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil These things are only initialized if you start streaming video, but they are also used in the disconnect function. So just init them always during probe time. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-core.c | 4 drivers/media/usb/usbvision/

[PATCH 05/12] usbvision: remove g/s_audio and for radio remove enum/g/s_input

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The g/s_audio ioctls didn't do anything, so remove them all for both video and radio nodes and remove V4L2_CAP_AUDIO. The enum/g/s_input ioctls are invalid for radio nodes, so remove them from the radio ioctl_ops. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/

[PATCH 06/12] usbvision: the radio device node has wrong caps

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil The radio device node had the same caps as the video node. Fix this. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/drivers/media/usb/usbvision/usbvision-vid

[PATCH 02/12] usbvision: remove power_on_at_open and timed power off

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil This causes lots of problems and is *very* slow as well. One of the main problems is that this prohibits the use of the control framework since subdevs will be unloaded on power off which is not allowed as long as they are used by a usb device. Apparently the reason for doing

[PATCH 03/12] usbvision: convert to the control framework

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Convert this driver to the control framework and struct v4l2_fh (needed for handling control events). Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 69 ++- drivers/media/usb/usbvision/usbvision.h | 2 + 2 files

[PATCH 04/12] usbvision: return valid error in usbvision_register_video()

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil Don't return -1, return a proper error code. Signed-off-by: Hans Verkuil --- drivers/media/usb/usbvision/usbvision-video.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/media/usb/usbvision/usbvision-video.c b/drivers/media/usb/usbvision/usbv

[PATCH 00/12] usbvision: convert to control framework

2015-07-20 Thread Hans Verkuil
From: Hans Verkuil This patch series converts the usbvision driver to the control framework. It also fixes a bunch of other bugs, making it a bit more v4l2-compliance friendly. This driver still needs a lot of work to make it pass v4l2-compliance, but this is a start. Regards, Hans Ha

[PATCH 01/12] [media] coda: make NV12 format default

2015-07-20 Thread Hans Verkuil
From: Philipp Zabel The chroma interleaved NV12 format has higher memory bandwidth efficiency because the chroma planes can be read/written with longer burst lengths. Use NV12 as default format if available and consistently sort it first. Signed-off-by: Philipp Zabel Signed-off-by: Hans Verkuil

Re: Adding support for three new Hauppauge HVR-1275 variants - testers reqd.

2015-07-20 Thread Steven Toth
On Mon, Jul 20, 2015 at 2:00 AM, Tony Chang(Wincomm) wrote: > Dear : Steven > > Sorry for my poor english !! I don’t know how to install it > > According your feedback.. > > diff --git a/drivers/media/pci/cx23885/Kconfig > b/drivers/media/pci/cx23885/Kconfigindex 2e1b88c..3e6398f 100644 > > I don'

[PATCH] tc358743: remove unused variable

2015-07-20 Thread Hans Verkuil
The bt pointer was never used, remove it. Signed-off-by: Hans Verkuil diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c index 4e8811c..9447d88 100644 --- a/drivers/media/i2c/tc358743.c +++ b/drivers/media/i2c/tc358743.c @@ -1316,7 +1316,6 @@ static int tc358743_s_dv_timing

Re: [PATCH v4 1/1] V4L2: platform: Add Renesas R-Car JPEG codec driver.

2015-07-20 Thread Hans Verkuil
On 07/19/2015 04:40 PM, Mikhail Ulyanov wrote: > Hi Hans, > > I've made some changes to driver(mostly minor, like sequence v4l2_buf > field filling and so on)to make it pass "v4l2-compliance -s" test(MMAP > part), but suddenly get stuck with USERPTR part. First there is WARN > about zero bytesuse

Re: [PATCH 7/7] sound/usb: Update ALSA driver to use Managed Media Controller API

2015-07-20 Thread Takashi Iwai
On Mon, 20 Jul 2015 10:47:46 +0200, Dan Carpenter wrote: > > On Tue, Jul 14, 2015 at 06:34:06PM -0600, Shuah Khan wrote: > > + ret = media_entity_setup_link(link, flags); > > + if (ret) { > > + dev_err(mctl->media_dev->dev, > > + "Cou

[PATCH] v4l2-ctl-modes: use reduced fps only with reduced blanking v2

2015-07-20 Thread Prashant Laddha
In the absence of reduced blanking v2, the clock granularity is not sufficient enough to allow pixel clock reduction done by factor of 1000 / 1001 in case of reduced fps. Cc: Hans Verkuil Signed-off-by: Prashant Laddha --- utils/v4l2-ctl/v4l2-ctl-modes.cpp | 2 +- 1 file changed, 1 insertion(+)

Re: [PATCH 7/7] sound/usb: Update ALSA driver to use Managed Media Controller API

2015-07-20 Thread Dan Carpenter
On Tue, Jul 14, 2015 at 06:34:06PM -0600, Shuah Khan wrote: > + ret = media_entity_setup_link(link, flags); > + if (ret) { > + dev_err(mctl->media_dev->dev, > + "Couldn't change tuner link", > + "%s-

Re: [PATCH 6/7] media: au0828 change to use Managed Media Controller API

2015-07-20 Thread Dan Carpenter
Sorry for this nit-pick and some time after you sent this patch. It's not a redo the patch" complaint, it's something that could be fixed later. On Tue, Jul 14, 2015 at 06:34:05PM -0600, Shuah Khan wrote: > @@ -131,10 +132,12 @@ static void au0828_unregister_media_device(struct > au0828_dev *dev

Re: [PATCH] v4l2-mem2mem: drop lock in v4l2_m2m_fop_mmap

2015-07-20 Thread Mikhail Ulyanov
Hi Hans, 2015-07-20 10:58 GMT+03:00 Hans Verkuil : > > Since vb2_fop_mmap doesn't take the lock, neither should v4l2_m2m_fop_mmap. > > Signed-off-by: Hans Verkuil [snip] Tested-by: Mikhail Ulyanov -- W.B.R, Mikhail. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in

[PATCH] v4l2-mem2mem: drop lock in v4l2_m2m_fop_mmap

2015-07-20 Thread Hans Verkuil
The v4l2_m2m_fop_mmap function takes the core mutex, but this will result in a potential circular locking dependency: [ 262.517164] == [ 262.517166] [ INFO: possible circular locking dependency detected ] [ 262.517169] 4.2.0-rc2-koryphon #844