cron job: media_tree daily build: ERRORS

2017-05-26 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 May 27 05:00:25 CEST 2017 media-tree git hash:36bcba973ad478042d1ffc6e89afd92e8bd17030 media_build

Re: [PATCH v5 3/7] media: i2c: max2175: Add MAX2175 support

2017-05-26 Thread Sakari Ailus
On Fri, May 26, 2017 at 02:47:32PM +, Ramesh Shanmugasundaram wrote: > Hi Sakari, > > Thanks for the review comments on the patches. Sorry for the late response > as I was caught up with another work. No worries. > > I will incorporate your comments and rebase it on top of your branch. I >

Re: [PATCH] davinci: vpif_capture: fix default pixel format for BT.656/BT.1120 video

2017-05-26 Thread Kevin Hilman
Sekhar Nori writes: > For both BT.656 and BT.1120 video, the pixel format > used by VPIF is Y/CbCr 4:2:2 in semi-planar format > (Luma in one plane and Chroma in another). This > corresponds to NV16 pixel format. > > This is documented in section 36.2.3 of OMAP-L138 > Technical

Re: [PATCH v2 1/2] media: entity: Add pad_from_fwnode entity operation

2017-05-26 Thread Sakari Ailus
Hejssan, On Wed, May 24, 2017 at 04:07:36PM +0200, Niklas Söderlund wrote: > Hi Sakari, > > Thanks for your feedback. > > On 2017-05-24 16:21:37 +0300, Sakari Ailus wrote: > > Hi Niklas, > > > > On Wed, May 24, 2017 at 02:09:06AM +0200, Niklas Söderlund wrote: > > > From: Niklas Söderlund

[PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-26 Thread Hyungwoo Yang
This patch adds driver for Omnivision's ov13858 sensor, the driver supports following features: - manual exposure/analog gain - two link frequencies - VBLANK support - media controller support - runtime pm support Signed-off-by: Hyungwoo Yang ---

[regression] Build failure on ubuntu 16.04 LTS

2017-05-26 Thread Vincent McIntyre
$ cat /etc/lsb-release DISTRIB_ID=Ubuntu DISTRIB_RELEASE=16.04 DISTRIB_CODENAME=xenial DISTRIB_DESCRIPTION="Ubuntu 16.04.2 LTS" $ uname -a Linux testbox 4.8.0-53-generic #56~16.04.1-Ubuntu SMP Tue May 16 01:18:56 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux $ git remote -v origin

RE: [PATCH 1/1] [media] i2c: add support for OV13858 sensor

2017-05-26 Thread Yang, Hyungwoo
Hi Sakari, I've submitted V2 yesterday. If possible, can you review that one also ? I'm learning many things from your review comments. I think in V2, I've addressed most of comments except raw bayer format. For ray bayer format, for now, I intentionally don't support crop since it requires

Re: [PATCH 1/1] [media] i2c: add support for OV13858 sensor

2017-05-26 Thread Sakari Ailus
Hi Hyungwoo, On Wed, May 24, 2017 at 11:13:50PM +, Yang, Hyungwoo wrote: ... > > > +static inline int ov13858_write_reg_list(struct ov13858 *ov13858, > > > > I'd drop inline. > > if it's not mandatory for upstream, I prefer to keep inline for people who > want to port this with a

Re: [patch, libv4l]: add sdlcam example for testing digital still camera functionality

2017-05-26 Thread Pavel Machek
Hi! > Add simple SDL-based application for capturing photos. Manual > focus/gain/exposure can be set, flash can be controlled and > autofocus/autogain can be selected if camera supports that. > > It is already useful for testing autofocus/autogain improvements to > the libraries on Nokia N900. >

[PATCH v6] dw9714: Initial driver for dw9714 VCM

2017-05-26 Thread Rajmohan Mani
DW9714 is a 10 bit DAC, 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: Rajmohan Mani --- Changes in v6: - Addressed review comments from Sakari on v5 patch

RE: [PATCH v5] dw9714: Initial driver for dw9714 VCM

2017-05-26 Thread Mani, Rajmohan
Hi Sakari, > -Original Message- > From: Sakari Ailus [mailto:sakari.ai...@iki.fi] > Sent: Friday, May 26, 2017 1:20 AM > To: Mani, Rajmohan > Cc: linux-media@vger.kernel.org; mche...@kernel.org; hverk...@xs4all.nl; > tf...@chromium.org; s.nawro...@samsung.com;

[PATCH 10/11] atomisp: remove sh_css_irq - it contains nothing

2017-05-26 Thread Alan Cox
We won't be adding abstractions or moving them here so kill it. Signed-off-by: Alan Cox --- .../staging/media/atomisp/pci/atomisp2/Makefile|1 - .../atomisp/pci/atomisp2/css2400/sh_css_irq.c | 16 2 files changed, 17 deletions(-) delete

[PATCH 11/11] atomisp: de-duplicate sh_css_mmu_set_page_table_base_index

2017-05-26 Thread Alan Cox
Between the ISP2400 and ISP2401 code base this function moved file. The merge of the drivers left us with two version in ifdefs. Resolve this down to a single copy. Signed-off-by: Alan Cox --- .../pci/atomisp2/css2400/ia_css_mmu_private.h |2 --

[PATCH 09/11] atomisp: Unify lut free logic

2017-05-26 Thread Alan Cox
ISP2401 introduced a helper for this which we can use just as well on the ISP2400 and remove some more noise differences. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c|7 --- .../atomisp/pci/atomisp2/css2400/sh_css_params.c |

[PATCH 08/11] atomisp: Unify load_preview_binaries for the most part

2017-05-26 Thread Alan Cox
ISP2401 introduced a rather sensible change to cut through the structure spaghetti. Adopt that for the ISP2400 as well. It makes no difference to the actual code other than readability. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c| 40

[PATCH 07/11] atomisp: unify sh_css_hmm_buffer_record_acquire

2017-05-26 Thread Alan Cox
The ISP2401 version of this function returns a pointer to the buffer, whilst the ISP2400 version returns a boolean if a slot is found. We can trivially unify the code to use the ISP2401 version. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c

[PATCHv2 06/11] atomisp: eliminate dead code under HAS_RES_MGR

2017-05-26 Thread Alan Cox
This define is never set and these code paths are never used so they can go away. Signed-off-by: Alan Cox --- .../css2400/isp/modes/interface/isp_const.h| 16 - .../css2400/isp/modes/interface/isp_exprs.h| 23 --

[PATCHv2 04/11] atomisp2: tidy up confused ifdefs

2017-05-26 Thread Alan Cox
The two drivers were machine merged and in this case the machine output was to say the least not optimal. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c| 26 1 file changed, 5 insertions(+), 21 deletions(-) diff

[PATCH 05/11] atomisp2: off by one in atomisp_s_input()

2017-05-26 Thread Alan Cox
From: Dan Carpenter The isp->inputs[] array has isp->input_cnt elements which have been initialized so this > should be >=. This bug is harmless. The check against ATOM_ISP_MAX_INPUTS prevents us from reading beyond the end of the array. The uninitialized elements

[PATCHv2 03/11] atomisp2: remove HRT_UNSCHED

2017-05-26 Thread Alan Cox
HRT_UNSCHED is never defined or set in the driver, so this is dead code that can be retired, simplifying the code a bit further. Signed-off-by: Alan Cox --- .../css2400/runtime/debug/src/ia_css_debug.c |7 --- .../atomisp2/css2400/runtime/spctrl/src/spctrl.c

[PATCHv2 02/11] atomisp: remove NUM_OF_BLS

2017-05-26 Thread Alan Cox
With the removal of the HAS_BL bootloader code the value of NUM_OF_BLS is an invariant zero. So let's get rid of it. Signed-off-by: Alan Cox --- .../atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff

[PATCHv2 01/11] atompisp: HAS_BL is never defined so lose it

2017-05-26 Thread Alan Cox
Kill off the HAS_BL define and the code and includes it brackets. We never define HAS_BL or use that functionality. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c| 101

[PATCH 05/12] hdlcdrv: Fix division by zero when bitrate is unset

2017-05-26 Thread Alan Cox
The code attempts to check for out of range calibration. What it forgets to do is check for the 0 bitrate case. As a result the range check itself oopses the kernel. Found by Andrey Konovalov using Syzkaller. Signed-off-by: Alan Cox --- drivers/net/hamradio/hdlcdrv.c |

[PATCH 04/12] atomisp2: tidy up confused ifdefs

2017-05-26 Thread Alan Cox
The two drivers were machine merged and in this case the machine output was to say the least not optimal. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c| 26 1 file changed, 5 insertions(+), 21 deletions(-) diff

[PATCH 03/12] atomisp2: remove HRT_UNSCHED

2017-05-26 Thread Alan Cox
HRT_UNSCHED is never defined or set in the driver, so this is dead code that can be retired, simplifying the code a bit further. Signed-off-by: Alan Cox --- .../css2400/runtime/debug/src/ia_css_debug.c |7 --- .../atomisp2/css2400/runtime/spctrl/src/spctrl.c

[PATCH 02/12] atomisp: remove NUM_OF_BLS

2017-05-26 Thread Alan Cox
With the removal of the HAS_BL bootloader code the value of NUM_OF_BLS is an invariant zero. So let's get rid of it. Signed-off-by: Alan Cox --- .../atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff

[PATCH 01/12] atompisp: HAS_BL is never defined so lose it

2017-05-26 Thread Alan Cox
Kill off the HAS_BL define and the code and includes it brackets. We never define HAS_BL or use that functionality. Signed-off-by: Alan Cox --- .../media/atomisp/pci/atomisp2/css2400/sh_css.c| 101

RE: [PATCH v5 3/7] media: i2c: max2175: Add MAX2175 support

2017-05-26 Thread Ramesh Shanmugasundaram
Hi Sakari, Thanks for the review comments on the patches. Sorry for the late response as I was caught up with another work. I will incorporate your comments and rebase it on top of your branch. I see it is not there in media-tree master yet. Please let me know if there is a change in plan.

Re: [PATCH 1/3] [media] si2157: get chip id during probing

2017-05-26 Thread Steven Toth
>> ep: 81 l:9 08260080ff5901 >> >> here you see all the from the device. You need to be able to see the traffic on the physical I2C bus in order to help diagnose issues like this. You're going to want to see ACKS/NAKS, clocks and other I2C bus activity. You'll need to solder down

[PATCH 1/1] ad5820: unregister async sub-device

2017-05-26 Thread Sakari Ailus
The async sub-device was not unregistered in ad5820_remove() as it should have been; do it now. Also remove the now-redundant v4l2_device_unregister_subdev(). Signed-off-by: Sakari Ailus --- drivers/media/i2c/ad5820.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [RFC PATCH 6/7] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-05-26 Thread Daniel Vetter
On Fri, May 26, 2017 at 12:34 PM, Hans Verkuil wrote: >>> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig >>> index 78d7fc0ebb57..dd771ce8a3d0 100644 >>> --- a/drivers/gpu/drm/Kconfig >>> +++ b/drivers/gpu/drm/Kconfig >>> @@ -120,6 +120,9 @@ config

[PATCH] davinci: vpif_capture: fix default pixel format for BT.656/BT.1120 video

2017-05-26 Thread Sekhar Nori
For both BT.656 and BT.1120 video, the pixel format used by VPIF is Y/CbCr 4:2:2 in semi-planar format (Luma in one plane and Chroma in another). This corresponds to NV16 pixel format. This is documented in section 36.2.3 of OMAP-L138 Technical Reference Manual, SPRUH77A. The VPIF driver

Re: [RFC PATCH 6/7] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-05-26 Thread Hans Verkuil
On 05/26/2017 09:18 AM, Daniel Vetter wrote: > On Thu, May 25, 2017 at 05:06:25PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> This adds support for the DisplayPort CEC-Tunneling-over-AUX >> feature that is part of the DisplayPort 1.3 standard. >> >>

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Hans Verkuil
On 05/26/2017 09:15 AM, Daniel Vetter wrote: > On Thu, May 25, 2017 at 05:06:26PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Implement support for this DisplayPort feature. >> >> The cec device is created whenever it detects an adapter that >> has this feature.

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Hans Verkuil
On 05/26/2017 12:13 PM, Jani Nikula wrote: > On Thu, 25 May 2017, Hans Verkuil wrote: >> @@ -4179,6 +4181,33 @@ intel_dp_check_mst_status(struct intel_dp *intel_dp) >> return -EINVAL; >> } >> >> +static bool >> +intel_dp_check_cec_status(struct intel_dp *intel_dp) >>

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Jani Nikula
On Thu, 25 May 2017, Hans Verkuil wrote: > @@ -4179,6 +4181,33 @@ intel_dp_check_mst_status(struct intel_dp *intel_dp) > return -EINVAL; > } > > +static bool > +intel_dp_check_cec_status(struct intel_dp *intel_dp) > +{ > + bool handled = false; > + > + for

Re: [RFC PATCH 5/7] drm/cec: Add CEC over Aux register definitions

2017-05-26 Thread Jani Nikula
On Thu, 25 May 2017, Hans Verkuil wrote: > From: Clint Taylor > > Adding DPCD register definitions from the DP 1.3 specification for CEC > over AUX support. > > V2: Add DP_ prefix to all defines. > V3: missed prefixes from the ESI1 defines > > Cc:

[PATCH 1/1] v4l2-ctrls.c: Implement unlocked variant of v4l2_ctrl_handler_setup()

2017-05-26 Thread Sakari Ailus
Sometimes the caller is already holding the control handler mutex and using it to serialise something. Provide an unlocked variant of the same function to be used in those cases. Signed-off-by: Sakari Ailus --- drivers/media/v4l2-core/v4l2-ctrls.c | 21

Re: [PATCH] ov5670: Add Omnivision OV5670 5M sensor support

2017-05-26 Thread Sakari Ailus
Hi Chiranjeevi, On Fri, May 26, 2017 at 01:19:59AM +, Rapolu, Chiranjeevi wrote: > >> +/* Analog gain controls from sensor */ > >> +#define ANALOG_GAIN_MIN 0 > >> +#define ANALOG_GAIN_MAX 8191 > >> +#define ANALOG_GAIN_STEP1 > >> +#define ANALOG_GAIN_DEFAULT

Re: [PATCH v5] dw9714: Initial driver for dw9714 VCM

2017-05-26 Thread Sakari Ailus
Hi Rajmohan, Thankd for the update. A few more comments I missed earlier. On Thu, May 25, 2017 at 06:50:35PM -0700, Rajmohan Mani wrote: > DW9714 is a 10 bit DAC, designed for linear > control of voice coil motor. > > This driver creates a V4L2 subdevice and > provides control to set the

Re: [RFC PATCH 6/7] drm: add support for DisplayPort CEC-Tunneling-over-AUX

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 05:06:25PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > This adds support for the DisplayPort CEC-Tunneling-over-AUX > feature that is part of the DisplayPort 1.3 standard. > > Unfortunately, not all DisplayPort/USB-C to HDMI adapters with

Re: [RFC PATCH 7/7] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2017-05-26 Thread Daniel Vetter
On Thu, May 25, 2017 at 05:06:26PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > Implement support for this DisplayPort feature. > > The cec device is created whenever it detects an adapter that > has this feature. It is only removed when a new adapter is connected

Re: [PATCH v3 0/4] r8a7793 Gose video input support

2017-05-26 Thread Simon Horman
On Fri, May 19, 2017 at 03:07:00PM +0200, Ulrich Hecht wrote: > Hi! > > This is a by-the-datasheet implementation of analog and digital video input > on the Gose board. > > This revision adds new bindings that distinguish between ADV7180 variants > with three and six input ports. There are

Re: [PATCH v2 1/1] [media] i2c: Add Omnivision OV5670 5M sensor support

2017-05-26 Thread kbuild test robot
/i2c-Add-Omnivision-OV5670-5M-sensor-support/20170526-134545 base: git://linuxtv.org/media_tree.git master config: ia64-allmodconfig (attached as .config) compiler: ia64-linux-gcc (GCC) 6.2.0 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin