Re: [PATCH v2 2/2] x86/mm/pat, drivers/media/ivtv: move pat warn and replace WARN() with pr_warn()

2015-06-25 Thread Ingo Molnar
* Luis R. Rodriguez mcg...@do-not-panic.com wrote: From: Luis R. Rodriguez mcg...@suse.com On built-in kernels this warning will always splat as this is part of the module init. Fix that by shifting the PAT requirement check out under the code that does the quasi-probe for the device. This

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

2015-06-25 Thread Ingo Molnar
* Luis R. Rodriguez mcg...@do-not-panic.com wrote: From: Luis R. Rodriguez mcg...@suse.com 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

[PATCH] [media] pctv452e: Replace memset with eth_zero_addr

2015-06-25 Thread Vaishali Thakkar
Use eth_zero_addr to assign the zero address to the given address array instead of memset when second argument is address of zero. Note that the 6 in the third argument of memset appears to represent an ethernet address size (ETH_ALEN). The Coccinelle semantic patch that makes this change is as

[PATCH] [media] ttpci: Replace memset with eth_zero_addr

2015-06-25 Thread Vaishali Thakkar
Use eth_zero_addr to assign the zero address to the given address array instead of memset when second argument is address of zero. Note that the 6 in the third argument of memset appears to represent an ethernet address size (ETH_ALEN). The Coccinelle semantic patch that makes this change is as

Re: [PATCH 4/5] [media] adv7604: Deliver resolution change events to userspace

2015-06-25 Thread Lars-Peter Clausen
On 06/25/2015 12:21 PM, Sakari Ailus wrote: Hi Lars-Peter, On Wed, Jun 24, 2015 at 06:50:30PM +0200, Lars-Peter Clausen wrote: Use the new v4l2_subdev_notify_event() helper function to deliver the resolution change event to userspace via the v4l2 subdev event queue as well as to the bridge

Re: [PATCH 2/2] [media] videobuf2: add trace events

2015-06-25 Thread Steven Rostedt
On Thu, 25 Jun 2015 12:01:27 +0200 Philipp Zabel p.za...@pengutronix.de wrote: diff --git a/include/trace/events/v4l2.h b/include/trace/events/v4l2.h index 89d0497..3d15cf1 100644 --- a/include/trace/events/v4l2.h +++ b/include/trace/events/v4l2.h @@ -175,9 +175,108 @@ SHOW_FIELD

Re: [PATCH 5/5] [media] adv7842: Deliver resolution change events to userspace

2015-06-25 Thread Sakari Ailus
Hi Lars-peter, On Wed, Jun 24, 2015 at 06:50:31PM +0200, Lars-Peter Clausen wrote: Use the new v4l2_subdev_notify_event() helper function to deliver the resolution change event to userspace via the v4l2 subdev event queue as well as to the bridge driver using the callback notify mechanism.

Re: [PATCH 02/13] dmaengine: Introduce dma_request_slave_channel_compat_reason()

2015-06-25 Thread Arnd Bergmann
On Wednesday 24 June 2015 21:54:01 Vinod Koul wrote: It would be nice to find another name for the dma_request_slave_channel_compat() so with the new name we could have chance to rearrange the parameters: (dev, name, mask, fn, fn_param) We would end up with the following APIs, all

Re: [PATCH 1/2] [media] v4l2-mem2mem: set the queue owner field just as vb2_ioctl_reqbufs does

2015-06-25 Thread Sylwester Nawrocki
Hello Philipp, On 25/06/15 12:01, Philipp Zabel wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/v4l2-core/v4l2-mem2mem.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-mem2mem.c

Re: [PATCH 4/5] [media] adv7604: Deliver resolution change events to userspace

2015-06-25 Thread Sakari Ailus
Hi Lars-Peter, On Wed, Jun 24, 2015 at 06:50:30PM +0200, Lars-Peter Clausen wrote: Use the new v4l2_subdev_notify_event() helper function to deliver the resolution change event to userspace via the v4l2 subdev event queue as well as to the bridge driver using the callback notify mechanism.

Re: [PATCH 3/5] [media] Add helper function for subdev event notifications

2015-06-25 Thread Lars-Peter Clausen
On 06/25/2015 11:47 AM, Sakari Ailus wrote: Hi Lars-Peter, On Wed, Jun 24, 2015 at 06:50:29PM +0200, Lars-Peter Clausen wrote: Add a new helper function called v4l2_subdev_notify_event() which will deliver the specified event to both the v4l2 subdev event queue as well as to the notify

[GIT PULL for v4.2-rc1] media updates

2015-06-25 Thread Mauro Carvalho Chehab
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v4.2-1 For the following media updates: - Lots of improvements at the DVB API DocBook documentation. Now, the frontend and the network APIs are fully in sync with the Kernel and looks

Re: i.MX6 video capture support in mainline

2015-06-25 Thread Philipp Zabel
Hi Javier, Am Dienstag, den 23.06.2015, 12:20 +0200 schrieb Javier Martin: [...] media-ctl -l 'IPU0 SMFC0:1-imx-ipuv3-camera.2:0[1]' The last command will fail like this: imx-ipuv3 240.ipu: invalid link 'IPU0 SMFC0'(5):1 - 'imx-ipuv3-camera.2'(2):0 Unable to parse link: Invalid

[PATCH 1/2] [media] v4l2-mem2mem: set the queue owner field just as vb2_ioctl_reqbufs does

2015-06-25 Thread Philipp Zabel
Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/v4l2-core/v4l2-mem2mem.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-mem2mem.c b/drivers/media/v4l2-core/v4l2-mem2mem.c index dc853e5..511caaa 100644 ---

[PATCH 2/2] [media] videobuf2: add trace events

2015-06-25 Thread Philipp Zabel
Add videobuf2 specific vb2_qbuf and vb2_dqbuf trace events that mirror the v4l2_qbuf and v4l2_dqbuf trace events, only they include additional information about queue fill state and are emitted right before the buffer is enqueued in the driver or userspace is woken up. This allows to make sense of

[PATCH 1/1] usb/airspy: removing unneeded goto

2015-06-25 Thread Maninder Singh
This patch removes unneded goto, reported by coccinelle. Signed-off-by: Maninder Singh maninder...@samsung.com Reviewed-by: Akhilesh Kumar akhiles...@samsung.com --- drivers/media/usb/airspy/airspy.c |3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/media/usb/airspy/airspy.c

XC5000C 0x14b4 status

2015-06-25 Thread Unembossed Name
Hi, I was working on a Linux driver for a hybrid TV-tuner with SAA7134 PCI bridge, XC5000C RF tuner and Si2168 DVB demodulator by a combining all existent at that time drivers together. During that work, I had an issue with XC5000C. Episodically, after attaching to DVB and reading

Re: [PATCH 1/2] [media] v4l2-mem2mem: set the queue owner field just as vb2_ioctl_reqbufs does

2015-06-25 Thread Kamil Debski
Hi Philipp, On 25 June 2015 at 11:01, Philipp Zabel p.za...@pengutronix.de wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de Please add the patch description no matter how simple it is and how well the subject covers the content of the patch. Best wishes, Kamil ---

Re: [PATCH 1/2] [media] v4l2-mem2mem: set the queue owner field just as vb2_ioctl_reqbufs does

2015-06-25 Thread Philipp Zabel
Hi Sylwester, Am Donnerstag, den 25.06.2015, 15:11 +0200 schrieb Sylwester Nawrocki: Hello Philipp, On 25/06/15 12:01, Philipp Zabel wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de --- drivers/media/v4l2-core/v4l2-mem2mem.c | 9 - 1 file changed, 8 insertions(+), 1

Re: [PATCH 1/2] [media] v4l2-mem2mem: set the queue owner field just as vb2_ioctl_reqbufs does

2015-06-25 Thread Philipp Zabel
Hi Kamil, Am Donnerstag, den 25.06.2015, 15:10 +0100 schrieb Kamil Debski: Hi Philipp, On 25 June 2015 at 11:01, Philipp Zabel p.za...@pengutronix.de wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de Please add the patch description no matter how simple it is and how well the

Re: [PATCH 2/2] [media] videobuf2: add trace events

2015-06-25 Thread Philipp Zabel
Am Donnerstag, den 25.06.2015, 09:07 -0400 schrieb Steven Rostedt: On Thu, 25 Jun 2015 12:01:27 +0200 Philipp Zabel p.za...@pengutronix.de wrote: diff --git a/include/trace/events/v4l2.h b/include/trace/events/v4l2.h index 89d0497..3d15cf1 100644 --- a/include/trace/events/v4l2.h +++

Re: [PATCH v2 2/2] x86/mm/pat, drivers/media/ivtv: move pat warn and replace WARN() with pr_warn()

2015-06-25 Thread Luis R. Rodriguez
On Thu, Jun 25, 2015 at 08:51:47AM +0200, Ingo Molnar wrote: * Luis R. Rodriguez mcg...@do-not-panic.com wrote: From: Luis R. Rodriguez mcg...@suse.com On built-in kernels this warning will always splat as this is part of the module init. Fix that by shifting the PAT requirement

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

2015-06-25 Thread Luis R. Rodriguez
On Thu, Jun 25, 2015 at 08:49:22AM +0200, Ingo Molnar wrote: * Luis R. Rodriguez mcg...@do-not-panic.com wrote: From: Luis R. Rodriguez mcg...@suse.com WARN() may confuse users, fix that. ipath_init_one() is part the device's probe so this would only be triggered if a corresponding

Re: [PATCH 5/7] [media] omap3isp: remove unused var

2015-06-25 Thread Laurent Pinchart
Hi Mauro, On Wednesday 24 June 2015 07:49:09 Mauro Carvalho Chehab wrote: drivers/media/platform/omap3isp/isppreview.c:932:6: warning: variable ‘features’ set but not used [-Wunused-but-set-variable] Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com Acked-by: Laurent Pinchart

Re: [patch] [media] v4l: xilinx: missing error code

2015-06-25 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Wednesday 24 June 2015 17:28:31 Dan Carpenter wrote: We should set ret on this error path instead of returning success. Fixes: df3305156f98 ('[media] v4l: xilinx: Add Xilinx Video IP core') Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Acked-by:

cron job: media_tree daily build: OK

2015-06-25 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: Fri Jun 26 04:00:24 CEST 2015 git branch: test git hash: faebbd8f134f0c054f372982c8ddd1bbcc41b440 gcc

Re: [PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.

2015-06-25 Thread Paul Bolle
On Wed, 2015-06-24 at 16:11 +0100, Peter Griffin wrote: --- /dev/null +++ b/drivers/media/tsin/c8sectpfe/Makefile +c8sectpfe-y += c8sectpfe-core.o c8sectpfe-common.o c8sectpfe-dvb.o + +obj-$(CONFIG_DVB_C8SECTPFE) += c8sectpfe.o + +ifneq ($(CONFIG_DVB_C8SECTPFE),) + c8sectpfe-y +=

Re: [PATCH] v4l2-event: v4l2_event_queue: do nothing if vdev == NULL

2015-06-25 Thread Sakari Ailus
Hi Hans, On Tue, Jun 23, 2015 at 11:20:23AM +0200, Hans Verkuil wrote: If the vdev pointer == NULL, then just return. This makes it easier for subdev drivers to use this function without having to check if the sd-devnode pointer is NULL or not. Do you have an example of when this would be

Re: Build regressions/improvements in v4.1

2015-06-25 Thread Russell King - ARM Linux
On Tue, Jun 23, 2015 at 09:50:00AM +0200, Geert Uytterhoeven wrote: Hi Russell, On Mon, Jun 22, 2015 at 11:18 PM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Mon, Jun 22, 2015 at 10:52:13PM +0200, Geert Uytterhoeven wrote: On Mon, Jun 22, 2015 at 10:48 PM, Geert Uytterhoeven

Re: Build regressions/improvements in v4.1

2015-06-25 Thread Russell King - ARM Linux
On Thu, Jun 25, 2015 at 10:30:07AM +0100, Russell King - ARM Linux wrote: On Thu, Jun 25, 2015 at 10:18:15AM +0100, Russell King - ARM Linux wrote: On Tue, Jun 23, 2015 at 09:50:00AM +0200, Geert Uytterhoeven wrote: As for the build errors you're reporting, that doesn't seem to be anything

Re: [PATCH] v4l2-event: v4l2_event_queue: do nothing if vdev == NULL

2015-06-25 Thread Lars-Peter Clausen
On 06/25/2015 11:12 AM, Sakari Ailus wrote: Hi Hans, On Tue, Jun 23, 2015 at 11:20:23AM +0200, Hans Verkuil wrote: If the vdev pointer == NULL, then just return. This makes it easier for subdev drivers to use this function without having to check if the sd-devnode pointer is NULL or not. Do

[PATCH 09/15] media: soc_camera pad-aware driver initialisation

2015-06-25 Thread William Towle
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 (for composite) underneath. Building rcar_vin

[PATCH 10/15] media: rcar_vin: Use correct pad number in try_fmt

2015-06-25 Thread William Towle
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 rob.tay...@codethink.co.uk ---

[PATCH 02/15] media: soc_camera: rcar_vin: Add BT.709 24-bit RGB888 input support

2015-06-25 Thread William Towle
This adds V4L2_MBUS_FMT_RGB888_1X24 input format support which is used by the ADV7612 chip. Signed-off-by: Koji Matsuoka koji.matsuoka...@renesas.com Signed-off-by: Simon Horman horms+rene...@verge.net.au Signed-off-by: Yoshihiro Kaneko ykaneko0...@gmail.com Modified to use MEDIA_BUS_FMT_*

Re: Build regressions/improvements in v4.1

2015-06-25 Thread Russell King - ARM Linux
On Thu, Jun 25, 2015 at 10:18:15AM +0100, Russell King - ARM Linux wrote: On Tue, Jun 23, 2015 at 09:50:00AM +0200, Geert Uytterhoeven wrote: As for the build errors you're reporting, that doesn't seem to be anything new. It seems to be down to a missing dependency between ARM_PTDUMP and MMU,

[PATCH 07/15] ARM: shmobile: lager dts: specify default-input for ADV7612

2015-06-25 Thread William Towle
Set 'default-input' property for ADV7612. Enables image/video capture without the need to have userspace specifying routing. Signed-off-by: Ian Molton ian.mol...@codethink.co.uk Tested-by: William Towle william.to...@codethink.co.uk --- arch/arm/boot/dts/r8a7790-lager.dts |1 + 1 file

[PATCH 04/15] media: adv7604: chip info and formats for ADV7612

2015-06-25 Thread William Towle
Add support for the ADV7612 chip as implemented on Renesas' Lager board to adv7604.c, including lists for formats/colourspace/timing selection and an IRQ handler. Signed-off-by: William Towle william.to...@codethink.co.uk --- drivers/media/i2c/adv7604.c | 102

[PATCH 01/15] ARM: shmobile: lager dts: Add entries for VIN HDMI input support

2015-06-25 Thread William Towle
Add DT entries for vin0, vin0_pins, and adv7612 Signed-off-by: William Towle william.to...@codethink.co.uk Signed-off-by: Rob Taylor rob.tay...@codethink.co.uk --- arch/arm/boot/dts/r8a7790-lager.dts | 41 ++- 1 file changed, 40 insertions(+), 1 deletion(-)

[PATCH 03/15] media: adv7180: add of match table

2015-06-25 Thread William Towle
From: Ben Dooks ben.do...@codethink.co.uk Add a proper of match id for use when the device is being bound via device tree, to avoid having to use the i2c old-style binding of the device. Signed-off-by: Ben Dooks ben.do...@codethink.co.uk Signed-off-by: William.Towle william.to...@codethink.co.uk

HDMI and Composite capture on Lager, for kernel 4.1, version 4

2015-06-25 Thread William Towle
Version 4. Obsoletes version 3, as seen at: http://permalink.gmane.org/gmane.linux.drivers.video-input-infrastructure/91931 This version has the bus_info field set in rcar_vin.c, where the value used can be constructed uniquely. Elsewhere a number of small optimisations have been

[PATCH 08/15] v4l: subdev: Add pad config allocator and init

2015-06-25 Thread William Towle
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 subdev pad operations. Signed-off-by: Laurent

[PATCH 05/15] media: adv7604: document support for ADV7612 dual HDMI input decoder

2015-06-25 Thread William Towle
From: Ian Molton ian.mol...@codethink.co.uk This documentation accompanies the patch adding support for the ADV7612 dual HDMI decoder / repeater chip. Signed-off-by: Ian Molton ian.mol...@codethink.co.uk Reviewed-by: William Towle william.to...@codethink.co.uk Acked-by: Hans Verkuil

[PATCH 12/15] media: soc_camera: Fill std field in enum_input

2015-06-25 Thread William Towle
From: Hans Verkuil hans.verk...@cisco.com Fill in the std field from the video_device tvnorms field in enum_input. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Rob Taylor rob.tay...@codethink.co.uk --- drivers/media/platform/soc_camera/soc_camera.c |3 +++ 1 file

[PATCH 13/15] media: soc_camera: Fix error reporting in expbuf

2015-06-25 Thread William Towle
From: Hans Verkuil hans.verk...@cisco.com Remove unnecessary check and fix the error code for vb1 drivers. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Signed-off-by: Rob Taylor rob.tay...@codethink.co.uk --- drivers/media/platform/soc_camera/soc_camera.c |8 ++-- 1 file changed,

[PATCH 15/15] media: rcar_vin: Reject videobufs that are too small for current format

2015-06-25 Thread William Towle
From: Rob Taylor rob.tay...@codethink.co.uk In videobuf_setup reject buffers that are too small for the configured format. Fixes v4l2-compliance issue. Signed-off-by: Rob Taylor rob.tay...@codethink.co.uk Reviewed-by: William Towle william.to...@codethink.co.uk Acked-by: Hans Verkuil

[PATCH 14/15] media: rcar_vin: fill in bus_info field

2015-06-25 Thread William Towle
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 william.to...@codethink.co.uk ---

[PATCH 11/15] media: soc_camera: soc_scale_crop: Use correct pad number in try_fmt

2015-06-25 Thread William Towle
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 william.to...@codethink.co.uk Reviewed-by: Rob Taylor

[PATCH 06/15] media: adv7604: ability to read default input port from DT

2015-06-25 Thread William Towle
From: Ian Molton ian.mol...@codethink.co.uk Adds support to the adv7604 driver for specifying the default input port in the Device tree. If no value is provided, the driver will be unable to select an input without help from userspace. Tested-by: William Towle william.to...@codethink.co.uk

Re: [PATCH] v4l2-event: v4l2_event_queue: do nothing if vdev == NULL

2015-06-25 Thread Sakari Ailus
Hi Lars-Peter, On Thu, Jun 25, 2015 at 11:22:02AM +0200, Lars-Peter Clausen wrote: On 06/25/2015 11:12 AM, Sakari Ailus wrote: Hi Hans, On Tue, Jun 23, 2015 at 11:20:23AM +0200, Hans Verkuil wrote: If the vdev pointer == NULL, then just return. This makes it easier for subdev drivers to

Re: [PATCH 3/5] [media] Add helper function for subdev event notifications

2015-06-25 Thread Sakari Ailus
Hi Lars-Peter, On Wed, Jun 24, 2015 at 06:50:29PM +0200, Lars-Peter Clausen wrote: Add a new helper function called v4l2_subdev_notify_event() which will deliver the specified event to both the v4l2 subdev event queue as well as to the notify callback. The former is typically used by userspace