cron job: media_tree daily build: OK

2019-05-16 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 May 17 05:00:12 CEST 2019 media-tree git hash:fc8670d1f72b746ff3a5fe441f1fca4c4dba0e6f media_build gi

[PATCH v3] media:staging/intel-ipu3: update minimal GDC envelope size to 4

2019-05-16 Thread bingbu . cao
From: Bingbu Cao The ipu3 GDC function need some envelope to do filtering and the minimal envelope size(GDC in - out) for ipu3 should be 4. Current value 4 was defined for older version GDC, this patch correct it. Signed-off-by: Bingbu Cao Reviewed-by: Tomasz Figa --- drivers/staging/media/i

[PATCH v2] media:staging/intel-ipu3: update minimal GDC envelope size to 4

2019-05-16 Thread bingbu . cao
From: Bingbu Cao The ipu3 GDC function need some envelope to do filtering and the minimal envelope size(GDC in - out) for ipu3 should be 4. Current value 4 was defined for older version GDC, this patch correct it. Signed-off-by: Bingbu Cao --- drivers/staging/media/ipu3/ipu3-css.c | 14 ++-

Re: [PATCH v6 3/5] media: dt-bindings: media: Add vendor prefix for allegro

2019-05-16 Thread Rob Herring
On Mon, May 13, 2019 at 12:21 PM Michael Tretter wrote: > > Add vendor prefix for Allegro DVT, a provider of H.264/AVC, H.265/HEVC, > AVS2, VP9 and AV1 compliance test suites and H.264/AVC, H.265/HEVC, and > VP9 encoder, codec and decoder hardware (RTL) IPs. > > Signed-off-by: Michael Tretter > R

Re: [PATCH v4 4/4] media: vsp1: Provide partition overlap algorithm

2019-05-16 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Mon, May 13, 2019 at 09:13:55PM +0100, Kieran Bingham wrote: > To improve image quality, entities involved within the image partition > algorithm may extend their partition window to account for their input > requirements and to take consideration of the num

Re: Proposed updates and guidelines for MPEG-2, H.264 and H.265 stateless support

2019-05-16 Thread Paul Kocialkowski
Hi, Le jeudi 16 mai 2019 à 14:24 -0400, Nicolas Dufresne a écrit : > Le mercredi 15 mai 2019 à 22:59 +0200, Paul Kocialkowski a écrit : > > Hi, > > > > Le mercredi 15 mai 2019 à 14:54 -0400, Nicolas Dufresne a écrit : > > > Le mercredi 15 mai 2019 à 19:42 +0200, Paul Kocialkowski a écrit : > > >

Re: Proposed updates and guidelines for MPEG-2, H.264 and H.265 stateless support

2019-05-16 Thread Nicolas Dufresne
Le mercredi 15 mai 2019 à 22:59 +0200, Paul Kocialkowski a écrit : > Hi, > > Le mercredi 15 mai 2019 à 14:54 -0400, Nicolas Dufresne a écrit : > > Le mercredi 15 mai 2019 à 19:42 +0200, Paul Kocialkowski a écrit : > > > Hi, > > > > > > Le mercredi 15 mai 2019 à 10:42 -0400, Nicolas Dufresne a écr

Re: [RFC] media: uapi: Add VP8 low-level decoder API compound controls.

2019-05-16 Thread Nicolas Dufresne
Le mercredi 13 février 2019 à 18:15 -0300, Ezequiel Garcia a écrit : > From: Pawel Osciak > > These controls are to be used with the new low-level decoder API for VP8 > to provide additional parameters for the hardware that cannot parse the > input stream. > > Signed-off-by: Pawel Osciak > [eze

Re: [PATCH v6 12/13] media: tvp5150: add support to limit tv norms on connector

2019-05-16 Thread Laurent Pinchart
Hi Marco, Thank you for the patch. On Mon, Apr 15, 2019 at 02:44:12PM +0200, Marco Felsch wrote: > The tvp5150 accepts NTSC(M,J,4.43), PAL (B,D,G,H,I,M,N) and SECAM video > data and is able to auto-detect the input signal. The auto-detection > does not work if the connector does not receive an in

Re: [PATCH v6 06/13] media: dt-bindings: tvp5150: Add input port connectors DT bindings

2019-05-16 Thread Laurent Pinchart
Hi Marco, Thank you for the patch. On Mon, Apr 15, 2019 at 02:44:06PM +0200, Marco Felsch wrote: > The TVP5150/1 decoders support different video input sources to their > AIP1A/B pins. > > Possible configurations are as follows: > - Analog Composite signal connected to AIP1A. > - Analog Comp

Re: [PATCH v6 05/13] media: tvp5150: add input source selection of_graph support

2019-05-16 Thread Laurent Pinchart
Hello Marco, Thank you for the patch. On Tue, May 14, 2019 at 03:25:45PM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 15 Apr 2019 14:44:05 +0200 Marco Felsch escreveu: > > > This patch adds the of_graph support to describe the tvp connections. > > Physical the TVP5150 has three ports: AIP1A, AI

Re: [RFC] media: uapi: Add VP8 low-level decoder API compound controls.

2019-05-16 Thread Nicolas Dufresne
Le lundi 25 février 2019 à 14:34 +0900, Tomasz Figa a écrit : > On Fri, Feb 15, 2019 at 9:06 AM Ezequiel Garcia > wrote: > > On Wed, 2019-02-13 at 21:35 -0500, Nicolas Dufresne wrote: > > > Le mer. 13 févr. 2019 à 16:23, Ezequiel Garcia > > > a écrit : > > > > Hi, > > > > > > > > On Wed, 2019-0

Re: [PATCH v6 03/13] media: v4l2-fwnode: add initial connector parsing support

2019-05-16 Thread Laurent Pinchart
Hello Marco, Thank you for the patch. On Tue, May 14, 2019 at 03:20:04PM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 6 May 2019 12:10:41 +0200 Hans Verkuil escreveu: > > On 4/15/19 2:44 PM, Marco Felsch wrote: > > > The patch adds the initial connector parsing code, so we can move from a > > >

Re: [PATCH v6 02/13] media: v4l2-fwnode: add v4l2_fwnode_connector

2019-05-16 Thread Laurent Pinchart
Hi Marco, Thank you for the patch. On Mon, Apr 15, 2019 at 02:44:02PM +0200, Marco Felsch wrote: > Currently every driver needs to parse the connector endpoints by it self. s/it self/itself/ > This is the initial work to make this generic. The generic connector has > some common fields and some

Re: [PATCH v6 01/13] dt-bindings: connector: analog: add tv norms property

2019-05-16 Thread Laurent Pinchart
Hi Marco, Thank you for the patch. On Mon, Apr 15, 2019 at 02:44:01PM +0200, Marco Felsch wrote: > Some connectors no matter if in- or output supports only a limited > range of tv norms. It doesn't matter if the hardware behind that > connector supports more than the listed formats since the user

Re: [PATCH v2 4/8] rcar-vin: Do not sync subdevice format when opening the video device

2019-05-16 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2019-05-16 14:26:44 +0300, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Thu, May 16, 2019 at 03:14:13AM +0200, Niklas Söderlund wrote: > > The format is already synced when the subdevice is bound, there is no > > need to do do

Re: [PATCH 2/3] rcar-vin: Add control for alpha component

2019-05-16 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2019-05-16 13:01:38 +0300, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Thu, May 16, 2019 at 02:47:45AM +0200, Niklas Söderlund wrote: > > In preparation to adding support for RGB pixel formats with an alpha > > component add

Re: [PATCH 3/3] rcar-vin: Add support for RGB formats with alpha component

2019-05-16 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2019-05-16 13:08:22 +0300, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Thu, May 16, 2019 at 02:47:46AM +0200, Niklas Söderlund wrote: > > The R-Car VIN module supports V4L2_PIX_FMT_ARGB555 and > > V4L2_PIX_FMT_ABGR32 pixel fo

Re: [PATCH v4 3/4] media: vsp1: Split out pre-filter calculation

2019-05-16 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Mon, May 13, 2019 at 09:13:54PM +0100, Kieran Bingham wrote: > The 'mp' value is used through several calculations in determining the > scaling factors of the UDS. This determines the pre-scaling filter > binning value. Factor this out so that it can be reus

Re: [PATCH v2 8/8] rcar-vin: Merge Gen2 and Gen3 file operations

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 03:14:17AM +0200, Niklas Söderlund wrote: > After the rework of the Gen2 file operations it's now trivial to merge > the Gen2 and Gen3 versions. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Ulrich Hecht > --- > drivers/media/pl

Re: [PATCH v2 6/8] rcar-vin: Merge helpers dealing with powering the parallel subdevice

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 03:14:15AM +0200, Niklas Söderlund wrote: > The two power helpers are now only dealing with the parallel subdevice, > merge them into a single rvin_power_parallel() helper to reduce code > duplication. > > Signed-off-by: Niklas Söderlun

Re: [PATCH v2 7/8] rcar-vin: Fold rvin_initialize_device() into rvin_open()

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 03:14:16AM +0200, Niklas Söderlund wrote: > The function no longer serve a purpose as most tasks it performed have > been refactored, fold what remains of it into the only caller. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Ulri

Re: [PATCH v2 5/8] rcar-vin: Move pm_runtime_{get,put} out of helpers

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 03:14:14AM +0200, Niklas Söderlund wrote: > The helpers rvin_power_{on,off} deal with both VIN and the parallel > subdevice power. This makes it hard to merge the Gen2 and Gen3 > open/release functions. Move the VIN power handling direct

Re: [PATCH v2 2/8] rcar-vin: Remove unneeded calls to pm_runtime_{enable,disable}

2019-05-16 Thread Laurent Pinchart
On Thu, May 16, 2019 at 09:27:03AM +0200, Geert Uytterhoeven wrote: > On Thu, May 16, 2019 at 3:49 AM Niklas Söderlund wrote: > > Runtime PM is already enabled unconditionally when the driver is probed > > and disabled when it's removed. There is no point in doing it again for > > Gen2 when opening

Re: [PATCH v2 1/8] rcar-vin: Do not call pm_runtime_{resume,suspend}()

2019-05-16 Thread Laurent Pinchart
On Thu, May 16, 2019 at 09:23:41AM +0200, Geert Uytterhoeven wrote: > On Thu, May 16, 2019 at 3:46 AM Niklas Söderlund wrote: > > The driver does not implement runtime resume and suspend function so > > there is little point in trying to call them. This is a leftover from > > the drivers soc_camera

Re: [PATCH v2 4/8] rcar-vin: Do not sync subdevice format when opening the video device

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 03:14:13AM +0200, Niklas Söderlund wrote: > The format is already synced when the subdevice is bound, there is no > need to do do it every time the video device is opened. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Ulrich Hecht

Re: [PATCH] rcar-csi2: Fix coccinelle warning for PTR_ERR_OR_ZERO()

2019-05-16 Thread Simon Horman
On Thu, May 16, 2019 at 02:35:38AM +0200, Niklas Söderlund wrote: > Use the PTR_ERR_OR_ZERO() macro instead of construct: > > if (IS_ERR(foo)) > return PTR_ERR(foo); > > return 0; > > Reported-by: kbuild test robot > Fixes: 3ae854cafd76 ("rcar-csi2: Use standby mode instead of r

Re: [PATCH v2 3/8] rcar-vin: Allow interrupting lock when trying to open the video device

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 03:14:12AM +0200, Niklas Söderlund wrote: > The user should be allowed to break waiting for the lock when opening > the video device. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart > -

Re: [PATCH] media: fdp1: Support M3N and E3 platforms

2019-05-16 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Wed, May 15, 2019 at 04:39:12PM +0100, Kieran Bingham wrote: > New Gen3 R-Car platforms incorporate the FDP1 with an updated version > register. No code change is required to support these targets, but they > will currently report an error stating that the d

Re: [PATCH 3/3] rcar-vin: Add support for RGB formats with alpha component

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 02:47:46AM +0200, Niklas Söderlund wrote: > The R-Car VIN module supports V4L2_PIX_FMT_ARGB555 and > V4L2_PIX_FMT_ABGR32 pixel formats. Add the hardware register setup and > allow the alpha component to be changed while streaming using t

Re: [PATCH 2/3] rcar-vin: Add control for alpha component

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 02:47:45AM +0200, Niklas Söderlund wrote: > In preparation to adding support for RGB pixel formats with an alpha > component add a control to allow the user to control which alpha value > should be used. > > Signed-off-by: Niklas Söderl

Re: [PATCH 1/3] rcar-vin: Rename VNDMR_DTMD_ARGB1555 to VNDMR_DTMD_ARGB

2019-05-16 Thread Laurent Pinchart
Hi Niklas, Thank you for the patch. On Thu, May 16, 2019 at 02:47:44AM +0200, Niklas Söderlund wrote: > The value have nothing to do with ARGB1555, it controls if the alpha > component should be filled in for ARGB1555 or ARGB888. Rename it to > reflect this. > > Signed-off-by: Niklas Söderlund

Re: [PATCH 1/3] rcar-vin: Rename VNDMR_DTMD_ARGB1555 to VNDMR_DTMD_ARGB

2019-05-16 Thread Ulrich Hecht
> On May 16, 2019 at 2:47 AM Niklas Söderlund > wrote: > > > The value have nothing to do with ARGB1555, it controls if the alpha > component should be filled in for ARGB1555 or ARGB888. Rename it to > reflect this. > > Signed-off-by: Niklas Söderlund > --- > drivers/media/platform/rcar-vi

Re: [RFC PATCH V1 0/6] media: platform: Add support for Face Detection (FD) on mt8183 SoC

2019-05-16 Thread Jerry-ch Chen
Hi Hans: Thanks for your comments. On Mon, 2019-05-13 at 11:14 +0200, Hans Verkuil wrote: > On 4/23/19 12:44 PM, Jerry-ch Chen wrote: > > Hello, > > > > This RFC patch series is adding Face Detection (FD) driver on Mediatek > > mt8183 SoC. It belongs to the first Mediatek's camera driver series

Re: [RFC PATCH] vb2: add V4L2_BUF_FLAG_M2M_HOLD_CAPTURE_BUF

2019-05-16 Thread Alexandre Courbot
On Tue, May 14, 2019 at 7:59 PM Hans Verkuil wrote: > > On 5/14/19 12:01 PM, Alexandre Courbot wrote: > > From: Hans Verkuil > > Date: Mon, May 13, 2019 at 6:36 PM > > To: Linux Media Mailing List, Paul Kocialkowski, Tomasz Figa, Nicolas > > Dufresne, Alexandre Courbot, Boris Brezillon, Maxime Ri

Re: [RFC PATCH v2 3/7] media: v4l2: Add extended buffer operations

2019-05-16 Thread Tomasz Figa
On Wed, May 8, 2019 at 10:58 PM Hans Verkuil wrote: > > Hi Boris, > > On 4/12/19 10:57 AM, Boris Brezillon wrote: > > On Thu, 4 Apr 2019 10:16:56 +0200 > > Boris Brezillon wrote: > > > > > >> +/** > >> + * struct v4l2_ext_buffer - extended video buffer info > >> + * @index: id number of the buff

Re: [PATCH] rcar-csi2: Fix coccinelle warning for PTR_ERR_OR_ZERO()

2019-05-16 Thread Geert Uytterhoeven
On Thu, May 16, 2019 at 3:50 AM Niklas Söderlund wrote: > Use the PTR_ERR_OR_ZERO() macro instead of construct: > > if (IS_ERR(foo)) > return PTR_ERR(foo); > > return 0; > > Reported-by: kbuild test robot > Fixes: 3ae854cafd76 ("rcar-csi2: Use standby mode instead of resetting") >

Re: [PATCH] rcar-csi2: Fix coccinelle warning for PTR_ERR_OR_ZERO()

2019-05-16 Thread Ulrich Hecht
> On May 16, 2019 at 2:35 AM Niklas Söderlund > wrote: > > > Use the PTR_ERR_OR_ZERO() macro instead of construct: > > if (IS_ERR(foo)) > return PTR_ERR(foo); > > return 0; > > Reported-by: kbuild test robot > Fixes: 3ae854cafd76 ("rcar-csi2: Use standby mode instead of re

Re: [PATCH] dt-bindings: rcar-{csi2,vin}: Rename bindings documentation files

2019-05-16 Thread Ulrich Hecht
> On May 16, 2019 at 2:38 AM Niklas Söderlund > wrote: > > > Renesas media binding documentation files uses a naming schema of > 'renesas,.txt'. Rename VIN and CSI-2 files to match this > pattern. > > Signed-off-by: Niklas Söderlund > --- > .../media/{renesas,rcar-csi2.txt => renesas,csi2.

Re: [PATCH v2 6/8] rcar-vin: Merge helpers dealing with powering the parallel subdevice

2019-05-16 Thread Ulrich Hecht
> On May 16, 2019 at 3:14 AM Niklas Söderlund > wrote: > > > The two power helpers are now only dealing with the parallel subdevice, > merge them into a single rvin_power_parallel() helper to reduce code > duplication. > > Signed-off-by: Niklas Söderlund > --- > drivers/media/platform/rcar

Re: [PATCH v2 2/8] rcar-vin: Remove unneeded calls to pm_runtime_{enable,disable}

2019-05-16 Thread Geert Uytterhoeven
Hi Niklas, On Thu, May 16, 2019 at 3:49 AM Niklas Söderlund wrote: > Runtime PM is already enabled unconditionally when the driver is probed > and disabled when it's removed. There is no point in doing it again for > Gen2 when opening and closing the video device. > > Signed-off-by: Niklas Söderl

Re: [PATCH v2 1/8] rcar-vin: Do not call pm_runtime_{resume,suspend}()

2019-05-16 Thread Geert Uytterhoeven
Hi Niklas, On Thu, May 16, 2019 at 3:46 AM Niklas Söderlund wrote: > The driver does not implement runtime resume and suspend function so > there is little point in trying to call them. This is a leftover from > the drivers soc_camera beginnings. > > Signed-off-by: Niklas Söderlund > Reviewed-by

Re: [PATCH] dt-bindings: rcar-{csi2,vin}: Rename bindings documentation files

2019-05-16 Thread Geert Uytterhoeven
On Thu, May 16, 2019 at 3:49 AM Niklas Söderlund wrote: > Renesas media binding documentation files uses a naming schema of > 'renesas,.txt'. Rename VIN and CSI-2 files to match this > pattern. > > Signed-off-by: Niklas Söderlund Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s,