cron job: media_tree daily build: WARNINGS

2017-11-24 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 Nov 25 05:00:17 CET 2017 media-tree git hash:30b4e122d71cbec2944a5f8b558b88936ee42f10 media_build

[PATCH] [media] v4l: mt9v032: Disable clock on error paths

2017-11-24 Thread Alexey Khoroshilov
mt9v032_power_on() leaves clk enabled in case of errors, but it is not expected by its callers. There is a similar problem in mt9v032_registered(). Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov ---

Re: [PATCH] v4l: vsp1: Fix function declaration/definition mismatch

2017-11-24 Thread Kieran Bingham
Hi Eugeniu, Thankyou for the patch, Laurent - Any comments on this? Otherwise I'll bundle this in with my suspend/resume patch for a pull request. On 20/08/17 13:40, Eugeniu Rosca wrote: > From: Eugeniu Rosca > > Cppcheck v1.81 complains that the parameter names of

Re: [PATCH] media: ov7670: use v4l2_async_unregister_subdev()

2017-11-24 Thread Kieran Bingham
Hi Akinobu, Thankyou for the patch. On 24/11/17 14:40, Akinobu Mita wrote: > The sub-device for ov7670 is registered by v4l2_async_register_subdev(). > So it should be unregistered by v4l2_async_unregister_subdev() instead of > v4l2_device_unregister_subdev(). > > Cc: Jonathan Corbet

Re: [GIT PULL] SAA716x DVB driver

2017-11-24 Thread Tycho Lürsen
Hi Mauro, afaik the last communication about submission of this driver was about two months ago. This driver is important to me, because I own several TurboSight cards that are saa716x based. I want to submit a patch that supports my cards. Of course I can only do so when you accept this

[PATCH] media: ov7670: add V4L2_CID_TEST_PATTERN control

2017-11-24 Thread Akinobu Mita
The ov7670 has the test pattern generator features. This makes use of it through V4L2_CID_TEST_PATTERN control. Cc: Jonathan Corbet Cc: Sakari Ailus Cc: Mauro Carvalho Chehab Signed-off-by: Akinobu Mita

[PATCH] media: ov7670: use v4l2_async_unregister_subdev()

2017-11-24 Thread Akinobu Mita
The sub-device for ov7670 is registered by v4l2_async_register_subdev(). So it should be unregistered by v4l2_async_unregister_subdev() instead of v4l2_device_unregister_subdev(). Cc: Jonathan Corbet Cc: Sakari Ailus Cc: Mauro Carvalho Chehab

Re: [PATCH v1 4/4] media: ov5640: add support of RGB565 and YUYV formats

2017-11-24 Thread Sakari Ailus
Hi Hugues, On Thu, Nov 16, 2017 at 02:41:42PM +0100, Hugues Fruchet wrote: > Add RGB565 (LE & BE) and YUV422 YUYV format in addition > to existing YUV422 UYVY format. > > Signed-off-by: Hugues Fruchet > --- > drivers/media/i2c/ov5640.c | 79 >

Re: [PATCH v1 3/4] media: ov5640: add support of DVP parallel interface

2017-11-24 Thread Sakari Ailus
Hi Hugues, On Thu, Nov 16, 2017 at 02:41:41PM +0100, Hugues Fruchet wrote: > @@ -2185,7 +2262,11 @@ static int ov5640_s_stream(struct v4l2_subdev *sd, int > enable) > goto out; > } > > - ret = ov5640_set_stream(sensor, enable); > +

[PATCH] media: dvb_frontend: dvb_unregister_frontend must not call dvb_detach for fe->ops.release

2017-11-24 Thread Wolfgang Rohdewald
because ops.release was never dvb_attached. Which makes sense because f->ops.release does not attach anything. Now, rmmod dvb_usb_pctv452e correctly sets counters for stb6100/stb0899 to 0. Before, stb0899 got a counter -1, and for my 4 receivers I got 3 OOPses like Nov 24 14:40:41 s5 kernel: [

Re: [PATCH v3] drm: bridge: synopsys/dw-hdmi: Enable cec clock

2017-11-24 Thread Pierre-Hugues Husson
Hi, >> On 11/20/2017 06:00 PM, Hans Verkuil wrote: >>> I didn't see this merged for 4.15, is it too late to include this? >>> All other changes needed to get CEC to work on rk3288 and rk3399 are all >>> merged. >> >> Sorry for the late reply. I was out last week. >> >> Dave recently sent the

Re: [PATCH 2/2] media: venus: venc: Apply inloop deblocking filter

2017-11-24 Thread Stanimir Varbanov
Hi Loic, Thanks for the patch! On 11/24/2017 11:34 AM, Loic Poulain wrote: > Deblocking filter allows to reduce blocking artifacts and improve > visual quality. This is configurable via the V4L2 API but eventually > not applied to the encoder. > > Note that alpha and beta deblocking values are

Re: [PATCH 1/2] media: venus: venc: configure entropy mode

2017-11-24 Thread Stanimir Varbanov
Hi Loic, Thanks for the patch! On 11/24/2017 11:34 AM, Loic Poulain wrote: > H264 entropy mode can be selected via V4L2 API but is eventually not > applied. Configure encoder with selected mode, CALVC (def) or CABAC. > > Note that hw/firmware also expects a CABAC model configuration which >

[PATCH 3/3] media: cec: move cec autorepeat handling to rc-core

2017-11-24 Thread Sean Young
CEC autorepeat is different than other protocols. Autorepeat is triggered by the first repeated user control pressed CEC message, rather than a fixed REP_DELAY. This change also does away with the KEY_UP event directly after the first KEY_DOWN event, which was used to stop autorepeat from

[PATCH 0/3] Improve CEC autorepeat handling

2017-11-24 Thread Sean Young
Due to the slowness of the CEC bus, autorepeat handling rather special on CEC. If the repeated user control pressed message is received, a keydown repeat should be sent immediately. By handling this in the input layer, we can remove some ugly code from cec, which also sends a keyup event after

[PATCH 2/3] input: handle case whether first repeated key triggers repeat

2017-11-24 Thread Sean Young
In the CEC protocol, as soon as the first repeated key is received, the autorepeat should start. We introduce a special value 3 for this situation. Signed-off-by: Sean Young --- Documentation/input/input.rst | 4 +++- drivers/input/input.c | 17 +++-- 2 files

[PATCH 1/3] input: remove redundant check for EV_REP

2017-11-24 Thread Sean Young
The caller input_pass_values has already checked this bit. Signed-off-by: Sean Young --- drivers/input/input.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/input/input.c b/drivers/input/input.c index 762bfb9487dc..ecc41d65b82a 100644 ---

[PATCH] Override sat_number by value from channel file if it not specified

2017-11-24 Thread Maksym Veremeyenko
Hi, attached patch fix using SAT_NUMBER value from channel file. please apply -- Maksym Veremeyenko From fe0193f8115ad256acfd2be42f5fed0d465dff55 Mon Sep 17 00:00:00 2001 From: Maksym Veremeyenko Date: Thu, 23 Nov 2017 14:15:28 +0100 Subject: [PATCH 2/4] Override sat_number by

[PATCH] Fix selecting satellite number 0

2017-11-24 Thread Maksym Veremeyenko
Hi, attached patch fixing zapping to satellite #0 without it specifying -S 0 for *dvbv5-zap* does not work and sat_number remains a negative default value. as result, no settings where made in *dvbsat_diseqc_set_input* ( https://git.linuxtv.org/v4l-utils.git/tree/lib/libdvbv5/dvb-sat.c#n526 )

[PATCH 1/2] media: venus: venc: configure entropy mode

2017-11-24 Thread Loic Poulain
H264 entropy mode can be selected via V4L2 API but is eventually not applied. Configure encoder with selected mode, CALVC (def) or CABAC. Note that hw/firmware also expects a CABAC model configuration which currently doesn't have existing V4L2 API control. For now, use model_0 which seems always

[PATCH 2/2] media: venus: venc: Apply inloop deblocking filter

2017-11-24 Thread Loic Poulain
Deblocking filter allows to reduce blocking artifacts and improve visual quality. This is configurable via the V4L2 API but eventually not applied to the encoder. Note that alpha and beta deblocking values are 32-bit signed (-6;+6). Signed-off-by: Loic Poulain ---

Re: notifier is skipped in some situations

2017-11-24 Thread Tomasz Figa
On Fri, Nov 24, 2017 at 6:17 PM, Sakari Ailus wrote: > Hi Tomasz, > > On Fri, Nov 24, 2017 at 06:03:26PM +0900, Tomasz Figa wrote: >> Hi Sakari, >> >> We have the following graph: >> >> ISP (registers notifier for v4l2_dev) >> >> | >> >> PHY (registers

Re: notifier is skipped in some situations

2017-11-24 Thread Sakari Ailus
Hi Tomasz, On Fri, Nov 24, 2017 at 06:03:26PM +0900, Tomasz Figa wrote: > Hi Sakari, > > We have the following graph: > > ISP (registers notifier for v4l2_dev) > > | > > PHY (registers notifier for v4l2_subdev, just like sensors for > flash/focuser) > > | \ > > sensor0

Re: notifier is skipped in some situations

2017-11-24 Thread Tomasz Figa
Hi Sakari, We have the following graph: ISP (registers notifier for v4l2_dev) | PHY (registers notifier for v4l2_subdev, just like sensors for flash/focuser) | \ sensor0 sensor1 ... Both ISP and PHY are completely separate drivers not directly aware of each other,

Re: notifier is skipped in some situations

2017-11-24 Thread Sakari Ailus
Hi Jacob, On Fri, Nov 24, 2017 at 09:00:14AM +0800, Jacob Chen wrote: > Hi Sakari, > > I encountered a problem when using async sub-notifiers. > > It's like that: > There are two notifiers, and they are waiting for one subdev. > When this subdev is probing, only one notifier is

[PATCH/RFC] not use a DiSEqC switch

2017-11-24 Thread Maksym Veremeyenko
Hi, there is a code in function *dvbsat_diseqc_set_input*: [...] /* Negative numbers means to not use a DiSEqC switch */ if (parms->p.sat_number < 0) return 0; [...] if it mean /there is no DiSEqC switch/ then LNB's *polarity* and *band* settings still should

Re: [PATCH v3] drm: bridge: synopsys/dw-hdmi: Enable cec clock

2017-11-24 Thread Hans Verkuil
On 11/24/2017 09:04 AM, Archit Taneja wrote: > Hi, > > On 11/20/2017 06:00 PM, Hans Verkuil wrote: >> I didn't see this merged for 4.15, is it too late to include this? >> All other changes needed to get CEC to work on rk3288 and rk3399 are all >> merged. > > Sorry for the late reply. I was out

Re: [PATCH v3] drm: bridge: synopsys/dw-hdmi: Enable cec clock

2017-11-24 Thread Archit Taneja
Hi, On 11/20/2017 06:00 PM, Hans Verkuil wrote: I didn't see this merged for 4.15, is it too late to include this? All other changes needed to get CEC to work on rk3288 and rk3399 are all merged. Sorry for the late reply. I was out last week. Dave recently sent the second pull request for