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
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
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
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
> ---
>
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
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'
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
> > >
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
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
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
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
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
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:
> >
> >>
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
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
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
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
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
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,
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
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
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
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
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
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
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
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
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
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
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
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
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
> >
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
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
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
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
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
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
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
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!
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
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
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
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
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
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
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
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
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
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,
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
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
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
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:
> >
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)
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
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
> >
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.
>
>
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
>
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:
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:
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:
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
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
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:
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,
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
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
>
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)
>
>
83 matches
Mail list logo