cron job: media_tree daily build: WARNINGS

2017-05-17 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: Thu May 18 05:00:23 CEST 2017 media-tree git hash:3622d3e77ecef090b5111e3c5423313f11711dfa media_build gi

[PATCH] Staging: media: fix missing blank line coding style issue in atomisp_tpg.c

2017-05-17 Thread Manny Vindiola
This is a patch to the atomisp_tpg.c file that fixes up a missing blank line warning found by the checkpatch.pl tool Signed-off-by: Manny Vindiola --- drivers/staging/media/atomisp/pci/atomisp2/atomisp_tpg.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/media/atomisp/pci/at

Re: [PATCH v1 1/3] of: base: Provide of_graph_get_port_parent()

2017-05-17 Thread Kuninori Morimoto
Hi Kieran > >> From: Kieran Bingham > >> > >> When handling endpoints, the v4l2 async framework needs to identify the > >> parent device of a port endpoint. > >> > >> Adapt the existing of_graph_get_remote_port_parent() such that a caller > >> can obtain the parent of a port without parsing the

[PATCH] media: platform: coda: remove variable self assignment

2017-05-17 Thread Gustavo A. R. Silva
Remove variable self assignment. Addresses-Coverity-ID: 1408817 Signed-off-by: Gustavo A. R. Silva --- drivers/media/platform/coda/coda-common.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/platform/coda/coda-common.c b/drivers/media/platform/coda/coda-common.c index d523e99

Re: [PATCH v3 2/2] v4l: vsp1: Provide a writeback video device

2017-05-17 Thread Kieran Bingham
Hi Geert, On 16/05/17 16:30, Geert Uytterhoeven wrote: > Hi Kieran, > > On Tue, May 9, 2017 at 6:39 PM, Kieran Bingham > wrote: >> When the VSP1 is used in an active display pipeline, the output of the >> WPF can supply the LIF entity directly and simultaneously write to >> memory. >> >> Support

Re: [PATCH] rc-core: cleanup rc_register_device (v2)

2017-05-17 Thread Sean Young
Hi David, On Wed, May 03, 2017 at 12:04:00PM +0200, David Härdeman wrote: > The device core infrastructure is based on the presumption that > once a driver calls device_add(), it must be ready to accept > userspace interaction. > > This requires splitting rc_setup_rx_device() into two functions >

Re: [PATCH v1 1/3] of: base: Provide of_graph_get_port_parent()

2017-05-17 Thread Kieran Bingham
On 17/05/17 17:36, Rob Herring wrote: > On Wed, May 17, 2017 at 10:03 AM, Kieran Bingham wrote: >> From: Kieran Bingham >> >> When handling endpoints, the v4l2 async framework needs to identify the >> parent device of a port endpoint. >> >> Adapt the existing of_graph_get_remote_port_parent() suc

Re: [PATCH v3.1 1/2] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-05-17 Thread Sakari Ailus
Hi Kieran, On Wed, May 17, 2017 at 04:38:14PM +0100, Kieran Bingham wrote: > From: Kieran Bingham > > Return NULL, if a null entity is parsed for it's v4l2_subdev > > Signed-off-by: Kieran Bingham > --- > include/media/v4l2-subdev.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >

Adding a cropping capability to the atmel-isc driver

2017-05-17 Thread Patrick Doyle
>From reading the discussion of the comparison of the selection API with the cropping API, (at https://linuxtv.org/downloads/v4l-dvb-apis/uapi/v4l/selection-api-005.html) it seems that the cropping API has been deprecated in favor of the selection API, so if I want to add a cropping capability to t

[PATCH 2/5] [media] sir_ir: use dev managed resources

2017-05-17 Thread Sean Young
Several error paths do not free up resources. This simplifies the code and fixes this. Signed-off-by: Sean Young --- drivers/media/rc/sir_ir.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc/sir_ir.c index c27d6b4..1ee41ad

[PATCH 5/5] [media] staging: remove todo and replace with lirc_zilog todo

2017-05-17 Thread Sean Young
The lirc_zilog driver is the last remaining lirc driver, so the existing todo is no longer relevant. Signed-off-by: Sean Young --- drivers/staging/media/lirc/TODO| 47 ++ drivers/staging/media/lirc/TODO.lirc_zilog | 36 --- 2 files chan

[PATCH 0/5] sir_ir fixes and update todo

2017-05-17 Thread Sean Young
Just some minor cleanups. Sean Young (5): [media] sir_ir: attempt to free already free_irq [media] sir_ir: use dev managed resources [media] sir_ir: remove init_port and drop_port functions [media] sir_ir: remove init_chrdev and init_sir_ir functions [media] staging: remove todo and repl

[PATCH 1/5] [media] sir_ir: attempt to free already free_irq

2017-05-17 Thread Sean Young
If the probe fails (e.g. port already in use), rmmod causes null deref. Signed-off-by: Sean Young --- drivers/media/rc/sir_ir.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc/sir_ir.c index 90a5f8f..c27d6b4 100644 --- a/driver

[PATCH 3/5] [media] sir_ir: remove init_port and drop_port functions

2017-05-17 Thread Sean Young
These functions are too short and removing them makes the code more readable. Signed-off-by: Sean Young --- drivers/media/rc/sir_ir.c | 27 +-- 1 file changed, 5 insertions(+), 22 deletions(-) diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc/sir_ir.c index 1ee41

[PATCH 4/5] [media] sir_ir: remove init_chrdev and init_sir_ir functions

2017-05-17 Thread Sean Young
Inlining these functions into the probe function makes it much more readable. Signed-off-by: Sean Young --- drivers/media/rc/sir_ir.c | 58 ++- 1 file changed, 22 insertions(+), 36 deletions(-) diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc

Re: [PATCH v1 1/3] of: base: Provide of_graph_get_port_parent()

2017-05-17 Thread Rob Herring
On Wed, May 17, 2017 at 10:03 AM, Kieran Bingham wrote: > From: Kieran Bingham > > When handling endpoints, the v4l2 async framework needs to identify the > parent device of a port endpoint. > > Adapt the existing of_graph_get_remote_port_parent() such that a caller > can obtain the parent of a p

[PATCH v3.1 1/2] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham Return NULL, if a null entity is parsed for it's v4l2_subdev Signed-off-by: Kieran Bingham --- include/media/v4l2-subdev.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h index 5f1669c45642..72d7

[PATCH v5 3/3] [media] platform: video-mux: include temporary mmio-mux support

2017-05-17 Thread Philipp Zabel
As long as the mux framework is not merged, add temporary mmio-mux support to the video-mux driver itself. This patch is to be reverted once the "mux: minimal mux subsystem" and "mux: mmio-based syscon mux controller" patches are merged. Signed-off-by: Philipp Zabel --- This patch allows the vide

[PATCH v5 2/3] platform: add video-multiplexer subdevice driver

2017-05-17 Thread Philipp Zabel
This driver can handle SoC internal and external video bus multiplexers, controlled by mux controllers provided by the mux controller framework, such as MMIO register bitfields or GPIOs. The subdevice passes through the mbus configuration of the active input to the output side. Signed-off-by: Sasc

[PATCH v5 1/3] dt-bindings: Add bindings for video-multiplexer device

2017-05-17 Thread Philipp Zabel
Add bindings documentation for the video multiplexer device. Signed-off-by: Sascha Hauer Signed-off-by: Philipp Zabel Signed-off-by: Steve Longerbeam Acked-by: Sakari Ailus Reviewed-by: Sebastian Reichel Acked-by: Rob Herring --- No changes since v4 [1]. [1] https://patchwork.kernel.org/pat

[PATCH v1 1/3] of: base: Provide of_graph_get_port_parent()

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham When handling endpoints, the v4l2 async framework needs to identify the parent device of a port endpoint. Adapt the existing of_graph_get_remote_port_parent() such that a caller can obtain the parent of a port without parsing the remote-endpoint first. Signed-off-by: Kieran

[PATCH v1 2/3] device property: Add fwnode_graph_get_port_parent

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham V4L2 async notifiers can pass the endpoint fwnode rather than the device fwnode. Provide a helper to obtain the parent device fwnode without first parsing the remote-endpoint as per fwnode_graph_get_remote_port_parent. Signed-off-by: Kieran Bingham --- drivers/base/proper

[PATCH v1 3/3] v4l: async: Match parent devices

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham Devices supporting multiple endpoints on a single device node must set their subdevice fwnode to the endpoint to allow distinct comparisons. Adapt the match_fwnode call to compare against the provided fwnodes first, but also to search for a comparison against the parent fwno

[PATCH v1 0/3] v4l: async: Match parent devices

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham To support using endpoints in the V4L2 Async subdev framework, we need to be able to parse a fwnode for the device from the endpoint. To enable this, provide an alternative of_graph_get_remote_port_parent which walks up the port hierarchy without parsing the 'remote-endpoint

Re: [PATCH v3 1/2] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-05-17 Thread Kieran Bingham
On 17/05/17 15:25, Geert Uytterhoeven wrote: > Hi Kieran, > > On Wed, May 17, 2017 at 4:13 PM, Kieran Bingham wrote: >> --- a/include/media/v4l2-subdev.h >> +++ b/include/media/v4l2-subdev.h >> @@ -829,7 +829,7 @@ struct v4l2_subdev { >> }; >> >> #define media_entity_to_v4l2_subdev(ent) \ >>

Re: [PATCH v3 1/2] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-05-17 Thread Geert Uytterhoeven
Hi Kieran, On Wed, May 17, 2017 at 4:13 PM, Kieran Bingham wrote: > --- a/include/media/v4l2-subdev.h > +++ b/include/media/v4l2-subdev.h > @@ -829,7 +829,7 @@ struct v4l2_subdev { > }; > > #define media_entity_to_v4l2_subdev(ent) \ > - container_of(ent, struct v4l2_subdev, entity) > +

[PATCH v3 1/2] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham Return NULL, if a null entity is parsed for it's v4l2_subdev Signed-off-by: Kieran Bingham --- include/media/v4l2-subdev.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/media/v4l2-subdev.h b/include/media/v4l2-subdev.h index 5f1669c45642..d43f

[PATCH v3 2/2] media: i2c: adv748x: add adv748x driver

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham Provide basic support for the ADV7481 and ADV7482. The driver is modelled with 2 subdevices to allow simultaneous streaming from the AFE (Analog front end) and HDMI inputs. Presently the HDMI is hardcoded to link to the TXA CSI bus, whilst the AFE is linked to the TXB CSI b

[PATCH v3 0/2] ADV748x HDMI/Analog video receiver

2017-05-17 Thread Kieran Bingham
From: Kieran Bingham This is a driver for the Analog Devices ADV748x device, and follows on from a previous posting by Niklas Söderlund [0] of an earlier incarnation of this driver. Aside from a few bug fixes, and considerable refactoring this driver: - is refactored to multiple object files -

[GIT PULL FOR v4.12] Hang in sir_ir

2017-05-17 Thread Sean Young
Hi Mauro, 0day found an issue which causes an infinite loop in the interrupt handler. Thanks, Sean The following changes since commit 3622d3e77ecef090b5111e3c5423313f11711dfa: [media] ov2640: print error if devm_*_optional*() fails (2017-04-25 07:08:21 -0300) are available in the git repos

Re: Sensor sub-device - what are the mandatory ops?

2017-05-17 Thread Dave Stevenson
Hi Sakari . Thanks for taking the time to respond. On 17 May 2017 at 00:04, Sakari Ailus wrote: > Hi Dave, > > On Thu, May 11, 2017 at 04:51:27PM +0100, Dave Stevenson wrote: >> Hi All. >> >> As previously discussed, I'm working on a V4L2 driver for the >> CSI-2/CCP2 receiver on BCM283x, as used

Re: [PATCH 0/4] Support for Synopsys DW CSI-2 Host

2017-05-17 Thread Mauro Carvalho Chehab
Em Wed, 17 May 2017 00:48:17 +0300 Sakari Ailus escreveu: > Hi Ramiro, > > On Tue, Mar 07, 2017 at 02:37:47PM +, Ramiro Oliveira wrote: > > This patchset adds support for the DW CSI-2 Host and also for a video > > device associated with it. > > > > The first 2 patches are only for the DW C

Re: [RFC PATCH v2 1/4] media: i2c: adv748x: add adv748x driver

2017-05-17 Thread Kieran Bingham
Hi Sakari, Ok - V3 is now closer :) I'm just trying to do a first spin of adding fwnode_graph_get_port_parent(), then I can post a v3 series. On 16/05/17 23:26, Sakari Ailus wrote: > Hi Kieran, > > On Tue, May 16, 2017 at 01:56:05PM +0100, Kieran Bingham wrote: > ... >> +static int adv748

Re: [PATCH 2/4] media: platform: dwc: Support for DW CSI-2 Host

2017-05-17 Thread Sakari Ailus
Hi Hans, On Wed, May 17, 2017 at 09:00:59AM +0200, Hans Verkuil wrote: > Hi Sakari, > > Can you comment on this? You are much more a CSI sensor expert than I am. Sure. Thanks for the ping. > > On 16/05/17 20:18, Ramiro Oliveira wrote: > > Hi Hans, > > > > Thank you very much for your feedbac

[PATCH v1 5/5] configure.ac: fix build of v4l-utils on uclinux

2017-05-17 Thread Hugues Fruchet
Build of v4-utils is conditional to "linux_os=yes" which was not set in case of uclinux, fix this. Signed-off-by: Hugues Fruchet --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 26dc18d..1af7408 100644 --- a/configure.ac +++ b

[PATCH v1 2/5] configure.ac: revisit v4l2-ctl/compliance using libv4l variable naming

2017-05-17 Thread Hugues Fruchet
USE_V4L2_CTL and USE_V4L2_COMPLIANCE are used to trig the fact that v4l2-ctl and v4l2-compliance are using libv4l2, change namings to not confuse with overall v4l2-ctl/compliance utilities building. Signed-off-by: Hugues Fruchet --- configure.ac | 8 1 file changed, 4 insertions(+), 4 d

[PATCH v1 3/5] configure.ac: revisit --disable-libv4l to --disable-dyn-libv4l

2017-05-17 Thread Hugues Fruchet
--disable-libv4l is not disabling libv4l compilation, but only dynamic library support of libv4l libraries. Signed-off-by: Hugues Fruchet --- configure.ac | 16 lib/libv4l1/Makefile.am | 2 +- lib/libv4l2/Makefile.am | 2 +- lib/libv4l2rds/Makefile

[PATCH v1 1/5] configure.ac: fix wrong summary if --disable-v4l2-ctl-stream-to

2017-05-17 Thread Hugues Fruchet
If --disable-v4l2-ctl-stream-to option is set, summary shows: v4l2-ctl uses libv4l : no due to USE_V4L2_CTL set to "no", fix this. Signed-off-by: Hugues Fruchet --- configure.ac | 1 - 1 file changed, 1 deletion(-) diff --git a/configure.ac b/configure.ac index 36b0c71..e8b86a5 100644 ---

[PATCH v1 0/5] v4l-utils build on buildroot with no-MMU devices

2017-05-17 Thread Hugues Fruchet
In order to pass V4L2 compliancy tests on STM32 devices -which are MMU less-, compliancy utilities -at least v4l2-compliance and cec-compliance- have to be built. Unfortunately the support of shared libraries (dlopen()) and fork() is a must have in v4l-utils. This have been fixed by: - revisiting

[PATCH v1 4/5] configure.ac: add --disable-libv4l option

2017-05-17 Thread Hugues Fruchet
Add an option to disable libv4l libraries and plugins compilation. If system is not supporting dynamic shared libraries, this option is automatically set. dlopen() is no more a mandatory dependency (warning is kept). lib/ and contrib/ folders are no more built with this option set because of libv4l

Re: [PATCH 2/4] media: platform: dwc: Support for DW CSI-2 Host

2017-05-17 Thread Hans Verkuil
Hi Sakari, Can you comment on this? You are much more a CSI sensor expert than I am. On 16/05/17 20:18, Ramiro Oliveira wrote: > Hi Hans, > > Thank you very much for your feedback. > > On 5/8/2017 11:38 AM, Hans Verkuil wrote: >> Hi Ramiro, >> >> My sincere apologies for the long delay in revie