PFA the diff between v2 and v3 for your convenience
On 12/12/18 11:49 AM, Oleksandr Andrushchenko wrote:
From: Oleksandr Andrushchenko
This is the ABI for the two halves of a para-virtualized
camera driver which extends Xen's reach multimedia capabilities even
farther enabling it for video con
If the operation in rvin_set_channel_routing() is successful the 'ret'
variable contains the runtime PM use count for the VIN master device.
The intention is not to return the use count to the caller but to return
0 on success else none zero.
Fix this by always returning 0 if the operation is succ
To support suspend/resume rvin_{start,stop}_streaming() needs to be
accessible from the suspend and resume callbacks. Up until now the only
users of these functions have been the callbacks in struct vb2_ops so
the arguments to the functions are not suitable for use by the driver it
self.
Fix this
To be able to properly support suspend and resume the VIN and all
subdevices involved in a running capture needs to be stopped before the
system is suspended. Likewise the whole pipeline needs to be started
once the system is resumed if it was running.
Achieve this by using the existing rvin_{star
Hi,
This series add suspend and resume support for rcar-vin pipelines. It
suspends all active pipelines and implicitly adds support for
suspend/resume to rcar-csi2. It is tested on Gen3 and Gen2 and based
on-top of latest media-tree.
Patch 1/4 fixes a bug in the driver which prevents suspend/r
In preparation of suspend/resume support cache the chsel value when we
write it to the register so it can be restored on resume if needed.
Signed-off-by: Niklas Söderlund
---
drivers/media/platform/rcar-vin/rcar-dma.c | 2 ++
drivers/media/platform/rcar-vin/rcar-vin.h | 2 ++
2 files changed, 4
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 Dec 14 05:00:17 CET 2018
media-tree git hash:d2b4387f3bdf016e266d23cf657465f557721488
media_build git
On Thu, Dec 13, 2018 at 11:03 PM Christoph Hellwig wrote:
>
> On Thu, Dec 13, 2018 at 12:13:29PM +0900, Tomasz Figa wrote:
> > Putting aside the problem of memory without struct page, one thing to
> > note here that what is a contiguous DMA range for device X, may not be
> > mappable contiguously
On 12/14/2018 06:24 AM, Laurent Pinchart wrote:
Hello Bingbu,
On Wednesday, 12 December 2018 06:55:53 EET Bingbu Cao wrote:
On 12/11/2018 09:43 PM, Laurent Pinchart wrote:
On Tuesday, 11 December 2018 15:34:49 EET Laurent Pinchart wrote:
On Wednesday, 5 December 2018 02:30:46 EET Mani, Raj
On Fri, Dec 14, 2018 at 6:10 AM wrote:
>
> Hi Chen-Yu,
>
> On Fri, Nov 30, 2018 at 03:58:43PM +0800, Chen-Yu Tsai wrote:
> > The CSI (camera sensor interface) controller found on the H3 (and H5)
> > is a reduced version of the one found on the A31. It only has 1 channel,
> > instead of 4 channels
On Wed, 12 Dec 2018 at 04:09, Eddie James wrote:
>
> The Video Engine (VE) embedded in the Aspeed AST2400 and AST2500 SOCs
> can capture and compress video data from digital or analog sources. With
> the Aspeed chip acting a service processor, the Video Engine can capture
> the host processor grap
Hi Sean,
Is the v5 OK?
Thanks,
Patrick.
Patrick Lerda wrote:
media: add support for RCMM infrared remote controls.
Signed-off-by: Patrick Lerda
---
MAINTAINERS| 5 +
drivers/media/rc/Kconfig | 7 ++
drivers/media/rc/Makefile | 1 +
Hello Bingbu,
On Wednesday, 12 December 2018 06:55:53 EET Bingbu Cao wrote:
> On 12/11/2018 09:43 PM, Laurent Pinchart wrote:
> > On Tuesday, 11 December 2018 15:34:49 EET Laurent Pinchart wrote:
> >> On Wednesday, 5 December 2018 02:30:46 EET Mani, Rajmohan wrote:
> >>
> >> [snip]
> >>
> >>> I
Hi Chen-Yu,
On Fri, Nov 30, 2018 at 03:58:43PM +0800, Chen-Yu Tsai wrote:
> The CSI (camera sensor interface) controller found on the H3 (and H5)
> is a reduced version of the one found on the A31. It only has 1 channel,
> instead of 4 channels supporting time-multiplexed BT.656 on the A31.
> Sinc
Hi Fabrizio,
Thank you for the patch.
On Thursday, 13 December 2018 22:20:24 EET Fabrizio Castro wrote:
> Document RZ/G1 and RZ/G2 support.
>
> Signed-off-by: Fabrizio Castro
Reviewed-by: Laurent Pinchart
And applied to my tree.
> ---
> Documentation/devicetree/bindings/media/renesas,vsp1.
Hi Fabrizio,
Thank you for the patch.
On Thursday, 13 December 2018 22:20:33 EET Fabrizio Castro wrote:
> Document RZ/G2 support.
>
> Signed-off-by: Fabrizio Castro
Reviewed-by: Laurent Pinchart
And applied to my tree.
> ---
> Documentation/devicetree/bindings/media/renesas,fcp.txt | 5 +++
The pull request you sent on Wed, 12 Dec 2018 13:54:03 -0200:
> git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media
> tags/media/v4.20-5
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/14a996c383129525e55bab07e4857d08f6b61dda
Thank you!
--
Deet-doot-do
On Thu, Dec 13, 2018 at 10:54:28PM +0200, sakari.ai...@iki.fi wrote:
> Hi Maxime,
>
> One more small note.
>
> On Fri, Dec 07, 2018 at 02:55:30PM +0100, Maxime Ripard wrote:
>
> ...
>
> > + /**
> > +* @wakeup:
> > +*
> > +* Time, in picoseconds, that a transmitter drives a Mark-1
Hi Maxime,
One more small note.
On Fri, Dec 07, 2018 at 02:55:30PM +0100, Maxime Ripard wrote:
...
> + /**
> + * @wakeup:
> + *
> + * Time, in picoseconds, that a transmitter drives a Mark-1
> + * state prior to a Stop state in order to initiate an exit
> + * from U
Hi Maxime,
On Fri, Dec 07, 2018 at 02:55:30PM +0100, Maxime Ripard wrote:
> Now that we have some infrastructure for it, allow the MIPI D-PHY phy's to
> be configured through the generic functions through a custom structure
> added to the generic union.
>
> The parameters added here are the ones
Hi Maxime,
On Fri, Dec 07, 2018 at 02:55:29PM +0100, Maxime Ripard wrote:
> The phy framework is only allowing to configure the power state of the PHY
> using the init and power_on hooks, and their power_off and exit
> counterparts.
>
> While it works for most, simple, PHYs supported so far, some
Some little style fixup work.
Signed-off-by: Sakari Ailus
---
drivers/media/i2c/ov9640.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/drivers/media/i2c/ov9640.c b/drivers/media/i2c/ov9640.c
index c183273fd332..295dcc5992c9 100644
--- a/drivers/media/i2c/ov9640.
According to the RZ/G2 User's manual, RZ/G2E and R-Car E3 CSI-2
blocks are identical, therefore use R-Car E3 definitions to add
RZ/G2E support.
Signed-off-by: Fabrizio Castro
---
drivers/media/platform/rcar-vin/rcar-csi2.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/media/pla
Add the compatible string for RZ/G2E (a.k.a. R8A774C0) to the list
of SoCs supported by rcar-vin driver.
Signed-off-by: Fabrizio Castro
---
Documentation/devicetree/bindings/media/rcar_vin.txt | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/b
Document RZ/G1 and RZ/G2 support.
Signed-off-by: Fabrizio Castro
---
Documentation/devicetree/bindings/media/renesas,vsp1.txt | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/renesas,vsp1.txt
b/Documentation/devicetree/bindings/me
According to the RZ/G2 User's manual, RZ/G2E and R-Car E3 VIN
blocks are identical, therefore use R-Car E3 definitions to add
RZ/G2E support.
Signed-off-by: Fabrizio Castro
---
drivers/media/platform/rcar-vin/rcar-core.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/media/platf
Add the compatible string for RZ/G2E (a.k.a. R8A774C0) to the
list of supported SoCs.
Signed-off-by: Fabrizio Castro
---
Documentation/devicetree/bindings/media/renesas,rcar-csi2.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/media/ren
Document RZ/G2 support.
Signed-off-by: Fabrizio Castro
---
Documentation/devicetree/bindings/media/renesas,fcp.txt | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.txt
b/Documentation/devicetree/bindings/media/renesas,f
Em Thu, 13 Dec 2018 18:35:15 +0100
Hans Verkuil escreveu:
> On 12/13/18 6:14 PM, Mauro Carvalho Chehab wrote:
> > Em Thu, 13 Dec 2018 14:41:11 +0100
> > hverkuil-ci...@xs4all.nl escreveu:
> >
> >> From: Hans Verkuil
> >>
> >> Add support for properties. In this initial implementation properti
Em Thu, 13 Dec 2018 18:13:03 +0100
Hans Verkuil escreveu:
> On 12/13/18 5:41 PM, Mauro Carvalho Chehab wrote:
> > Em Thu, 13 Dec 2018 14:41:10 +0100
> > hverkuil-ci...@xs4all.nl escreveu:
> >
> >> From: Hans Verkuil
> >>
> >> Extend the topology struct with a properties array.
> >>
> >> Add a
On 12/13/18 6:14 PM, Mauro Carvalho Chehab wrote:
> Em Thu, 13 Dec 2018 14:41:11 +0100
> hverkuil-ci...@xs4all.nl escreveu:
>
>> From: Hans Verkuil
>>
>> Add support for properties. In this initial implementation properties
>> can be added to entities and pads. In addition, properties can be
>> n
Em Thu, 13 Dec 2018 14:41:13 +0100
hverkuil-ci...@xs4all.nl escreveu:
> From: Hans Verkuil
>
> Add properties to entities and pads to be able to test the
> properties API.
Looks OK to me.
>
> Signed-off-by: Hans Verkuil
> ---
> drivers/media/platform/vimc/vimc-common.c | 50
Em Thu, 13 Dec 2018 14:41:12 +0100
hverkuil-ci...@xs4all.nl escreveu:
> From: Hans Verkuil
>
> Add functions to add properties to entities, pads and other
> properties. This can be extended to include interfaces and links
> in the future when needed.
>
> Signed-off-by: Hans Verkuil
> ---
> dr
Em Thu, 13 Dec 2018 14:41:13 -0200
Mauro Carvalho Chehab escreveu:
> Em Thu, 13 Dec 2018 14:41:10 +0100
> hverkuil-ci...@xs4all.nl escreveu:
>
> > From: Hans Verkuil
> >
> > Extend the topology struct with a properties array.
> >
> > Add a new media_v2_prop structure to store property informa
Em Thu, 13 Dec 2018 14:41:11 +0100
hverkuil-ci...@xs4all.nl escreveu:
> From: Hans Verkuil
>
> Add support for properties. In this initial implementation properties
> can be added to entities and pads. In addition, properties can be
> nested.
>
> Most of the changes are straightforward, but I h
On 12/13/18 5:41 PM, Mauro Carvalho Chehab wrote:
> Em Thu, 13 Dec 2018 14:41:10 +0100
> hverkuil-ci...@xs4all.nl escreveu:
>
>> From: Hans Verkuil
>>
>> Extend the topology struct with a properties array.
>>
>> Add a new media_v2_prop structure to store property information.
>>
>> Signed-off-by:
Em Thu, 13 Dec 2018 14:41:10 +0100
hverkuil-ci...@xs4all.nl escreveu:
> From: Hans Verkuil
>
> Extend the topology struct with a properties array.
>
> Add a new media_v2_prop structure to store property information.
>
> Signed-off-by: Hans Verkuil
> ---
> include/uapi/linux/media.h | 56
From: Petr Cvek
This patch removes the dependency on an obsoleted soc_camera from ov9640
driver and changes the code to be a standalone v4l2 async subdevice.
It also adds GPIO allocations for power and reset signals (as they are not
handled by soc_camera now).
The values for waiting on GPIOs (re
From: Petr Cvek
Control handlers registration lacked error path with
v4l2_ctrl_handler_free() call. Fix it by using goto to alread existing
v4l2_ctrl_handler_free() call.
Signed-off-by: Petr Cvek
---
drivers/media/i2c/ov9640.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
From: Petr Cvek
The driver uses variables to store frame resolutions and to indexing
various arrays. These should be unsigned.
Signed-off-by: Petr Cvek
---
drivers/media/i2c/ov9640.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/media/i2c/ov9640.c b
From: Petr Cvek
Initial part of ov9640 transition from soc_camera subsystem to a standalone
v4l2 subdevice. The soc_camera version seems to be used only in Palm Zire72
and in (the future) HTC Magician. On these two devices the support is
broken as pxa_camera driver doesn't use soc_camera anymore.
From: Petr Cvek
Add missing SPDX identifiers to .c and .h files of the sensor driver.
Signed-off-by: Petr Cvek
---
drivers/media/i2c/ov9640.c | 5 +
drivers/media/i2c/ov9640.h | 5 +
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/media/i2c/ov9640.c b/drivers/med
From: Petr Cvek
Some returns were adjoined to unrelated code blocks. This patch adds
a space inbetween.
Signed-off-by: Petr Cvek
---
drivers/media/i2c/ov9640.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/media/i2c/ov9640.c b/drivers/media/i2c/ov9640.c
index 08f3f8247759..2
From: Petr Cvek
This patch series transfer the ov9640 driver from the soc_camera subsystem
into a standalone v4l2 driver. There is no changes except the required
v4l2_async calls, GPIO allocation, deletion of now unused variables,
a change from mdelay() to msleep() and an addition of SPDX identif
From: Petr Cvek
An array which defines sensor's supported formats is not written anywhere,
so it can be constant.
Signed-off-by: Petr Cvek
---
drivers/media/i2c/ov9640.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/i2c/ov9640.c b/drivers/media/i2c/ov9640.c
From: Petr Cvek
The soc_camera drivers are marked as orphaned. Add Petr Cvek as a new
maintainer for ov9640 driver after its switch from the soc_camera.
Signed-off-by: Petr Cvek
---
MAINTAINERS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index a2e686819e
On Thu, Dec 13, 2018 at 12:13:29PM +0900, Tomasz Figa wrote:
> Putting aside the problem of memory without struct page, one thing to
> note here that what is a contiguous DMA range for device X, may not be
> mappable contiguously for device Y and it would still need something
> like a scatter list
From: Hans Verkuil
Add functions to add properties to entities, pads and other
properties. This can be extended to include interfaces and links
in the future when needed.
Signed-off-by: Hans Verkuil
---
drivers/media/media-entity.c | 64 +
include/media/media-entity.h | 264 ++
From: Hans Verkuil
The main changes since RFCv4 are:
- Dropped all the indexing code that I added to make it easier to
traverse the topology. I still think that's a good idea, but that
can be done in a future patch.
- Split the second patch into two: the first part adds the core support,
t
From: Hans Verkuil
Add support for properties. In this initial implementation properties
can be added to entities and pads. In addition, properties can be
nested.
Most of the changes are straightforward, but I had to make some changes
to the way entities are initialized, since the struct has to
From: Hans Verkuil
Add properties to entities and pads to be able to test the
properties API.
Signed-off-by: Hans Verkuil
---
drivers/media/platform/vimc/vimc-common.c | 50 +++
1 file changed, 50 insertions(+)
diff --git a/drivers/media/platform/vimc/vimc-common.c
b/driv
From: Hans Verkuil
Extend the topology struct with a properties array.
Add a new media_v2_prop structure to store property information.
Signed-off-by: Hans Verkuil
---
include/uapi/linux/media.h | 56 ++
1 file changed, 56 insertions(+)
diff --git a/includ
My @samusung.com address is going to cease existing soon, so change it to
an address which can actually be used to contact me.
Adding Sylwester Nawrocki, who still has access to a wide spectrum
of Exynos-based hardware.
Signed-off-by: Andrzej Pietrasiewicz
---
MAINTAINERS | 3 ++-
1 file change
My @samusung.com address is going to cease existing soon, so change it to
an address which can actually be used to contact me.
Signed-off-by: Andrzej Pietrasiewicz
---
drivers/media/common/videobuf2/videobuf2-dma-sg.c | 2 +-
drivers/media/platform/s5p-jpeg/jpeg-core.c | 4 ++--
drivers/me
Hi Sakari,
On Thursday, 13 December 2018 15:02:03 EET Sakari Ailus wrote:
> On Thu, Dec 13, 2018 at 02:59:50PM +0200, Laurent Pinchart wrote:
> > On Thursday, 13 December 2018 12:40:06 EET Sakari Ailus wrote:
> > > The size argument of the alloc memop, which allocates buffer memory, is
> > > page
On Thu, Dec 13, 2018 at 03:03:47PM +0200, Laurent Pinchart wrote:
> On Thursday, 13 December 2018 15:00:23 EET Sakari Ailus wrote:
> > Hi Laurent,
> >
> > On Thu, Dec 13, 2018 at 02:57:46PM +0200, Laurent Pinchart wrote:
> > > Hi Sakari,
> > >
> > > Thank you for the patch.
> > >
> > > On Thursd
On Thu, Dec 13, 2018 at 02:49:44PM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> On Thursday, 13 December 2018 12:40:04 EET Sakari Ailus wrote:
> > PAGE_ALIGN() may wrap the buffer size around to 0. Prevent this by
> > checking that the aligned value is not smaller
On Thursday, 13 December 2018 15:00:23 EET Sakari Ailus wrote:
> Hi Laurent,
>
> On Thu, Dec 13, 2018 at 02:57:46PM +0200, Laurent Pinchart wrote:
> > Hi Sakari,
> >
> > Thank you for the patch.
> >
> > On Thursday, 13 December 2018 12:40:05 EET Sakari Ailus wrote:
> > > buf->size is an unsigned
On Thu, Dec 13, 2018 at 02:59:50PM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> On Thursday, 13 December 2018 12:40:06 EET Sakari Ailus wrote:
> > The size argument of the alloc memop, which allocates buffer memory, is
> > page aligned. Document it as such, as code
Hi Laurent,
On Thu, Dec 13, 2018 at 02:57:46PM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> On Thursday, 13 December 2018 12:40:05 EET Sakari Ailus wrote:
> > buf->size is an unsigned long; casting that to int will lead to an
> > overflow if buf->size exceeds INT_
Hi Steve,
On Tue, 2018-10-16 at 17:00 -0700, Steve Longerbeam wrote:
> The function ipu_csi_init_interface() was inverting the F-bit for
> NTSC case, in the CCIR_CODE_1/2 registers. The result being that
> for NTSC bottom-top field order, the CSI would swap fields and
> capture in top-bottom order
Hi Sakari,
Thank you for the patch.
On Thursday, 13 December 2018 12:40:06 EET Sakari Ailus wrote:
> The size argument of the alloc memop, which allocates buffer memory, is
> page aligned. Document it as such, as code elsewhere has not taken this
> into account.
>
> Signed-off-by: Sakari Ailus
Hi Sakari,
Thank you for the patch.
On Thursday, 13 December 2018 12:40:05 EET Sakari Ailus wrote:
> buf->size is an unsigned long; casting that to int will lead to an
> overflow if buf->size exceeds INT_MAX.
>
> Fix this by changing the type to unsigned long instead. This is possible
> as the b
On 12/12/18 9:43 AM, Hans Verkuil wrote:
> On 12/12/18 9:18 AM, Mauro Carvalho Chehab wrote:
>>> /* ioctls */
>>> @@ -368,6 +446,8 @@ struct media_v2_topology {
>>> #define MEDIA_IOC_ENUM_ENTITIES_IOWR('|', 0x01, struct
>>> media_entity_desc)
>>> #define MEDIA_IOC_ENUM_LINKS _IOWR('|'
Hi Sakari,
Thank you for the patch.
On Thursday, 13 December 2018 12:40:04 EET Sakari Ailus wrote:
> PAGE_ALIGN() may wrap the buffer size around to 0. Prevent this by
> checking that the aligned value is not smaller than the unaligned one.
>
> Note on backporting to stable: the file used to be
commit c0decac19da3906d9b66291e57b7759489e1170f
Author: Mauro Carvalho Chehab
Date: Mon Sep 10 08:19:14 2018 -0400
media: use strscpy() instead of strlcpy()
The implementation of strscpy() is more robust and safer.
That's now the recommended way to copy NUL terminated str
Hi Sakari,
On Thursday, 13 December 2018 13:45:43 EET Sakari Ailus wrote:
> From: Rajmohan Mani
>
> This patch adds the details about the IPU3 Imaging Unit driver (both CIO2
> and IMGU).
>
> Signed-off-by: Rajmohan Mani
> Signed-off-by: Sakari Ailus
> ---
> Hi Laurent,
>
> Here's the origina
On 12/12/18 7:28 PM, Jonas Karlman wrote:
> Hi Hans,
>
> Since this function only return DEQUEUED and DONE buffers,
> it cannot be used to find a capture buffer that is both used for
> frame output and is part of the frame reference list.
> E.g. a bottom field referencing a top field that is alrea
Em Thu, 13 Dec 2018 14:03:40 +0200
sakari.ai...@iki.fi escreveu:
> Hi Mauro,
>
> Here's the ipu3 staging driver plus the META_OUTPUT buffer type needed to
> pass the parameters for the device. If you think this there's still time to
> get this to 4.21, then please pull. The non-staging patches ha
Hi Bingbu,
On Thu, Dec 13, 2018 at 01:35:08PM +0800, Bingbu Cao wrote:
>
>
> On 12/12/2018 07:49 PM, Sakari Ailus wrote:
> > Address the following warnings by setting the entity's function to an
> > appropriate value.
> >
> > [5.043377] ipu3-cio2 :00:14.3: Entity type for entity ipu3-cs
Hi Mauro,
Here's the ipu3 staging driver plus the META_OUTPUT buffer type needed to
pass the parameters for the device. If you think this there's still time to
get this to 4.21, then please pull. The non-staging patches have been
around for more than half a year and they're relatively simple.
Not
From: Rajmohan Mani
This patch adds the details about the IPU3 Imaging Unit driver (both CIO2
and IMGU).
Signed-off-by: Rajmohan Mani
Signed-off-by: Sakari Ailus
---
Hi Laurent,
Here's the original documentation patch with my changes folded into it.
The metadata format patch is needed before
On 12/13/18 11:40 AM, Sakari Ailus wrote:
> The size argument of the alloc memop, which allocates buffer memory, is
> page aligned. Document it as such, as code elsewhere has not taken this
> into account.
>
> Signed-off-by: Sakari Ailus
Reviewed-by: Hans Verkuil
Regards,
Hans
> ---
On 12/13/18 11:40 AM, Sakari Ailus wrote:
> buf->size is an unsigned long; casting that to int will lead to an
> overflow if buf->size exceeds INT_MAX.
>
> Fix this by changing the type to unsigned long instead. This is possible
> as the buf->size is always aligned to PAGE_SIZE, and therefore the
On 12/13/18 11:40 AM, Sakari Ailus wrote:
> PAGE_ALIGN() may wrap the buffer size around to 0. Prevent this by
> checking that the aligned value is not smaller than the unaligned one.
>
> Note on backporting to stable: the file used to be under
> drivers/media/v4l2-core, it was moved to the curren
Hi Mauro,
Here's the ipu3 staging driver plus the META_OUTPUT buffer type needed to
pass the parameters for the device. If you think this there's still time to
get this to 4.21, then please pull. The non-staging patches have been
around for more than half a year and they're relatively simple.
Not
Hi Mauro,
Here's the ipu3 staging driver plus the META_OUTPUT buffer type needed to
pass the parameters for the device. If you think this there's still time to
get this to 4.21, then please pull. The non-staging patches have been
around for more than half a year and they're relatively simple.
Not
Hi Laurent,
On Thu, Dec 13, 2018 at 01:01:55PM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> Thank you for the patch.
>
> On Thursday, 13 December 2018 11:51:07 EET Sakari Ailus wrote:
> > Add a MAINTAINERS entry for the ImgU driver.
> >
> > Signed-off-by: Sakari Ailus
> > ---
> > MAINTAINE
Hi Sakari,
Thank you for the patch.
On Thursday, 13 December 2018 11:51:07 EET Sakari Ailus wrote:
> Add a MAINTAINERS entry for the ImgU driver.
>
> Signed-off-by: Sakari Ailus
> ---
> MAINTAINERS | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> in
Address comments on the documentation after Yong's original patch.
Signed-off-by: Sakari Ailus
---
.../media/uapi/v4l/pixfmt-meta-intel-ipu3.rst | 2 +-
Documentation/media/v4l-drivers/ipu3.rst | 343 -
drivers/staging/media/ipu3/TODO| 11
Hi Laurent,
Thanks for the review.
On Thu, Dec 13, 2018 at 12:41:36PM +0200, Laurent Pinchart wrote:
> Hi Sakari,
>
> On Thursday, 13 December 2018 11:38:26 EET Sakari Ailus wrote:
> > Hi Laurent,
> >
> > I'm sending a separate patch to address the comments.
> >
> > On Fri, Nov 30, 2018 at 12:
Hi Sakari,
On Thursday, 13 December 2018 11:38:26 EET Sakari Ailus wrote:
> Hi Laurent,
>
> I'm sending a separate patch to address the comments.
>
> On Fri, Nov 30, 2018 at 12:50:36AM +0200, Laurent Pinchart wrote:
> > On Tuesday, 30 October 2018 00:22:56 EET Yong Zhi wrote:
> >> From: Rajmohan
The size argument of the alloc memop, which allocates buffer memory, is
page aligned. Document it as such, as code elsewhere has not taken this
into account.
Signed-off-by: Sakari Ailus
---
include/media/videobuf2-core.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/inclu
Hi folks,
Most drivers already have limits to size such that you don't hit these,
but if you do, then mayhem will follow. The two first are cc'd to stable.
Sakari Ailus (3):
videobuf2-core: Prevent size alignment wrapping buffer size to 0
videobuf2-dma-sg: Prevent size from overflowing
vide
PAGE_ALIGN() may wrap the buffer size around to 0. Prevent this by
checking that the aligned value is not smaller than the unaligned one.
Note on backporting to stable: the file used to be under
drivers/media/v4l2-core, it was moved to the current location after 4.14.
Signed-off-by: Sakari Ailus
buf->size is an unsigned long; casting that to int will lead to an
overflow if buf->size exceeds INT_MAX.
Fix this by changing the type to unsigned long instead. This is possible
as the buf->size is always aligned to PAGE_SIZE, and therefore the size
will never have values lesser than 0.
Note on
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: Thu Dec 13 10:12:56 CET 2018
media-tree git hash:d2b4387f3bdf016e266d23cf657465f557721488
media_build git
From: Yong Zhi
These meta formats are used on Intel IPU3 ImgU video queues
to carry 3A statistics and ISP pipeline parameters.
V4L2_META_FMT_IPU3_3A
V4L2_META_FMT_IPU3_PARAMS
Signed-off-by: Yong Zhi
Signed-off-by: Chao C Li
Signed-off-by: Rajmohan Mani
Signed-off-by: Sakari Ailus
---
drive
From: Yong Zhi
Introduce functions to load and install ImgU FW blobs.
Signed-off-by: Yong Zhi
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-css-fw.c | 264 +++
drivers/staging/media/ipu3/ipu3-css-fw.h | 188 ++
2 files changed,
Address a few false positive compiler warnings related to uninitialised
variables. While at it, use bool where bool is needed and %u to print an
unsigned integer.
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
From: Yong Zhi
Add IPU3-specific meta formats for processing parameters and
3A statistics.
V4L2_META_FMT_IPU3_PARAMS
V4L2_META_FMT_IPU3_STAT_3A
Signed-off-by: Yong Zhi
Signed-off-by: Sakari Ailus
---
Documentation/media/uapi/v4l/meta-formats.rst | 1 +
.../media/uapi/v4l/pixfmt-me
From: Yong Zhi
Implement video driver that utilizes v4l2, vb2 queue support
and media controller APIs. The driver exposes single subdevice and
six nodes.
Signed-off-by: Yong Zhi
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-v4l2.c | 1086
1 f
Hi folks,
This is effectively Yong's v8 patchset, with a few changes since:
- Addressed most of Laurent's comments on the driver documentation. Some
have been postponed and added to TODO.
- Added a MAINTAINERS entry.
- Removed uAPI definitions (formats etc.) added by the patches originally
Document the interface for metadata output, including
V4L2_BUF_TYPE_META_OUTPUT buffer type and V4L2_CAP_META_OUTPUT capability
bits.
Signed-off-by: Sakari Ailus
Acked-by: Hans Verkuil
Reviewed-by: Tomasz Figa
Tested-by: Tian Shu Qiu
---
Documentation/media/uapi/v4l/buffer.rst | 3 +
From: Yong Zhi
This patch implements the functions to initialize
and configure IPU3 h/w such as clock, irq and power.
Signed-off-by: Yong Zhi
Signed-off-by: Tomasz Figa
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-css.c | 537 ++
drivers/sta
From: Yong Zhi
A collection of routines that are mainly used
to calculate the parameters for accelerator cluster.
Signed-off-by: Yong Zhi
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-css-params.c | 2915 ++
drivers/staging/media/ipu3/ipu3-css-params.
Add a MAINTAINERS entry for the ImgU driver.
Signed-off-by: Sakari Ailus
---
MAINTAINERS | 8
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 3e9f1710ed13e..9ed5cff35e075 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7587,6 +7587,14 @@ S: Maintained
The firmware binary is located under "intel" directory in the
linux-firmware repository. Reflect this in the driver.
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-css-fw.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/ipu3/ipu3-css-fw
From: Yong Zhi
Add macros and enums used for IPU3 firmware interface.
Signed-off-by: Yong Zhi
Signed-off-by: Rajmohan Mani
Reviewed-by: Laurent Pinchart
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-abi.h | 661 ++
1 file changed, 661 insert
From: Yong Zhi
This provides helper library to be used by v4l2 level to program
imaging pipelines and control the streaming.
Signed-off-by: Yong Zhi
Signed-off-by: Sakari Ailus
---
drivers/staging/media/ipu3/ipu3-css.c | 1740 +
1 file changed, 1740 insertions(
1 - 100 of 117 matches
Mail list logo
