cron job: media_tree daily build: WARNINGS

2017-05-29 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: Tue May 30 05:00:17 CEST 2017 media-tree git hash:36bcba973ad478042d1ffc6e89afd92e8bd17030 media_build

Re: [PATCH v7 16/34] [media] add Omnivision OV5640 sensor driver

2017-05-29 Thread Steve Longerbeam
Hi Sakari, On 05/29/2017 08:55 AM, Sakari Ailus wrote: Hi Steve, A few comments below. On Wed, May 24, 2017 at 05:29:31PM -0700, Steve Longerbeam wrote: This driver is based on ov5640_mipi.c from Freescale imx_3.10.17_1.0.0_beta branch, modified heavily to bring forward to latest interfaces

Re: [PATCH v7 2/3] [media] add mux and video interface bridge entity functions

2017-05-29 Thread Sakari Ailus
Hi Philipp, On Mon, May 29, 2017 at 05:12:10PM +0200, Philipp Zabel wrote: > Signed-off-by: Philipp Zabel > > - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX > > Signed-off-by: Steve Longerbeam > --- > Changes since [1]: > - Reword the

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

2017-05-29 Thread Yang, Hyungwoo
Hi Sakari, Thank you for the reviews. I have addressed all of your comments and please review the "v4" I already submitted. -Hyungwoo -Original Message- > From: Sakari Ailus [mailto:sakari.ai...@iki.fi] > Sent: Monday, May 29, 2017 12:30 AM > To: Yang, Hyungwoo

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-05-29 Thread Jacek Anaszewski
Hi Mauro, This patch seems to have lost somehow. Could you help merging it? Thanks, Jacek Anaszewski On 05/29/2017 09:29 AM, Alexandre Courbot wrote: > Hi everyone, > > On Thu, Apr 27, 2017 at 4:35 AM, Jacek Anaszewski > wrote: >> On 04/26/2017 04:54 AM, Alexandre

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

2017-05-29 Thread Yang, Hyungwoo
Hi Tomasz, I left my comments. Thanks, Hyungwoo -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Monday, May 29, 2017 12:35 AM > To: Yang, Hyungwoo > Cc: Sakari Ailus ; linux-media@vger.kernel.org; >

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

2017-05-29 Thread Daniel Vetter
On Fri, May 26, 2017 at 12:20:48PM +0200, Hans Verkuil wrote: > 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

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

2017-05-29 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 ---

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Steve Longerbeam
Hi Sakari, On 05/29/2017 08:36 AM, Sakari Ailus wrote: Hi Hans, On Mon, May 29, 2017 at 03:46:08PM +0200, Hans Verkuil wrote: Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: - video-mux: switched to Philipp's latest video-mux driver and updated bindings docs,

Re: [PATCH v2 5/7] [media] vimc: cap: Support several image formats

2017-05-29 Thread Helen Koike
Hi Hans, Thank you for your review. I just have a question for this one. On 2017-05-08 08:53 AM, Hans Verkuil wrote: On 04/08/2017 12:37 AM, Helen Koike wrote: Allow user space to change the image format as the frame size, the pixel format, colorspace, quantization, field YCbCr encoding and

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Hans Verkuil
On 05/29/2017 07:23 PM, Steve Longerbeam wrote: Hi Hans, thanks for the reply... On 05/29/2017 06:46 AM, Hans Verkuil wrote: Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: What is the status as of v7? From what I can tell patch 2/34 needs an Ack from Rob

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Steve Longerbeam
Hi Hans, thanks for the reply... On 05/29/2017 06:46 AM, Hans Verkuil wrote: Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: What is the status as of v7? From what I can tell patch 2/34 needs an Ack from Rob Herring, Yes still missing that Ack. I think the

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Hans Verkuil
On 05/29/2017 05:36 PM, Sakari Ailus wrote: Hi Hans, On Mon, May 29, 2017 at 03:46:08PM +0200, Hans Verkuil wrote: Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: - video-mux: switched to Philipp's latest video-mux driver and updated bindings docs, that makes use

Re: [PATCH v7 16/34] [media] add Omnivision OV5640 sensor driver

2017-05-29 Thread Sakari Ailus
Hi Steve, A few comments below. On Wed, May 24, 2017 at 05:29:31PM -0700, Steve Longerbeam wrote: > This driver is based on ov5640_mipi.c from Freescale imx_3.10.17_1.0.0_beta > branch, modified heavily to bring forward to latest interfaces and code > cleanup. > > Signed-off-by: Steve

[PATCH 00/15] make more driver use devm_of_platform_populate()

2017-05-29 Thread Benjamin Gaignard
Number of calls to of_platform_populate() aren't unbalanced by a call to of_platform_depopulate() that could generate issue will loading/unloading the drivers. Make those drivers use devm_of_platform_populate() fix the problem without need to add remove function. In some case replacing

[PATCH 15/15] media: exynos4-is: use devm_of_platform_populate()

2017-05-29 Thread Benjamin Gaignard
Usage of devm_of_platform_populate() simplify driver code and save somes lines Signed-off-by: Benjamin Gaignard CC: Kyungmin Park CC: Sylwester Nawrocki CC: Mauro Carvalho Chehab CC: Kukjin

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Sakari Ailus
Hi Hans, On Mon, May 29, 2017 at 03:46:08PM +0200, Hans Verkuil wrote: > Hi Steve, > > On 05/25/2017 02:29 AM, Steve Longerbeam wrote: > >In version 7: > > > >- video-mux: switched to Philipp's latest video-mux driver and updated > > bindings docs, that makes use of the mmio-mux framework. > >

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Hans Verkuil
On 05/29/2017 04:21 PM, Hans Verkuil wrote: On 05/29/2017 04:15 PM, Philipp Zabel wrote: On Mon, 2017-05-29 at 15:46 +0200, Hans Verkuil wrote: Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: - video-mux: switched to Philipp's latest video-mux driver and updated

[PATCH v7 1/3] dt-bindings: Add bindings for video-multiplexer device

2017-05-29 Thread Philipp Zabel
Add bindings documentation for the video multiplexer device. Signed-off-by: Sascha Hauer Signed-off-by: Philipp Zabel Signed-off-by: Steve Longerbeam Acked-by: Sakari Ailus Reviewed-by:

[PATCH v7 3/3] [media] platform: add video-multiplexer subdevice driver

2017-05-29 Thread Philipp Zabel
This driver can handle SoC internal and external video bus multiplexers, controlled by mux controllers provided by the mux controller framework, such as MMIO register bitfields or GPIOs. The subdevice passes through the mbus configuration of the active input to the output side. Since the mux

[PATCH v7 2/3] [media] add mux and video interface bridge entity functions

2017-05-29 Thread Philipp Zabel
Signed-off-by: Philipp Zabel - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX Signed-off-by: Steve Longerbeam --- Changes since [1]: - Reword the video interface bridge description, do not use "bus format" to avoid confiusion with media

[PATCH 1/1] v4l2-ctrls: Correctly destroy mutex in v4l2_ctrl_handler_free()

2017-05-29 Thread Sakari Ailus
The mutex that was initialised in v4l2_ctrl_handler_init_class() was not destroyed in v4l2_ctrl_handler_free(). Do that. Additionally, explicitly refer to the ctrl handler's mutex in mutex initialisation for clarity. Signed-off-by: Sakari Ailus ---

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Hans Verkuil
On 05/29/2017 04:15 PM, Philipp Zabel wrote: On Mon, 2017-05-29 at 15:46 +0200, Hans Verkuil wrote: Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: - video-mux: switched to Philipp's latest video-mux driver and updated bindings docs, that makes use of the mmio-mux

Re: [PATCH 1/1] v4l: fwnode: Use correct argument type for nop v4l2_flash_init()

2017-05-29 Thread kbuild test robot
Hi Sakari, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.12-rc3 next-20170529] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Sakari-Ailus/v4l-fwnode-Use-correct

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Philipp Zabel
On Mon, 2017-05-29 at 15:46 +0200, Hans Verkuil wrote: > Hi Steve, > > On 05/25/2017 02:29 AM, Steve Longerbeam wrote: > > In version 7: > > > > - video-mux: switched to Philipp's latest video-mux driver and updated > >bindings docs, that makes use of the mmio-mux framework. > > > > -

Re: [PATCH v7 15/34] add mux and video interface bridge entity functions

2017-05-29 Thread Philipp Zabel
On Mon, 2017-05-29 at 15:37 +0200, Hans Verkuil wrote: > On 05/25/2017 02:29 AM, Steve Longerbeam wrote: > > From: Philipp Zabel > > > > Signed-off-by: Philipp Zabel > > > > - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX > > > > Signed-off-by:

Re: [PATCH v7 15/34] add mux and video interface bridge entity functions

2017-05-29 Thread Hans Verkuil
On 05/29/2017 03:51 PM, Philipp Zabel wrote: On Mon, 2017-05-29 at 15:37 +0200, Hans Verkuil wrote: On 05/25/2017 02:29 AM, Steve Longerbeam wrote: From: Philipp Zabel Signed-off-by: Philipp Zabel - renamed MEDIA_ENT_F_MUX to

[GIT PULL FOR v4.13] Fixes, mostly for rcar-vin

2017-05-29 Thread Hans Verkuil
Important: this pull request assumes that this pull request: https://patchwork.linuxtv.org/patch/41530/ was merged first! Regards, Hans The following changes since commit 575cafad3859ae42e83538b1c60d2beb51bd845f: v4l: Remove V4L2 OF framework in favour of V4L2 fwnode framework

Re: [PATCH v7 00/34] i.MX Media Driver

2017-05-29 Thread Hans Verkuil
Hi Steve, On 05/25/2017 02:29 AM, Steve Longerbeam wrote: In version 7: - video-mux: switched to Philipp's latest video-mux driver and updated bindings docs, that makes use of the mmio-mux framework. - mmio-mux: includes Philipp's temporary patch that adds mmio-mux support to video-mux

Re: [PATCH v7 16/34] [media] add Omnivision OV5640 sensor driver

2017-05-29 Thread Hans Verkuil
On 05/25/2017 02:29 AM, Steve Longerbeam wrote: This driver is based on ov5640_mipi.c from Freescale imx_3.10.17_1.0.0_beta branch, modified heavily to bring forward to latest interfaces and code cleanup. Signed-off-by: Steve Longerbeam Acked-by: Hans Verkuil

Re: [PATCH v7 15/34] add mux and video interface bridge entity functions

2017-05-29 Thread Hans Verkuil
On 05/25/2017 02:29 AM, Steve Longerbeam wrote: From: Philipp Zabel Signed-off-by: Philipp Zabel - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX Signed-off-by: Steve Longerbeam ---

Re: [RFC v2 3/3] dt: bindings: Add a binding for referencing EEPROM from camera sensors

2017-05-29 Thread Sakari Ailus
Hi Daniel, On Mon, May 29, 2017 at 02:39:01PM +0200, Daniel Mack wrote: > On 05/29/2017 02:20 PM, Sakari Ailus wrote: > > On Mon, May 08, 2017 at 12:24:18PM -0500, Rob Herring wrote: > >> On Fri, May 05, 2017 at 11:48:30AM +0300, Sakari Ailus wrote: > >>> Many camera sensor devices contain EEPROM

Re: [RFC v2 3/3] dt: bindings: Add a binding for referencing EEPROM from camera sensors

2017-05-29 Thread Daniel Mack
On 05/29/2017 02:20 PM, Sakari Ailus wrote: > On Mon, May 08, 2017 at 12:24:18PM -0500, Rob Herring wrote: >> On Fri, May 05, 2017 at 11:48:30AM +0300, Sakari Ailus wrote: >>> Many camera sensor devices contain EEPROM chips that describe the >>> properties of a given unit --- the data is specific

Re: [RFC v2 3/3] dt: bindings: Add a binding for referencing EEPROM from camera sensors

2017-05-29 Thread Sakari Ailus
Hi Rob, On Mon, May 08, 2017 at 12:24:18PM -0500, Rob Herring wrote: > On Fri, May 05, 2017 at 11:48:30AM +0300, Sakari Ailus wrote: > > Many camera sensor devices contain EEPROM chips that describe the > > properties of a given unit --- the data is specific to a given unit can > > thus is not

Re: [PATCH 6/7] [media] soc_camera: rcar_vin: use proper name for the R-Car SoC

2017-05-29 Thread Wolfram Sang
>Why "soc_camera:" in the subject? I used 'git log $file' and copied the most recent entry. Do I need to resend? signature.asc Description: PGP signature

Re: [PATCH v7 2/2] media: rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

2017-05-29 Thread Sakari Ailus
Hi Niklas, A few comments below. On Wed, May 24, 2017 at 02:13:53AM +0200, Niklas Söderlund wrote: > From: Niklas Söderlund > > A V4L2 driver for Renesas R-Car MIPI CSI-2 receiver. The driver > supports the rcar-vin driver on R-Car Gen3 SoCs where

Re: [PATCH v4 2/2] cec: add STM32 cec driver (fwd)

2017-05-29 Thread Julia Lawall
nel.org, r...@kernel.org, hans.verk...@cisco.com CC: Benjamin Gaignard <benjamin.gaign...@linaro.org> Hi Benjamin, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.12-rc3 next-20170529] [if your patch is applied to the wrong git tree, please drop us a note to

Re: [PATCH v7 2/2] media: rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

2017-05-29 Thread Hans Verkuil
On 05/24/2017 02:13 AM, Niklas Söderlund wrote: From: Niklas Söderlund A V4L2 driver for Renesas R-Car MIPI CSI-2 receiver. The driver supports the rcar-vin driver on R-Car Gen3 SoCs where separate CSI-2 hardware blocks are connected between the video

Re: [PATCH v7 1/2] media: rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver documentation

2017-05-29 Thread Sakari Ailus
Hi Niklas, On Wed, May 24, 2017 at 02:13:52AM +0200, Niklas Söderlund wrote: > From: Niklas Söderlund > > Documentation for Renesas R-Car MIPI CSI-2 receiver. The CSI-2 receivers > are located between the video sources (CSI-2 transmitters) and the video >

Re: [PATCH v2 2/2] v4l: async: add subnotifier registration for subdevices

2017-05-29 Thread Sakari Ailus
Hi Niklas, On Wed, May 24, 2017 at 02:07:27AM +0200, Niklas Söderlund wrote: ... > diff --git a/drivers/media/v4l2-core/v4l2-async.c > b/drivers/media/v4l2-core/v4l2-async.c > index c16200c88417b151..e1e181db90f789c0 100644 > --- a/drivers/media/v4l2-core/v4l2-async.c > +++

Re: [PATCH v2 2/2] media: entity: Add media_entity_pad_from_fwnode() function

2017-05-29 Thread Hans Verkuil
On 05/24/2017 02:09 AM, Niklas Söderlund wrote: From: Niklas Söderlund This is a wrapper around the media entity pad_from_fwnode operation. Signed-off-by: Niklas Söderlund --- drivers/media/media-entity.c | 39

Re: [PATCH v2 2/2] v4l: async: add subnotifier registration for subdevices

2017-05-29 Thread Hans Verkuil
On 05/24/2017 02:07 AM, Niklas Söderlund wrote: From: Niklas Söderlund When registered() of v4l2_subdev_internal_ops is called the subdevice When the registered() callback of... have access to the master devices v4l2_dev and it's called with the

Re: [PATCH v2 1/2] v4l: async: check for v4l2_dev in v4l2_async_notifier_register()

2017-05-29 Thread Hans Verkuil
On 05/24/2017 02:07 AM, Niklas Söderlund wrote: From: Niklas Söderlund Add a check for v4l2_dev to v4l2_async_notifier_register() as to fail as early as possible since this will fail later in v4l2_async_test_notify(). Signed-off-by: Niklas Söderlund

[PATCH v5 1/2] dt-bindings: media: stm32 cec driver

2017-05-29 Thread Benjamin Gaignard
Add bindings documentation for stm32 CEC driver. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/st,stm32-cec.txt| 19 +++ 1 file changed, 19 insertions(+) create mode 100644

[PATCH v5 0/2] cec: STM32 driver

2017-05-29 Thread Benjamin Gaignard
version 5: - remove cec notifier (to be added for drm driver release) version 4: - rebased on Hans cec-config branch - rework bindings commit message - add notifier support - update KConfig version 2: - fix typo in compagnie name - add yannick sign-off - use cec_message instead of custom struct

[PATCH v5 2/2] cec: add STM32 cec driver

2017-05-29 Thread Benjamin Gaignard
This patch add cec driver for STM32 platforms. cec hardware block isn't not always used with hdmi so cec notifier is not implemented. That will be done later when STM32 DSI driver will be available. Driver compliance has been tested with cec-ctl and cec-compliance tools. Signed-off-by: Benjamin

[GIT PULL v2 FOR v4.13] V4L2 fwnode support

2017-05-29 Thread Sakari Ailus
Hi Mauro, This pull request introduces the V4L2 fwnode framework which has equivalent functionality to V4L2 OF framework. The V4L2 OF framework users are converted to use the V4L2 fwnode framework and the redundant V4L2 OF framework is removed. since v1: - Merge patch to the series ("v4l: flash

[PATCH 1/1] v4l: fwnode: Use correct argument type for nop v4l2_flash_init()

2017-05-29 Thread Sakari Ailus
The device_node -> fwnode change didn't reach the nop implementation of v4l2_flash_init(), do that change now. Fixes: ("v4l: flash led class: Use fwnode_handle instead of device_node in init") Signed-off-by: Sakari Ailus --- include/media/v4l2-flash-led-class.h |

Re: [PATCH v4 0/2] cec: STM32 driver

2017-05-29 Thread Hans Verkuil
On 05/29/2017 11:07 AM, Benjamin Gaignard wrote: 2017-05-29 10:54 GMT+02:00 Hans Verkuil : Hi Benjamin, On 05/29/2017 10:34 AM, Benjamin Gaignard wrote: version 4: - rebased on Hans cec-config branch - rework bindings commit message - add notifier support I really

Re: [PATCH 7/7] [media] v4l: rcar_fdp1: use proper name for the R-Car SoC

2017-05-29 Thread Geert Uytterhoeven
On Sun, May 28, 2017 at 11:30 AM, Wolfram Sang wrote: > It is 'R-Car', not 'RCar'. No code or binding changes, only descriptive text. > > Signed-off-by: Wolfram Sang Acked-by: Geert Uytterhoeven

Re: [PATCH 6/7] [media] soc_camera: rcar_vin: use proper name for the R-Car SoC

2017-05-29 Thread Geert Uytterhoeven
On Sun, May 28, 2017 at 11:30 AM, Wolfram Sang wrote: > It is 'R-Car', not 'RCar'. No code or binding changes, only descriptive text. > > Signed-off-by: Wolfram Sang Acked-by: Geert Uytterhoeven

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

2017-05-29 Thread Hans Verkuil
On 05/19/2017 03:07 PM, 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 numerous variants of this chip, but

Re: [PATCH v4 0/2] cec: STM32 driver

2017-05-29 Thread Benjamin Gaignard
2017-05-29 10:54 GMT+02:00 Hans Verkuil : > Hi Benjamin, > > On 05/29/2017 10:34 AM, Benjamin Gaignard wrote: >> >> version 4: >> - rebased on Hans cec-config branch >> - rework bindings commit message >> - add notifier support > > > I really don't like this change. This forced

Re: [PATCH v4 0/2] cec: STM32 driver

2017-05-29 Thread Hans Verkuil
Hi Benjamin, On 05/29/2017 10:34 AM, Benjamin Gaignard wrote: version 4: - rebased on Hans cec-config branch - rework bindings commit message - add notifier support I really don't like this change. This forced me to think about this a bit more, and I think this requires another change as

Re: [PATCH 6/7] [media] soc_camera: rcar_vin: use proper name for the R-Car SoC

2017-05-29 Thread Sergei Shtylyov
Hello! Why "soc_camera:" in the subject? The 'soc_camera" driver has been removed (replaced by a "normal" V4L2 driver). MBR, Sergei

[PATCH v4 1/2] dt-bindings: media: stm32 cec driver

2017-05-29 Thread Benjamin Gaignard
Add bindings documentation for stm32 CEC driver. Signed-off-by: Benjamin Gaignard --- version 4: - rework commit message - add hdmi-phandle optional property .../devicetree/bindings/media/st,stm32-cec.txt | 22 ++ 1 file changed, 22

[PATCH v4 2/2] cec: add STM32 cec driver

2017-05-29 Thread Benjamin Gaignard
This patch add cec driver for STM32 platforms. cec hardware block isn't not always used with hdmi so cec notifier is not implemented. That will be done later when STM32 DSI driver will be available. Driver compliance has been tested with cec-ctl and cec-compliance tools. Signed-off-by: Benjamin

[PATCH v4 0/2] cec: STM32 driver

2017-05-29 Thread Benjamin Gaignard
version 4: - rebased on Hans cec-config branch - rework bindings commit message - add notifier support - update KConfig version 2: - fix typo in compagnie name - add yannick sign-off - use cec_message instead of custom struct in cec device - add monitor mode I don't change the split between irq

Re: [PATCH v2 15/17] rcar-vin: register the video device at probe time

2017-05-29 Thread Niklas Söderlund
Hi Hans, On 2017-05-29 09:55:07 +0200, Hans Verkuil wrote: > On 05/29/2017 09:49 AM, Niklas Söderlund wrote: > > Hi Hans, > > > > Thanks for taking the time to look at this :-) > > > > On 2017-05-29 08:56:31 +0200, Hans Verkuil wrote: > > > On 05/29/2017 08:52 AM, Hans Verkuil wrote: > > > > Hi

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

2017-05-29 Thread Hans Verkuil
On 05/29/2017 09:32 AM, Pavel Machek wrote: On Mon 2017-05-29 08:13:22, Hans Verkuil wrote: Hi Pavel, On 05/21/2017 12:33 PM, Pavel Machek wrote: Add simple SDL-based application for capturing photos. Manual focus/gain/exposure can be set, flash can be controlled and autofocus/autogain can be

Re: [PATCH v2 15/17] rcar-vin: register the video device at probe time

2017-05-29 Thread Hans Verkuil
On 05/29/2017 09:49 AM, Niklas Söderlund wrote: Hi Hans, Thanks for taking the time to look at this :-) On 2017-05-29 08:56:31 +0200, Hans Verkuil wrote: On 05/29/2017 08:52 AM, Hans Verkuil wrote: Hi Niklas, On 05/24/2017 02:15 AM, Niklas Söderlund wrote: From: Niklas Söderlund

Re: [PATCH v2 15/17] rcar-vin: register the video device at probe time

2017-05-29 Thread Niklas Söderlund
Hi Hans, Thanks for taking the time to look at this :-) On 2017-05-29 08:56:31 +0200, Hans Verkuil wrote: > On 05/29/2017 08:52 AM, Hans Verkuil wrote: > > Hi Niklas, > > > > On 05/24/2017 02:15 AM, Niklas Söderlund wrote: > > > From: Niklas Söderlund > >

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

2017-05-29 Thread Tomasz Figa
On Mon, May 29, 2017 at 3:49 PM, Yang, Hyungwoo wrote: > > Hello Tomasz, > > Here's my comments. > > Thanks, > Hyungwoo > > -Original Message- >> From: Tomasz Figa [mailto:tf...@chromium.org] >> Sent: Sunday, May 28, 2017 7:56 PM >> To: Yang, Hyungwoo

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

2017-05-29 Thread Pavel Machek
On Mon 2017-05-29 08:13:22, Hans Verkuil wrote: > Hi Pavel, > > On 05/21/2017 12:33 PM, Pavel Machek wrote: > >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

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

2017-05-29 Thread Sakari Ailus
Hi Hyugnwoo, On Sun, May 28, 2017 at 11:26:46PM +, Yang, Hyungwoo wrote: > > > +/* Mode : resolution and related config */ > > > +struct ov13858_mode > > > +{ > > > + /* Frame width */ > > > + u32 width; > > > + /* Frame height */ > > > + u32 height; > > > + > > > + /* V-timing */ > > > +

Re: [PATCH] [media] s5p-jpeg: fix recursive spinlock acquisition

2017-05-29 Thread Alexandre Courbot
Hi everyone, On Thu, Apr 27, 2017 at 4:35 AM, Jacek Anaszewski wrote: > On 04/26/2017 04:54 AM, Alexandre Courbot wrote: >> On Wed, Apr 26, 2017 at 4:15 AM, Jacek Anaszewski >> wrote: >>> Hi Alexandre, >>> >>> Thanks for the patch. >>> >>>

Re: [PATCH v2 15/17] rcar-vin: register the video device at probe time

2017-05-29 Thread Hans Verkuil
On 05/29/2017 08:52 AM, Hans Verkuil wrote: Hi Niklas, On 05/24/2017 02:15 AM, Niklas Söderlund wrote: From: Niklas Söderlund The driver registers the video device from the async complete callback and unregistered in the async unbind callback. This

Re: [PATCH v2 15/17] rcar-vin: register the video device at probe time

2017-05-29 Thread Hans Verkuil
Hi Niklas, On 05/24/2017 02:15 AM, Niklas Söderlund wrote: From: Niklas Söderlund The driver registers the video device from the async complete callback and unregistered in the async unbind callback. This creates problems if if the subdevice is bound,

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

2017-05-29 Thread Yang, Hyungwoo
Hello Tomasz, Here's my comments. Thanks, Hyungwoo -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Sunday, May 28, 2017 7:56 PM > To: Yang, Hyungwoo > Cc: Sakari Ailus ; linux-media@vger.kernel.org; >

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

2017-05-29 Thread Hans Verkuil
Hi Pavel, On 05/21/2017 12:33 PM, Pavel Machek wrote: 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