cron job: media_tree daily build: OK

2018-06-01 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 Jun 2 05:00:14 CEST 2018 media-tree git hash:a00031c159748f322f771f3c1d5ed944cba4bd30 media_build

Re: [PATCH v3 03/12] media: ov5640: Remove the clocks registers initialization

2018-06-01 Thread Sam Bobrowicz
>> On May 21, 2018, at 12:39 AM, Maxime Ripard >> wrote: >> >>> On Fri, May 18, 2018 at 07:42:34PM -0700, Sam Bobrowicz wrote: On Fri, May 18, 2018 at 3:35 AM, Daniel Mack wrote: On Thursday, May 17, 2018 10:53 AM, Maxime Ripard wrote: Part of the hardcoded initialization

[PATCH 2/2] rockchip/rga: Remove unrequired wait in .job_abort

2018-06-01 Thread Ezequiel Garcia
As per the documentation, job_abort is not required to wait until the current job finishes. It is redundant to do so, as the core will perform the wait operation. Remove the wait infrastructure completely. Signed-off-by: Ezequiel Garcia --- drivers/media/platform/rockchip/rga/rga.c | 13

[PATCH 1/2] rockchip/rga: Fix broken .start_streaming

2018-06-01 Thread Ezequiel Garcia
Currently, rga_buf_start_streaming() is expecting pm_runtime_get_sync to return zero on success, which is wrong. As per the documentation, pm_runtime_get_sync increments the device's usage counter and return its result. This means it will typically return a positive integer on success and a

[PATCH 0/2] rockchip/rga: A fix and a cleanup

2018-06-01 Thread Ezequiel Garcia
Decided to test v4l2transform filters and found these two issues. Without the first commit, start_streaming fails. The second commit is just a cleanup, removing a seemingly redundant operation. Tested on RK3288 Radxa Rock2 with these kind of pipelines: gst-launch-1.0 videotestsrc !

Re: [PATCH v6 05/13] clk: imx7d: reset parent for mipi csi root

2018-06-01 Thread Stephen Boyd
Quoting Rui Miguel Silva (2018-05-22 07:52:37) > To guarantee that we do not get Overflow in image FIFO the outer bandwidth has > to be faster than inputer bandwidth. For that it must be possible to set a > faster frequency clock. So set new parent to sys_pfd3 clock for the mipi csi > block. > >

Re: [PATCH v6 04/13] clk: imx7d: fix mipi dphy div parent

2018-06-01 Thread Stephen Boyd
Quoting Rui Miguel Silva (2018-05-22 07:52:36) > Fix the mipi dphy root divider to mipi_dphy_pre_div, this would remove a > orphan > clock and set the correct parent. > > before: > cat clk_orphan_summary > enable prepare protect >clock

Re: i.MX6 IPU CSI analog video input on Ventana

2018-06-01 Thread Philipp Zabel
On Fri, 2018-05-25 at 16:21 -0700, Steve Longerbeam wrote: > Hi Philipp, > > On 05/24/2018 11:32 PM, Philipp Zabel wrote: > > On Thu, 2018-05-24 at 11:12 -0700, Steve Longerbeam wrote: > > [...] > > > > The following is required as well. Now the question is why we can't skip > > > > writing those

Re: [PATCH v2 10/10] media: imx.rst: Update doc to reflect fixes to interlaced capture

2018-06-01 Thread Philipp Zabel
On Thu, 2018-05-31 at 17:30 -0700, Steve Longerbeam wrote: > Also add an example pipeline for unconverted capture with interweave > on SabreAuto. > > Signed-off-by: Steve Longerbeam > --- > Documentation/media/v4l-drivers/imx.rst | 51 > - > 1 file changed, 37

Re: [PATCH v2 07/10] media: imx-csi: Allow skipping odd chroma rows for YVU420

2018-06-01 Thread Philipp Zabel
On Thu, 2018-05-31 at 17:30 -0700, Steve Longerbeam wrote: > Skip writing U/V components to odd rows for YVU420 in addition to > YUV420 and NV12. > > Signed-off-by: Steve Longerbeam > --- > drivers/staging/media/imx/imx-media-csi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git

Re: [PATCH v2 06/10] media: imx: Fix field setting logic in try_fmt

2018-06-01 Thread Philipp Zabel
On Thu, 2018-05-31 at 17:30 -0700, Steve Longerbeam wrote: > The logic for setting field type in try_fmt at CSI and PRPENCVF > entities wasn't quite right. The behavior should be: > > - No restrictions on field type at sink pads (except ANY, which is filled > with current sink pad field by

Re: [PATCH v2 04/10] media: imx: interweave only for sequential input/interlaced output fields

2018-06-01 Thread Philipp Zabel
Hi Steve, On Thu, 2018-05-31 at 17:30 -0700, Steve Longerbeam wrote: > IDMAC interlaced scan, a.k.a. interweave, should be enabled at the > IDMAC output pads only if the input field type is 'seq-bt' or 'seq-tb', > and the IDMAC output pad field type is 'interlaced*'. Move this > determination to

Re: [PATCH v2 02/10] gpu: ipu-csi: Check for field type alternate

2018-06-01 Thread Philipp Zabel
Hi Steve, On Thu, 2018-05-31 at 17:30 -0700, Steve Longerbeam wrote: > When the CSI is receiving from a bt.656 bus, include a check for > field type 'alternate' when determining whether to set CSI clock > mode to CCIR656_INTERLACED or CCIR656_PROGRESSIVE. > > Signed-off-by: Steve Longerbeam >

Re: [PATCH v2 01/10] media: imx-csi: Pass sink pad field to ipu_csi_init_interface

2018-06-01 Thread Philipp Zabel
On Thu, 2018-05-31 at 17:30 -0700, Steve Longerbeam wrote: > The output pad's field type was being passed to ipu_csi_init_interface(), > in order to deal with field type 'alternate' at the sink pad, which > is not understood by ipu_csi_init_interface(). > > Remove that code and pass the sink pad

[PATCH] gpu: ipu-v3: Allow negative offsets for interlaced scanning

2018-06-01 Thread Philipp Zabel
The IPU also supports interlaced buffers that start with the bottom field. To achieve this, the the base address EBA has to be increased by a stride length and the interlace offset ILO has to be set to the negative stride. Signed-off-by: Philipp Zabel --- drivers/gpu/ipu-v3/ipu-cpmem.c | 8

Re: i.MX6 IPU CSI analog video input on Ventana

2018-06-01 Thread Philipp Zabel
Hi Krzysztof, On Fri, 2018-06-01 at 12:02 +0200, Krzysztof Hałasa wrote: > Steve Longerbeam writes: > > > I tend to agree, I've found conflicting info out there regarding > > PAL vs. NTSC field order. And I've never liked having to guess > > at input analog standard based on input # lines. I

Re: i.MX6 IPU CSI analog video input on Ventana

2018-06-01 Thread Krzysztof Hałasa
Steve Longerbeam writes: > I tend to agree, I've found conflicting info out there regarding > PAL vs. NTSC field order. And I've never liked having to guess > at input analog standard based on input # lines. I will go ahead > and remove the field order override code. I've merged your current

Re: [RESEND PATCH V2 2/2] media: ak7375: Add ak7375 lens voice coil driver

2018-06-01 Thread Sakari Ailus
On Fri, Jun 01, 2018 at 05:54:30PM +0800, Bing Bu Cao wrote: > > > +AKM AK7375 LENS VOICE COIL DRIVER > > > +M: Tianshu Qiu > > > +L: linux-media@vger.kernel.org > > > +T: git git://linuxtv.org/media_tree.git > > > +S: Maintained > > > +F: drivers/media/i2c/ak7375.c

Re: [RESEND PATCH V2 2/2] media: ak7375: Add ak7375 lens voice coil driver

2018-06-01 Thread Bing Bu Cao
On 2018年06月01日 17:34, Sakari Ailus wrote: Hi Bingbu, A few comments below. On Fri, May 25, 2018 at 05:55:35PM +0800, bingbu@intel.com wrote: From: Bingbu Cao Add a V4L2 sub-device driver for the ak7375 lens voice coil. This is a voice coil module using the I2C bus to control the

Re: [RESEND PATCH V2 2/2] media: ak7375: Add ak7375 lens voice coil driver

2018-06-01 Thread Sakari Ailus
On Fri, May 25, 2018 at 05:55:35PM +0800, bingbu@intel.com wrote: > +static int ak7375_i2c_write(struct ak7375_device *ak7375, > + u8 addr, u16 data, int size) > +{ > + struct i2c_client *client = v4l2_get_subdevdata(>sd); > + int ret; > + u8 buf[3]; > + > + if (size != 1

Re: [RESEND PATCH V2 2/2] media: ak7375: Add ak7375 lens voice coil driver

2018-06-01 Thread Sakari Ailus
Hi Bingbu, A few comments below. On Fri, May 25, 2018 at 05:55:35PM +0800, bingbu@intel.com wrote: > From: Bingbu Cao > > Add a V4L2 sub-device driver for the ak7375 lens voice coil. > This is a voice coil module using the I2C bus to control the > focus position. > > Signed-off-by:

[PATCH v3 2/2] media: dw9807: Add dw9807 vcm driver

2018-06-01 Thread Sakari Ailus
From: Alan Chiang DW9807 is a 10 bit DAC from Dongwoon, designed for linear control of voice coil motor. This driver creates a V4L2 subdevice and provides control to set the desired focus. Signed-off-by: Alan Chiang Signed-off-by: Andy Yeh Reviewed-by: Jacopo Mondi Reviewed-by: Tomasz Figa

[PATCH v3 1/2] media: dt-bindings: Add bindings for Dongwoon DW9807 voice coil

2018-06-01 Thread Sakari Ailus
From: Alan Chiang Dongwoon DW9807 is a voice coil lens driver. Signed-off-by: Alan Chiang Signed-off-by: Andy Yeh Reviewed-by: Rob Herring Signed-off-by: Sakari Ailus --- Documentation/devicetree/bindings/media/i2c/dongwoon,dw9807.txt | 9 + 1 file changed, 9 insertions(+) create

[PATCH v3 0/2] Dongwoon dw9807

2018-06-01 Thread Sakari Ailus
Just posting this after squashing the vcm-only bit to the compatible string. Alan Chiang (2): media: dt-bindings: Add bindings for Dongwoon DW9807 voice coil media: dw9807: Add dw9807 vcm driver .../bindings/media/i2c/dongwoon,dw9807.txt | 9 + MAINTAINERS

Re: [ANN] Meeting to discuss improvements to support MC-based cameras on generic apps

2018-06-01 Thread Javier Martinez Canillas
On Thu, May 31, 2018 at 4:19 PM, Hans Verkuil wrote: > On 05/31/18 15:58, Hans Verkuil wrote: >> On 05/31/18 15:22, Mauro Carvalho Chehab wrote: >>> Em Mon, 28 May 2018 10:43:51 -0300 >>> Mauro Carvalho Chehab escreveu: >>> Em Thu, 17 May 2018 16:07:08 -0300 Mauro Carvalho Chehab