Re: [PATCH v6 2/2] media: i2c: Add the ov7740 image sensor driver

2017-12-05 Thread Yang, Wenyou
Hi Sakari, On 2017/12/5 18:45, Sakari Ailus wrote: Hi Wenyou, On Mon, Dec 04, 2017 at 02:58:58PM +0800, Wenyou Yang wrote: The ov7740 (color) image sensor is a high performance VGA CMOS image snesor, which supports for output formats: RAW RGB and YUV and image sizes: VGA, and QVGA, CIF and an

cron job: media_tree daily build: ERRORS

2017-12-05 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: Wed Dec 6 05:00:16 CET 2017 media-tree git hash:781b045baefdabf7e0bc9f33672ca830d3db9f27 media_build git

[PATCH v7 1/2] media: ov7740: Document device tree bindings

2017-12-05 Thread Wenyou Yang
Add the device tree binding documentation for the ov7740 sensor driver. Signed-off-by: Wenyou Yang Acked-by: Rob Herring --- Changes in v7: - Add Acked-by tag. Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: - Explicitly document the "remote-endpoint" property. Ch

[PATCH v7 0/2] media: ov7740: Add a V4L2 sensor-level driver

2017-12-05 Thread Wenyou Yang
Add a Video4Linux2 sensor-level driver for the OmniVision OV7740 VGA camera image sensor. Changes in v7: - Add Acked-by tag. - Fix the wrong handle of default register configuration. - Add the missed assignment of ov7740->frmsize. Changes in v6: - Remove unnecessary #include . - Remove unnec

[PATCH v7 2/2] media: i2c: Add the ov7740 image sensor driver

2017-12-05 Thread Wenyou Yang
The ov7740 (color) image sensor is a high performance VGA CMOS image snesor, which supports for output formats: RAW RGB and YUV and image sizes: VGA, and QVGA, CIF and any size smaller. Signed-off-by: Songjun Wu Signed-off-by: Wenyou Yang --- Changes in v7: - Fix the wrong handle of default re

RE: [PATCH v4 00/12] Intel IPU3 ImgU patchset

2017-12-05 Thread Mani, Rajmohan
Hi Tomasz, Hans, > Subject: Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset > > On 12/05/2017 04:22 AM, Tomasz Figa wrote: > > Hi Raj, > > > > On Tue, Dec 5, 2017 at 9:13 AM, Mani, Rajmohan > wrote: > >> Hi Hans, > >> > >> Thanks for your patience and sharing your thoughts on this. > >> > >>> Subj

[RESEND PATCH] partial revert of "[media] tvp5150: add HW input connectors support"

2017-12-05 Thread Javier Martinez Canillas
Commit f7b4b54e6364 ("[media] tvp5150: add HW input connectors support") added input signals support for the tvp5150, but the approach was found to be incorrect so the corresponding DT binding commit 82c2ffeb217a ("[media] tvp5150: document input connectors DT bindings") was reverted. This left th

Re: [PATCH] string.h: work around for increased stack usage

2017-12-05 Thread Andrew Morton
On Tue, 5 Dec 2017 22:51:19 +0100 Arnd Bergmann wrote: > The hardened strlen() function causes rather large stack usage in at > least one file in the kernel, in particular when CONFIG_KASAN is enabled: > > drivers/media/usb/em28xx/em28xx-dvb.c: In function 'em28xx_dvb_init': > drivers/media/usb

[PATCH] string.h: work around for increased stack usage

2017-12-05 Thread Arnd Bergmann
The hardened strlen() function causes rather large stack usage in at least one file in the kernel, in particular when CONFIG_KASAN is enabled: drivers/media/usb/em28xx/em28xx-dvb.c: In function 'em28xx_dvb_init': drivers/media/usb/em28xx/em28xx-dvb.c:2062:1: error: the frame size of 3256 bytes is

[PATCH] v4l: rcar-csi2: Don't bail out from probe on no ep

2017-12-05 Thread Jacopo Mondi
When rcar-csi interface is not connected to any endpoint, it fails and bails out from probe before registering its own video subdevice. This prevents rcar-vin registered notifier from completing and no subdevice is ever registered, also for other properly connected csi interfaces. Fix this not ret

Re: [PATCH 1/1] v4l: fwnode: Add debug prints for V4L2 endpoint property parsing

2017-12-05 Thread Niklas Söderlund
Hi Sakari, Thanks for your patch. On 2017-12-04 23:45:01 +0200, Sakari Ailus wrote: > Print debug info as standard V4L2 endpoint are parsed. > > Signed-off-by: Sakari Ailus Reviewed-by: Niklas Söderlund > --- > drivers/media/v4l2-core/v4l2-fwnode.c | 101 > +++--

Re: [PATCH] c8sectpfe: fix potential NULL pointer dereference in c8sectpfe_timer_interrupt

2017-12-05 Thread Gustavo A. R. Silva
Hello, On 11/21/2017 02:22 AM, Patrice CHOTARD wrote: Hi Gustavo On 11/20/2017 03:00 PM, Gustavo A. R. Silva wrote: _channel_ is being dereferenced before it is null checked, hence there is a potential null pointer dereference. Fix this by moving the pointer dereference after _channel_ has bee

[PATCH] media: platform: sti: Adopt SPDX identifier

2017-12-05 Thread Benjamin Gaignard
Add SPDX identifiers to files under sti directory Signed-off-by: Benjamin Gaignard --- drivers/media/platform/sti/bdisp/bdisp-debug.c | 2 +- drivers/media/platform/sti/bdisp/bdisp-filter.h | 2 +- drivers/media/platform/sti/bdisp/bdisp-hw.c | 2 +- drivers/med

[PATCH] media: platform: stm32: Adopt SPDX identifier

2017-12-05 Thread Benjamin Gaignard
Add SPDX identifiers to files under stm32 directory Signed-off-by: Benjamin Gaignard --- drivers/media/platform/stm32/stm32-cec.c | 5 + drivers/media/platform/stm32/stm32-dcmi.c | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-cec.c

[PATCH] media: imx274: Silence uninitialized variable warning

2017-12-05 Thread Dan Carpenter
Smatch complains that "err" can be uninitialized if we have a zero size write. The flow analysis is a little complicated so I'm not sure if that's possible or not, but it's harmless to set this to zero and it makes the code easier to read. Signed-off-by: Dan Carpenter diff --git a/drivers/media

Re: [PATCH v4 2/5] media: dt-bindings: Add bindings for TDA1997X

2017-12-05 Thread Sakari Ailus
On Wed, Nov 29, 2017 at 01:19:54PM -0800, Tim Harvey wrote: > Cc: Rob Herring > Cc: Sakari Ailus > Signed-off-by: Tim Harvey Acked-by: Sakari Ailus -- Sakari Ailus e-mail: sakari.ai...@iki.fi

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Guennadi Liakhovetski
On Tue, 5 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > On Tuesday, 5 December 2017 15:44:34 EET Guennadi Liakhovetski wrote: > > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > > On Tuesday, 5 December 2017 12:56:53 EET Guennadi Liakhovetski wrote: > > >> On Tue, 5 Dec 2017, Laurent Pinch

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 5 December 2017 15:44:34 EET Guennadi Liakhovetski wrote: > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > On Tuesday, 5 December 2017 12:56:53 EET Guennadi Liakhovetski wrote: > >> On Tue, 5 Dec 2017, Laurent Pinchart wrote: > >>> On Wednesday, 8 November 2017 18:00:14 E

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Guennadi Liakhovetski
On Tue, 5 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > On Tuesday, 5 December 2017 12:56:53 EET Guennadi Liakhovetski wrote: > > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > > On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: > > >> Some UVC video cameras contain

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

2017-12-05 Thread Sakari Ailus
On Wed, Nov 29, 2017 at 08:32:34PM +0100, Niklas Söderlund wrote: > 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 > grabbers (VIN) on Gen3 of Renesas R-Car SoC. > > Each CSI-2 device is connected

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 5 December 2017 12:56:53 EET Guennadi Liakhovetski wrote: > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: > >> Some UVC video cameras contain metadata in their payload headers. This > >> patch extrac

Re: [PATCH v4 3/5] staging: Introduce NVIDIA Tegra video decoder driver

2017-12-05 Thread Hans Verkuil
On 12/05/17 13:17, Dmitry Osipenko wrote: > Hi Hans, > > On 04.12.2017 17:04, Hans Verkuil wrote: >> Hi Dmitry, >> >> As you already mention in the TODO, this should become a v4l2 codec driver. >> >> Good existing examples are the coda, qcom/venus and mtk-vcodec drivers. >> >> One thing that is no

Re: [PATCH v4 3/5] staging: Introduce NVIDIA Tegra video decoder driver

2017-12-05 Thread Dmitry Osipenko
Hi Hans, On 04.12.2017 17:04, Hans Verkuil wrote: > Hi Dmitry, > > As you already mention in the TODO, this should become a v4l2 codec driver. > > Good existing examples are the coda, qcom/venus and mtk-vcodec drivers. > > One thing that is not clear from this code is if the tegra hardware is a

Re: camss: camera controls missing on vfe interfaces

2017-12-05 Thread Daniel Mack
Hi, FWIW, we are using the attached trivial patch in our tree now. Not sure if this is interesting for upstream, but it does solve our problem. Let me know what you think. Thanks, Daniel On Monday, November 27, 2017 12:50 AM, Daniel Mack wrote: > Hi Todor, everyone, > > On Monday, November 20

RE: [PATCH v4 07/12] [media] cxd2880: Add top level of the driver

2017-12-05 Thread Takiguchi, Yasunari
Dear Sean Hi, Thanks for your review. We will refer to your comments and consider how to respond for them. > > + u8 rdata[2]; > > + int ret; > > + > > + if ((!tnrdmd) || (!pre_bit_err) || (!pre_bit_count)) > > + return -EINVAL; > > + > > + if (tnrdmd->diver_mode == CXD2880_TNRD

Re: [PATCH 2/8] [media] uvc_video: use ktime_t for timestamps

2017-12-05 Thread Arnd Bergmann
On Tue, Dec 5, 2017 at 1:58 AM, Laurent Pinchart wrote: > Hi Arnd, > > On Tuesday, 5 December 2017 02:37:27 EET Laurent Pinchart wrote: >> On Monday, 27 November 2017 15:19:54 EET Arnd Bergmann wrote: >> > uvc_video_get_ts() returns a 'struct timespec', but all its users >> > really want a nanosec

Re: [PATCH v5 02/12] intel-ipu3: Add user space ABI definitions

2017-12-05 Thread Sakari Ailus
Hi Yong, Thanks for the update. I went through the header again, some comments below. On Fri, Dec 01, 2017 at 10:32:12PM -0600, Yong Zhi wrote: > The UAPI header defines the structures and macros > to be used by user space. > > Signed-off-by: Rajmohan Mani > Signed-off-by: Yong Zhi > --- > i

Re: [PATCH v6 0/9] i2c: document DMA handling and add helpers for it

2017-12-05 Thread Jonathan Cameron
On Mon, 4 Dec 2017 22:05:41 + Mark Brown wrote: > On Sun, Dec 03, 2017 at 08:43:47PM +0100, Wolfram Sang wrote: > > > > It's a bit different in that it's much more likely that a SPI controller > > > will actually do DMA than an I2C one since the speeds are higher and > > > there's frequent a

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Guennadi Liakhovetski
Hi Laurent, Thanks for a review. On Tue, 5 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > Thank you for the patch. We're getting very close, I only have small > comments, > please see below. > > On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: > > Some UVC video

Re: [PATCH v6 2/2] media: i2c: Add the ov7740 image sensor driver

2017-12-05 Thread Sakari Ailus
Hi Wenyou, On Mon, Dec 04, 2017 at 02:58:58PM +0800, Wenyou Yang wrote: > The ov7740 (color) image sensor is a high performance VGA CMOS > image snesor, which supports for output formats: RAW RGB and YUV > and image sizes: VGA, and QVGA, CIF and any size smaller. > > Signed-off-by: Songjun Wu >

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 5 December 2017 10:06:02 EET Guennadi Liakhovetski wrote: > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > On Tuesday, 5 December 2017 02:24:30 EET Laurent Pinchart wrote: > >> On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: > > [snip] > > > >>> +

Re: [PATCH 1/2] uvcvideo: Factor out video device registration to a function

2017-12-05 Thread Guennadi Liakhovetski
On Tue, 5 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > On Tuesday, 5 December 2017 11:14:18 EET Guennadi Liakhovetski wrote: > > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > > The function will then be used to register the video device for metadata > > > capture. > > > > > > Signed-of

Re: [PATCH 1/2] uvcvideo: Factor out video device registration to a function

2017-12-05 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 5 December 2017 11:14:18 EET Guennadi Liakhovetski wrote: > On Tue, 5 Dec 2017, Laurent Pinchart wrote: > > The function will then be used to register the video device for metadata > > capture. > > > > Signed-off-by: Laurent Pinchart > > --- > > > > drivers/media/usb/u

Re: [PATCH 1/2] uvcvideo: Factor out video device registration to a function

2017-12-05 Thread Guennadi Liakhovetski
Hi Laurent, On Tue, 5 Dec 2017, Laurent Pinchart wrote: > The function will then be used to register the video device for metadata > capture. > > Signed-off-by: Laurent Pinchart > --- > drivers/media/usb/uvc/uvc_driver.c | 66 > +++--- > drivers/media/usb/uvc/u

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-05 Thread Guennadi Liakhovetski
Hi Laurent, On Tue, 5 Dec 2017, Laurent Pinchart wrote: > Hi Guennadi, > > On Tuesday, 5 December 2017 02:24:30 EET Laurent Pinchart wrote: > > On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: > > [snip] > > > > +static void uvc_video_decode_meta(struct uvc_streaming *st