cron job: media_tree daily build: ERRORS

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: Tue Aug 8 05:00:16 CEST 2017 media-tree git hash:da48c948c263c9d87dfc64566b3373a858cc8aa2 media_build

Re: [PATCH] media: i2c: OV5647: gate clock lane before stream on

Hi all, 2017-08-07 22:48 GMT+08:00 Luis Oliveira : > Hi all, > > I'm new here, I got to be Maintainer of this driver by the old Maintainer > recommendation. Still getting the hang of it :) > > On 07-Aug-17 13:26, Philipp Zabel wrote: >> Hi Jacob, >> >> On Mon,

Re: [PATCH v2 1/2] dt-bindings: media: Add Cadence MIPI-CSI2 RX Device Tree bindings

Hi Maxime, Thank you for the patch. On Thursday 20 Jul 2017 11:23:01 Maxime Ripard wrote: > The Cadence MIPI-CSI2 RX controller is a CSI2RX bridge that supports up to > 4 CSI-2 lanes, and can route the frames to up to 4 streams, depending on > the hardware implementation. > > It can operate

Re: [PATCH v2 2/2] v4l: cadence: Add Cadence MIPI-CSI2 RX driver

Maxime Ripard wrote on Thu [2017-Jul-20 11:23:02 +0200]: > The Cadence CSI-2 RX Controller is an hardware block meant to be used as a > bridge between a CSI-2 bus and pixel grabbers. > > It supports operating with internal or external D-PHY, with up to 4 lanes,

[PATCH] [media] vs6624: constify vs6624_default_fmt

The structure vs6624_default_fmt is only copied into another structure field, so it can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/i2c/vs6624.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v2 1/2] dt-bindings: media: Add Cadence MIPI-CSI2 RX Device Tree bindings

Maxime Ripard wrote on Thu [2017-Jul-20 11:23:01 +0200]: > The Cadence MIPI-CSI2 RX controller is a CSI2RX bridge that supports up to > 4 CSI-2 lanes, and can route the frames to up to 4 streams, depending on > the hardware implementation. > > It can operate

Re: [PATCH 2/2] cec: fix remote control passthrough

On Mon, Aug 07, 2017 at 03:31:24PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > The 'Press and Hold' operation was not correctly implemented, in > particular the requirement that the repeat doesn't start until > the second identical keypress arrives. The REP_DELAY

Re: [PATCH 2/2] v4l2-flash-led-class: Create separate sub-devices for indicators

On Wed, Jul 19, 2017 at 02:02:46PM +0200, Pavel Machek wrote: > On Tue 2017-07-18 21:41:07, Sakari Ailus wrote: > > The V4L2 flash interface allows controlling multiple LEDs through a single > > sub-devices if, and only if, these LEDs are of different types. This > > approach scales badly for

Re: [PATCH 1/2] rc-main: support CEC protocol keypress timeout

On Mon, Aug 07, 2017 at 03:31:23PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > The CEC protocol has a keypress timeout of 550ms. Add support for this. > > Note: this really should be defined in a protocol struct. That is a good point, the names of the protocol

Re: [PATCH v2 2/2] v4l: cadence: Add Cadence MIPI-CSI2 RX driver

Hi Maxime, Thank you for the patch. On Thursday 20 Jul 2017 11:23:02 Maxime Ripard wrote: > The Cadence CSI-2 RX Controller is an hardware block meant to be used as a > bridge between a CSI-2 bus and pixel grabbers. > > It supports operating with internal or external D-PHY, with up to 4 lanes,

[PATCH 6/6] [media] rc: saa7134: raw decoder can support any protocol

Any protocol for which we have a software decoder, can be enabled. Without this only the protocol of the default keymap can be used (rc-5). Signed-off-by: Sean Young --- drivers/media/pci/saa7134/saa7134-input.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 3/6] [media] rc: simplify ir_raw_event_store_edge()

Since commit 12749b198fa4 ("[media] rc: saa7134: add trailing space for timely decoding"), the workaround of inserting reset events is no longer needed. Verified on a HVR-1150. Fixes: 3f5c4c73322e ("[media] rc: fix ghost keypresses with certain hw") Signed-off-by: Sean Young ---

[PATCH 1/6] [media] rc-core: improve ir_raw_store_edge() handling

The gpio-ir-recv driver does many spurious wakeups (once per edge); the saa7134 driver has special handling to only wakeup 15ms after the first edge. Make this part of rc-core so gpio-ir-recv also benefits from this (so a rc-5 keypress now causes 3 wakeups rather than 24). Signed-off-by: Sean

[PATCH 4/6] [media] rc: ensure we do not read out of bounds

If rc_validate_filter() is called for CEC or XMP, then we would read beyond the end of the array. Suggested-by: Hans Verkuil Signed-off-by: Sean Young --- drivers/media/rc/rc-main.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[PATCH 2/6] [media] rc: saa7134: add trailing space for timely decoding

The gpio-ir-recv driver adds timeouts which the saa7134 lacks; this causes keypress not to arrive, and to only arrive once more IR is received. This is what the commit below calls "ghost keypresses", and it does not solve the issue completely. This makes the IR on the HVR-1150 much more reliable

[PATCH 0/6] Various RC fixes

Some improvements to the saa7134 raw decoder, more efficient gpio-ir-recv and a few cosmetic changes. Sean Young (6): [media] rc-core: improve ir_raw_store_edge() handling [media] rc: saa7134: add trailing space for timely decoding [media] rc: simplify ir_raw_event_store_edge() [media]

Re: [PATCH] keytable.c: add support for the CEC protocol

On Mon, Aug 07, 2017 at 02:52:01PM +0200, Hans Verkuil wrote: > The CEC protocol wasn't known, so 'Supported protocols:' would just say > 'other' instead of 'cec'. > > Signed-off-by: Hans Verkuil Acked-by: Sean Young > --- > diff --git

[PATCH] [media] staging: atomisp: imx: remove dead code

Making some functions 'static' has uncovered a few functions that have no caller, through the gcc warnings: atomisp/i2c/imx/imx.c::12: error: 'imx_t_focus_vcm' defined but not used [-Werror=unused-function] atomisp/i2c/imx/imx.c:1103:12: error: 'imx_vcm_init' defined but not used

Re: [PATCH] media: i2c: OV5647: gate clock lane before stream on

Hi Philipp, 2017-08-07 16:17 GMT+08:00 Philipp Zabel : > Hi Jacob, > > On Mon, 2017-08-07 at 15:11 +0800, Jacob Chen wrote: >> Hi all, >> >> 2017-07-25 10:34 GMT+08:00 Jacob Chen : >> > According to datasheet, BIT5 in reg-0x4800 are used to >> >

Re: Revert "[media] et8ek8: Export OF device ID as module aliases"

Hi! > Just hit this issue too. This seems not to have made it into v4.13 as of > today yet, any chance to get it still in? If Mauro does not react, I guess we can push the patche through akpm. Hmm. Or directly to the Linus. -rc really is about fixing regressions, such as this. Best regards,

EMAIL UPDATE

Please be advised that we will be performing a scheduled email maintenance within the next 24hrs, during this maintenance you will be require to update your email account via link https://tinyurl.com/y9ubogy4 --- This email has been checked for viruses by Avast antivirus software.

Re: [PATCHv3 3/4] drm/bridge: dw-hdmi: add cec driver

On 08/03/2017 12:11 AM, Hans Verkuil wrote: From: Russell King Add a CEC driver for the dw-hdmi hardware. Queued to drm-misc-next after fixing up some minor Makefile/Kconfig conflicts. Thanks, Archit Reviewed-by: Neil Armstrong

Re: [PATCHv3 4/4] drm/bridge: dw-hdmi: remove CEC engine register definitions

On 08/03/2017 12:11 AM, Hans Verkuil wrote: From: Russell King We don't need the CEC engine register definitions, so let's remove them. Queued to drm-misc-next. Thanks, Archit Signed-off-by: Russell King Acked-by: Hans Verkuil

Re: [PATCH] media: i2c: OV5647: gate clock lane before stream on

Hi Jacob, On Mon, 2017-08-07 at 19:06 +0800, Jacob Chen wrote: [...] > >> > --- a/drivers/media/i2c/ov5647.c > >> > +++ b/drivers/media/i2c/ov5647.c > >> > @@ -253,6 +253,10 @@ static int ov5647_stream_on(struct v4l2_subdev *sd) > >> > { > >> > int ret; > >> > > >> > + ret =

[PATCH] keytable.c: add support for the CEC protocol

The CEC protocol wasn't known, so 'Supported protocols:' would just say 'other' instead of 'cec'. Signed-off-by: Hans Verkuil --- diff --git a/utils/keytable/keytable.c b/utils/keytable/keytable.c index 634f4561..55abfc19 100644 --- a/utils/keytable/keytable.c +++

Re: [PATCH] media: i2c: OV5647: gate clock lane before stream on

Hi all, I'm new here, I got to be Maintainer of this driver by the old Maintainer recommendation. Still getting the hang of it :) On 07-Aug-17 13:26, Philipp Zabel wrote: > Hi Jacob, > > On Mon, 2017-08-07 at 19:06 +0800, Jacob Chen wrote: > [...] > --- a/drivers/media/i2c/ov5647.c >

Re: [PATCH 03/12] [media] s5p-g2d: constify v4l2_m2m_ops structures

On 08/06/2017 10:25 AM, Julia Lawall wrote: The v4l2_m2m_ops structures are only passed as the only argument to v4l2_m2m_init, which is declared as const. Thus the v4l2_m2m_ops structures themselves can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall

[PATCH 2/2] cec: fix remote control passthrough

From: Hans Verkuil The 'Press and Hold' operation was not correctly implemented, in particular the requirement that the repeat doesn't start until the second identical keypress arrives. The REP_DELAY value also had to be adjusted (see the comment in the code) to achieve

[PATCH 1/2] rc-main: support CEC protocol keypress timeout

From: Hans Verkuil The CEC protocol has a keypress timeout of 550ms. Add support for this. Note: this really should be defined in a protocol struct. Signed-off-by: Hans Verkuil --- drivers/media/rc/rc-main.c | 17 +++-- 1 file

[PATCH 0/2] cec: improve RC passthrough behavior

From: Hans Verkuil The CEC remote control passthrough behavior was not correct in the case of the 'Press and Hold' auto-repeat feature. This patch series fixes this. The first patch teaches rc about the CEC protocol keypress timeout (550 ms). The second improves the

[PATCH] staging: media: atomisp: use kvmalloc/kvzalloc

Use kvmalloc()/kvzalloc() instead of atomisp_kernel_malloc() /atomisp_kernel_zalloc(). Signed-off-by: Geliang Tang --- .../media/atomisp/pci/atomisp2/atomisp_cmd.c | 31 +- .../media/atomisp/pci/atomisp2/atomisp_cmd.h | 2 --

Re: [PATCH] media: i2c: OV5647: gate clock lane before stream on

Hi all, 2017-07-25 10:34 GMT+08:00 Jacob Chen : > According to datasheet, BIT5 in reg-0x4800 are used to > enable/disable clock lane gate. > > It's wrong to make clock lane free running before > sensor stream on was called, while the mipi phy > are not initialized. > >

[GIT PULL FOR v4.14] cec: little improvements

Better logging, better documentation, add a convenience define. Regards, Hans The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: media: fix warning on v4l2_subdev_call() result interpreted as bool (2017-07-26 13:43:17 -0400) are available in the git

Re: [PATCH][resend] media: ti-vpe: cal: use of_graph_get_remote_endpoint()

Hi Hans > > From: Kuninori Morimoto > > > > Now, we can use of_graph_get_remote_endpoint(). Let's use it. > > I'm not sure why this is resent. It's part of a pending pull request > so I expect it to be merged this week. Sorry, I didn't know that. Thank you

Re: [PATCH] keytable: ensure udev rule fires on rc input device

Hi Sean! On Sun, Aug 06, 2017 at 09:56:55AM +0100, Sean Young wrote: > The rc device is created before the input device, so if ir-keytable runs > too early the input device does not exist yet. > > Ensure that rule fires on creation of a rc device's input device. > > Note that this also prevents

Re: [PATCH] media: i2c: OV5647: gate clock lane before stream on

Hi Jacob, On Mon, 2017-08-07 at 15:11 +0800, Jacob Chen wrote: > Hi all, > > 2017-07-25 10:34 GMT+08:00 Jacob Chen : > > According to datasheet, BIT5 in reg-0x4800 are used to > > enable/disable clock lane gate. > > > > It's wrong to make clock lane free running before > >

Re: [PATCH 11/12] [media] exynos4-is: constify v4l2_m2m_ops structures

On 08/06/2017 10:25 AM, Julia Lawall wrote: The v4l2_m2m_ops structures are only passed as the only argument to v4l2_m2m_init, which is declared as const. Thus the v4l2_m2m_ops structures themselves can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall

[PATCH] [media] staging/imx: always select VIDEOBUF2_DMA_CONTIG

I ran into a rare build error during randconfig testing: drivers/staging/media/imx/imx-media-capture.o: In function `capture_stop_streaming': imx-media-capture.c:(.text+0x224): undefined reference to `vb2_buffer_done' drivers/staging/media/imx/imx-media-capture.o: In function

UVC property auto update

Hi all, I have some USB-3.0 cameras that use UVC. These cameras offer auto updates for various properties. An example of such a property would be gain, that will be adjusted when activating the auto-gain property. These property changes are not queried by the UVC driver, unless it already has

Re: [PATCH v3 10/23] media: camss: Add VFE files

Hi Sakari, On 4.08.2017 21:02, Sakari Ailus wrote: > Hi Todor, > > Todor Tomov wrote: >> Hi Sakari, >> >> Thank you for the review. >> >> On 20.07.2017 17:59, Sakari Ailus wrote: >>> Hi Todor, >>> >>> On Mon, Jul 17, 2017 at 01:33:36PM +0300, Todor Tomov wrote: These files control the VFE

Re: [PATCH][resend] media: ti-vpe: cal: use of_graph_get_remote_endpoint()

On 07/08/17 04:13, Kuninori Morimoto wrote: > > From: Kuninori Morimoto > > Now, we can use of_graph_get_remote_endpoint(). Let's use it. I'm not sure why this is resent. It's part of a pending pull request so I expect it to be merged this week. Regards,

[PATCHv2] cec-api: log the reason for the -EINVAL in cec_s_mode

If cec_debug >= 1 then log why the requested mode returned -EINVAL. It can be hard to debug this since -EINVAL can be returned for many reasons. So this should help. Signed-off-by: Hans Verkuil --- Changes since v1: Removed some error injection code that inadvertently

Re: [PATCH 06/12] [media] exynos-gsc: constify v4l2_m2m_ops structures

On 08/06/2017 10:25 AM, Julia Lawall wrote: The v4l2_m2m_ops structures are only passed as the only argument to v4l2_m2m_init, which is declared as const. Thus the v4l2_m2m_ops structures themselves can be const. Done with the help of Coccinelle. Signed-off-by: Julia

Re: [PATCH] [media] solo6x10: export hardware GPIO pins 8:31 to gpiolib interface

Hi Anton, Nothing serious, just some purist nitpicking below. On Wed, Aug 02, 2017 at 06:17:02PM +0400, Anton Sviridenko wrote: > 24 GPIO pins from 32 available on solo6x10 chips are exported > to gpiolib. First 8 GPIOs are reserved for internal use on capture card > boards, GPIOs in range 8:15