[GIT PULL FOR v4.10] Smiapp runtime PM fixes

2016-12-08 Thread Sakari Ailus
Hi Mauro, These patches fix a compiler warning for the smiapp driver and also make power management work again without runtime PM. Please pull. The following changes since commit 365fe4e0ce218dc5ad10df17b150a366b6015499: [media] mn88472: fix chip id check on probe (2016-12-01 12:47:22

Re: Regression: tvp5150 refactoring breaks all em28xx devices

2016-12-08 Thread Mauro Carvalho Chehab
Hi Laurent and Devin, Em Thu, 08 Dec 2016 00:59:17 +0200 Laurent Pinchart escreveu: > Hi Devin, > > On Wednesday 07 Dec 2016 12:47:01 Devin Heitmueller wrote: > > Hello Javier, Mauro, Laurent, > > > > I hope all is well with you. Mauro, Laurent: you guys

Re: [PATCH RFC] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Mauro, Thank you for the patch. On Thursday 08 Dec 2016 06:55:58 Mauro Carvalho Chehab wrote: > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, > depending on the type of bus set via the

Re: [PATCH RFC] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 08 Dec 2016 15:02:28 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > On Thursday 08 Dec 2016 10:53:33 Mauro Carvalho Chehab wrote: > > Em Thu, 08 Dec 2016 14:25:38 +0200 Laurent Pinchart escreveu: > > > On Thursday 08 Dec 2016 06:55:58 Mauro

Re: [GIT PULL FOR v4.11] Remove FSF postal address

2016-12-08 Thread Mauro Carvalho Chehab
Hi Sakari, Em Thu, 8 Dec 2016 13:09:20 +0200 Sakari Ailus escreveu: > Hi Felipe, > > On Thu, Dec 08, 2016 at 08:51:20AM -0200, Felipe Sanches wrote: > > but why? > > Please see my reply here: > > Please

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Mauro, On Thursday 08 Dec 2016 11:38:34 Mauro Carvalho Chehab wrote: > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, > depending on the type of bus set via the .set_fmt() subdev callback. > >

Re: [PATCH 4/5] media: entity: Split graph walk iteration into two functions

2016-12-08 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 25 Nov 2016 15:55:45 Sakari Ailus wrote: > With media_entity_graph_walk_next() getting more and more complicated (and > especially so with has_routing() support added), split the function into > two. > > Signed-off-by: Sakari Ailus

Re: [PATCH RFC] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Mauro, On Thursday 08 Dec 2016 11:41:43 Mauro Carvalho Chehab wrote: > Em Thu, 08 Dec 2016 15:02:28 +0200 Laurent Pinchart escreveu: > > On Thursday 08 Dec 2016 10:53:33 Mauro Carvalho Chehab wrote: > >> Em Thu, 08 Dec 2016 14:25:38 +0200 Laurent Pinchart escreveu: > >>> On Thursday 08 Dec

Re: [PATCH 2/5] media: entity: Be vocal about failing sanity checks

2016-12-08 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 25 Nov 2016 15:55:43 Sakari Ailus wrote: > Commit 3801bc7d1b8d ("[media] media: Media Controller fix to not let > stream_count go negative") added a sanity check for negative stream_count, > but a failure of the check remained silent. Make sure the

Re: [PATCH 3/5] media: Rename graph and pipeline structs and functions

2016-12-08 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 25 Nov 2016 15:55:44 Sakari Ailus wrote: > The media_entity_pipeline_start() and media_entity_pipeline_stop() > functions are renamed as media_pipeline_start() and media_pipeline_stop(), > respectively. The reason is two-fold: the pipeline struct is,

Re: [PATCH 1/5] media: entity: Fix stream count check

2016-12-08 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 25 Nov 2016 15:55:42 Sakari Ailus wrote: > There's a sanity check for the stream count remaining positive or zero on > error path, but instead of performing the check on the traversed entity it > is performed on the entity where traversal ends. Fix

Re: [GIT PULL FOR v4.11] Remove FSF postal address

2016-12-08 Thread Sakari Ailus
Hi Mauro, On Thu, Dec 08, 2016 at 10:59:47AM -0200, Mauro Carvalho Chehab wrote: > Hi Sakari, > > Em Thu, 8 Dec 2016 13:09:20 +0200 > Sakari Ailus escreveu: > > > Hi Felipe, > > > > On Thu, Dec 08, 2016 at 08:51:20AM -0200, Felipe Sanches wrote: > > > but why? > > > >

Re: [PATCH v2 3/3] uvcvideo: add a metadata device node

2016-12-08 Thread Guennadi Liakhovetski
Hi Laurent, One more question: On Tue, 6 Dec 2016, Laurent Pinchart wrote: > Hi Guennadi, > > On Tuesday 06 Dec 2016 11:39:22 Guennadi Liakhovetski wrote: > > On Tue, 6 Dec 2016, Laurent Pinchart wrote: > > > On Monday 05 Dec 2016 23:13:53 Guennadi Liakhovetski wrote: > > >> On Tue, 6 Dec

Re: [PATCH 5/5] media: entity: Add debug information to graph walk

2016-12-08 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 25 Nov 2016 15:55:46 Sakari Ailus wrote: > Use dev_dbg() to tell about the progress of the graph traversal algorithm. > This is intended to make debugging of the algorithm easier. > > Signed-off-by: Sakari Ailus > ---

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 08 Dec 2016 15:41:59 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > On Thursday 08 Dec 2016 11:38:34 Mauro Carvalho Chehab wrote: > > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > > support") added a logic that overrides

Re: [PATCH RFC] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Mauro, On Thursday 08 Dec 2016 10:53:33 Mauro Carvalho Chehab wrote: > Em Thu, 08 Dec 2016 14:25:38 +0200 Laurent Pinchart escreveu: > > On Thursday 08 Dec 2016 06:55:58 Mauro Carvalho Chehab wrote: > >> changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > >> support")

[PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, depending on the type of bus set via the .set_fmt() subdev callback. This is known to cause trobules on devices that don't use a V4L2 subdev devnode, and

Re: [PATCH v2 3/3] uvcvideo: add a metadata device node

2016-12-08 Thread Laurent Pinchart
Hi Guennadi, On Thursday 08 Dec 2016 14:34:46 Guennadi Liakhovetski wrote: > On Tue, 6 Dec 2016, Laurent Pinchart wrote: > > On Tuesday 06 Dec 2016 11:39:22 Guennadi Liakhovetski wrote: > >> On Tue, 6 Dec 2016, Laurent Pinchart wrote: > >>> On Monday 05 Dec 2016 23:13:53 Guennadi Liakhovetski

Re: [PATCH RFC] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 08 Dec 2016 14:25:38 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > Thank you for the patch. > > On Thursday 08 Dec 2016 06:55:58 Mauro Carvalho Chehab wrote: > > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > > support")

uvcvideo logging kernel warnings on device disconnect

2016-12-08 Thread Dave Stevenson
Hi All. I'm working with a USB webcam which has been seen to spontaneously disconnect when in use. That's a separate issue, but when it does it throws a load of warnings into the kernel log if there is a file handle on the device open at the time, even if not streaming. I've reproduced this

[PATCH v2] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
commit 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, depending on the type of bus set via the .set_fmt() subdev callback. This is known to cause trobules on devices that don't use a V4L2 subdev devnode, and a

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

2016-12-08 Thread Shuah Khan
Hi Sakari, On 12/07/2016 03:27 PM, Sakari Ailus wrote: > Hi Shuah, > > On Wed, Dec 07, 2016 at 01:03:59PM -0700, Shuah Khan wrote: >> Hi Sakari, >> >> On 12/07/2016 03:52 AM, Sakari Ailus wrote: >>> Hi Shuah, >>> >>> On Mon, Dec 05, 2016 at 05:38:23PM -0700, Shuah Khan wrote: On 12/05/2016

[GIT PULL FOR v4.11] Remove FSF postal address

2016-12-08 Thread Sakari Ailus
Hi Mauro, This pull request contains a single patch, one that removes the FSF postal address from the headers in source code files. The patch is rebased from what I posted for review, no other changes. Please pull. The following changes since commit 365fe4e0ce218dc5ad10df17b150a366b6015499:

[GIT PULL FOR v4.10] cec: fix report_current_latency

2016-12-08 Thread Hans Verkuil
CEC bug fix. Regards, Hans The following changes since commit 365fe4e0ce218dc5ad10df17b150a366b6015499: [media] mn88472: fix chip id check on probe (2016-12-01 12:47:22 -0200) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git cecfix for you to

[PATCH RFC] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, depending on the type of bus set via the .set_fmt() subdev callback. This is known to cause trobules on devices that don't use a V4L2 subdev devnode, and

Re: [git:media_tree/master] [media] Add Cinergy S2 rev.4 support

2016-12-08 Thread Enrico Mioso
Hello guys. First of all, thank you for your great work. I am writing to you to understand if there where problems with a patch I sent something like 17 octobrer to add an USB id. Where there any problems? I saw it was accepted but can't find it in the kernel. I admit I didn't check in media

Re: [GIT PULL FOR v4.11] Remove FSF postal address

2016-12-08 Thread Felipe Sanches
but why? 2016-12-08 6:08 GMT-02:00 Sakari Ailus : > Hi Mauro, > > This pull request contains a single patch, one that removes the FSF postal > address from the headers in source code files. The patch is rebased from > what I posted for review, no other changes. > > Please

Re: [git:media_tree/master] [media] Add Cinergy S2 rev.4 support

2016-12-08 Thread Sakari Ailus
Hi Enrico, On Thu, Dec 08, 2016 at 10:40:06AM +0100, Enrico Mioso wrote: > Hello guys. > > First of all, thank you for your great work. > > I am writing to you to understand if there where problems with a patch I sent > something like 17 octobrer to add an USB id. > Where there any problems? >

Re: [GIT PULL FOR v4.11] Remove FSF postal address

2016-12-08 Thread Sakari Ailus
Hi Felipe, On Thu, Dec 08, 2016 at 08:51:20AM -0200, Felipe Sanches wrote: > but why? Please see my reply here: -- Kind regards, Sakari Ailus e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk -- To unsubscribe from this

Re: [PATCH v3 3/3] ARM: multi_v7_defconfig: enable STMicroelectronics HVA debugfs

2016-12-08 Thread Jean Christophe TROTIN
Hi Hans, You're right: the HVA DEBUGFS config option shouldn't be enabled by default. So, "[PATCH v3 3/3] ARM: multi_v7_defconfig: enable STMicroelectronics HVA debugfs" must be ignored. It will be taken into account in our configuration fragment. Do you have any other remark about the patches

Re: [PATCH 5/9] [media] coda: get VDOA device using dt phandle

2016-12-08 Thread Philipp Zabel
Am Donnerstag, den 08.12.2016, 16:24 +0100 schrieb Michael Tretter: > Signed-off-by: Michael Tretter > --- > drivers/media/platform/coda/coda-common.c | 43 > +++ > drivers/media/platform/coda/coda.h| 1 + > 2 files changed, 44

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
On Thursday 08 Dec 2016 12:16:08 Mauro Carvalho Chehab wrote: > Em Thu, 08 Dec 2016 15:41:59 +0200 Laurent Pinchart escreveu: > > On Thursday 08 Dec 2016 11:38:34 Mauro Carvalho Chehab wrote: > > > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > > > support") added a

[PATCH 2/9] [media] coda: add i.MX6 VDOA driver

2016-12-08 Thread Michael Tretter
From: Philipp Zabel The i.MX6 Video Data Order Adapter's (VDOA) sole purpose is to convert from a custom macroblock tiled format produced by the CODA960 decoder into linear formats that can be used for scanout. Signed-off-by: Philipp Zabel

[PATCH 1/9] ARM: dts: imx6qdl: Add VDOA compatible and clocks properties

2016-12-08 Thread Michael Tretter
From: Philipp Zabel This adds a compatible property and the correct clock for the i.MX6Q Video Data Order Adapter. Signed-off-by: Philipp Zabel Signed-off-by: Michael Tretter --- arch/arm/boot/dts/imx6qdl.dtsi | 2 ++

[PATCH 4/9] [media] coda: correctly set capture compose rectangle

2016-12-08 Thread Michael Tretter
From: Philipp Zabel Correctly store the rectangle of valid video data in the destination q_data before rounding up to macroblock size. This fixes the output of VIDIOC_G_SELECTION for the capture side compose rectangle. Signed-off-by: Philipp Zabel

[PATCH 3/9] ARM: dts: imx6qdl: Add VDOA phandle to CODA node

2016-12-08 Thread Michael Tretter
The CODA driver should use the VDOA to transform the tiled format to raster-ordered format, if the platform has a VDOA. Link the CODA and VDOA nodes to tell the CODA driver that it can use the VDOA. Signed-off-by: Michael Tretter ---

Re: [PATCH 9/9] [media] coda: support YUYV output if VDOA is used

2016-12-08 Thread Philipp Zabel
Am Donnerstag, den 08.12.2016, 16:24 +0100 schrieb Michael Tretter: > The VDOA is able to transform the NV12 custom macroblock tiled format of > the CODA to YUYV format. If and only if the VDOA is available, the > driver can also provide YUYV support. > > While the driver is configured to produce

[PATCH 6/9] [media] coda: add debug output about tiling

2016-12-08 Thread Michael Tretter
From: Philipp Zabel In order to make the VDOA work correctly, the CODA must produce frames in tiled format. Print this information in the debug output. Also print the color format in fourcc instead of the numeric value. Signed-off-by: Philipp Zabel

[PATCH 8/9] [media] coda: use VDOA for un-tiling custom macroblock format

2016-12-08 Thread Michael Tretter
If the CODA driver is configured to produce NV12 output and the VDOA is available, the VDOA can be used to transform the custom macroblock tiled format to a raster-ordered format for scanout. In this case, set the output format of the CODA to the custom macroblock tiled format, disable the

[PATCH 7/9] [media] coda: fix frame index to returned error

2016-12-08 Thread Michael Tretter
display_idx refers to the frame that will be returned in the next round. The currently processed frame is ctx->display_idx and errors should be reported for this frame. Signed-off-by: Michael Tretter --- drivers/media/platform/coda/coda-bit.c | 2 +- 1 file changed, 1

[PATCH 9/9] [media] coda: support YUYV output if VDOA is used

2016-12-08 Thread Michael Tretter
The VDOA is able to transform the NV12 custom macroblock tiled format of the CODA to YUYV format. If and only if the VDOA is available, the driver can also provide YUYV support. While the driver is configured to produce YUYV output, the CODA must be configured to produce NV12 macroblock tiled

[PATCH 5/9] [media] coda: get VDOA device using dt phandle

2016-12-08 Thread Michael Tretter
Signed-off-by: Michael Tretter --- drivers/media/platform/coda/coda-common.c | 43 +++ drivers/media/platform/coda/coda.h| 1 + 2 files changed, 44 insertions(+) diff --git a/drivers/media/platform/coda/coda-common.c

Re: [PATCH v2 3/3] uvcvideo: add a metadata device node

2016-12-08 Thread Guennadi Liakhovetski
On Thu, 8 Dec 2016, Laurent Pinchart wrote: > Hi Guennadi, > > On Thursday 08 Dec 2016 14:34:46 Guennadi Liakhovetski wrote: > > On Tue, 6 Dec 2016, Laurent Pinchart wrote: > > > On Tuesday 06 Dec 2016 11:39:22 Guennadi Liakhovetski wrote: > > >> On Tue, 6 Dec 2016, Laurent Pinchart wrote: > >

Re: [PATCH 7/9] [media] coda: fix frame index to returned error

2016-12-08 Thread Philipp Zabel
Am Donnerstag, den 08.12.2016, 16:24 +0100 schrieb Michael Tretter: > display_idx refers to the frame that will be returned in the next round. > The currently processed frame is ctx->display_idx and errors should be > reported for this frame. > > Signed-off-by: Michael Tretter

Re: [GIT PULL FOR v4.11] Remove FSF postal address

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 8 Dec 2016 16:47:28 +0200 Sakari Ailus escreveu: > > Ok, if you're doing massive changes on some driver, be my > > guest and remove the FSF address from it. Otherwise, just live > > it as-is. > > This is a cleanup. The patch removes 628 instances of the postal

Re: [PATCH 8/9] [media] coda: use VDOA for un-tiling custom macroblock format

2016-12-08 Thread Philipp Zabel
Am Donnerstag, den 08.12.2016, 16:24 +0100 schrieb Michael Tretter: > If the CODA driver is configured to produce NV12 output and the VDOA is > available, the VDOA can be used to transform the custom macroblock tiled > format to a raster-ordered format for scanout. > > In this case, set the

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Devin, On Thursday 08 Dec 2016 12:50:19 Devin Heitmueller wrote: > Hi Mauro, Laurent, > > I tried out Mauro's latest patch (9:46am EST), and it appears to at > least partially address the issue, but still doesn't work. In fact, > whereas before I was getting stable video with a chroma issue,

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Hi Devin, Em Thu, 8 Dec 2016 12:50:19 -0500 Devin Heitmueller escreveu: > Hi Mauro, Laurent, > > I tried out Mauro's latest patch (9:46am EST), and it appears to at > least partially address the issue, but still doesn't work. In fact, > whereas before I was

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Devin Heitmueller
Hi Mauro, Laurent, I tried out Mauro's latest patch (9:46am EST), and it appears to at least partially address the issue, but still doesn't work. In fact, whereas before I was getting stable video with a chroma issue, with the patch applied I'm now getting no video at all (i.e. tvtime is

Re: [PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 8 Dec 2016 17:46:53 -0200 Mauro Carvalho Chehab escreveu: > commit 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, > depending on the type of bus set via the .set_fmt()

Re: [PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 8 Dec 2016 17:51:02 -0200 Mauro Carvalho Chehab escreveu: > Em Thu, 8 Dec 2016 17:46:53 -0200 > Mauro Carvalho Chehab escreveu: > > > commit 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > > support") added a logic

Re: [PATCH 3/3] [media] em28xx: don't store usb_device at struct em28xx

2016-12-08 Thread kbuild test robot
Hi Mauro, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on next-20161208] [cannot apply to v4.9-rc8] [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/linux/commits/Mauro-Carvalho

Re: [PATCH 3/3] [media] em28xx: don't store usb_device at struct em28xx

2016-12-08 Thread kbuild test robot
Hi Mauro, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on next-20161208] [cannot apply to v4.9-rc8] [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/linux/commits/Mauro-Carvalho

Re: [PATCH 2/3] [media] em28xx: use usb_interface for dev_foo() calls

2016-12-08 Thread Mauro Carvalho Chehab
Em Thu, 8 Dec 2016 23:28:14 +0200 Antti Palosaari escreveu: > Tested both patch 1 and 2 individually and bug is fixed. > > Tested-by: Antti Palosaari Thanks for testing it! > > However, some loggings are wrong as error level used instead of info. If > you has

[PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
commit 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, depending on the type of bus set via the .set_fmt() subdev callback. This is known to cause trobules on devices that don't use a V4L2 subdev devnode, and a

[PATCH 3/3] [media] em28xx: don't store usb_device at struct em28xx

2016-12-08 Thread Mauro Carvalho Chehab
From: Mauro Carvalho Chehab Now that we're storing usb_interface at em28xx struct, there's no good reason to keep storing usb_device, as we can get it from usb_interface. So, get rid of it. Signed-off-by: Mauro Carvalho Chehab Signed-off-by:

[PATCH 2/3] [media] em28xx: use usb_interface for dev_foo() calls

2016-12-08 Thread Mauro Carvalho Chehab
From: Mauro Carvalho Chehab The usb_device->dev is not the right device for dev_foo() calls. Instead, it should use usb_interface->dev. Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 1/3] [media] em28xx: don't change the device's name

2016-12-08 Thread Mauro Carvalho Chehab
From: Mauro Carvalho Chehab Changing the device name, causes it to be unable to remove the sysfs file, causing troubles if a device is removed and then re-inserted. [ 1010.310320] WARNING: CPU: 3 PID: 119 at fs/sysfs/dir.c:31 sysfs_warn_dup+0x7b/0x90 [ 1010.310323]

Re: [PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread kbuild test robot
Hi Mauro, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.9-rc8 next-20161208] [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/linux/commits/Mauro-Carvalho-Chehab/tvp5150-don-t

Re: [PATCH 3/3] [media] em28xx: don't store usb_device at struct em28xx

2016-12-08 Thread kbuild test robot
Hi Mauro, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on next-20161208] [cannot apply to v4.9-rc8] [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/linux/commits/Mauro-Carvalho

Re: [PATCH 2/3] [media] em28xx: use usb_interface for dev_foo() calls

2016-12-08 Thread Antti Palosaari
Tested both patch 1 and 2 individually and bug is fixed. Tested-by: Antti Palosaari However, some loggings are wrong as error level used instead of info. If you has colors enabled those log levels are printed with different colors, red is error and so. These for example are

[PATCH 1/6] v4l: tvp5150: Compile tvp5150_link_setup out if !CONFIG_MEDIA_CONTROLLER

2016-12-08 Thread Laurent Pinchart
The function is only referenced as a handler in the tvp5150_sd_media_ops structure, which is only used when CONFIG_MEDIA_CONTROLLER is set. Don't define the function and the structure when the configuration option is unset to avoid an unused function warning. Signed-off-by: Laurent Pinchart

[PATCH 4/6] v4l: tvp5150: Don't reset device in get/set format handlers

2016-12-08 Thread Laurent Pinchart
The tvp5150 doesn't support format setting through the subdev pad API and thus implements the set format handler as a get format operation. The single handler, tvp5150_fill_fmt(), resets the device by calling tvp5150_reset(). This causes malfunction as the device can be reset at will, possibly

[PATCH 0/6] Fix tvp5150 regression with em28xx

2016-12-08 Thread Laurent Pinchart
Hello, This patch series fixes a regression reported by Devin Heitmueller that affects a large number of em28xx. The problem was introduced by commit 13d52fe40f1f7bbad49128e8ee6a2fe5e13dd18d Author: Mauro Carvalho Chehab Date: Tue Jan 26 06:59:39 2016 -0200

Re: [PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Mauro, On Thursday 08 Dec 2016 21:16:07 Mauro Carvalho Chehab wrote: > Em Fri, 09 Dec 2016 00:33:22 +0200 Laurent Pinchart escreveu: > > Hi Mauro, > > > > I've just sent a series of patches ("[PATCH 0/6] Fix tvp5150 regression > > with em28xx") that should fix this problem properly. I

[GIT PULL v2 FOR v4.11] Remove FSF postal address

2016-12-08 Thread Sakari Ailus
Hi Mauro, This pull request contains a single patch, one that removes the FSF postal address from the headers in source code files. The patch is rebased from what I posted for review. The changes since that are done to the commit message as you requested. It now looks like this:

Re: [PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Mauro Carvalho Chehab
Em Fri, 09 Dec 2016 00:33:22 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > I've just sent a series of patches ("[PATCH 0/6] Fix tvp5150 regression with > em28xx") that should fix this problem properly. I unfortunately haven't been > able to test it with

Re: [PATCH v4l-utils v7 4/7] mediactl: Add media_device creation helpers

2016-12-08 Thread Jacek Anaszewski
Hi Sakari, On 11/24/2016 01:17 PM, Sakari Ailus wrote: Hi Jacek, Thanks for the patchset. On Wed, Oct 12, 2016 at 04:35:19PM +0200, Jacek Anaszewski wrote: Add helper functions that allow for easy instantiation of media_device object basing on whether the media device contains v4l2 subdev

Re: [PATCH v3] [media] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

2016-12-08 Thread Laurent Pinchart
Hi Mauro, I've just sent a series of patches ("[PATCH 0/6] Fix tvp5150 regression with em28xx") that should fix this problem properly. I unfortunately haven't been able to test it with an em28xx device as I don't own any. On Thursday 08 Dec 2016 17:46:53 Mauro Carvalho Chehab wrote: > commit

Re: uvcvideo logging kernel warnings on device disconnect

2016-12-08 Thread Laurent Pinchart
Hi Dave, (CC'ing LKML and Greg KH) On Thursday 08 Dec 2016 12:31:55 Dave Stevenson wrote: > Hi All. > > I'm working with a USB webcam which has been seen to spontaneously > disconnect when in use. That's a separate issue, but when it does it > throws a load of warnings into the kernel log if

Re: [PATCH v4l-utils v7 4/7] mediactl: Add media_device creation helpers

2016-12-08 Thread Jacek Anaszewski
Hhi Sakari, On 12/09/2016 12:05 AM, Sakari Ailus wrote: Hi Jacek, On Thu, Dec 08, 2016 at 11:04:20PM +0100, Jacek Anaszewski wrote: Hi Sakari, On 11/24/2016 01:17 PM, Sakari Ailus wrote: Hi Jacek, Thanks for the patchset. On Wed, Oct 12, 2016 at 04:35:19PM +0200, Jacek Anaszewski wrote:

Re: uvcvideo logging kernel warnings on device disconnect

2016-12-08 Thread Greg KH
On Fri, Dec 09, 2016 at 01:09:21AM +0200, Laurent Pinchart wrote: > Hi Dave, > > (CC'ing LKML and Greg KH) > > On Thursday 08 Dec 2016 12:31:55 Dave Stevenson wrote: > > Hi All. > > > > I'm working with a USB webcam which has been seen to spontaneously > > disconnect when in use. That's a

Re: [PATCH v6 0/5] davinci: VPIF: add DT support

2016-12-08 Thread Kevin Hilman
Hi Javier, Javier Martinez Canillas writes: > On Wed, Dec 7, 2016 at 3:30 PM, Kevin Hilman wrote: >> Prepare the groundwork for adding DT support for davinci VPIF drivers. >> This series does some fixups/cleanups and then adds the DT binding and >> DT

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

2016-12-08 Thread Sakari Ailus
Hi Shuah, On Thu, Dec 08, 2016 at 07:46:03AM -0700, Shuah Khan wrote: > Hi Sakari, > > On 12/07/2016 03:27 PM, Sakari Ailus wrote: > > Hi Shuah, > > > > On Wed, Dec 07, 2016 at 01:03:59PM -0700, Shuah Khan wrote: > >> Hi Sakari, > >> > >> On 12/07/2016 03:52 AM, Sakari Ailus wrote: > >>> Hi

[PATCH 3/4] si2157: Si2141/2151 tuner support.

2016-12-08 Thread CrazyCat
Support for new tuner version. Signed-off-by: CrazyCat --- drivers/media/tuners/si2157.c | 71 ++ drivers/media/tuners/si2157_priv.h | 2 ++ 2 files changed, 67 insertions(+), 6 deletions(-) diff --git

[PATCH 1/4] dvb-usb-cxusb: New RC map for Geniatech Mygica T230.

2016-12-08 Thread CrazyCat
Updated RC map for Geniatech DVB-T/T2 sticks. Signed-off-by: CrazyCat --- drivers/media/usb/dvb-usb/cxusb.c | 42 +-- 1 file changed, 40 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/dvb-usb/cxusb.c

[PATCH 2/4] si2168: Si2168-D60 support.

2016-12-08 Thread CrazyCat
Support for new demod version. Signed-off-by: CrazyCat --- drivers/media/dvb-frontends/si2168.c | 4 drivers/media/dvb-frontends/si2168_priv.h | 2 ++ 2 files changed, 6 insertions(+) diff --git a/drivers/media/dvb-frontends/si2168.c

[PATCH 4/4] dvb-usb-cxusb: Geniatech Mygica T230C support.

2016-12-08 Thread CrazyCat
Updated Geniatech DVB-T/T2 stick support. Signed-off-by: CrazyCat --- drivers/media/usb/dvb-usb/cxusb.c | 136 ++ 1 file changed, 136 insertions(+) diff --git a/drivers/media/usb/dvb-usb/cxusb.c b/drivers/media/usb/dvb-usb/cxusb.c index

Re: [PATCH 2/2] uvcvideo: Document Intel SR300 Depth camera INZI format

2016-12-08 Thread Daniel Johnson
> In addition to my previous comments, wouldn't it make more sense to create a > multiplanar format for this instead of bundling the two separate images into a > single plane ? Unfortunately that would break userspace at this point as multiple libraries are already depending on a patch that

[PATCH 2/6] v4l: tvp5150: Don't inline the tvp5150_selmux() function

2016-12-08 Thread Laurent Pinchart
The function is large and called in several places, don't inline it. Signed-off-by: Laurent Pinchart --- drivers/media/i2c/tvp5150.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c

[PATCH 5/6] v4l: tvp5150: Fix comment regarding output pin muxing

2016-12-08 Thread Laurent Pinchart
The FID/GLCO/VLK/HVLK and INTREQ/GPCL/VBLK pins are muxed differently depending on whether the input is an S-Video or composite signal. The comment that explains the logic doesn't reflect the code. It appears that the comment is incorrect, as disabling the output data bus in composite mode makes

[PATCH 6/6] v4l: tvp5150: Don't override output pinmuxing at stream on/off time

2016-12-08 Thread Laurent Pinchart
The s_stream() handler incorrectly writes the whole MISC_CTL register to enable or disable the outputs, overriding the output pinmuxing configuration. Fix it to only touch the output enable bits. The CONF_SHARED_PIN register is also written by the same function, resulting in muxing the INTREQ

[PATCH 3/6] v4l: tvp5150: Add missing break in set control handler

2016-12-08 Thread Laurent Pinchart
A break is missing resulting in the hue control enabling or disabling the decode completely. Fix it. Fixes: c43875f66140 ("[media] tvp5150: replace MEDIA_ENT_F_CONN_TEST by a control") Signed-off-by: Laurent Pinchart --- drivers/media/i2c/tvp5150.c | 1 + 1

Re: [PATCH v4l-utils v7 4/7] mediactl: Add media_device creation helpers

2016-12-08 Thread Sakari Ailus
Hi Jacek, On Thu, Dec 08, 2016 at 11:04:20PM +0100, Jacek Anaszewski wrote: > Hi Sakari, > > On 11/24/2016 01:17 PM, Sakari Ailus wrote: > >Hi Jacek, > > > >Thanks for the patchset. > > > >On Wed, Oct 12, 2016 at 04:35:19PM +0200, Jacek Anaszewski wrote: > >>Add helper functions that allow for

cron job: media_tree daily build: ERRORS

2016-12-08 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 Dec 9 05:00:17 CET 2016 media-tree git hash:365fe4e0ce218dc5ad10df17b150a366b6015499 media_build