cron job: media_tree daily build: WARNINGS

2017-11-17 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: Sat Nov 18 05:00:19 CET 2017 media-tree git hash:30b4e122d71cbec2944a5f8b558b88936ee42f10 media_build

[PATCH] media: imx: Remove incorrect check for queue state in start_streaming

2017-11-17 Thread Ian Jamison
It is possible to call STREAMON without the minimum number of buffers queued. In this case the vb2_queue state will be set to streaming but the start_streaming vb2_op will not be called. Later when enough buffers are queued, start_streaming will be called but vb2_is_streaming will already return

Re: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-17 Thread Rob Herring
On Thu, Nov 16, 2017 at 06:22:49PM +, Fabrizio Castro wrote: > Add compatible strings for r8a7743 and r8a7745. No driver change > is needed as "renesas,rcar-gen2-vin" will activate the right code. > However, it is good practice to document compatible strings for the > specific SoC as this

Re: [PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number list

2017-11-17 Thread Rob Herring
On Thu, Nov 16, 2017 at 06:22:48PM +, Fabrizio Castro wrote: > Change the sorting of the part numbers from descending to ascending to > match with other documentation. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das > --- >

Grüße an Dich

2017-11-17 Thread Meinze Klaus Peter
In einer kurzen Einführung Ich bin ein Rechtsanwalt Meinze Klaus Peter, aus Germany, ich lebe jetzt in London, ich habe Ihnen eine E-Mail über Ihre verstorbene Familie geschickt, aber ich habe keine Antwort von Ihnen erhalten, der Verstorbene ist ein Bürger Ihres Landes mit der gleicher Nachname

Re: [PATCH] media: cx231xx: add support for TheImagingSource DFG/USB2pro

2017-11-17 Thread jacopo mondi
Hi Romain, On Fri, Nov 17, 2017 at 04:38:55PM +0100, Romain Reignier wrote: > Hello, > > This is my first patch to the kernel so please be indulgent if I have done > anything wrong and help me produce a better submission. With this formatting (email text + patch below) applying with 'git am'

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Gustavo Padovan : > 2017-11-17 Hans Verkuil : > > > On 15/11/17 18:10, Gustavo Padovan wrote: > > > From: Gustavo Padovan > > > > > > Receive in-fence from userspace and add support for waiting on them > > >

Re: [PATCH] media: rc: double keypresses due to timeout expiring to early

2017-11-17 Thread Matthias Reichl
On Fri, Nov 17, 2017 at 03:52:50PM +0100, Matthias Reichl wrote: > Hi Sean! > > On Thu, Nov 16, 2017 at 09:54:51PM +, Sean Young wrote: > > Since commit d57ea877af38 ("media: rc: per-protocol repeat period"), > > double keypresses are reported on the ite-cir driver. This is due > > two

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Hans Verkuil : > On 15/11/17 18:10, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Receive in-fence from userspace and add support for waiting on them > > before queueing the buffer to the driver. Buffers can't be queued to

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Philippe Ombredanne
On Fri, Nov 17, 2017 at 6:01 PM, Mauro Carvalho Chehab wrote: > Em Fri, 17 Nov 2017 16:01:41 +0100 > Philippe Ombredanne escreveu: > >> On Fri, Nov 17, 2017 at 3:58 PM, Mauro Carvalho Chehab >> wrote: >> > Em Fri, 17 Nov

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Mauro Carvalho Chehab : > Em Fri, 17 Nov 2017 11:12:48 -0200 > Gustavo Padovan escreveu: > > > > > /* > > > > * If streamon has been called, and we haven't yet called > > > > * start_streaming() since not enough

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Greg Kroah-Hartman
On Fri, Nov 17, 2017 at 03:01:02PM -0200, Mauro Carvalho Chehab wrote: > Em Fri, 17 Nov 2017 16:01:41 +0100 > Philippe Ombredanne escreveu: > > > On Fri, Nov 17, 2017 at 3:58 PM, Mauro Carvalho Chehab > > wrote: > > > Em Fri, 17 Nov 2017 15:18:26

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 16:01:41 +0100 Philippe Ombredanne escreveu: > On Fri, Nov 17, 2017 at 3:58 PM, Mauro Carvalho Chehab > wrote: > > Em Fri, 17 Nov 2017 15:18:26 +0100 > > Greg Kroah-Hartman escreveu: > > > >>

Re: [PATCH 6/6] media: usb: add SPDX identifiers to some code I wrote

2017-11-17 Thread Laurent Pinchart
Hi Philippe, On Friday, 17 November 2017 17:08:51 EET Philippe Ombredanne wrote: > On Fri, Nov 17, 2017 at 4:01 PM, Mauro Carvalho Chehab wrote: > > Em Fri, 17 Nov 2017 12:54:15 +0100 Philippe Ombredanne escreveu: > >> On Fri, Nov 17, 2017 at 11:21 AM, Mauro Carvalho Chehab wrote: > >> > As we're

[PATCH v4 9/9] v4l: vsp1: Reduce display list body size

2017-11-17 Thread Kieran Bingham
The display list originally allocated a body of 256 entries to store all of the register lists required for each frame. This has now been separated into fragments for constant stream setup, and runtime updates. Empirical testing shows that the body0 now uses a maximum of 41 registers for each

[PATCH v4 8/9] v4l: vsp1: Move video configuration to a cached dlb

2017-11-17 Thread Kieran Bingham
We are now able to configure a pipeline directly into a local display list body. Take advantage of this fact, and create a cacheable body to store the configuration of the pipeline in the video object. vsp1_video_pipeline_run() is now the last user of the pipe->dl object. Convert this function to

[PATCH v4 5/9] v4l: vsp1: Use reference counting for bodies

2017-11-17 Thread Kieran Bingham
Extend the display list body with a reference count, allowing bodies to be kept as long as a reference is maintained. This provides the ability to keep a cached copy of bodies which will not change, so that they can be re-applied to multiple display lists. Signed-off-by: Kieran Bingham

[PATCH v4 3/9] v4l: vsp1: Provide a body pool

2017-11-17 Thread Kieran Bingham
Each display list allocates a body to store register values in a dma accessible buffer from a dma_alloc_wc() allocation. Each of these results in an entry in the TLB, and a large number of display list allocations adds pressure to this resource. Reduce TLB pressure on the IPMMUs by allocating

[PATCH v4 7/9] v4l: vsp1: Adapt entities to configure into a body

2017-11-17 Thread Kieran Bingham
Currently the entities store their configurations into a display list. Adapt this such that the code can be configured into a body directly, allowing greater flexibility and control of the content. All users of vsp1_dl_list_write() are removed in this process, thus it too is removed. A helper,

[PATCH v4 6/9] v4l: vsp1: Refactor display list configure operations

2017-11-17 Thread Kieran Bingham
The entities provide a single .configure operation which configures the object into the target display list, based on the vsp1_entity_params selection. This restricts us to a single function prototype for both static configuration (the pre-stream INIT stage) and the dynamic runtime stages for

[PATCH v4 2/9] v4l: vsp1: Protect bodies against overflow

2017-11-17 Thread Kieran Bingham
The body write function relies on the code never asking it to write more than the entries available in the list. Currently with each list body containing 256 entries, this is fine, but we can reduce this number greatly saving memory. In preparation of this add a level of protection to catch any

[PATCH v4 4/9] v4l: vsp1: Convert display lists to use new body pool

2017-11-17 Thread Kieran Bingham
Adapt the dl->body0 object to use an object from the body pool. This greatly reduces the pressure on the TLB for IPMMU use cases, as all of the lists use a single allocation for the main body. The CLU and LUT objects pre-allocate a pool containing three bodies, allowing a userspace update before

[PATCH v4 1/9] v4l: vsp1: Reword uses of 'fragment' as 'body'

2017-11-17 Thread Kieran Bingham
Throughout the codebase, the term 'fragment' is used to represent a display list body. This term duplicates the 'body' which is already in use. The datasheet references these objects as a body, therefore replace all mentions of a fragment with a body, along with the corresponding pluralised

[PATCH v4 0/9] vsp1: TLB optimisation and DL caching

2017-11-17 Thread Kieran Bingham
Each display list currently allocates an area of DMA memory to store register settings for the VSP1 to process. Each of these allocations adds pressure to the IPMMU TLB entries. We can reduce the pressure by pre-allocating larger areas and dividing the area across multiple bodies represented as a

[PATCH] media: cx231xx: add support for TheImagingSource DFG/USB2pro

2017-11-17 Thread Romain Reignier
Hello, This is my first patch to the kernel so please be indulgent if I have done anything wrong and help me produce a better submission. This is a patch to add the support for The Imaging Source DFG/USB2pro USB capture device. It is based on the Conexant CX23102 chip do the patch only

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Greg Kroah-Hartman
On Fri, Nov 17, 2017 at 12:58:47PM -0200, Mauro Carvalho Chehab wrote: > Em Fri, 17 Nov 2017 15:18:26 +0100 > Greg Kroah-Hartman escreveu: > > > There is no need to #define the license of the driver, just put it in > > the MODULE_LICENSE() line directly as a text

Re: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2017-11-17 Thread Simon Horman
On Thu, Nov 16, 2017 at 06:22:49PM +, Fabrizio Castro wrote: > Add compatible strings for r8a7743 and r8a7745. No driver change > is needed as "renesas,rcar-gen2-vin" will activate the right code. > However, it is good practice to document compatible strings for the > specific SoC as this

Re: [PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number list

2017-11-17 Thread Simon Horman
On Thu, Nov 16, 2017 at 06:22:48PM +, Fabrizio Castro wrote: > Change the sorting of the part numbers from descending to ascending to > match with other documentation. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das

Re: [PATCH 6/6] media: usb: add SPDX identifiers to some code I wrote

2017-11-17 Thread Philippe Ombredanne
On Fri, Nov 17, 2017 at 4:01 PM, Mauro Carvalho Chehab wrote: > Em Fri, 17 Nov 2017 12:54:15 +0100 > Philippe Ombredanne escreveu: > >> On Fri, Nov 17, 2017 at 11:21 AM, Mauro Carvalho Chehab >> wrote: >> > As we're now

Re: [PATCH v2 5/8] v4l: vsp1: Refactor display list configure operations

2017-11-17 Thread Kieran Bingham
Hi Laurent, Just a query on your bikeshedding here. Choose your colours wisely :) -- Kieran On 12/09/17 20:19, Laurent Pinchart wrote: > Hi Kieran, > > On Tuesday, 12 September 2017 00:16:50 EEST Kieran Bingham wrote: >> On 17/08/17 19:13, Laurent Pinchart wrote: >>> On Monday 14 Aug 2017

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Philippe Ombredanne
On Fri, Nov 17, 2017 at 3:58 PM, Mauro Carvalho Chehab wrote: > Em Fri, 17 Nov 2017 15:18:26 +0100 > Greg Kroah-Hartman escreveu: > >> There is no need to #define the license of the driver, just put it in >> the MODULE_LICENSE() line directly

Re: [PATCH 6/6] media: usb: add SPDX identifiers to some code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 12:54:15 +0100 Philippe Ombredanne escreveu: > On Fri, Nov 17, 2017 at 11:21 AM, Mauro Carvalho Chehab > wrote: > > As we're now using SPDX identifiers, on several > > media drivers I wrote, add the proper SPDX, identifying > >

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 15:18:26 +0100 Greg Kroah-Hartman escreveu: > There is no need to #define the license of the driver, just put it in > the MODULE_LICENSE() line directly as a text string. > > This allows tools that check that the module license matches the source

[PATCH v2] media: ABS macro parameter parenthesization

2017-11-17 Thread dgopstein
From: Dan Gopstein Two definitions of the ABS (absolute value) macro fail to parenthesize their parameter properly. This can lead to a bad expansion for low-precedence expression arguments. Add parens to protect against troublesome arguments. For example: ABS(1-2) currently

Re: [PATCH] media: rc: double keypresses due to timeout expiring to early

2017-11-17 Thread Matthias Reichl
Hi Sean! On Thu, Nov 16, 2017 at 09:54:51PM +, Sean Young wrote: > Since commit d57ea877af38 ("media: rc: per-protocol repeat period"), > double keypresses are reported on the ite-cir driver. This is due > two factors: that commit reduced the timeout used for some protocols > (it became

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Philippe Ombredanne
On Fri, Nov 17, 2017 at 3:18 PM, Greg Kroah-Hartman wrote: > There is no need to #define the license of the driver, just put it in > the MODULE_LICENSE() line directly as a text string. > > This allows tools that check that the module license matches the source > code

Re: [RFCv1 PATCH 1/6] v4l2-ctrls: v4l2_ctrl_add_handler: add from_other_dev

2017-11-17 Thread Hans Verkuil
On 17/11/17 14:41, Mauro Carvalho Chehab wrote: > Em Mon, 13 Nov 2017 15:34:03 +0100 > Hans Verkuil escreveu: > >> From: Hans Verkuil >> >> Add a 'bool from_other_dev' argument: set to true if the two >> handlers refer to different devices (e.g. it is

Re: [PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Hans Verkuil
On 17/11/17 15:18, Greg Kroah-Hartman wrote: > There is no need to #define the license of the driver, just put it in > the MODULE_LICENSE() line directly as a text string. > > This allows tools that check that the module license matches the source > code license to work properly, as there is no

Re: [RFC v5 06/11] [media] vb2: add explicit fence user API

2017-11-17 Thread Hans Verkuil
On 17/11/17 14:53, Mauro Carvalho Chehab wrote: > Em Fri, 17 Nov 2017 14:29:23 +0100 > Hans Verkuil escreveu: > >> On 15/11/17 18:10, Gustavo Padovan wrote: >>> From: Gustavo Padovan >>> >>> Turn the reserved2 field into fence_fd that we will

[PATCH] media: coda: fix comparision of decoded frames' indexes

2017-11-17 Thread Martin Kepplinger
At this point the driver looks the currently decoded frame's index and compares is to VPU-specific state values. Directly before this if and else statements the indexes are read (index for decoded and for displayed frame). Now what is saved in ctx->display_idx is an older value at this point!

Re: [PATCH v1 04/10] ARM: dts: r7s72100: Add Capture Engine Unit (CEU)

2017-11-17 Thread Simon Horman
On Wed, Nov 15, 2017 at 11:55:57AM +0100, Jacopo Mondi wrote: > Add Capture Engine Unit (CEU) node to device tree. Other patches in this series (which are not for my tree) appear to warrant updating. Accordingly I am marking this patch as "Changes Requested" and am expecting it to be reposted at

[PATCH] media: usbvision: remove unneeded DRIVER_LICENSE #define

2017-11-17 Thread Greg Kroah-Hartman
There is no need to #define the license of the driver, just put it in the MODULE_LICENSE() line directly as a text string. This allows tools that check that the module license matches the source code license to work properly, as there is no need to unwind the unneeded dereference. Cc: Hans

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Hans Verkuil
On 15/11/17 18:10, Gustavo Padovan wrote: > From: Gustavo Padovan > > Receive in-fence from userspace and add support for waiting on them > before queueing the buffer to the driver. Buffers can't be queued to the > driver before its fences signal. And a buffer

Re: [RFC v5 06/11] [media] vb2: add explicit fence user API

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 14:29:23 +0100 Hans Verkuil escreveu: > On 15/11/17 18:10, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Turn the reserved2 field into fence_fd that we will use to send > > an in-fence to the kernel and return an

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 11:12:48 -0200 Gustavo Padovan escreveu: > > > /* > > >* If streamon has been called, and we haven't yet called > > >* start_streaming() since not enough buffers were queued, and > > >* we now have reached the minimum number of queued

Re: [PATCH v7 01/25] rcar-vin: add Gen3 devicetree bindings documentation

2017-11-17 Thread Rob Herring
On Wed, Nov 15, 2017 at 4:58 PM, Niklas Söderlund wrote: > Hi Rob, > > Thanks for your feedback, much appreciated! > > On 2017-11-15 14:02:26 -0600, Rob Herring wrote: >> On Sat, Nov 11, 2017 at 01:38:11AM +0100, Niklas Söderlund wrote: >> > Document the devicetree

Re: [RFCv1 PATCH 1/6] v4l2-ctrls: v4l2_ctrl_add_handler: add from_other_dev

2017-11-17 Thread Mauro Carvalho Chehab
Em Mon, 13 Nov 2017 15:34:03 +0100 Hans Verkuil escreveu: > From: Hans Verkuil > > Add a 'bool from_other_dev' argument: set to true if the two > handlers refer to different devices (e.g. it is true when > inheriting controls from a subdev into a

Re: [PATCH v2 6/8] v4l: vsp1: Adapt entities to configure into a body

2017-11-17 Thread Kieran Bingham
Hi Laurent, On 12/09/17 20:18, Laurent Pinchart wrote: > Hi Kieran, > > On Tuesday, 12 September 2017 00:42:09 EEST Kieran Bingham wrote: >> On 17/08/17 18:58, Laurent Pinchart wrote: >>> On Monday 14 Aug 2017 16:13:29 Kieran Bingham wrote: Currently the entities store their configurations

Re: [RFC v5 10/11] [media] vb2: add out-fence support to QBUF

2017-11-17 Thread Hans Verkuil
On 15/11/17 18:10, Gustavo Padovan wrote: > From: Gustavo Padovan > > If V4L2_BUF_FLAG_OUT_FENCE flag is present on the QBUF call we create > an out_fence and send its fd to userspace on the fence_fd field as a > return arg for the QBUF call. > > The fence is

Re: [RFC v5 06/11] [media] vb2: add explicit fence user API

2017-11-17 Thread Hans Verkuil
On 15/11/17 18:10, Gustavo Padovan wrote: > From: Gustavo Padovan > > Turn the reserved2 field into fence_fd that we will use to send > an in-fence to the kernel and return an out-fence from the kernel to > userspace. > > Two new flags were added,

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 11:08:01 -0200 Gustavo Padovan escreveu: > 2017-11-17 Mauro Carvalho Chehab : > > > Em Fri, 17 Nov 2017 15:49:23 +0900 > > Alexandre Courbot escreveu: > > > > > > @@ -178,6 +179,12 @@ static int

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Mauro Carvalho Chehab : > Em Wed, 15 Nov 2017 15:10:53 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > Receive in-fence from userspace and add support for waiting on them > > before

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Mauro Carvalho Chehab : > Em Fri, 17 Nov 2017 15:49:23 +0900 > Alexandre Courbot escreveu: > > > > @@ -178,6 +179,12 @@ static int vb2_queue_or_prepare_buf(struct > > > vb2_queue *q, struct v4l2_buffer *b, > > > return

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Alexandre Courbot : > Hi Gustavo, > > I am coming a bit late in this series' review, so apologies if some of my > comments have already have been discussed in an earlier revision. > > On Thursday, November 16, 2017 2:10:53 AM JST, Gustavo Padovan wrote: > >

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Mauro Carvalho Chehab
Em Fri, 17 Nov 2017 15:49:23 +0900 Alexandre Courbot escreveu: > > @@ -178,6 +179,12 @@ static int vb2_queue_or_prepare_buf(struct > > vb2_queue *q, struct v4l2_buffer *b, > > return -EINVAL; > > } > > > > + if ((b->fence_fd != 0 && b->fence_fd != -1)

Re: [RFC v5 07/11] [media] vb2: add in-fence support to QBUF

2017-11-17 Thread Mauro Carvalho Chehab
Em Wed, 15 Nov 2017 15:10:53 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > Receive in-fence from userspace and add support for waiting on them > before queueing the buffer to the driver. Buffers can't be queued to the > driver

Re: [RFC v5 03/11] [media] vb2: add 'ordered_in_driver' property to queues

2017-11-17 Thread Gustavo Padovan
Hi Mauro, 2017-11-17 Mauro Carvalho Chehab : > Em Wed, 15 Nov 2017 15:10:49 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > We use ordered_in_driver property to optimize for the case where > >

Re: [RFC v5 06/11] [media] vb2: add explicit fence user API

2017-11-17 Thread Mauro Carvalho Chehab
Em Wed, 15 Nov 2017 15:10:52 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > Turn the reserved2 field into fence_fd that we will use to send > an in-fence to the kernel and return an out-fence from the kernel to > userspace. > >

Re: [RFC v5 01/11] [media] v4l: add V4L2_CAP_ORDERED to the uapi

2017-11-17 Thread Gustavo Padovan
2017-11-17 Mauro Carvalho Chehab : > Em Wed, 15 Nov 2017 15:10:47 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > When using explicit synchronization userspace needs to know if > > the queue can

Re: [RFC v5 03/11] [media] vb2: add 'ordered_in_driver' property to queues

2017-11-17 Thread Mauro Carvalho Chehab
Em Wed, 15 Nov 2017 15:10:49 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > We use ordered_in_driver property to optimize for the case where > the driver can deliver the buffers in an ordered fashion. When it > is ordered we can

Re: [RFC v5 01/11] [media] v4l: add V4L2_CAP_ORDERED to the uapi

2017-11-17 Thread Mauro Carvalho Chehab
Em Wed, 15 Nov 2017 15:10:47 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > When using explicit synchronization userspace needs to know if > the queue can deliver everything back in the same order, so we added > a new capability

Re: [PATCH 6/6] media: usb: add SPDX identifiers to some code I wrote

2017-11-17 Thread Philippe Ombredanne
On Fri, Nov 17, 2017 at 11:21 AM, Mauro Carvalho Chehab wrote: > As we're now using SPDX identifiers, on several > media drivers I wrote, add the proper SPDX, identifying > the license I meant. > > As we're now using the short license, it doesn't make sense to > keep the

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

2017-11-17 Thread Hans Verkuil
On 11/11/17 01:25, Niklas Söderlund wrote: > 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 sources and the video > grabbers (VIN). > > Driver is based on a

Re: [RFC v5 10/11] [media] vb2: add out-fence support to QBUF

2017-11-17 Thread Gustavo Padovan
2017-11-17 Alexandre Courbot : > On Thursday, November 16, 2017 2:10:56 AM JST, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > If V4L2_BUF_FLAG_OUT_FENCE flag is present on the QBUF call we create > > an out_fence and send its fd

Re: [PATCH v7 02/25] rcar-vin: register the video device at probe time

2017-11-17 Thread Hans Verkuil
On 11/11/17 01:38, Niklas Söderlund wrote: > 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, unbound and later rebound. The second time "unbound and later rebound": that's

Re: [PATCH v7 23/25] rcar-vin: extend {start,stop}_streaming to work with media controller

2017-11-17 Thread Hans Verkuil
On 11/11/17 01:38, Niklas Söderlund wrote: > The procedure to start or stop streaming using the non-MC single > subdevice and the MC graph and multiple subdevices are quite different. > Create a new function to abstract which method is used based on which > mode the driver is running in and add

Re: [RFC v5 09/11] [media] vb2: add infrastructure to support out-fences

2017-11-17 Thread Gustavo Padovan
2017-11-17 Alexandre Courbot : > On Thursday, November 16, 2017 2:10:55 AM JST, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Add vb2_setup_out_fence() and the needed members to struct vb2_buffer. > > > > v3: > > - Do not

Re: [PATCH v7 22/25] rcar-vin: add link notify for Gen3

2017-11-17 Thread Hans Verkuil
On 11/11/17 01:38, Niklas Söderlund wrote: > Add the ability to process media device link change request. Link > enabling is a bit complicated on Gen3, whether or not it's possible to > enable a link depends on what other links already are enabled. On Gen3 > the 8 VINs are split into two

Re: [RFC v5 08/11] [media] vb2: add videobuf2 dma-buf fence helpers

2017-11-17 Thread Gustavo Padovan
2017-11-17 Alexandre Courbot : > On Friday, November 17, 2017 4:02:56 PM JST, Alexandre Courbot wrote: > > On Thursday, November 16, 2017 2:10:54 AM JST, Gustavo Padovan wrote: > > > From: Javier Martinez Canillas > > > > > > Add a

Re: [PATCH v7 20/25] rcar-vin: add chsel information to rvin_info

2017-11-17 Thread Hans Verkuil
On 11/11/17 01:38, Niklas Söderlund wrote: > Each Gen3 SoC has a limited set of predefined routing possibilities for > which CSI-2 device and virtual channel can be routed to which VIN > instance. Prepare to store this information in the struct rvin_info. > > Signed-off-by: Niklas Söderlund

Re: [RFC v5 03/11] [media] vb2: add 'ordered_in_driver' property to queues

2017-11-17 Thread Gustavo Padovan
2017-11-17 Alexandre Courbot : > On Thursday, November 16, 2017 2:10:49 AM JST, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > We use ordered_in_driver property to optimize for the case where > > the driver can deliver the buffers

Re: [PATCH v7 14/25] rcar-vin: add function to manipulate Gen3 CHSEL value

2017-11-17 Thread Hans Verkuil
On 11/11/17 01:38, Niklas Söderlund wrote: > On Gen3 the CSI-2 routing is controlled by the VnCSI_IFMD register. One > feature of this register is that it's only present in the VIN0 and VIN4 > instances. The register in VIN0 controls the routing for VIN0-3 and the > register in VIN4 controls

Re: [PATCH/RFC 1/2] v4l: v4l2-dev: Add infrastructure to protect device unplug race

2017-11-17 Thread Hans Verkuil
Hi Laurent, On 16/11/17 01:33, Laurent Pinchart wrote: > Device unplug being asynchronous, it naturally races with operations > performed by userspace through ioctls or other file operations on video > device nodes. > > This leads to potential access to freed memory or to other resources >

[PATCH 1/6] edac: adjust GPLv2 license and SPDX identifiers at the code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several EDAC drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

[PATCH 4/6] media: rc: add SPDX identifiers to the code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

[PATCH 3/6] media: i2c: add SPDX identifiers to the code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

[PATCH 2/6] media: rc keymaps: add SPDX identifiers to the code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several RC keymap files I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to identify GPL v2, as this is the minimal

[PATCH 6/6] media: usb: add SPDX identifiers to some code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by: Mauro

[PATCH 5/6] media: tuners: add SPDX identifiers to the code I wrote

2017-11-17 Thread Mauro Carvalho Chehab
As we're now using SPDX identifiers, on the several media drivers I wrote, add the proper SPDX, identifying the license I meant. As we're now using the short license, it doesn't make sense to keep the original license text. Also, fix MODULE_LICENSE to properly identify GPL v2. Signed-off-by:

Re: [PATCH v1 03/10] v4l: platform: Add Renesas CEU driver

2017-11-17 Thread jacopo mondi
Hi Sakari! On Fri, Nov 17, 2017 at 02:36:51AM +0200, Sakari Ailus wrote: > Hi Jacopo, > > On Wed, Nov 15, 2017 at 03:25:11PM +0100, jacopo mondi wrote: > > Hi Sakari, > >thanks for review! > > You're welcome! > > > On Wed, Nov 15, 2017 at 02:45:51PM +0200, Sakari Ailus wrote: > > > Hi Jacopo,

Re: [PATCH v1 06/10] sh: sh7722: Rename CEU clock

2017-11-17 Thread jacopo mondi
Hi Geert, On Wed, Nov 15, 2017 at 02:13:43PM +0100, Geert Uytterhoeven wrote: > Hi Jacopo, > > On Wed, Nov 15, 2017 at 11:55 AM, Jacopo Mondi > wrote: > > Rename CEU clock to match the new platform driver name used in Migo-R. > > > > There are no other sh7722 based

Re: [PATCH v1 08/10] media: i2c: ov772x: Remove soc_camera dependencies

2017-11-17 Thread jacopo mondi
Hi Sakari! On Fri, Nov 17, 2017 at 02:43:15AM +0200, Sakari Ailus wrote: > Hi Jacopo, > > On Wed, Nov 15, 2017 at 11:56:01AM +0100, Jacopo Mondi wrote: > > [snip] > > +#include > > #include > > #include > > #include > > @@ -25,8 +26,8 @@ > > #include > > > > #include > > -#include >

Re: [linux-sunxi] Cedrus driver

2017-11-17 Thread Maxime Ripard
Hi Nicolas, On Thu, Nov 16, 2017 at 02:59:55PM -0500, Nicolas Dufresne wrote: > Le jeudi 16 novembre 2017 à 12:02 +0100, Maxime Ripard a écrit : > > Assuming that the request API is in, we'd need to: > > - Finish the MPEG4 support > > - Work on more useful codecs (H264 comes to my mind) > >