[GIT PULL FOR v4.3] Helper to abstract vma handling in media layer

2015-07-24 Thread Hans Verkuil
From Jan Kara's cover letter: I'm sending the seventh version of my patch series to abstract vma handling from the various media drivers. Since the previous version there are just minor cleanups and fixes (see detailed changelog at the end of the email). After this patch set drivers have to know

[GIT PULL FOR v4.3] usbvision/zoran/fsl-viu: convert to the control framework

2015-07-24 Thread Hans Verkuil
I've cleaned up some old patch series of mine that convert the usbvision, zoran and fsl-viu drivers to the control framework. I have tested the zoran and usbvision drivers, the fsl-viu driver has been tested back in 2013: http://www.spinics.net/lists/linux-media/msg61898.html I said these patch

Re: [PATCH] [media] DocBook: Fix typo in intro.xml

2015-07-24 Thread Hans Verkuil
On 07/14/2015 08:38 PM, Jonathan Corbet wrote: On Tue, 14 Jul 2015 08:36:50 +0900 Masanari Iida standby2...@gmail.com wrote: This patch fix spelling typos in intro.xml. This xml file is not created from comments within source, I fix the xml file. Applied to the docs tree, thanks. Jon,

Re: [PATCH 1/1] v4l: subdev: Serialise open and release internal ops

2015-07-24 Thread Hans Verkuil
Hi Sakari, On 07/22/2015 06:14 PM, Sakari Ailus wrote: By default, serialise open and release internal ops using a mutex. The underlying problem is that a large proportion of the drivers do use v4l2_fh_is_singular() in their open() handlers (struct v4l2_subdev_internal_ops).

[RFC PATCH 0/7] Simplify first open/last close checks

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com This patch series makes a number of API improvements to simplify checking if an open or close of a filehandle was the first or last. Regards, Hans Hans Verkuil (7): v4l2-fh: change int to bool for v4l2_fh_is_singular(_file) v4l2-fh:

[RFC PATCH 1/7] v4l2-fh: change int to bool for v4l2_fh_is_singular(_file)

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com This function really returns a bool, so use that as the return type instead of int. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/v4l2-core/v4l2-fh.c | 6 +++--- include/media/v4l2-fh.h | 10 +- 2 files changed,

[RFC PATCH 2/7] v4l2-fh: v4l2_fh_add/del now return whether it was the first or last fh.

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Simplify driver code by letting v4l2_fh_add/del return true if the filehandle was the first open or the last close. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/v4l2-core/v4l2-fh.c | 10 -- include/media/v4l2-fh.h

[RFC PATCH 3/7] v4l2-fh: add v4l2_fh_open_is_first and v4l2_fh_release_is_last

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Add new helper functions that report back if this was the first open or last close. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/v4l2-core/v4l2-fh.c | 25 ++--- include/media/v4l2-fh.h | 27

[RFC PATCH 7/7] cpia2/si470x: simplify open

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com With the new v4l2_fh_open_is_first function the check whether a file open is the first is now much easier. Implement this in these two drivers. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/radio/si470x/radio-si470x-i2c.c | 21

[RFC PATCH 6/7] am437x/exynos4-is/marvell-ccic/sh_vou: simplify release()

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com Use the _vb2_fop_release() return code to determine if this was the last close or not. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/platform/am437x/am437x-vpfe.c | 13 +++--

[RFC PATCH 5/7] vb2: _vb2_fop_release returns if this was the last fh.

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com _vb2_fop_release now returns true if this was the last open fh. This will simplify drivers that call it and that need to check if it was the last close. Signed-off-by: Hans Verkuil hans.verk...@cisco.com --- drivers/media/v4l2-core/videobuf2-core.c | 7

[RFC PATCH 4/7] am437x-vpfe/fimc-capture: always return 0 on close

2015-07-24 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com When the filehandle is closed always return 0 and ignore the return code from _vb2_fop_release(). Currently _vb2_fop_release() always returns 0, but this will change in the next patch where _vb2_fop_release() will return a boolean telling the caller if

Re: [PATCH 07/13] media: soc_camera pad-aware driver initialisation

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: Add detection of source pad number for drivers aware of the media controller API, so that the combination of soc_camera and rcar_vin can create device nodes to support modern drivers such as adv7604.c (for HDMI on Lager) and the converted adv7180.c

Re: [PATCH 03/13] media: adv7604: fix probe of ADV7611/7612

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: Prior to commit f862f57d ([media] media: i2c: ADV7604: Migrate to regmap), the local variable 'val' contained the combined register reads used in the chipset version ID test. Restore this expectation so that the comparison works as it used to. ---

Re: [PATCH 12/13] media: rcar_vin: fill in bus_info field

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: From: Rob Taylor rob.tay...@codethink.co.uk Adapt rcar_vin_querycap() so that cap-bus_info is populated with something meaningful/unique. Signed-off-by: Rob Taylor rob.tay...@codethink.co.uk Signed-off-by: William Towle

Re: [PATCH 09/13] media: soc_camera: soc_scale_crop: Use correct pad number in try_fmt

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: From: Rob Taylor rob.tay...@codethink.co.uk Fix calls to subdev try_fmt function to use valid pad numbers, fixing the case where subdevs (eg. ADV7612) have valid pad numbers that are non-zero. Signed-off-by: William Towle

Re: [Linux-kernel] [PATCH 03/13] media: adv7604: fix probe of ADV7611/7612

2015-07-24 Thread Ben Dooks
On 23/07/15 13:21, William Towle wrote: Prior to commit f862f57d ([media] media: i2c: ADV7604: Migrate to regmap), the local variable 'val' contained the combined register reads used in the chipset version ID test. Restore this expectation so that the comparison works as it used to. Forgot

Re: [PATCH v3 4/5] [media] imx-ipu: Add ipu media common code

2015-07-24 Thread Hans Verkuil
Hi Philip, A quick review: On 07/16/2015 06:24 PM, Philipp Zabel wrote: From: Sascha Hauer s.ha...@pengutronix.de Add video4linux API routines common to drivers for units that accept or provide video data via the i.MX IPU IDMAC channels, such as scaler or deinterlacer drivers.

Re: [PATCH v3 5/5] [media] imx-ipu: Add i.MX IPUv3 scaler driver

2015-07-24 Thread Hans Verkuil
Hi Philipp, A quick review of this driver: On 07/16/2015 06:24 PM, Philipp Zabel wrote: From: Sascha Hauer s.ha...@pengutronix.de This patch adds support for hardware accelerated scaling and color space conversion between memory buffers using the IPUv3 IC. Since the maximum output size of

Re: [PATCH 08/13] media: rcar_vin: Use correct pad number in try_fmt

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: Fix rcar_vin_try_fmt's use of an inappropriate pad number when calling the subdev set_fmt function - for the ADV7612, IDs should be non-zero. Signed-off-by: William Towle william.to...@codethink.co.uk Reviewed-by: Rob Taylor

Re: [PATCH 04/14] tw9910: init priv-scale and update standard

2015-07-24 Thread Hans Verkuil
Guennadi, I want to make a pull request for this patch series. This patch is the only outstanding one. Or do you have to review more patches? I only got Acks for patches 1 and 2. Regards, Hans On 06/22/2015 09:29 AM, Hans Verkuil wrote: On 06/22/2015 09:21 AM, Guennadi Liakhovetski

Re: [PATCH 04/13] media: adv7604: reduce support to first (digital) input

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: Using adv7611_read_cable_det() for ADV7612 means that full support for '.max_port = ADV7604_PAD_HDMI_PORT_B,' isn't available due to the need for multiple port reads to determine cable detection, and an agreed mechanism for communicating the

Re: [PATCH 05/13] v4l: subdev: Add pad config allocator and init

2015-07-24 Thread Hans Verkuil
On 07/23/2015 02:21 PM, William Towle wrote: From: Laurent Pinchart laurent.pinch...@linaro.org Add a new subdev operation to initialize a subdev pad config array, and a helper function to allocate and initialize the array. This can be used by bridge drivers to implement try format based on

cron job: media_tree daily build: ERRORS

2015-07-24 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: Sat Jul 25 04:00:18 CEST 2015 git branch: test git hash: 4dc102b2f53d63207fa12a6ad49c7b6448bc3301 gcc

Re: [PATCH] [media] DocBook: Fix typo in intro.xml

2015-07-24 Thread Jonathan Corbet
On Fri, 24 Jul 2015 10:33:51 +0200 Hans Verkuil hverk...@xs4all.nl wrote: Jon, would you mind if I take this patch and let it go through the media tree? I'd like to apply a patch on top of this one that removes the mention of devfs. Fine, I'll drop it. It makes more sense in general to take

Re: [PATCH] [media] vmc: Virtual Media Controller core, capture and sensor

2015-07-24 Thread Hans Verkuil
Hi Helen, Thank you for creating this driver! Much appreciated! I do have some comments, see my notes below... On 07/18/2015 04:42 PM, Helen Fornazier wrote: First version of the Virtual Media Controller. Add a simple version of the core of the driver, the capture and sensor nodes in the

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

2015-07-24 Thread Steven Toth
What happens if you revert this back to the code from my original patch, and include your changes listed below? IE: /* Tri-state the TS bus */ si2168_set_ts_mode(fe, 1); 1) Do you still lock no signal lock issues. MythTV says 'no lock' (though I don't know if such a message is

[GIT PULL FOR v4.3] Various fixes, R-Car JPEG Processing Unit driver

2015-07-24 Thread Hans Verkuil
The following changes since commit 4dc102b2f53d63207fa12a6ad49c7b6448bc3301: [media] dvb_core: Replace memset with eth_zero_addr (2015-07-22 13:32:21 -0300) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v4.3d for you to fetch changes up to

Re: [PATCH] [media] DocBook: Fix typo in intro.xml

2015-07-24 Thread Hans Verkuil
On 07/24/2015 03:10 PM, Jonathan Corbet wrote: On Fri, 24 Jul 2015 10:33:51 +0200 Hans Verkuil hverk...@xs4all.nl wrote: Jon, would you mind if I take this patch and let it go through the media tree? I'd like to apply a patch on top of this one that removes the mention of devfs. Fine,