cron job: media_tree daily build: ERRORS

2016-12-09 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 Dec 10 05:00:18 CET 2016 media-tree git hash:365fe4e0ce218dc5ad10df17b150a366b6015499 media_build

cx231xx: how to disable V4L2 interface?

2016-12-09 Thread Petri Gynther
Hi linux-media: For drivers/media/usb/cx231xx, I'd like to add a kernel config option: CONFIG_VIDEO_CX231XX_V4L2 that allows us to disable the V4L2 interface when only the DVB interface is needed. au0828 driver handles this cleanly: drivers/media/usb/au0828/Makefile ifeq

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

2016-12-09 Thread Antti Palosaari
On 12/09/2016 02:17 AM, CrazyCat wrote: Support for new demod version. Signed-off-by: CrazyCat Patch is correct. Could you still use your real name? regards Antti --- drivers/media/dvb-frontends/si2168.c | 4 drivers/media/dvb-frontends/si2168_priv.h | 2 ++

Omap3-isp isp_remove() access subdev.entity after media_device_cleanup()

2016-12-09 Thread Shuah Khan
Hi Sakari, I am looking at omap3 isp_remove() closely and I think there are a few issues there that could cause problems during unbind. isp_remove() tries to do media_entity_cleanup() after it unregisters media_device isp_remove() calls isp_unregister_entities() followed by

[PATCH v2 6/7] [media] coda: use VDOA for un-tiling custom macroblock format

2016-12-09 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 v2 3/7] [media] coda: correctly set capture compose rectangle

2016-12-09 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 v2 5/7] [media] coda: fix frame index to returned error

2016-12-09 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 Acked-by: Philipp Zabel ---

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

2016-12-09 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 v2 4/7] [media] coda: add debug output about tiling

2016-12-09 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 v2 2/7] [media] coda: add i.MX6 VDOA driver

2016-12-09 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 v2 1/7] ARM: dts: imx6qdl: Add VDOA compatible and clocks properties

2016-12-09 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 ---

[PATCH v2 0/7] Add support for Video Data Order Adapter

2016-12-09 Thread Michael Tretter
Hello, This is v2 of a patch series that adds support for the Video Data Order Adapter (VDOA) that can be found on Freescale i.MX6. It converts the macroblock tiled format produced by the CODA 960 video decoder to a raster-ordered format for scanout. Changes since v1: - Dropped patch 8/9 of v1

exposure controlled by v4l2-ctl

2016-12-09 Thread Nandor Balogh
Hi, I would like to talk to someone who knows how auto-exposure and exposure algorithms work which controlled by v4l2-ctl command. Or maybe source code of these would also help, I may understand it by reading the code, though I am not an expert. Thanks, Nandor -- To unsubscribe from this

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

2016-12-09 Thread Shuah Khan
On 12/09/2016 06:17 AM, Sakari Ailus wrote: > Hi Shuah, > > On Fri, Dec 09, 2016 at 01:44:07AM +0200, Sakari Ailus wrote: >> 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

[GIT PULL FOR v4.11] Media pipeline and graph walk cleanups and fixes

2016-12-09 Thread Sakari Ailus
Hi Mauro, This pull request contains fixes and cleanups to the media pipeline and graph walk code. Please pull. 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

[PATCH 1/1] smiapp: Fix error handling in power on sequence

2016-12-09 Thread Sakari Ailus
The error handling code in smiapp_power_on() returned in case of a failed I2C write instead of cleaning up the mess. Fix this. Signed-off-by: Sakari Ailus --- drivers/media/i2c/smiapp/smiapp-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v2 8/9] xilinx: Use a local media device pointer instead

2016-12-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 09 Dec 2016 16:53:41 Sakari Ailus wrote: > The function has a local variable that points to the media device; use > that instead of finding the media device under the entity. > > Signed-off-by: Sakari Ailus

Re: [PATCH v2 9/9] davinci: Use a local media device pointer instead

2016-12-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 09 Dec 2016 16:53:42 Sakari Ailus wrote: > The function has a local variable that points to the media device; use > that instead of finding the media device under the entity. > > Signed-off-by: Sakari Ailus

Re: [PATCH v2 7/9] omap3isp: Use a local media device pointer instead

2016-12-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 09 Dec 2016 16:53:40 Sakari Ailus wrote: > The function has a local variable that points to the media device; use > that instead of finding the media device under the entity. > > Signed-off-by: Sakari Ailus

Re: [PATCH v2 5/9] media: Use single quotes to quote entity names

2016-12-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 09 Dec 2016 16:53:38 Sakari Ailus wrote: > Instead of double quotes, use single quotes to quote entity names. Using > single quotes is consistent with the English language and is also in line > with the practices across the kernel. > >

[PATCH v2 6/9] media: entity: Add debug information to graph walk

2016-12-09 Thread Sakari Ailus
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 Reviewed-by: Laurent Pinchart --- drivers/media/media-entity.c |

[PATCH v2 3/9] media: Rename graph and pipeline structs and functions

2016-12-09 Thread Sakari Ailus
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, rightly, already called media_pipeline (rather than media_entity_pipeline) and what this

[PATCH v2 1/9] media: entity: Fix stream count check

2016-12-09 Thread Sakari Ailus
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 this. Fixes: commit 3801bc7d1b8d ("[media] media: Media Controller fix to not let stream_count

[PATCH v2 7/9] omap3isp: Use a local media device pointer instead

2016-12-09 Thread Sakari Ailus
The function has a local variable that points to the media device; use that instead of finding the media device under the entity. Signed-off-by: Sakari Ailus --- drivers/media/platform/omap3isp/ispvideo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2 4/9] media: entity: Split graph walk iteration into two functions

2016-12-09 Thread Sakari Ailus
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 Reviewed-by: Laurent Pinchart ---

[PATCH v2 8/9] xilinx: Use a local media device pointer instead

2016-12-09 Thread Sakari Ailus
The function has a local variable that points to the media device; use that instead of finding the media device under the entity. Signed-off-by: Sakari Ailus --- drivers/media/platform/xilinx/xilinx-dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH v2] Media pipeline and graph walk cleanups and fixes

2016-12-09 Thread Sakari Ailus
Hi folks, This is the second version of the patchset that contains a few cleanups and fixes for graph traversal and pipeline starting and stopping. The set prepares for further routing changes without still making

[PATCH v2 9/9] davinci: Use a local media device pointer instead

2016-12-09 Thread Sakari Ailus
The function has a local variable that points to the media device; use that instead of finding the media device under the entity. Signed-off-by: Sakari Ailus --- drivers/staging/media/davinci_vpfe/vpfe_video.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH v2 5/9] media: Use single quotes to quote entity names

2016-12-09 Thread Sakari Ailus
Instead of double quotes, use single quotes to quote entity names. Using single quotes is consistent with the English language and is also in line with the practices across the kernel. Signed-off-by: Sakari Ailus --- drivers/media/media-entity.c | 4 ++-- 1 file

[PATCH v2 2/9] media: entity: Be vocal about failing sanity checks

2016-12-09 Thread Sakari Ailus
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 failure is noticed. Signed-off-by: Sakari Ailus Reviewed-by:

[PATCH 1/2] si2168: implement ber statistics

2016-12-09 Thread Antti Palosaari
Implement DVBv5 BER. Signed-off-by: Antti Palosaari --- drivers/media/dvb-frontends/si2168.c | 46 +-- drivers/media/dvb-frontends/si2168_priv.h | 1 + 2 files changed, 45 insertions(+), 2 deletions(-) diff --git

[PATCH 2/2] si2168: implement ucb statistics

2016-12-09 Thread Antti Palosaari
Implement DVBv5 UCB. Only uncorrected blocks are currently counted. Signed-off-by: Antti Palosaari --- drivers/media/dvb-frontends/si2168.c | 24 1 file changed, 24 insertions(+) diff --git a/drivers/media/dvb-frontends/si2168.c

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

2016-12-09 Thread Sakari Ailus
Hi Shuah, On Fri, Dec 09, 2016 at 01:44:07AM +0200, Sakari Ailus wrote: > 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

[PATCH v1.5 2/6] v4l: rcar-fcp: Add an API to retrieve the FCP device

2016-12-09 Thread Ulrich Hecht
From: Laurent Pinchart The new rcar_fcp_get_device() function retrieves the struct device related to the FCP device. This is useful to handle DMA mapping through the right device. Signed-off-by: Laurent Pinchart

[PATCH v1.5 0/6] R-Car DU: Fix IOMMU operation when connected to VSP

2016-12-09 Thread Ulrich Hecht
Hi! This is a slightly updated version of Laurent's series that adds the fix suggested by Magnus Damm and connects the FCP devices on M3-W to their IPMMU. It also drops the patches that have already been picked up in the media tree. With this series and an assortment of patches from the

[PATCH v1.5 6/6] arm64: dts: r8a7796: Connect FCP devices to IPMMU

2016-12-09 Thread Ulrich Hecht
Signed-off-by: Ulrich Hecht --- arch/arm64/boot/dts/renesas/r8a7796.dtsi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi b/arch/arm64/boot/dts/renesas/r8a7796.dtsi index 52e81bb..f5496d4 100644 ---

[PATCH v1.5 1/6] v4l: rcar-fcp: Don't get/put module reference

2016-12-09 Thread Ulrich Hecht
From: Laurent Pinchart Direct callers of the FCP API hold a reference to the FCP module due to module linkage, there's no need to take another one manually. Take a reference to the device instead to ensure that it won't disappear behind the caller's

[PATCH v1.5 3/6] v4l: vsp1: Add API to map and unmap DRM buffers through the VSP

2016-12-09 Thread Ulrich Hecht
From: Laurent Pinchart The display buffers must be mapped for DMA through the device that performs memory access. Expose an API to map and unmap memory through the VSP device to be used by the DU. Signed-off-by: Laurent Pinchart

[PATCH v1.5 5/6] v4l: vsp1: Provide display list and VB2 queue with FCP device

2016-12-09 Thread Ulrich Hecht
Prevents IPMMU trap during boot on r8a7795/6 Salvator-X boards: ipmmu-vmsa febd.mmu: Unhandled faut: status 0x0101 iova 0x7f09a000 Code by Magnus Damm. Signed-off-by: Ulrich Hecht --- drivers/media/platform/vsp1/vsp1_dl.c| 12 +---

[PATCH v1.5 4/6] drm: rcar-du: Map memory through the VSP device

2016-12-09 Thread Ulrich Hecht
From: Laurent Pinchart For planes handled by a VSP instance, map the framebuffer memory through the VSP to ensure proper IOMMU handling. Signed-off-by: Laurent Pinchart ---

[PATCH v2 0/6] Fix tvp5150 regression with em28xx

2016-12-09 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

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

2016-12-09 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 v2 1/6] v4l: tvp5150: Compile tvp5150_link_setup out if !CONFIG_MEDIA_CONTROLLER

2016-12-09 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 v2 5/6] v4l: tvp5150: Fix comment regarding output pin muxing

2016-12-09 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 v2 4/6] v4l: tvp5150: Reset device at probe time, not in get/set format handlers

2016-12-09 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 v2 3/6] v4l: tvp5150: Add missing break in set control handler

2016-12-09 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

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

2016-12-09 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] [media] s5k4ecgx: select CRC32 helper

2016-12-09 Thread Arnd Bergmann
A rare randconfig build failure shows up in this driver when the CRC32 helper is not there: drivers/media/built-in.o: In function `s5k4ecgx_s_power': s5k4ecgx.c:(.text+0x9eb4): undefined reference to `crc32_le' This adds the 'select' that all other users of this function have. Fixes:

[PATCH] [v3] [media] dvb: avoid warning in dvb_net

2016-12-09 Thread Arnd Bergmann
With gcc-5 or higher on x86, we can get a bogus warning in the dvb-net code: drivers/media/dvb-core/dvb_net.c: In function 'dvb_net_ule': arch/x86/include/asm/string_32.h:78:22: error: '*((void *)_addr+4)' may be used uninitialized in this function [-Werror=maybe-uninitialized] The problem here

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

2016-12-09 Thread Antti Palosaari
On 12/09/2016 02:19 AM, CrazyCat wrote: 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

Re: [PATCH v6 2/5] [media] davinci: vpif_capture: remove hard-coded I2C adapter id

2016-12-09 Thread Sakari Ailus
Hi Kevin, On Wed, Dec 07, 2016 at 10:30:22AM -0800, Kevin Hilman wrote: > Remove hard-coded I2C adapter in favor of getting the > ID from platform_data. > > Signed-off-by: Kevin Hilman > --- > drivers/media/platform/davinci/vpif_capture.c | 5 - >

Re: [PATCH 0/6] Fix tvp5150 regression with em28xx

2016-12-09 Thread Mauro Carvalho Chehab
Em Fri, 9 Dec 2016 00:22:40 +0200 Laurent Pinchart escreveu: > Hello, > > This patch series fixes a regression reported by Devin Heitmueller that > affects a large number of em28xx. The problem was introduced by > > commit

Re: [PATCH v6 1/5] [media] davinci: VPIF: fix module loading, init errors

2016-12-09 Thread Sakari Ailus
On Wed, Dec 07, 2016 at 10:30:21AM -0800, Kevin Hilman wrote: > Fix problems with automatic module loading by adding MODULE_ALIAS. Also > fix various load-time errors cause by incorrect or not present > platform_data. Acked-by: Sakari Ailus -- Sakari Ailus

Re: uvcvideo logging kernel warnings on device disconnect

2016-12-09 Thread Greg KH
On Fri, Dec 09, 2016 at 11:14:41AM +0200, Laurent Pinchart wrote: > Hi Greg, > > On Friday 09 Dec 2016 10:11:13 Greg KH wrote: > > On Fri, Dec 09, 2016 at 10:59:24AM +0200, Laurent Pinchart wrote: > > > On Friday 09 Dec 2016 08:25:52 Greg KH wrote: > > >> On Fri, Dec 09, 2016 at 01:09:21AM +0200,

Re: uvcvideo logging kernel warnings on device disconnect

2016-12-09 Thread Laurent Pinchart
Hi Greg, On Friday 09 Dec 2016 10:11:13 Greg KH wrote: > On Fri, Dec 09, 2016 at 10:59:24AM +0200, Laurent Pinchart wrote: > > On Friday 09 Dec 2016 08:25:52 Greg KH wrote: > >> On Fri, Dec 09, 2016 at 01:09:21AM +0200, Laurent Pinchart wrote: > >>> On Thursday 08 Dec 2016 12:31:55 Dave Stevenson

Re: uvcvideo logging kernel warnings on device disconnect

2016-12-09 Thread Greg KH
On Fri, Dec 09, 2016 at 10:59:24AM +0200, Laurent Pinchart wrote: > Hi Greg, > > On Friday 09 Dec 2016 08:25:52 Greg KH wrote: > > On Fri, Dec 09, 2016 at 01:09:21AM +0200, Laurent Pinchart wrote: > > > On Thursday 08 Dec 2016 12:31:55 Dave Stevenson wrote: > > >> Hi All. > > >> > > >> I'm

Re: uvcvideo logging kernel warnings on device disconnect

2016-12-09 Thread Laurent Pinchart
Hi Greg, On Friday 09 Dec 2016 08:25:52 Greg KH wrote: > On Fri, Dec 09, 2016 at 01:09:21AM +0200, Laurent Pinchart wrote: > > 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