Re: [PATCH v2 1/2] media: add helpers for memory-to-memory media controller

2018-06-21 Thread Hans Verkuil
On 06/21/2018 10:38 PM, Ezequiel Garcia wrote: > A memory-to-memory pipeline device consists in three > entities: two DMA engine and one video processing entities. > The DMA engine entities are linked to a V4L interface. > > This commit add a new v4l2_m2m_{un}register_media_controller > API to reg

cron job: media_tree daily build: ERRORS

2018-06-21 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: Fri Jun 22 05:00:21 CEST 2018 media-tree git hash:f2809d20b9250c675fca8268a0f6274277cca7ff media_build gi

[PATCH v6 04/17] omap3isp: Add vb2_queue lock

2018-06-21 Thread Ezequiel Garcia
vb2_queue locks is now mandatory. Add it, remove driver ad-hoc locks, and implement wait_{prepare, finish}. Also, remove stream_lock mutex. Since the ioctls operations are now protected by the queue mutex, the stream_lock mutex is not needed. Signed-off-by: Ezequiel Garcia --- drivers/media/pla

Re: [PATCH v5 04/17] omap3isp: Add vb2_queue lock

2018-06-21 Thread Ezequiel Garcia
On Thu, 2018-06-21 at 10:18 +0200, Hans Verkuil wrote: > On 06/20/18 19:42, Ezequiel Garcia wrote: > > vb2_queue locks is now mandatory. Add it, remove driver ad-hoc > > locks, > > and implement wait_{prepare, finish}. > > > > Also, remove stream_lock mutex. Sicen the ioctls operations > > Sicen

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

2018-06-21 Thread Bing Bu Cao
On 2018年06月21日 19:25, Sakari Ailus wrote: > On Tue, Jun 19, 2018 at 02:01:47PM +0800, bingbu@intel.com wrote: >> +static int ak7375_probe(struct i2c_client *client) >> +{ >> +struct ak7375_device *ak7375_dev; >> +int val; >> + >> +ak7375_dev = devm_kzalloc(&client->dev, sizeof(*a

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

2018-06-21 Thread Bing Bu Cao
On 2018年06月21日 19:23, Sakari Ailus wrote: > Hi Bingbu, > > On Tue, Jun 19, 2018 at 02:01:47PM +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 positi

Re: Software-only image processing for Intel "complex" cameras

2018-06-21 Thread Mauro Carvalho Chehab
Em Fri, 22 Jun 2018 06:08:50 +0900 Mauro Carvalho Chehab escreveu: > Em Thu, 21 Jun 2018 18:58:37 + > escreveu: > > > > -Original Message- > > > From: Mauro Carvalho Chehab [mailto:mchehab+sams...@kernel.org] > > > Sent: Thursday, June 21, 2018 1:50 PM > > > To: Limonciello, Mario >

Re: Software-only image processing for Intel "complex" cameras

2018-06-21 Thread Mauro Carvalho Chehab
Em Thu, 21 Jun 2018 18:58:37 + escreveu: > > -Original Message- > > From: Mauro Carvalho Chehab [mailto:mchehab+sams...@kernel.org] > > Sent: Thursday, June 21, 2018 1:50 PM > > To: Limonciello, Mario > > Cc: pa...@ucw.cz; nico...@ndufresne.ca; linux-media@vger.kernel.org; > > sakari.

[PATCH v2 2/2] vim2m: add media device

2018-06-21 Thread Ezequiel Garcia
From: Hans Verkuil Request API requires a media node. Add one to the vim2m driver so we can use requests with it. Signed-off-by: Hans Verkuil Signed-off-by: Ezequiel Garcia --- drivers/media/platform/vim2m.c | 41 ++ 1 file changed, 37 insertions(+), 4 deletion

[PATCH v2 1/2] media: add helpers for memory-to-memory media controller

2018-06-21 Thread Ezequiel Garcia
A memory-to-memory pipeline device consists in three entities: two DMA engine and one video processing entities. The DMA engine entities are linked to a V4L interface. This commit add a new v4l2_m2m_{un}register_media_controller API to register this topology. For instance, a typical mem2mem devic

[PATCH v2 0/2] Memory-to-memory media controller topology

2018-06-21 Thread Ezequiel Garcia
As discussed on IRC, memory-to-memory need to be modeled properly in order to be supported by the media controller framework, and thus to support the Request API. First commit introduces a register/unregister API, that creates/destroys all the entities and pads needed, and links them. The second

Re: [PATCH 0/2] Memory-to-memory media controller topology

2018-06-21 Thread Ezequiel Garcia
On Thu, 2018-06-21 at 14:07 +0200, Hans Verkuil wrote: > On 06/20/18 21:44, Ezequiel Garcia wrote: > > As discussed on IRC, memory-to-memory need to be modeled > > properly in order to be supported by the media controller > > framework, and thus to support the Request API. > > > > The topology loo

RE: Software-only image processing for Intel "complex" cameras

2018-06-21 Thread Mario.Limonciello
> -Original Message- > From: Mauro Carvalho Chehab [mailto:mchehab+sams...@kernel.org] > Sent: Thursday, June 21, 2018 1:50 PM > To: Limonciello, Mario > Cc: pa...@ucw.cz; nico...@ndufresne.ca; linux-media@vger.kernel.org; > sakari.ai...@linux.intel.com; niklas.soderl...@ragnatech.se; > jer

Re: Software-only image processing for Intel "complex" cameras

2018-06-21 Thread Mauro Carvalho Chehab
Em Thu, 21 Jun 2018 13:41:37 + escreveu: > > -Original Message- > > From: Pavel Machek [mailto:pa...@ucw.cz] > > Sent: Wednesday, June 20, 2018 4:12 PM > > To: Nicolas Dufresne > > Cc: linux-media@vger.kernel.org; sakari.ai...@linux.intel.com; > > niklas.soderl...@ragnatech.se; jerry.

Re: Software-only image processing for Intel "complex" cameras

2018-06-21 Thread Mauro Carvalho Chehab
Em Wed, 20 Jun 2018 16:57:59 -0400 Nicolas Dufresne escreveu: > The IPU3 sensor produce a vendor specific form of bayer. If we manage > to implement support for this format, it would likely be done in > software. I don't think anyone can answer your other questions has no > one have ever implemen

[PATCH] media: coda: add h.264 level 4.2

2018-06-21 Thread Philipp Zabel
This enables reordering support for h.264 main profile level 4.2 streams. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-h264.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/coda/coda-h264.c b/drivers/media/platform/coda/coda-h264.c index 0e27412e

RE: Software-only image processing for Intel "complex" cameras

2018-06-21 Thread Mario.Limonciello
> -Original Message- > From: Pavel Machek [mailto:pa...@ucw.cz] > Sent: Wednesday, June 20, 2018 4:12 PM > To: Nicolas Dufresne > Cc: linux-media@vger.kernel.org; sakari.ai...@linux.intel.com; > niklas.soderl...@ragnatech.se; jerry.w...@intel.com; Limonciello, Mario > Subject: Re: Software-

Re: [PATCH 0/2] Memory-to-memory media controller topology

2018-06-21 Thread Hans Verkuil
On 06/20/18 21:44, Ezequiel Garcia wrote: > As discussed on IRC, memory-to-memory need to be modeled > properly in order to be supported by the media controller > framework, and thus to support the Request API. > > The topology looks like this: > > Device topology > - entity 1: source (1 pad, 1 l

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

2018-06-21 Thread Sakari Ailus
On Tue, Jun 19, 2018 at 02:01:47PM +0800, bingbu@intel.com wrote: > +static int ak7375_probe(struct i2c_client *client) > +{ > + struct ak7375_device *ak7375_dev; > + int val; > + > + ak7375_dev = devm_kzalloc(&client->dev, sizeof(*ak7375_dev), > + GFP_

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

2018-06-21 Thread Sakari Ailus
Hi Bingbu, On Tue, Jun 19, 2018 at 02:01:47PM +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. > > ak7375 can write multiple bytes of data at a

[GIT FIXES for 4.18] Cadence compile fixes, imx258 rotation property check

2018-06-21 Thread Sakari Ailus
Hi Mauro, Here are a few fixes for 4.18. Arnd's patches fix regular build errors and the imx258 patch is there to prevent writing firmware they'll only find was incomplete later on. Please pull. The following changes since commit e88f5e9ebd54bdf75c9833e3d9add7c2c0d39b0b: media: uvcvideo: Pre

[PATCH 1/1] v4l-common: Make v4l2_find_nearest_size more sparse-friendly

2018-06-21 Thread Sakari Ailus
This sparse warning is emitted by using v4l2_find_nearest_size in some cases. Fix it in the framework. >> drivers/media/i2c/ov5640.c:1394:14: sparse: incorrect type in assignment +(different base types) @@expected struct ov5640_mode_info const *mode @@ +got ststruct ov5640_mode_info const *mod

Re: [PATCH v4] media: ov5640: fix frame interval enumeration

2018-06-21 Thread Sakari Ailus
Hi Hugues, On Thu, Jun 21, 2018 at 10:53:39AM +0200, Hugues Fruchet wrote: > Driver must reject frame interval enumeration of unsupported resolution. > This was detected by v4l2-compliance format ioctl test: > v4l2-compliance Format ioctls: > info: found 2 frameintervals for pixel format 47455

[PATCH v4] media: ov5640: fix frame interval enumeration

2018-06-21 Thread Hugues Fruchet
Driver must reject frame interval enumeration of unsupported resolution. This was detected by v4l2-compliance format ioctl test: v4l2-compliance Format ioctls: info: found 2 frameintervals for pixel format 4745504a and size 176x144 fail: v4l2-test-formats.cpp(123):

Re: [PATCH v5 04/17] omap3isp: Add vb2_queue lock

2018-06-21 Thread Hans Verkuil
On 06/20/18 19:42, Ezequiel Garcia wrote: > vb2_queue locks is now mandatory. Add it, remove driver ad-hoc locks, > and implement wait_{prepare, finish}. > > Also, remove stream_lock mutex. Sicen the ioctls operations Sicen -> Since > are protected by the queue mutex, the stream_lock mutex is a

[PATCHv3 5/8] media.h: add MEDIA_ENT_F_DTV_ENCODER

2018-06-21 Thread Hans Verkuil
From: Hans Verkuil Add a new function for digital video encoders such as HDMI transmitters. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/mediactl/media-types.rst | 7 +++ include/uapi/linux/media.h| 3 ++- 2 files changed, 9 insertions(+), 1 deletion(-)

[PATCHv3 1/8] media: add 'index' to struct media_v2_pad

2018-06-21 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 --- drivers/media/media-devi

[PATCHv3 8/8] media/i2c: add missing entity functions

2018-06-21 Thread Hans Verkuil
From: Hans Verkuil Several drivers in media/i2c do not set the entity function. Correct this. Signed-off-by: Hans Verkuil --- drivers/media/i2c/adv7604.c | 1 + drivers/media/i2c/adv7842.c | 1 + drivers/media/i2c/et8ek8/et8ek8_driver.c | 1 + drivers/media/i2c/mt9m03

[PATCHv3 3/8] media: add flags field to struct media_v2_entity

2018-06-21 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 --- drivers/media/media-device.c | 1 + include/ua

[PATCHv3 6/8] ad9389b/adv7511: set proper media entity function

2018-06-21 Thread Hans Verkuil
From: Hans Verkuil These two drivers both have function MEDIA_ENT_F_DTV_ENCODER. Signed-off-by: Hans Verkuil --- drivers/media/i2c/ad9389b.c | 1 + drivers/media/i2c/adv7511.c | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/media/i2c/ad9389b.c b/drivers/media/i2c/ad9389b.c index

[PATCHv3 7/8] adv7180/tvp514x/tvp7002: fix entity function

2018-06-21 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 --- drivers/media/i2c/adv7180.c | 2 +- drivers/media/i2c/tvp514x.c | 2 +- drivers/media/i2c/tvp7002.c | 2 +- 3 files changed, 3 insertion

[PATCHv3 2/8] media-ioc-g-topology.rst: document new 'index' field

2018-06-21 Thread Hans Verkuil
From: Hans Verkuil Document the new struct media_v2_pad 'index' field. Signed-off-by: Hans Verkuil --- .../media/uapi/mediactl/media-ioc-g-topology.rst | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/Documentation/media/uapi/mediactl/media-ioc-g-topology.rs

[PATCHv3 4/8] media-ioc-g-topology.rst: document new 'flags' field

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

[PATCHv3 0/8] media/mc: fix inconsistencies

2018-06-21 Thread Hans Verkuil
From: Hans Verkuil This patch series sits on top of this pull request: https://patchwork.linuxtv.org/patch/50453/ That pull request cleans up the tables in the documentation, making it easier to add new entries. This series is v3 of my previous attempt: https://www.spinics.net/lists/linux-med