cron job: media_tree daily build: OK

2018-07-10 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 Jul 11 05:00:11 CEST 2018 media-tree git hash:666e994aa2278e948e2492ee9d81b4df241e7222 media_build

Re: [PATCH v8 1/3] uvcvideo: remove a redundant check

2018-07-10 Thread Laurent Pinchart
Hi Guennadi, Thank you for the patch. On Tuesday, 8 May 2018 18:07:42 EEST Guennadi Liakhovetski wrote: > Event subscribers cannot have a NULL file handle. They are only added > at a single location in the code, and the .fh pointer is used without > checking there. > > Signed-off-by: Guennadi

Re: [PATCH -next v3 2/2] media: ov772x: use SCCB helpers

2018-07-10 Thread Wolfram Sang
> > I think it would be even better to introduce a SSCB regmap layer > > and use that. > > I'm fine with introducing a SCCB regmap layer. I am fine with this approach, too. > But do we need to provide both a SCCB regmap and these SCCB helpers? I don't know much about the OV sensor drivers.

Wine Enthusiasts List

2018-07-10 Thread Sofia Thomas
-- Hi, Would you be interested in reaching out to "Wine Drinkers list " from USA? Our Databases:-1.Beer Drinkers List 2.Alcohol Drinkers List 3.Beverage Consumers 4.Liquor Drinkers List 5.Food Enthusiasts List

Re: [PATCH v2 0/2] media: i2c: ov5640: Re-work MIPI startup sequence

2018-07-10 Thread Steve Longerbeam
Hi Jacopo, Sorry to report my testing on SabreSD has same result as last time. This series fixes the LP-11 timeout at stream on but captured images are still blank. I tried the 640x480 mode with UYVY2X8. Here is the pad config: # media-ctl --get-v4l2 "'ov5640 1-003c':0"        

info!!

2018-07-10 Thread Lee Morrow
Top of the day to you, this is in respect of a very beneficial transaction which you would not want to let go reply for more details, Regards, Lee

[PATCH v2 1/2] media: i2c: ov5640: Re-work MIPI startup sequence

2018-07-10 Thread Jacopo Mondi
Rework the MIPI interface startup sequence with the following changes: - Remove MIPI bus initialization from the initial settings blob - At set_power(1) time power up MIPI Tx/Rx and set data and clock lanes in LP11 during 'sleep' and 'idle' with MIPI clock in non-continuous mode. - At s_stream

[PATCH v2 2/2] media: ov5640: Fix timings setup code

2018-07-10 Thread Jacopo Mondi
From: Samuel Bobrowicz The current code, when changing the mode and changing the scaling or sampling parameters, will look at the horizontal and vertical total size, which, since 5999f381e023 ("media: ov5640: Add horizontal and vertical totals") has been moved from the static register

Re: [PATCH] media: i2c: ov5640: Re-work MIPI startup sequence

2018-07-10 Thread jacopo mondi
Hi Steve, I've done some more testing, and finally got MIPI capture work properly on the i.MX6Q platform I'm testing on. Apparently commit: 476dec012f4c6545b0b7599cd9adba2ed819ad3b ("media: ov5640: Add horizontal and vertical totals") which several people, Jagan included, has reported

[PATCH v2 0/2] media: i2c: ov5640: Re-work MIPI startup sequence

2018-07-10 Thread Jacopo Mondi
Hello, this series fixes capture operations on i.MX6Q platforms (and possible other platforms reported not working) using MIPI CSI-2 interface. This iteration expands the v1 version with an additional fix, initially submitted by Maxime in his series: [PATCH v3 00/12] media: ov5640: Misc

Re: [PATCH] media: i2c: ov5640: Re-work MIPI startup sequence

2018-07-10 Thread Steve Longerbeam
On 07/10/2018 10:52 AM, jacopo mondi wrote: Hi Steve, I've done some more testing, and finally got MIPI capture work properly on the i.MX6Q platform I'm testing on. Apparently commit: 476dec012f4c6545b0b7599cd9adba2ed819ad3b ("media: ov5640: Add horizontal and vertical totals") which

Re: [PATCH -next v3 2/2] media: ov772x: use SCCB helpers

2018-07-10 Thread Akinobu Mita
2018年7月10日(火) 6:23 Sebastian Reichel : > > Hi, > > On Mon, Jul 09, 2018 at 06:14:43PM +0200, Wolfram Sang wrote: > > > static int ov772x_read(struct i2c_client *client, u8 addr) > > > { > > > - int ret; > > > - u8 val; > > > - > > > - ret = i2c_master_send(client, , 1); > > > - if (ret <

Re: [PATCH -next v3 1/2] i2c: add SCCB helpers

2018-07-10 Thread Wolfram Sang
> even when not used. I think it will also cause the compiler to emit warnings > for unused functions. I don't think that's a good idea. Where is my brown paper bag? :/ > > But if you insist on drivers/i2c/i2c-sccb.c, then it should be a > > seperate module, I'd think? > > Given how small the

Re: [PATCH -next v3 1/2] i2c: add SCCB helpers

2018-07-10 Thread Laurent Pinchart
Hi Wolfram, On Tuesday, 10 July 2018 15:07:47 EEST Wolfram Sang wrote: > >> +static inline int sccb_read_byte(struct i2c_client *client, u8 addr) > >> +{ > >> + int ret; > >> + union i2c_smbus_data data; > >> + > >> + i2c_lock_bus(client->adapter, I2C_LOCK_SEGMENT); > >> + > >> + ret =

Re: [PATCH] media: imx208: Add imx208 camera sensor driver

2018-07-10 Thread kbuild test robot
/0day-ci/linux/commits/Ping-chung-Chen/media-imx208-Add-imx208-camera-sensor-driver/20180710-153546 base: git://linuxtv.org/media_tree.git master config: i386-allyesconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 reproduce: # save the attached .config to linux build tree

Re: [PATCH -next v3 1/2] i2c: add SCCB helpers

2018-07-10 Thread Wolfram Sang
> > +static inline int sccb_read_byte(struct i2c_client *client, u8 addr) > > +{ > > + int ret; > > + union i2c_smbus_data data; > > + > > + i2c_lock_bus(client->adapter, I2C_LOCK_SEGMENT); > > + > > + ret = __i2c_smbus_xfer(client->adapter, client->addr, client->flags, > > +

Re: [PATCH 16/16] media: imx: add mem2mem device

2018-07-10 Thread Pavel Machek
Hi! > Add a single imx-media mem2mem video device that uses the IPU IC PP > (image converter post processing) task for scaling and colorspace > conversion. > On i.MX6Q/DL SoCs with two IPUs currently only the first IPU is used. > > The hardware only supports writing to destination buffers up to

Re: [PATCH] media: imx208: Add imx208 camera sensor driver

2018-07-10 Thread kbuild test robot
/0day-ci/linux/commits/Ping-chung-Chen/media-imx208-Add-imx208-camera-sensor-driver/20180710-153546 base: git://linuxtv.org/media_tree.git master config: x86_64-allmodconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 reproduce: # save the attached .config to linux build

Re: [PATCH -next v3 1/2] i2c: add SCCB helpers

2018-07-10 Thread Laurent Pinchart
Hi Akinobu, Thank you for the patch. On Monday, 9 July 2018 18:41:13 EEST Akinobu Mita wrote: > This adds Serial Camera Control Bus (SCCB) helpers (sccb_is_available, > sccb_read_byte, and sccb_write_byte) that are intended to be used by some > of Omnivision sensor drivers. > > The ov772x

hi

2018-07-10 Thread Sgt Sherri Gallagher
Please reply me back.I am Sgt.Sherri

[GIT PULL FOR v4.19] media/mc: fix inconsistencies

2018-07-10 Thread Hans Verkuil
This is the pull request of v6 of my patch series to fix the mc inconsistencies. Regards, Hans The following changes since commit 666e994aa2278e948e2492ee9d81b4df241e7222: media: platform: s5p-mfc: simplify getting .drvdata (2018-07-04 11:45:40 -0400) are available in the Git

[PATCHv6 00/12] media/mc: fix inconsistencies

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil This series is v6 of my previous attempt: https://www.mail-archive.com/linux-media@vger.kernel.org/msg133178.html The goal is to fix the inconsistencies between the 'old' and 'new' MC API. I hate the terms 'old' and 'new', there is nothing wrong IMHO with using an 'old'

[PATCHv6 08/12] ad9389b/adv7511: set proper media entity function

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil These two drivers both have function MEDIA_ENT_F_DV_ENCODER. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus Reviewed-by: Laurent Pinchart --- drivers/media/i2c/ad9389b.c | 1 + drivers/media/i2c/adv7511.c | 1 + 2 files changed, 2 insertions(+) diff --git

[PATCHv6 12/12] media-ioc-enum-entities.rst/-g-topology.rst: clarify ID/name usage

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Mention that IDs should not be hardcoded in applications and that the entity name must be unique within the media topology. Signed-off-by: Hans Verkuil Reviewed-by: Laurent Pinchart --- .../uapi/mediactl/media-ioc-enum-entities.rst | 9 +---

[PATCHv6 11/12] media-ioc-enum-links.rst: improve pad index description

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Make it clearer that the index starts at 0, and that it won't change since future new pads will never renumber existing pad indices. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/mediactl/media-ioc-enum-links.rst | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCHv6 03/12] media: add flags field to struct media_v2_entity

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil The v2 entity structure never exposed the entity flags, which made it impossible to detect connector or default entities. It is really trivial to just expose this information, so implement this. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus ---

[PATCHv6 10/12] media/i2c: add missing entity functions

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Several drivers in media/i2c do not set the entity function. Correct this. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus Reviewed-by: Laurent Pinchart --- drivers/media/i2c/et8ek8/et8ek8_driver.c | 1 + drivers/media/i2c/mt9m032.c | 1 +

[PATCHv6 05/12] media: rename MEDIA_ENT_F_DTV_DECODER to MEDIA_ENT_F_DV_DECODER

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil The use of 'DTV' is very confusing since it normally refers to Digital TV e.g. DVB etc. Instead use 'DV' (Digital Video), which nicely corresponds to the DV Timings API used to configure such receivers and transmitters. We keep an alias to avoid breaking userspace

[PATCHv6 04/12] media-ioc-g-topology.rst: document new 'flags' field

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Document the new struct media_v2_entity 'flags' field. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus --- .../media/uapi/mediactl/media-ioc-g-topology.rst | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git

[PATCHv6 09/12] adv7180/tvp514x/tvp7002: fix entity function

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil The entity function was ORed with the flags field instead of assigned to the function field. Correct this. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus Reviewed-by: Laurent Pinchart --- drivers/media/i2c/adv7180.c | 2 +- drivers/media/i2c/tvp514x.c | 2 +-

[PATCHv6 06/12] media.h: add MEDIA_ENT_F_DV_ENCODER

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Add a new function for digital video encoders such as HDMI transmitters. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus Reviewed-by: Laurent Pinchart --- Documentation/media/uapi/mediactl/media-types.rst | 7 +++ include/uapi/linux/media.h|

[PATCHv6 02/12] media-ioc-g-topology.rst: document new 'index' field

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Document the new struct media_v2_pad 'index' field. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus --- .../media/uapi/mediactl/media-ioc-g-topology.rst | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git

[PATCHv6 07/12] media.h: reorder video en/decoder functions

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil Keep the function defines in numerical order: 0x6000 comes after 0x2000, so move it back. Signed-off-by: Hans Verkuil Reviewed-by: Laurent Pinchart --- include/uapi/linux/media.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git

[PATCHv6 01/12] media: add 'index' to struct media_v2_pad

2018-07-10 Thread Hans Verkuil
From: Hans Verkuil The v2 pad structure never exposed the pad index, which made it impossible to call the MEDIA_IOC_SETUP_LINK ioctl, which needs that information. It is really trivial to just expose this information, so implement this. Signed-off-by: Hans Verkuil Acked-by: Sakari Ailus ---

RE: [PATCH] media: imx208: Add imx208 camera sensor driver

2018-07-10 Thread Yeh, Andy
Hi PC, Thanks for the patch. Cc in Grant, and Intel Jim/Jason > -Original Message- > From: Chen, Ping-chung > Sent: Tuesday, July 10, 2018 3:16 PM > To: linux-media@vger.kernel.org > Cc: sakari.ai...@linux.intel.com; Yeh, Andy ; > tf...@chromium.org; Chen, Ping-chung > Subject: [PATCH]

Re: [PATCH] media: i2c: ov5640: Re-work MIPI startup sequence

2018-07-10 Thread jacopo mondi
Hi Steve, thanks for testing! On Mon, Jul 09, 2018 at 02:52:09PM -0700, Steve Longerbeam wrote: > Hi Jacopo, > > I tested this patch on the i.MX6Q SabreSD with the OV5640 module. > It fixes the LP-11 timeout at stream on, but the captured images > are completely blank/black. Intersting that

[PATCH] media: imx208: Add imx208 camera sensor driver

2018-07-10 Thread Ping-chung Chen
From: "Chen, Ping-chung" Add a V4L2 sub-device driver for the Sony IMX208 image sensor. This is a camera sensor using the I2C bus for control and the CSI-2 bus for data. Signed-off-by: Ping-Chung Chen --- drivers/media/i2c/Kconfig | 11 + drivers/media/i2c/Makefile | 1 +