cron job: media_tree daily build: WARNINGS

2019-08-29 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 Aug 30 05:00:12 CEST 2019 media-tree git hash:968bce2f59ce462d296af28610628fe7e03b120a media_build gi

Singaporean Mr. Teo En Ming's Refugee Seeking Attempts

2019-08-29 Thread Turritopsis Dohrnii Teo En Ming
Subject: Singaporean Mr. Teo En Ming's Refugee Seeking Attempts In reverse chronological order: [1] Petition to the Government of Taiwan for Refugee Status, 5th August 2019 Monday Photo #1: At the building of the National Immigration Agency, Ministry of the Interior, Taipei, Taiwan, 5th August

Re: [PATCH] media: vimc: upon streaming, check that the pipeline starts with a source entity

2019-08-29 Thread kbuild test robot
Hi Dafna, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [cannot apply to v5.3-rc6 next-20190829] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci

Re: [PATCH 6/7] media: don't do an unsigned int with a 31 bit shift

2019-08-29 Thread Kees Cook
On Thu, Aug 22, 2019 at 04:39:33PM -0300, Mauro Carvalho Chehab wrote: > Doing something like: > > i32 foo = 1, bar; > > bar = foo << 31; > > has an undefined behavior in C, as warned by cppcheck, as we're > shifting a signed integer. > > Instead, force the numbers to be unsigned, i

Re: [PATCH v2 09/10] media: i2c: ov5670: Report native size and crop bounds

2019-08-29 Thread Jacopo Mondi
Hi Hans, On Thu, Aug 29, 2019 at 03:39:43PM +0200, Hans Verkuil wrote: > On 8/29/19 3:18 PM, Jacopo Mondi wrote: > > Hi Hans > > > > On Thu, Aug 29, 2019 at 02:55:30PM +0200, Hans Verkuil wrote: > >> On 8/29/19 2:40 PM, Jacopo Mondi wrote: > >>> HI Hans, > >>> > >>> On Thu, Aug 29, 2019 at 12:20:1

Re: [PATCH v9 07/13] media: tvp5150: add FORMAT_TRY support for get/set selection handlers

2019-08-29 Thread Marco Felsch
On 19-08-29 11:56, Hans Verkuil wrote: > On 8/22/19 10:05 AM, Marco Felsch wrote: > > Since commit 10d5509c8d50 ("[media] v4l2: remove g/s_crop from video ops") > > the 'which' field for set/get_selection must be FORMAT_ACTIVE. There is > > no way to try different selections. The patch adds a helpe

Re: [PATCH v2 06/10] media: v4l2-fwnode: Add helper to register controls from fw

2019-08-29 Thread Hans Verkuil
On 8/29/19 5:05 PM, Laurent Pinchart wrote: > Hi Hans, > > On Thu, Aug 29, 2019 at 03:04:42PM +0200, Hans Verkuil wrote: >> On 8/29/19 2:45 PM, Jacopo Mondi wrote: >>> On Thu, Aug 29, 2019 at 12:31:37PM +0200, Hans Verkuil wrote: On 8/27/19 11:23 AM, Jacopo Mondi wrote: > Add the 'v4l2_fw

[PATCH v4l-utils] ir-ctl: warn if keycode has multiple definitions

2019-08-29 Thread Sean Young
Signed-off-by: Sean Young --- utils/ir-ctl/ir-ctl.c | 52 --- 1 file changed, 29 insertions(+), 23 deletions(-) diff --git a/utils/ir-ctl/ir-ctl.c b/utils/ir-ctl/ir-ctl.c index 28d8d97a..e884b4f8 100644 --- a/utils/ir-ctl/ir-ctl.c +++ b/utils/ir-ctl/ir-ctl

Re: [PATCH v2 06/10] media: v4l2-fwnode: Add helper to register controls from fw

2019-08-29 Thread Laurent Pinchart
Hi Hans, On Thu, Aug 29, 2019 at 03:04:42PM +0200, Hans Verkuil wrote: > On 8/29/19 2:45 PM, Jacopo Mondi wrote: > > On Thu, Aug 29, 2019 at 12:31:37PM +0200, Hans Verkuil wrote: > >> On 8/27/19 11:23 AM, Jacopo Mondi wrote: > >>> Add the 'v4l2_fwnode_register_controls()' helper to v4l2-fwnode. Th

Re: [PATCH v2 09/10] media: i2c: ov5670: Report native size and crop bounds

2019-08-29 Thread Hans Verkuil
On 8/29/19 3:18 PM, Jacopo Mondi wrote: > Hi Hans > > On Thu, Aug 29, 2019 at 02:55:30PM +0200, Hans Verkuil wrote: >> On 8/29/19 2:40 PM, Jacopo Mondi wrote: >>> HI Hans, >>> >>> On Thu, Aug 29, 2019 at 12:20:18PM +0200, Hans Verkuil wrote: On 8/27/19 11:23 AM, Jacopo Mondi wrote: > Repo

Re: [PATCH v2 09/10] media: i2c: ov5670: Report native size and crop bounds

2019-08-29 Thread Jacopo Mondi
Hi Hans On Thu, Aug 29, 2019 at 02:55:30PM +0200, Hans Verkuil wrote: > On 8/29/19 2:40 PM, Jacopo Mondi wrote: > > HI Hans, > > > > On Thu, Aug 29, 2019 at 12:20:18PM +0200, Hans Verkuil wrote: > >> On 8/27/19 11:23 AM, Jacopo Mondi wrote: > >>> Report the native pixel array size and the crop bou

Re: [PATCH] media: vimc: upon streaming, check that the pipeline starts with a source entity

2019-08-29 Thread André Almeida
Hello Dafna, Thanks for your patch! On 8/29/19 8:00 AM, Dafna Hirschfeld wrote: > Userspace can disable links and create pipelines that > do not start with a source entity. Trying to stream > from such a pipeline should fail with -EPIPE > currently this is not handled and cause kernel crash. > >

Re: [PATCH v2 06/10] media: v4l2-fwnode: Add helper to register controls from fw

2019-08-29 Thread Hans Verkuil
On 8/29/19 2:45 PM, Jacopo Mondi wrote: > Hi Hans, > > On Thu, Aug 29, 2019 at 12:31:37PM +0200, Hans Verkuil wrote: >> On 8/27/19 11:23 AM, Jacopo Mondi wrote: >>> Add the 'v4l2_fwnode_register_controls()' helper to v4l2-fwnode. The >>> function parses the device node and endpoint firmware proper

Re: [PATCH v2 09/10] media: i2c: ov5670: Report native size and crop bounds

2019-08-29 Thread Hans Verkuil
On 8/29/19 2:40 PM, Jacopo Mondi wrote: > HI Hans, > > On Thu, Aug 29, 2019 at 12:20:18PM +0200, Hans Verkuil wrote: >> On 8/27/19 11:23 AM, Jacopo Mondi wrote: >>> Report the native pixel array size and the crop bounds for the ov5670 >>> sensor driver. >>> >>> Signed-off-by: Jacopo Mondi >>> ---

Re: [PATCH v2 06/10] media: v4l2-fwnode: Add helper to register controls from fw

2019-08-29 Thread Jacopo Mondi
Hi Laurent, On Tue, Aug 27, 2019 at 04:16:20PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Tue, Aug 27, 2019 at 11:23:34AM +0200, Jacopo Mondi wrote: > > Add the 'v4l2_fwnode_register_controls()' helper to v4l2-fwnode. The > > function parses the device node and

Re: [PATCH v9 01/13] dt-bindings: connector: analog: add sdtv standards property

2019-08-29 Thread Marco Felsch
On 19-08-29 12:02, Hans Verkuil wrote: > On 8/22/19 10:05 AM, Marco Felsch wrote: > > Some connectors no matter if in- or output supports only a limited > > range of sdtv standards. It doesn't matter if the hardware behind that > > connector supports more than the listed formats since the users are

Re: [PATCH v9 02/13] media: v4l2-fwnode: add v4l2_fwnode_connector

2019-08-29 Thread Marco Felsch
On 19-08-29 11:59, Hans Verkuil wrote: > On 8/22/19 10:05 AM, Marco Felsch wrote: > > Currently every driver needs to parse the connector endpoints by it self. > > This is the initial work to make this generic. A generic connector has > > common members and connector specific members. The common me

Re: [PATCH v2 02/10] media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_LOCATION

2019-08-29 Thread Jacopo Mondi
Hi Laurent, On Tue, Aug 27, 2019 at 03:50:08PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Tue, Aug 27, 2019 at 11:23:28AM +0200, Jacopo Mondi wrote: > > Add documentation for the V4L2_CID_CAMERA_SENSOR_LOCATION camera > > control. The newly added read-only cont

Re: [PATCH v2 01/10] media: dt-bindings: Document 'location' property

2019-08-29 Thread Jacopo Mondi
Hi Laurent, On Tue, Aug 27, 2019 at 03:21:26PM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > Thank you for the patch. > > On Tue, Aug 27, 2019 at 11:23:27AM +0200, Jacopo Mondi wrote: > > Add the 'location' device property, used to specify the camera device > > mounting position. The property is

Re: [PATCH v2 06/10] media: v4l2-fwnode: Add helper to register controls from fw

2019-08-29 Thread Jacopo Mondi
Hi Hans, On Thu, Aug 29, 2019 at 12:31:37PM +0200, Hans Verkuil wrote: > On 8/27/19 11:23 AM, Jacopo Mondi wrote: > > Add the 'v4l2_fwnode_register_controls()' helper to v4l2-fwnode. The > > function parses the device node and endpoint firmware properties to > > which a v4l2 control is associated

Re: [PATCH v2 09/10] media: i2c: ov5670: Report native size and crop bounds

2019-08-29 Thread Jacopo Mondi
HI Hans, On Thu, Aug 29, 2019 at 12:20:18PM +0200, Hans Verkuil wrote: > On 8/27/19 11:23 AM, Jacopo Mondi wrote: > > Report the native pixel array size and the crop bounds for the ov5670 > > sensor driver. > > > > Signed-off-by: Jacopo Mondi > > --- > > drivers/media/i2c/ov5670.c | 20 +

Re: [GIT PULL FOR v5.4] Small fixes

2019-08-29 Thread Jenkins
From: buil...@linuxtv.org Pull request: https://patchwork.linuxtv.org/patch/58474/ Build log: https://builder.linuxtv.org/job/patchwork/13787/ Build time: 00:05:35 Link: https://lore.kernel.org/linux-media/7d8c241c-10a5-948a-4e0f-24962e3d7...@xs4all.nl gpg: Signature made Thu 29 Aug 2019 12:12:3

[GIT PULL FOR v5.4] Small fixes

2019-08-29 Thread Hans Verkuil
The following changes since commit 95c520690f5fafb2cda2ec17f8c76ab3422b0174: media: don't do a 31 bit shift on a signed int (2019-08-26 14:11:10 -0300) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.4p for you to fetch changes up to 9cb46b4a1ac7

[PATCH] media: vimc: upon streaming, check that the pipeline starts with a source entity

2019-08-29 Thread Dafna Hirschfeld
Userspace can disable links and create pipelines that do not start with a source entity. Trying to stream from such a pipeline should fail with -EPIPE currently this is not handled and cause kernel crash. Signed-off-by: Dafna Hirschfeld --- Hi, These are the commands to reproduce the crash: media

Re: [PATCH v2 06/10] media: v4l2-fwnode: Add helper to register controls from fw

2019-08-29 Thread Hans Verkuil
On 8/27/19 11:23 AM, Jacopo Mondi wrote: > Add the 'v4l2_fwnode_register_controls()' helper to v4l2-fwnode. The > function parses the device node and endpoint firmware properties to > which a v4l2 control is associated to and registers the control with the > provided handler. > > Signed-off-by: Ja

Re: [PATCH v2 09/10] media: i2c: ov5670: Report native size and crop bounds

2019-08-29 Thread Hans Verkuil
On 8/27/19 11:23 AM, Jacopo Mondi wrote: > Report the native pixel array size and the crop bounds for the ov5670 > sensor driver. > > Signed-off-by: Jacopo Mondi > --- > drivers/media/i2c/ov5670.c | 20 > 1 file changed, 20 insertions(+) > > diff --git a/drivers/media/i2c/o

Re: [PATCH v9 01/13] dt-bindings: connector: analog: add sdtv standards property

2019-08-29 Thread Hans Verkuil
On 8/22/19 10:05 AM, Marco Felsch wrote: > Some connectors no matter if in- or output supports only a limited > range of sdtv standards. It doesn't matter if the hardware behind that > connector supports more than the listed formats since the users are > restriced by a label e.g. to plug only a cam

Re: [PATCH v9 02/13] media: v4l2-fwnode: add v4l2_fwnode_connector

2019-08-29 Thread Hans Verkuil
On 8/22/19 10:05 AM, Marco Felsch wrote: > Currently every driver needs to parse the connector endpoints by it self. > This is the initial work to make this generic. A generic connector has > common members and connector specific members. The common members are: > - type > - label (optional) >

Re: [PATCH v9 07/13] media: tvp5150: add FORMAT_TRY support for get/set selection handlers

2019-08-29 Thread Hans Verkuil
On 8/22/19 10:05 AM, Marco Felsch wrote: > Since commit 10d5509c8d50 ("[media] v4l2: remove g/s_crop from video ops") > the 'which' field for set/get_selection must be FORMAT_ACTIVE. There is > no way to try different selections. The patch adds a helper function to > select the correct selection me

[PATCH v2 4/4] dvb-usb-gl861: support I2C read from tuner via demod

2019-08-29 Thread tskd08
From: Akihiro Tsukada Support Friio devices to read from tuner via demod tc90522, by enabling the config option. Signed-off-by: Akihiro Tsukada --- drivers/media/usb/dvb-usb-v2/gl861.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/usb/dvb-usb-v2/gl861.c b/drivers/media/usb

[PATCH v2 0/4] dvb-usb-gl861, tc90522: remove device-specific i2c algo

2019-08-29 Thread tskd08
From: Akihiro Tsukada This patch series removes and integrates the device(Friio)-specific i2c algo with the other, generic one that was recently patched ( https://patchwork.linuxtv.org/patch/58340/ ) to support the i2c messages used in Friio, for reading/writing to tuner. To fully complete the i

[PATCH v2 3/4] dvb-frontends/tc90522: extend i2c algo to support some devices

2019-08-29 Thread tskd08
From: Akihiro Tsukada This demod implements an i2c adapter for attached tuner and relays i2c messages from users (dvb adapters / bridge chips). Some of them, such as Friio dvb card using gl861, require each pair of i2c messages for one read to be issued as two separate transactions. This patch ad

[PATCH v2 1/4] dvb-usb-v2/gl861: remove device-specific i2c algo

2019-08-29 Thread tskd08
From: Akihiro Tsukada For Friio dvb cards, a dintinct I2C algo was provided to support some "long" I2C messages used in relaying tuner I2C via demod. Since the other (generic) i2c algo in this module has been patched to support those messages in a more generic way, this patch replaces and integra

[PATCH v2 2/4] dvb-usb-v2/gl861: remove an un-used header file

2019-08-29 Thread tskd08
From: Akihiro Tsukada The header contained just internal definitions and they are not used anymore. Signed-off-by: Akihiro Tsukada --- drivers/media/usb/dvb-usb-v2/gl861.c | 2 +- drivers/media/usb/dvb-usb-v2/gl861.h | 14 -- 2 files changed, 1 insertion(+), 15 deletions(-) delet