cron job: media_tree daily build: ERRORS

2015-02-03 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 Feb 4 04:00:17 CET 2015 git branch: test git hash: 4bad5d2d25099a42e146d7b18d2b98950ed287f5 gcc

Re: [possible BUG, cx23885] Dual tuner TV card, works using one tuner only, doesn't work if both tuners are used

2015-02-03 Thread Hans Verkuil
On 02/03/2015 08:32 PM, Steven Toth wrote: While I am the maintainer of the cx23885 driver, its currently undergoing a significant amount of churn related to Han's recent VB2 and other changes. I consider the current driver broken until the feedback on the mailing list dies down. I'm reluctant

Re: Vivid test device: adding YU12

2015-02-03 Thread Hans Verkuil
On 02/04/2015 01:42 AM, Miguel Casas-Sanchez wrote: On 02/02/15 23:32, Miguel Casas-Sanchez wrote: On 01/29/2015 03:44 AM, Miguel Casas-Sanchez wrote: Hi folks, I've been trying to add a triplanar format to those that vivid can generate, and didn't quite manage :(

[PATCH 0/5] Remove .ioctl from v4l2_file_operations

2015-02-03 Thread Hans Verkuil
All V4L2 drivers should use .unlocked_ioctl instead of .ioctl. There are only three drivers left that do not do that: pvrusb2, radio-bcm2048 and the uvc gadget driver. The pvrusb2 driver does its own locking as far as I can tell, so it can just switch to unlocked_ioctl. Ditto for radio-bcm2048.

Re: [PATCH 4/5] uvc gadget: set device_caps in querycap.

2015-02-03 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Tuesday 03 February 2015 13:47:25 Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com The V4L2 core will warn if this is not done. Unfortunately this driver wasn't updated. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Acked-by:

[PATCH 1/2] pvrusb2: use struct v4l2_fh

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com But using struct v4l2_fh both the prio handling and the linked list implementation in pvrusb2 can be removed since both are now done in the v4l2 core if you use struct v4l2_fh. Signed-off-by: Hans Verkuil hans.verk...@cisco.com ---

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 02:28:26PM +0100, Christian Gmeiner wrote: 2015-02-03 13:28 GMT+01:00 Russell King - ARM Linux li...@arm.linux.org.uk: What I've found with *my* etnaviv drm implementation (not Christian's - I found it impossible to work with Christian, especially with the endless

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 09:04:03AM -0500, Rob Clark wrote: Since I'm stuck w/ an iommu, instead of built in mmu, my plan was to drop use of dma-mapping entirely (incl the current call to dma_map_sg, which I just need until we can use drm_cflush on arm), and attach/detach iommu domains directly

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 03:17:27PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 09:04:03 Rob Clark wrote: Since I'm stuck w/ an iommu, instead of built in mmu, my plan was to drop use of dma-mapping entirely (incl the current call to dma_map_sg, which I just need until we can use

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 9:37 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Feb 03, 2015 at 09:04:03AM -0500, Rob Clark wrote: Since I'm stuck w/ an iommu, instead of built in mmu, my plan was to drop use of dma-mapping entirely (incl the current call to dma_map_sg, which I

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 9:41 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Feb 03, 2015 at 03:17:27PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 09:04:03 Rob Clark wrote: Since I'm stuck w/ an iommu, instead of built in mmu, my plan was to drop use of

Re: [PATCH 6/8] WmT: adv7604 driver compatibility

2015-02-03 Thread Laurent Pinchart
Hi Hans, On Monday 02 February 2015 11:09:27 Hans Verkuil wrote: On 02/02/2015 11:01 AM, Laurent Pinchart wrote: On Sunday 01 February 2015 12:26:11 Guennadi Liakhovetski wrote: On a second thought: On Sun, 1 Feb 2015, Guennadi Liakhovetski wrote: Hi Wills, Thanks for the patch.

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 9:52 AM, Arnd Bergmann a...@arndb.de wrote: On Tuesday 03 February 2015 14:41:09 Russell King - ARM Linux wrote: On Tue, Feb 03, 2015 at 03:17:27PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 09:04:03 Rob Clark wrote: Since I'm stuck w/ an iommu, instead

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 15:22:05 Russell King - ARM Linux wrote: On Tue, Feb 03, 2015 at 03:52:48PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 14:41:09 Russell King - ARM Linux wrote: I'd go as far as saying that the DMA API on top of IOMMU is more intended to be for a

Re: [PATCH 6/8] WmT: adv7604 driver compatibility

2015-02-03 Thread Hans Verkuil
On 02/03/15 16:22, Laurent Pinchart wrote: Hi Hans, On Monday 02 February 2015 11:09:27 Hans Verkuil wrote: On 02/02/2015 11:01 AM, Laurent Pinchart wrote: On Sunday 01 February 2015 12:26:11 Guennadi Liakhovetski wrote: On a second thought: On Sun, 1 Feb 2015, Guennadi Liakhovetski

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 03:52:48PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 14:41:09 Russell King - ARM Linux wrote: I'd go as far as saying that the DMA API on top of IOMMU is more intended to be for a system IOMMU for the bus in question, rather than a device-level IOMMU.

Re: [PATCH 6/8] WmT: adv7604 driver compatibility

2015-02-03 Thread Lars-Peter Clausen
On 02/03/2015 04:24 PM, Hans Verkuil wrote: On 02/03/15 16:22, Laurent Pinchart wrote: Hi Hans, On Monday 02 February 2015 11:09:27 Hans Verkuil wrote: On 02/02/2015 11:01 AM, Laurent Pinchart wrote: On Sunday 01 February 2015 12:26:11 Guennadi Liakhovetski wrote: On a second thought: On

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 14:41:09 Russell King - ARM Linux wrote: On Tue, Feb 03, 2015 at 03:17:27PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 09:04:03 Rob Clark wrote: Since I'm stuck w/ an iommu, instead of built in mmu, my plan was to drop use of dma-mapping entirely

[PATCH] [media] V4L2: Delete an unnecessary check before the function call media_entity_put

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 15:27:38 +0100 The media_entity_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software.

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 09:44:57AM -0500, Rob Clark wrote: On Tue, Feb 3, 2015 at 9:37 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Feb 03, 2015 at 09:04:03AM -0500, Rob Clark wrote: Since I'm stuck w/ an iommu, instead of built in mmu, my plan was to drop use of

Re: [BUG] - Why anyone fix this problem?

2015-02-03 Thread Olli Salonen
Hi Francesco, You need to understand that many people write code for their own enjoyment. In other words, they often write code to scratch an itch. Thus it can sometimes happen that there really is no-one here who could help you. The person who wrote the code originally might have stopped

Re: [PATCH v2 08/15] media: blackfin: bfin_capture: use vb2_ioctl_* helpers

2015-02-03 Thread Hans Verkuil
On 02/03/15 09:27, Scott Jiang wrote: Hi Lad, 2015-01-23 6:18 GMT+08:00 Lad, Prabhakar prabhakar.cse...@gmail.com: this patch adds support to vb2_ioctl_* helpers. Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com --- drivers/media/platform/blackfin/bfin_capture.c | 108

Re: [PATCH v2 00/15] media: blackfin: bfin_capture enhancements

2015-02-03 Thread Scott Jiang
Hi Hans, On Thu, Jan 22, 2015 at 10:18 PM, Lad, Prabhakar prabhakar.cse...@gmail.com wrote: This patch series, enhances blackfin capture driver with vb2 helpers. Changes for v2: -- Only patches 5/15 and 8/15 as per Scott's suggestions. Lad, Prabhakar (15): media:

[PATCH 5/5] [media] netup_unidvb: NetUP Universal DVB-S/S2/T/T2/C PCI-E card driver

2015-02-03 Thread Kozlov Sergey
NetUP Dual Universal CI PCIe board driver. The board has - two CI slots - two I2C adapters - SPI master bus for accessing flash memory containing FPGA firmware Signed-off-by: Kozlov Sergey se...@netup.ru --- MAINTAINERS |9 +

Re: [PATCH v2 08/15] media: blackfin: bfin_capture: use vb2_ioctl_* helpers

2015-02-03 Thread Scott Jiang
Hi Lad, 2015-01-23 6:18 GMT+08:00 Lad, Prabhakar prabhakar.cse...@gmail.com: this patch adds support to vb2_ioctl_* helpers. Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com --- drivers/media/platform/blackfin/bfin_capture.c | 108 ++--- 1 file changed, 23

[PATCH 4/5] [media] cxd2841er: Sony CXD2841ER DVB-S/S2/T/T2/C demodulator driver

2015-02-03 Thread Kozlov Sergey
DVB-C/T/T2/S/S2 demodulator frontend driver Sony CXD2841ER chip. Signed-off-by: Kozlov Sergey se...@netup.ru --- MAINTAINERS |9 + drivers/media/dvb-frontends/Kconfig |7 + drivers/media/dvb-frontends/Makefile |1 +

Re: [RFC PATCH 2/2] [media] videobuf2: return -EPIPE from DQBUF after the last buffer

2015-02-03 Thread Laurent Pinchart
On Monday 02 February 2015 15:00:51 Hans Verkuil wrote: On 01/22/2015 12:28 PM, Philipp Zabel wrote: If the last buffer was dequeued from a capture queue, let poll return immediately and let DQBUF return -EPIPE to signal there will no more buffers to dequeue until STREAMOFF. This looks

Re: [PATCH 3/5] uvc gadget: switch to unlocked_ioctl.

2015-02-03 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Tuesday 03 February 2015 13:47:24 Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com Instead of .ioctl use unlocked_ioctl. While all the queue ops already use a lock, there was no lock to protect uvc_video, so add that one. There's more.

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Daniel Vetter
On Tue, Feb 3, 2015 at 10:42 PM, Arnd Bergmann a...@arndb.de wrote: Again assuming I'm not confused can't we just solve this by pushing the dma api abstraction down one layer for just the gpu, and let it use its private iommmu directly? Steps for binding a buffer would be: 1. dma_map_sg 2.

[PATCH 0/2] DVB: Deletion of a few unnecessary checks

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 16:50:07 +0100 Another update suggestion was taken into account after a patch was applied from static source code analysis. Markus Elfring (2): Delete an unnecessary check before the function call dvb_unregister_device

[PATCH 2/2] [media] DVB: Less function calls in dvb_ca_en50221_init() after error detection

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 16:47:48 +0100 The functions dvb_unregister_device and kfree could still be called by the dvb_ca_en50221_init() function in the case that a previous resource allocation failed. * Corresponding details could be improved by

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 11:22:01 Rob Clark wrote: On Tue, Feb 3, 2015 at 11:12 AM, Arnd Bergmann a...@arndb.de wrote: I agree for the case you are describing here. From what I understood from Rob was that he is looking at something more like: Fig 3

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 04:31:13PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 15:22:05 Russell King - ARM Linux wrote: Don't we already have those in the DMA API? dma_sync_*() ? dma_map_sg() - sets up the system MMU and deals with initial cache coherency handling. Device

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 15:54:04 Russell King - ARM Linux wrote: On Tue, Feb 03, 2015 at 04:31:13PM +0100, Arnd Bergmann wrote: The dma_map_* interfaces assign the virtual addresses internally, using typically either a global address space for all devices, or one address space per

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 11:22:01AM -0500, Rob Clark wrote: On Tue, Feb 3, 2015 at 11:12 AM, Arnd Bergmann a...@arndb.de wrote: I agree for the case you are describing here. From what I understood from Rob was that he is looking at something more like: Fig 3

Re: [PATCH 6/8] WmT: adv7604 driver compatibility

2015-02-03 Thread Laurent Pinchart
On Tuesday 03 February 2015 16:29:57 Lars-Peter Clausen wrote: On 02/03/2015 04:24 PM, Hans Verkuil wrote: On 02/03/15 16:22, Laurent Pinchart wrote: [snip] I can't help you much with that, but I could test changes using the rcar-vin driver with the adv7180 if needed (does the adv7180

Re: [PATCH 6/8] WmT: adv7604 driver compatibility

2015-02-03 Thread Laurent Pinchart
Hi Hans, On Tuesday 03 February 2015 16:24:19 Hans Verkuil wrote: On 02/03/15 16:22, Laurent Pinchart wrote: On Monday 02 February 2015 11:09:27 Hans Verkuil wrote: On 02/02/2015 11:01 AM, Laurent Pinchart wrote: On Sunday 01 February 2015 12:26:11 Guennadi Liakhovetski wrote: On a

[PATCH 1/2] [media] DVB: Delete an unnecessary check before the function call dvb_unregister_device

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 16:01:40 +0100 The dvb_unregister_device() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software.

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 11:12 AM, Arnd Bergmann a...@arndb.de wrote: I agree for the case you are describing here. From what I understood from Rob was that he is looking at something more like: Fig 3 CPU--L1cache--L2cache--Memory--IOMMU---iobus--device where the IOMMU controls one or more

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 05:12:40PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 15:54:04 Russell King - ARM Linux wrote: On Tue, Feb 03, 2015 at 04:31:13PM +0100, Arnd Bergmann wrote: The dma_map_* interfaces assign the virtual addresses internally, using typically either a

[PATCH] media: i2c: ADV7604: Rename adv7604 prefixes.

2015-02-03 Thread Pablo Anton
It is confusing which parts of the driver are adv7604 specific, adv7611 specific or common for both. This patch renames any adv7604 prefixes (both for functions and defines) to adv76xx whenever they are common. Signed-off-by: Pablo Anton pablo.an...@vodalys-labs.com Signed-off-by: Jean-Michel

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 08:48:56AM +0100, Daniel Vetter wrote: On Mon, Feb 02, 2015 at 03:30:21PM -0500, Rob Clark wrote: On Mon, Feb 2, 2015 at 11:54 AM, Daniel Vetter dan...@ffwll.ch wrote: My initial thought is for dma-buf to not try to prevent something than an exporter can actually

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Daniel Vetter
On Tue, Feb 03, 2015 at 12:28:14PM +, Russell King - ARM Linux wrote: On Tue, Feb 03, 2015 at 08:48:56AM +0100, Daniel Vetter wrote: On Mon, Feb 02, 2015 at 03:30:21PM -0500, Rob Clark wrote: On Mon, Feb 2, 2015 at 11:54 AM, Daniel Vetter dan...@ffwll.ch wrote: My initial thought is

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 7:28 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Tue, Feb 03, 2015 at 08:48:56AM +0100, Daniel Vetter wrote: On Mon, Feb 02, 2015 at 03:30:21PM -0500, Rob Clark wrote: On Mon, Feb 2, 2015 at 11:54 AM, Daniel Vetter dan...@ffwll.ch wrote: My initial

Re: [PATCHv2 1/2] regmap: add configurable lock class key for lockdep

2015-02-03 Thread Mark Brown
On Mon, Feb 02, 2015 at 12:31:38PM -0200, Mauro Carvalho Chehab wrote: Antti/Mark, Any news with regards to this? Please don't top post or send content free nags. I can't really remember what this is about but I don't think my review comments were ever addressed. signature.asc Description:

Re: [GIT FIXES FOR v3.19] smiapp compile fix for non-OF configuration

2015-02-03 Thread Sakari Ailus
Hi Mauro, On Thu, Jan 29, 2015 at 06:00:55PM -0200, Mauro Carvalho Chehab wrote: Em Tue, 27 Jan 2015 12:36:49 +0200 Sakari Ailus sakari.ai...@iki.fi escreveu: Hi Mauro, The recent smiapp OF support patches contained a small issue related to reading 64-bit numbers from the device

[PATCH 4/5] uvc gadget: set device_caps in querycap.

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com The V4L2 core will warn if this is not done. Unfortunately this driver wasn't updated. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/usb/gadget/function/uvc_v4l2.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 1/5] pvrusb2: replace .ioctl by .unlocked_ioctl.

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com As far as I can tell pvrusb2 does its own locking, so there is no need to use .ioctl. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/usb/pvrusb2/pvrusb2-v4l2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 5/5] v4l2-core: remove the old .ioctl BKL replacement

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com To keep V4L2 drivers that did not yet convert to unlocked_ioctl happy, the v4l2 core had a .ioctl file operation that took a V4L2 lock. The last drivers are now converted to unlocked_ioctl, so all this old code can now be removed. Signed-off-by: Hans

[PATCH 3/5] uvc gadget: switch to unlocked_ioctl.

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Instead of .ioctl use unlocked_ioctl. While all the queue ops already use a lock, there was no lock to protect uvc_video, so add that one. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/usb/gadget/function/f_uvc.c| 1 +

[PATCH 2/5] radio-bcm2048: use unlocked_ioctl instead of ioctl

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com This driver does its own locking, so there is no need to use ioctl instead of unlocked_ioctl. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH] [media] sp2: Delete an unnecessary check before the function call kfree

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 15:05:26 +0100 The kfree() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by:

Re: [PATCH 2/5] radio-bcm2048: use unlocked_ioctl instead of ioctl

2015-02-03 Thread Pali Rohár
On Tuesday 03 February 2015 13:47:23 Hans Verkuil wrote: From: Hans Verkuil hans.verk...@cisco.com This driver does its own locking, so there is no need to use ioctl instead of unlocked_ioctl. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Looks good, Acked-by: Pali Rohár

[PATCH 0/2] Drop g/s_priority ioctl ops

2015-02-03 Thread Hans Verkuil
Only pvrusb2 is still using the vidioc_g/s_priority ioctl ops. Add struct v4l2_fh support to pvrusb2, allowing us to drop those ioctl ops altogether. This patch series sits on top of the earlier 5 part patch series Remove .ioctl from v4l2_file_operations, but it is probably independent of that

[PATCH 2/2] v4l2-core: drop g/s_priority ops

2015-02-03 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com The handling of VIDIOC_G/S_PRIORITY is now entirely done by the V4L2 core, so we can drop the g/s_priority ioctl ops. We do have to make sure though that when S_PRIORITY is called we check that the driver used struct v4l2_fh. This check can be removed

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 09:04:03 Rob Clark wrote: On Tue, Feb 3, 2015 at 2:48 AM, Daniel Vetter dan...@ffwll.ch wrote: On Mon, Feb 02, 2015 at 03:30:21PM -0500, Rob Clark wrote: On Mon, Feb 2, 2015 at 11:54 AM, Daniel Vetter dan...@ffwll.ch wrote: My initial thought is for dma-buf to

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Christian Gmeiner
2015-02-03 13:28 GMT+01:00 Russell King - ARM Linux li...@arm.linux.org.uk: On Tue, Feb 03, 2015 at 08:48:56AM +0100, Daniel Vetter wrote: On Mon, Feb 02, 2015 at 03:30:21PM -0500, Rob Clark wrote: On Mon, Feb 2, 2015 at 11:54 AM, Daniel Vetter dan...@ffwll.ch wrote: My initial thought is

Re: Re: [BUG, workaround] HVR-2200/saa7164 problem with C7 power state

2015-02-03 Thread dcr...@telefonica.net
Mensaje original De : st...@kernellabs.com Fecha : 02/02/2015 - 16:39 (GMT) Para : dcr...@telefonica.net CC : linux-media@vger.kernel.org Asunto : Re: [BUG, workaround] HVR-2200/saa7164 problem with C7 power state Basically, it starts working but after a while I get an Event timed out

Re: [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 2:48 AM, Daniel Vetter dan...@ffwll.ch wrote: On Mon, Feb 02, 2015 at 03:30:21PM -0500, Rob Clark wrote: On Mon, Feb 2, 2015 at 11:54 AM, Daniel Vetter dan...@ffwll.ch wrote: My initial thought is for dma-buf to not try to prevent something than an exporter can

[PATCH] [media] stk-webcam: Delete an unnecessary check before the function call vfree

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 18:36:35 +0100 The vfree() function performs also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH] [media] au0828: Delete unnecessary checks before the function call video_unregister_device

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring elfr...@users.sourceforge.net Date: Tue, 3 Feb 2015 19:00:25 +0100 The video_unregister_device() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software.

Re: [PATCH 21/66] rtl2830: implement own I2C locking

2015-02-03 Thread Antti Palosaari
On 02/03/2015 07:53 PM, Mauro Carvalho Chehab wrote: Em Mon, 02 Feb 2015 21:33:24 +0100 Wolfram Sang w...@the-dreams.de escreveu: Ok, this may eventually work ok for now, but a further change at the I2C core could easily break it. So, we need to double check about such patch with the I2C

[PATCH] rc: img-ir: Add and enable sys clock for IR

2015-02-03 Thread Sifan Naeem
Gets a handle to the system clock, already described in the binding document, and calls the appropriate common clock framework functions to mark it prepared/enabled, the common clock framework initially enables the clock and doesn't disable it at least until the device/driver is removed. The

[PATCH 1/3] [media] rtl2830: declare functions as static

2015-02-03 Thread Mauro Carvalho Chehab
drivers/media/dvb-frontends/rtl2830.c:21:5: warning: no previous prototype for ‘rtl2830_bulk_write’ [-Wmissing-prototypes] int rtl2830_bulk_write(struct i2c_client *client, unsigned int reg, ^ drivers/media/dvb-frontends/rtl2830.c:33:5: warning: no previous prototype for

[PATCH 3/3] [media] rtl28xxu: properly initialize pdata

2015-02-03 Thread Mauro Carvalho Chehab
As complained by smatch: drivers/media/usb/dvb-usb-v2/rtl28xxu.c:1159 rtl2832u_tuner_attach() info: 'pdata' is not actually initialized (unreached code). Cc: Antti Palosaari cr...@iki.fi Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com diff --git

[PATCH 2/3] [media] rtl2832: declare functions as static

2015-02-03 Thread Mauro Carvalho Chehab
drivers/media/dvb-frontends/rtl2832.c:157:5: warning: no previous prototype for ‘rtl2832_bulk_write’ [-Wmissing-prototypes] int rtl2832_bulk_write(struct i2c_client *client, unsigned int reg, ^ drivers/media/dvb-frontends/rtl2832.c:169:5: warning: no previous prototype for

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Rob Clark
On Tue, Feb 3, 2015 at 11:58 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: Okay, but switching contexts is not something which the DMA API has any knowledge of (so it can't know which context to associate with which mapping.) While it knows which device, it has no knowledge (nor

Re: [PATCH 21/66] rtl2830: implement own I2C locking

2015-02-03 Thread Mauro Carvalho Chehab
Em Mon, 02 Feb 2015 21:33:24 +0100 Wolfram Sang w...@the-dreams.de escreveu: Ok, this may eventually work ok for now, but a further change at the I2C core could easily break it. So, we need to double check about such patch with the I2C maintainer. Jean, Are you ok with such

Re: [PATCH 5/5] lmedm04: add read snr, signal strength and ber call backs

2015-02-03 Thread Mauro Carvalho Chehab
Em Fri, 2 Jan 2015 13:56:31 + Malcolm Priestley tvbox...@gmail.com escreveu: This allows calling the original functions providing the streaming is off. Malcolm, I'm applying this patch series, as the driver has already some support for the legacy DVBv3 stats, but please port it to use

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Daniel Vetter
On Tue, Feb 03, 2015 at 05:36:59PM +0100, Arnd Bergmann wrote: On Tuesday 03 February 2015 11:22:01 Rob Clark wrote: On Tue, Feb 3, 2015 at 11:12 AM, Arnd Bergmann a...@arndb.de wrote: I agree for the case you are describing here. From what I understood from Rob was that he is looking at

Re: [PATCH 5/5] lmedm04: add read snr, signal strength and ber call backs

2015-02-03 Thread Malcolm Priestley
On 03/02/15 19:19, Mauro Carvalho Chehab wrote: Em Fri, 2 Jan 2015 13:56:31 + Malcolm Priestley tvbox...@gmail.com escreveu: This allows calling the original functions providing the streaming is off. Malcolm, I'm applying this patch series, as the driver has already some support for

Re: [PATCH 5/5] lmedm04: add read snr, signal strength and ber call backs

2015-02-03 Thread Malcolm Priestley
On 03/02/15 19:44, Mauro Carvalho Chehab wrote: Em Tue, 03 Feb 2015 19:31:16 + Malcolm Priestley tvbox...@gmail.com escreveu: On 03/02/15 19:19, Mauro Carvalho Chehab wrote: Em Fri, 2 Jan 2015 13:56:31 + Malcolm Priestley tvbox...@gmail.com escreveu: This allows calling the

Re: [PATCH 5/5] lmedm04: add read snr, signal strength and ber call backs

2015-02-03 Thread Mauro Carvalho Chehab
Em Tue, 03 Feb 2015 19:31:16 + Malcolm Priestley tvbox...@gmail.com escreveu: On 03/02/15 19:19, Mauro Carvalho Chehab wrote: Em Fri, 2 Jan 2015 13:56:31 + Malcolm Priestley tvbox...@gmail.com escreveu: This allows calling the original functions providing the streaming is

Re: [possible BUG, cx23885] Dual tuner TV card, works using one tuner only, doesn't work if both tuners are used

2015-02-03 Thread Steven Toth
While I am the maintainer of the cx23885 driver, its currently undergoing a significant amount of churn related to Han's recent VB2 and other changes. I consider the current driver broken until the feedback on the mailing list dies down. I'm reluctant to work on the driver while its considered

Re: [PATCH 5/5] lmedm04: add read snr, signal strength and ber call backs

2015-02-03 Thread Malcolm Priestley
On 03/02/15 19:31, Malcolm Priestley wrote: On 03/02/15 19:19, Mauro Carvalho Chehab wrote: Em Fri, 2 Jan 2015 13:56:31 + Malcolm Priestley tvbox...@gmail.com escreveu: This allows calling the original functions providing the streaming is off. Malcolm, I'm applying this patch

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Daniel Vetter
On Tue, Feb 03, 2015 at 12:35:34PM -0500, Rob Clark wrote: On Tue, Feb 3, 2015 at 11:58 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: Okay, but switching contexts is not something which the DMA API has any knowledge of (so it can't know which context to associate with which

Re: [PATCH] media:ic:Remove calls to msleep from the functions,ov7670_s_vflip and ov7670_s_hflip

2015-02-03 Thread Jonathan Corbet
On Mon, 2 Feb 2015 23:40:17 -0500 Nicholas Krause xerofo...@gmail.com wrote: Removes the no longer required calls to msleep for the duration of 10 miliseconds in the functions,ov7670_s_vflip and ov7670_s_hflip respectfully due to no need for a delayed sleep of this duration in either

Re: [PATCH] [media] dvb-usb: fix spaces after commas

2015-02-03 Thread Mauro Carvalho Chehab
Em Wed, 28 Jan 2015 16:11:22 +0100 Luis de Bethencourt l...@debethencourt.com escreveu: Please provide some description Signed-off-by: Luis de Bethencourt luis...@samsung.com Also, if you're touching on those printks, you should be fixing the scripts/checkpatch.pl warnings for that too:

Re: [git:media_tree/master] [media] lirc_dev: avoid potential null-dereference

2015-02-03 Thread Andy Shevchenko
On Tue, Feb 3, 2015 at 9:33 PM, Mauro Carvalho Chehab mche...@osg.samsung.com wrote: This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/cgit.cgi/media_tree.git tree: Subject: [media] lirc_dev: avoid potential

Re: [PATCH] add raw video support for Samsung SUR40 touchscreen

2015-02-03 Thread Florian Echtler
Sorry to bring this up again, but would it be acceptable to simply use dma-contig after all? Since the GFP_DMA flag is gone, this shouldn't be too big of an issue IMHO, and I was kind of hoping the patch could still be part of 3.20. Best, Florian On 29.01.2015 22:35, Florian Echtler wrote: I'm

Re: [BUG] - Why anyone fix this problem?

2015-02-03 Thread Olli Salonen
Well, if you suspect that some other change broke the driver, then you can try an older kernel (the support was introduced in kernel 3.10) or an older media_tree to see if that's indeed the case. The firmware is just one small piece of the puzzle. Maybe you can just replace the firmware or maybe

Re: [PATCH] media:dvb-frontends: Change setting of variable interval to the correct values in the function, hd29l2_get_frontend for the switch statement checking the frame header

2015-02-03 Thread Antti Palosaari
Yet another bad patch. Do not touch whole driver unless you understand and can test your patches! Antti On 02/03/2015 05:52 AM, Nicholas Krause wrote: This changes the switch statement checking the frame header of the pointer, c as a pointer to a structure of type,dtv_frontend_properties to

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Russell King - ARM Linux
On Tue, Feb 03, 2015 at 10:42:26PM +0100, Arnd Bergmann wrote: Right, if you have a private iommu, there is no problem. The tricky part is using a single driver for the system-level translation and the gpu private mappings when there is only one type of iommu in the system. You've got a

Re: [BUG] - Why anyone fix this problem?

2015-02-03 Thread Francesco Other
Maybe when Doron Cohen wrote the patch the device worked fine but now, after that someone change the code for their own enjoyment, it doesn't. If you read my question you will find that the device has signal lock but no data stream. There isn't need to write a code from scratch because I have the

Re: [PATCH] media: i2c: ADV7604: Migrate to regmap

2015-02-03 Thread Lars-Peter Clausen
On 02/01/2015 11:08 AM, Jean-Michel Hautbois wrote: Looks mostly good, some things in addition to what Hans already said. [...] - -static s32 adv_smbus_write_byte_data(struct adv7604_state *state, -enum adv7604_page page, u8 command, -

Re: [PATCH v3] dma-buf: cleanup dma_buf_export() to make it easily extensible

2015-02-03 Thread Daniel Thompson
On 28/01/15 12:54, Sumit Semwal wrote: At present, dma_buf_export() takes a series of parameters, which makes it difficult to add any new parameters for exporters, if required. Make it simpler by moving all these parameters into a struct, and pass the struct * as parameter to

Re: [PATCH] media: i2c: ADV7604: Migrate to regmap

2015-02-03 Thread Hans Verkuil
On 02/02/15 11:44, Hans Verkuil wrote: Hi Pablo, Jean-Michel, On 02/01/2015 11:08 AM, Jean-Michel Hautbois wrote: This is a preliminary patch in order to add support for ALSA. It replaces all current i2c access with regmap. Add the registers which will then be used too, as these are

Re: Vivid test device: adding YU12

2015-02-03 Thread Hans Verkuil
On 02/02/15 23:32, Miguel Casas-Sanchez wrote: On 01/29/2015 03:44 AM, Miguel Casas-Sanchez wrote: Hi folks, I've been trying to add a triplanar format to those that vivid can generate, and didn't quite manage :( So, I tried adding code for it like in the patch (with some dprintk() as well)

Re: [possible BUG, cx23885] Dual tuner TV card, works using one tuner only, doesn't work if both tuners are used

2015-02-03 Thread Steven Toth
Hi, I'm based in the US :( - Steve -- Steven Toth - Kernel Labs http://www.kernellabs.com On Tue, Feb 3, 2015 at 4:39 PM, dCrypt dcr...@telefonica.net wrote: Thanks Steve. If sending it really helps you in your development, I will gladly pay for the postage forth and back if you live in

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 21:04:35 Daniel Vetter wrote: - On many soc people love to reuse iommus with the same or similar interface all over the place. The solution thus far adopted on arm platforms is to write an iommu driver for those and then implement the dma-api on top of this

RE: [possible BUG, cx23885] Dual tuner TV card, works using one tuner only, doesn't work if both tuners are used

2015-02-03 Thread dCrypt
Thanks Steve. If sending it really helps you in your development, I will gladly pay for the postage forth and back if you live in Europe. When my recently purchased HVR-2200 is stable in my PVR (I could still use the Terratec disabling one tuner), I will consider donating the Terratec. BR

Re: [Linaro-mm-sig] [RFCv3 2/2] dma-buf: add helpers for sharing attacher constraints with dma-parms

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 12:35:34 Rob Clark wrote: I can't think of cases outside of GPU's.. if it were more common I'd be in favor of teaching dma api about multiple contexts, but right now I think that would just amount to forcing a lot of churn on everyone else for the benefit of