cron job: media_tree daily build: ERRORS

2017-05-19 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 May 20 05:00:19 CEST 2017 media-tree git hash:36bcba973ad478042d1ffc6e89afd92e8bd17030 media_build

Re: [PATCH] ARM64: defconfig: enable IR core, decoders and Meson IR device

2017-05-19 Thread Kevin Hilman
Neil Armstrong writes: > This patch enables the MEDIA Infrared RC Decoders and Meson Infrared > decoder for ARM64 defconfig. > These drivers are selected as modules by default. > > Signed-off-by: Neil Armstrong > --- >

Re: [PATCH v2 1/2] device property: Add fwnode_graph_get_port_parent

2017-05-19 Thread Sakari Ailus
Hi Kieran, On Fri, May 19, 2017 at 05:16:02PM +0100, Kieran Bingham wrote: > +struct fwnode_handle * > +fwnode_graph_get_port_parent(struct fwnode_handle *endpoint) > +{ > + return fwnode_call_ptr_op(endpoint, graph_get_port_parent); graph_get_port_parent op will actually get the parent of

Re: [PATCH 14/16] lirc_dev: cleanup includes

2017-05-19 Thread Sean Young
On Mon, May 01, 2017 at 06:04:47PM +0200, David Härdeman wrote: > Remove superfluous includes and defines. > > Signed-off-by: David Härdeman > --- > drivers/media/rc/lirc_dev.c | 12 +--- > 1 file changed, 1 insertion(+), 11 deletions(-) > > diff --git

[PATCH][V2] [media] rainshadow-cec: ensure exit_loop is intialized

2017-05-19 Thread Colin King
From: Colin Ian King exit_loop is not being initialized, so it contains garbage. Ensure it is initialized to false. Detected by CoverityScan, CID#1436409 ("Uninitialized scalar variable") Fixes: ea6a69defd3311 ("[media] rainshadow-cec: avoid -Wmaybe-uninitialized

NACK: [PATCH] [media] rainshadow-cec: ensure exit_loop is initialized

2017-05-19 Thread Colin Ian King
On 19/05/17 18:39, Colin King wrote: > From: Colin Ian King > > exit_loop is not being initialized, so it contains garbage. Ensure it is > initialized to false. > > Detected by CoverityScan, CID#1436409 ("Uninitialzed scalar variable") > > Fixes: ea6a69defd3311

[PATCH] [media] rainshadow-cec: ensure exit_loop is initialized

2017-05-19 Thread Colin King
From: Colin Ian King exit_loop is not being initialized, so it contains garbage. Ensure it is initialized to false. Detected by CoverityScan, CID#1436409 ("Uninitialzed scalar variable") Fixes: ea6a69defd3311 ("[media] rainshadow-cec: avoid -Wmaybe-uninitialized

Re: [PATCH 1/4] [media] tw5864, fc0011: better handle WARN_ON()

2017-05-19 Thread Anton Sviridenko
ok, I'll check this patch on tw5864 hardware bit later and report results here On Thu, May 18, 2017 at 10:48 PM, Andrey Utkin wrote: > On Thu, May 18, 2017 at 11:06:43AM -0300, Mauro Carvalho Chehab wrote: >> As such macro will check if the expression is true, it may

[PATCH v2 2/2] v4l: async: Match parent devices

2017-05-19 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

[PATCH v2 1/2] device property: Add fwnode_graph_get_port_parent

2017-05-19 Thread Kieran Bingham
From: Kieran Bingham 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 --- v2: - Rebase on

[PATCH v2 0/2] v4l: async: Match parent devices

2017-05-19 Thread Kieran Bingham
From: Kieran Bingham As devices become more complicated, it becomes necessary (and more accurate) to match devices based on their endpoint, as devices may have multiple subdevices. To support using endpoints in the V4L2 async subdev framework, while some

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

2017-05-19 Thread Kieran Bingham
Hi Sakari, >> >> Finding and matching the parent of the SD would allow a driver to register a >> notifier ASD with a dev node, and expect to match a subdevice that has >> registered it's subdev with an EP node. This would erroneously allow drivers >> to >> register a notifier that would match

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

2017-05-19 Thread Laurent Pinchart
Hi Kieran, On Friday 19 May 2017 14:34:33 Kieran Bingham wrote: > On 18/05/17 14:36, Laurent Pinchart wrote: > > On Wednesday 17 May 2017 16:03:38 Kieran Bingham wrote: > >> From: Kieran Bingham > >> > >> V4L2 async notifiers can pass the endpoint fwnode

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

2017-05-19 Thread Kieran Bingham
Hi Laurent, On 18/05/17 14:36, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Wednesday 17 May 2017 16:03:38 Kieran Bingham wrote: >> From: Kieran Bingham >> >> V4L2 async notifiers can pass the endpoint fwnode rather than the

[PATCH v3 3/4] media: adv7180: Add adv7180cp, adv7180st bindings

2017-05-19 Thread Ulrich Hecht
To differentiate between two classes of chip packages that have different numbers of input ports. Signed-off-by: Ulrich Hecht --- Documentation/devicetree/bindings/media/i2c/adv7180.txt | 15 +++ 1 file changed, 15 insertions(+) diff --git

[PATCH v3 1/4] ARM: dts: gose: add HDMI input

2017-05-19 Thread Ulrich Hecht
Identical to the setup on Koelsch. Signed-off-by: Ulrich Hecht Reviewed-by: Laurent Pinchart --- arch/arm/boot/dts/r8a7793-gose.dts | 68 -- 1 file changed, 66 insertions(+), 2 deletions(-)

[PATCH v3 4/4] ARM: dts: gose: add composite video input

2017-05-19 Thread Ulrich Hecht
Adds VIN, decoder and connector. Signed-off-by: Ulrich Hecht --- arch/arm/boot/dts/r8a7793-gose.dts | 59 ++ 1 file changed, 59 insertions(+) diff --git a/arch/arm/boot/dts/r8a7793-gose.dts

[PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings

2017-05-19 Thread Ulrich Hecht
Used to differentiate between models with 3 and 6 inputs. Signed-off-by: Ulrich Hecht --- drivers/media/i2c/adv7180.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c index bdbbf8c..78de7dd 100644 ---

[PATCH v3 0/4] r8a7793 Gose video input support

2017-05-19 Thread Ulrich Hecht
Hi! This is a by-the-datasheet implementation of analog and digital video input on the Gose board. This revision adds new bindings that distinguish between ADV7180 variants with three and six input ports. There are numerous variants of this chip, but since all that have "CP" in their names have

[PATCH v2 1/6] [media] atomisp: disable several warnings when W=1

2017-05-19 Thread Mauro Carvalho Chehab
The atomisp currently produce hundreds of warnings when W=1. It is a known fact that this driver is currently in bad shape, and there are lot of things to be done here. We don't want to be bothered by those "minor" stuff for now, while the driver doesn't receive a major cleanup. So, disable

[PATCH 4/6] [media] soc_camera: annotate a switch fall through

2017-05-19 Thread Mauro Carvalho Chehab
Clearly, hsync and vsinc bool vars are part of the return logic on the second case of the switch. Annotate that, in order to shut up gcc warnings. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/soc_camera/soc_mediabus.c | 1 + 1 file changed, 1

[PATCH 5/6] [media] s2255drv: avoid a switch fall through

2017-05-19 Thread Mauro Carvalho Chehab
On this driver, it can fall through a switch. I tried to annotate it, in order to shut up a gcc warning, but that didn't work, as the logic there is somewhat complex. So, instead, let's just repeat the code. gcc should likely optimize it anyway, and this makes the code better readable, IMHO.

[PATCH 3/6] [media] zoran: annotate switch fall through

2017-05-19 Thread Mauro Carvalho Chehab
There are two cases here that it does a switch fall through. Annotate it, in order to shut up gcc warnings. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/pci/zoran/zoran_driver.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 2/6] [media] av7110: avoid switch fall through

2017-05-19 Thread Mauro Carvalho Chehab
On two switches, this driver have unannotated switch fall through. in the first case, it falls through a return. On the second one, it prints undesired log messages on fall through. Solve that by copying the commands that it should be running. Gcc will very likely optimize it anyway, so this

[PATCH 6/6] [media] uvcvideo: annotate a switch fall through

2017-05-19 Thread Mauro Carvalho Chehab
Without annotations, gcc 7.1 will complain. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/uvc/uvc_video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/uvc/uvc_video.c b/drivers/media/usb/uvc/uvc_video.c index

Re: [PATCH v1] [media] atmel-isi: code cleanup

2017-05-19 Thread Hugues FRUCHET
Adding Songjun and Ludovic as Atmel maintainers, sorry for inconvenience. On 05/19/2017 12:04 PM, Hugues Fruchet wrote: > Ensure that ISI is clocked before starting sensor sub device. > Remove un-needed type check in try_fmt(). > Use clamp() macro for hardware capabilities. > Fix wrong tabulation

Re: [PATCH v1] ATMEL ISI code cleanup

2017-05-19 Thread Hugues FRUCHET
Adding Songjun and Ludovic as Atmel maintainers, sorry for inconvenience. On 05/19/2017 12:04 PM, Hugues Fruchet wrote: > Here is a bunch of small fixes found when upstreaming > the ST DCMI driver based on ATMEL ISI driver. > > Review remarks can be found here: >

Re: [PATCH 00/13] staging: media: atomisp queued up patches

2017-05-19 Thread Mauro Carvalho Chehab
Em Thu, 18 May 2017 17:40:27 +0100 Alan Cox escreveu: > On Thu, 2017-05-18 at 11:10 -0300, Mauro Carvalho Chehab wrote: > > Em Thu, 18 May 2017 15:50:09 +0200 > > Greg Kroah-Hartman escreveu: > > > > > > > > Hi Mauro, > > > > > > Here's the

[PATCH] [media] atomisp: disable several warnings when W=1

2017-05-19 Thread Mauro Carvalho Chehab
The atomisp currently produce hundreds of warnings when W=1. It is a known fact that this driver is currently in bad shape, and there are lot of things to be done here. We don't want to be bothered by those "minor" stuff for now, while the driver doesn't receive a major cleanup. So, disable

[PATCH v1] [media] atmel-isi: code cleanup

2017-05-19 Thread Hugues Fruchet
Ensure that ISI is clocked before starting sensor sub device. Remove un-needed type check in try_fmt(). Use clamp() macro for hardware capabilities. Fix wrong tabulation to space. Signed-off-by: Hugues Fruchet --- drivers/media/platform/atmel/atmel-isi.c | 24

[PATCH v1] ATMEL ISI code cleanup

2017-05-19 Thread Hugues Fruchet
Here is a bunch of small fixes found when upstreaming the ST DCMI driver based on ATMEL ISI driver. Review remarks can be found here: http://www.mail-archive.com/linux-media@vger.kernel.org/msg112338.html: - Ensure that ISI is clocked before starting sensor sub device. - Remove un-needed type

[libv4l]: How to do 10-bit support? Stand-alone conversions?

2017-05-19 Thread Pavel Machek
Hi! > I much rather do this as part of a longer series that actually adds 10 bit > support. > The problem is that adding support for 10 bit doesn't mean you can just use > LSIZE > all the time since you still need support for 8 bit as well. Heh. I was afraid to hear that. I can probably

Re: [RFC] V4L2 unified low-level decoder API

2017-05-19 Thread Randy Li
On 05/19/2017 04:08 PM, Hugues FRUCHET wrote: Hi all, Here is the latest st-delta MPEG2 code: [PATCH v6 0/3] Add support for MPEG-2 in DELTA video decoder http://www.mail-archive.com/linux-media@vger.kernel.org/msg112067.html [PATCH v2 0/3] Add a libv4l plugin for video bitstream parsing

Re: [RFC] V4L2 unified low-level decoder API

2017-05-19 Thread Hugues FRUCHET
Hi all, Here is the latest st-delta MPEG2 code: [PATCH v6 0/3] Add support for MPEG-2 in DELTA video decoder http://www.mail-archive.com/linux-media@vger.kernel.org/msg112067.html [PATCH v2 0/3] Add a libv4l plugin for video bitstream parsing

Re: [PATCH 3/4] [media] s5p-jpeg: don't return a random width/height

2017-05-19 Thread Andrzej Pietrasiewicz
W dniu 18.05.2017 o 16:06, Mauro Carvalho Chehab pisze: Gcc 7.1 complains about: drivers/media/platform/s5p-jpeg/jpeg-core.c: In function 's5p_jpeg_parse_hdr.isra.9': drivers/media/platform/s5p-jpeg/jpeg-core.c:1207:12: warning: 'width' may be used uninitialized in this function