cron job: media_tree daily build: OK

2018-06-08 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 Jun  9 05:00:42 CEST 2018
media-tree git hash:f2809d20b9250c675fca8268a0f6274277cca7ff
media_build git hash:   464ef972618cc9f845f07c1a4e8957ce2270cf91
v4l-utils git hash: c3b46c2c53d7d815a53c902cfb2ddd96c3732c5b
gcc version:i686-linux-gcc (GCC) 8.1.0
sparse version: 0.5.2
smatch version: 0.5.1
host hardware:  x86_64
host os:4.16.0-1-amd64

linux-git-arm-at91: OK
linux-git-arm-davinci: OK
linux-git-arm-multi: OK
linux-git-arm-pxa: OK
linux-git-arm-stm32: OK
linux-git-arm64: OK
linux-git-i686: OK
linux-git-mips: OK
linux-git-powerpc64: OK
linux-git-sh: OK
linux-git-x86_64: OK
Check COMPILE_TEST: OK
linux-2.6.36.4-i686: OK
linux-2.6.36.4-x86_64: OK
linux-2.6.37.6-i686: OK
linux-2.6.37.6-x86_64: OK
linux-2.6.38.8-i686: OK
linux-2.6.38.8-x86_64: OK
linux-2.6.39.4-i686: OK
linux-2.6.39.4-x86_64: OK
linux-3.0.101-i686: OK
linux-3.0.101-x86_64: OK
linux-3.1.10-i686: OK
linux-3.1.10-x86_64: OK
linux-3.2.101-i686: OK
linux-3.2.101-x86_64: OK
linux-3.3.8-i686: OK
linux-3.3.8-x86_64: OK
linux-3.4.113-i686: OK
linux-3.4.113-x86_64: OK
linux-3.5.7-i686: OK
linux-3.5.7-x86_64: OK
linux-3.6.11-i686: OK
linux-3.6.11-x86_64: OK
linux-3.7.10-i686: OK
linux-3.7.10-x86_64: OK
linux-3.8.13-i686: OK
linux-3.8.13-x86_64: OK
linux-3.9.11-i686: OK
linux-3.9.11-x86_64: OK
linux-3.10.108-i686: OK
linux-3.10.108-x86_64: OK
linux-3.11.10-i686: OK
linux-3.11.10-x86_64: OK
linux-3.12.74-i686: OK
linux-3.12.74-x86_64: OK
linux-3.13.11-i686: OK
linux-3.13.11-x86_64: OK
linux-3.14.79-i686: OK
linux-3.14.79-x86_64: OK
linux-3.15.10-i686: OK
linux-3.15.10-x86_64: OK
linux-3.16.56-i686: OK
linux-3.16.56-x86_64: OK
linux-3.17.8-i686: OK
linux-3.17.8-x86_64: OK
linux-3.18.102-i686: OK
linux-3.18.102-x86_64: OK
linux-3.19.8-i686: OK
linux-3.19.8-x86_64: OK
linux-4.0.9-i686: OK
linux-4.0.9-x86_64: OK
linux-4.1.51-i686: OK
linux-4.1.51-x86_64: OK
linux-4.2.8-i686: OK
linux-4.2.8-x86_64: OK
linux-4.3.6-i686: OK
linux-4.3.6-x86_64: OK
linux-4.4.109-i686: OK
linux-4.4.109-x86_64: OK
linux-4.5.7-i686: OK
linux-4.5.7-x86_64: OK
linux-4.6.7-i686: OK
linux-4.6.7-x86_64: OK
linux-4.7.10-i686: OK
linux-4.7.10-x86_64: OK
linux-4.8.17-i686: OK
linux-4.8.17-x86_64: OK
linux-4.9.91-i686: OK
linux-4.9.91-x86_64: OK
linux-4.10.17-i686: OK
linux-4.10.17-x86_64: OK
linux-4.11.12-i686: OK
linux-4.11.12-x86_64: OK
linux-4.12.14-i686: OK
linux-4.12.14-x86_64: OK
linux-4.13.16-i686: OK
linux-4.13.16-x86_64: OK
linux-4.14.42-i686: OK
linux-4.14.42-x86_64: OK
linux-4.15.14-i686: OK
linux-4.15.14-x86_64: OK
linux-4.16.8-i686: OK
linux-4.16.8-x86_64: OK
linux-4.17-i686: OK
linux-4.17-x86_64: OK
apps: OK
spec-git: OK
sparse: WARNINGS

Detailed results are available here:

http://www.xs4all.nl/~hverkuil/logs/Saturday.log

Full logs are available here:

http://www.xs4all.nl/~hverkuil/logs/Saturday.tar.bz2

The Media Infrastructure API from this daily build is here:

http://www.xs4all.nl/~hverkuil/spec/index.html


[PATCH v2] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Steve Longerbeam
The ADV748x CSI-2 subdevices are HDMI/AFE to MIPI CSI-2 bridges.

Signed-off-by: Steve Longerbeam 
Acked-by: Kieran Bingham 
---
Changes since v1:
- fix typo in commit message.
---
 drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c 
b/drivers/media/i2c/adv748x/adv748x-csi2.c
index 820b44e..469be87 100644
--- a/drivers/media/i2c/adv748x/adv748x-csi2.c
+++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
@@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct 
adv748x_csi2 *tx)
adv748x_csi2_set_virtual_channel(tx, 0);
 
adv748x_subdev_init(>sd, state, _csi2_ops,
-   MEDIA_ENT_F_UNKNOWN,
+   MEDIA_ENT_F_VID_IF_BRIDGE,
is_txa(tx) ? "txa" : "txb");
 
/* Ensure that matching is based upon the endpoint fwnodes */
-- 
2.7.4



Re: [PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Steve Longerbeam




On 06/08/2018 02:39 PM, Kieran Bingham wrote:

Hi Steve,

On 08/06/18 18:43, Steve Longerbeam wrote:

The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.

Just spotted this :D

s/HMDI/HDMI/


Sigh, sending v2.
Steve




Signed-off-by: Steve Longerbeam 
---
  drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c 
b/drivers/media/i2c/adv748x/adv748x-csi2.c
index 820b44e..469be87 100644
--- a/drivers/media/i2c/adv748x/adv748x-csi2.c
+++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
@@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct 
adv748x_csi2 *tx)
adv748x_csi2_set_virtual_channel(tx, 0);
  
  	adv748x_subdev_init(>sd, state, _csi2_ops,

-   MEDIA_ENT_F_UNKNOWN,
+   MEDIA_ENT_F_VID_IF_BRIDGE,
is_txa(tx) ? "txa" : "txb");
  
  	/* Ensure that matching is based upon the endpoint fwnodes */






Re: [PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Steve Longerbeam




On 06/08/2018 02:37 PM, Kieran Bingham wrote:

Hi Steve,

On 08/06/18 22:34, Steve Longerbeam wrote:

Hi Kieran,


On 06/08/2018 02:29 PM, Kieran Bingham wrote:

Hi Steve,

Thankyou for the patch.

On 08/06/18 18:43, Steve Longerbeam wrote:

The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.


Reading the documentation for MEDIA_ENT_F_VID_IF_BRIDGE, this seems reasonable.

Out of interest, have you stumbled across this as part of your other work on
CSI2 drivers - or have you been looking to test the ADV748x with your CSI2
receiver? I'd love to know if the driver works with other (non-renesas)
platforms!

This isn't really related to my other work on the i.MX CSI2 receiver driver
in imx-media. I've only tested this on Renesas (Salvator-X).

No problem. I was just curious :D
And this will get rid of that annoying warning message that I've been ignoring!


Yeah, darn those pesky warnings :)

Steve



Signed-off-by: Steve Longerbeam 

Acked-by: Kieran Bingham 



---
   drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c
b/drivers/media/i2c/adv748x/adv748x-csi2.c
index 820b44e..469be87 100644
--- a/drivers/media/i2c/adv748x/adv748x-csi2.c
+++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
@@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct
adv748x_csi2 *tx)
   adv748x_csi2_set_virtual_channel(tx, 0);
     adv748x_subdev_init(>sd, state, _csi2_ops,
-    MEDIA_ENT_F_UNKNOWN,
+    MEDIA_ENT_F_VID_IF_BRIDGE,
   is_txa(tx) ? "txa" : "txb");
     /* Ensure that matching is based upon the endpoint fwnodes */





Re: [PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Kieran Bingham
Hi Steve,

On 08/06/18 18:43, Steve Longerbeam wrote:
> The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.

Just spotted this :D

s/HMDI/HDMI/

Regards

Kieran

> 
> Signed-off-by: Steve Longerbeam 
> ---
>  drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c 
> b/drivers/media/i2c/adv748x/adv748x-csi2.c
> index 820b44e..469be87 100644
> --- a/drivers/media/i2c/adv748x/adv748x-csi2.c
> +++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
> @@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct 
> adv748x_csi2 *tx)
>   adv748x_csi2_set_virtual_channel(tx, 0);
>  
>   adv748x_subdev_init(>sd, state, _csi2_ops,
> - MEDIA_ENT_F_UNKNOWN,
> + MEDIA_ENT_F_VID_IF_BRIDGE,
>   is_txa(tx) ? "txa" : "txb");
>  
>   /* Ensure that matching is based upon the endpoint fwnodes */
> 



signature.asc
Description: OpenPGP digital signature


Re: [PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Kieran Bingham
Hi Steve,

On 08/06/18 22:34, Steve Longerbeam wrote:
> Hi Kieran,
> 
> 
> On 06/08/2018 02:29 PM, Kieran Bingham wrote:
>> Hi Steve,
>>
>> Thankyou for the patch.
>>
>> On 08/06/18 18:43, Steve Longerbeam wrote:
>>> The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.
>>>
>> Reading the documentation for MEDIA_ENT_F_VID_IF_BRIDGE, this seems 
>> reasonable.
>>
>> Out of interest, have you stumbled across this as part of your other work on
>> CSI2 drivers - or have you been looking to test the ADV748x with your CSI2
>> receiver? I'd love to know if the driver works with other (non-renesas)
>> platforms!
> 
> This isn't really related to my other work on the i.MX CSI2 receiver driver
> in imx-media. I've only tested this on Renesas (Salvator-X).

No problem. I was just curious :D
And this will get rid of that annoying warning message that I've been ignoring!

Regards

Kieran

> 
> Steve
> 
>>> Signed-off-by: Steve Longerbeam 
>> Acked-by: Kieran Bingham 
>>
>>
>>> ---
>>>   drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c
>>> b/drivers/media/i2c/adv748x/adv748x-csi2.c
>>> index 820b44e..469be87 100644
>>> --- a/drivers/media/i2c/adv748x/adv748x-csi2.c
>>> +++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
>>> @@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, 
>>> struct
>>> adv748x_csi2 *tx)
>>>   adv748x_csi2_set_virtual_channel(tx, 0);
>>>     adv748x_subdev_init(>sd, state, _csi2_ops,
>>> -    MEDIA_ENT_F_UNKNOWN,
>>> +    MEDIA_ENT_F_VID_IF_BRIDGE,
>>>   is_txa(tx) ? "txa" : "txb");
>>>     /* Ensure that matching is based upon the endpoint fwnodes */
>>>
> 



signature.asc
Description: OpenPGP digital signature


Re: [PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Steve Longerbeam

Hi Kieran,


On 06/08/2018 02:29 PM, Kieran Bingham wrote:

Hi Steve,

Thankyou for the patch.

On 08/06/18 18:43, Steve Longerbeam wrote:

The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.


Reading the documentation for MEDIA_ENT_F_VID_IF_BRIDGE, this seems reasonable.

Out of interest, have you stumbled across this as part of your other work on
CSI2 drivers - or have you been looking to test the ADV748x with your CSI2
receiver? I'd love to know if the driver works with other (non-renesas) 
platforms!


This isn't really related to my other work on the i.MX CSI2 receiver driver
in imx-media. I've only tested this on Renesas (Salvator-X).

Steve


Signed-off-by: Steve Longerbeam 

Acked-by: Kieran Bingham 



---
  drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c 
b/drivers/media/i2c/adv748x/adv748x-csi2.c
index 820b44e..469be87 100644
--- a/drivers/media/i2c/adv748x/adv748x-csi2.c
+++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
@@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct 
adv748x_csi2 *tx)
adv748x_csi2_set_virtual_channel(tx, 0);
  
  	adv748x_subdev_init(>sd, state, _csi2_ops,

-   MEDIA_ENT_F_UNKNOWN,
+   MEDIA_ENT_F_VID_IF_BRIDGE,
is_txa(tx) ? "txa" : "txb");
  
  	/* Ensure that matching is based upon the endpoint fwnodes */






Re: [PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Kieran Bingham
Hi Steve,

Thankyou for the patch.

On 08/06/18 18:43, Steve Longerbeam wrote:
> The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.
> 

Reading the documentation for MEDIA_ENT_F_VID_IF_BRIDGE, this seems reasonable.

Out of interest, have you stumbled across this as part of your other work on
CSI2 drivers - or have you been looking to test the ADV748x with your CSI2
receiver? I'd love to know if the driver works with other (non-renesas) 
platforms!

> Signed-off-by: Steve Longerbeam 

Acked-by: Kieran Bingham 


> ---
>  drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c 
> b/drivers/media/i2c/adv748x/adv748x-csi2.c
> index 820b44e..469be87 100644
> --- a/drivers/media/i2c/adv748x/adv748x-csi2.c
> +++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
> @@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct 
> adv748x_csi2 *tx)
>   adv748x_csi2_set_virtual_channel(tx, 0);
>  
>   adv748x_subdev_init(>sd, state, _csi2_ops,
> - MEDIA_ENT_F_UNKNOWN,
> + MEDIA_ENT_F_VID_IF_BRIDGE,
>   is_txa(tx) ? "txa" : "txb");
>  
>   /* Ensure that matching is based upon the endpoint fwnodes */
> 



signature.asc
Description: OpenPGP digital signature


[PATCH] media: i2c: adv748x: csi2: set entity function to video interface bridge

2018-06-08 Thread Steve Longerbeam
The ADV748x CSI-2 subdevices are HMDI/AFE to MIPI CSI-2 bridges.

Signed-off-by: Steve Longerbeam 
---
 drivers/media/i2c/adv748x/adv748x-csi2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/adv748x/adv748x-csi2.c 
b/drivers/media/i2c/adv748x/adv748x-csi2.c
index 820b44e..469be87 100644
--- a/drivers/media/i2c/adv748x/adv748x-csi2.c
+++ b/drivers/media/i2c/adv748x/adv748x-csi2.c
@@ -284,7 +284,7 @@ int adv748x_csi2_init(struct adv748x_state *state, struct 
adv748x_csi2 *tx)
adv748x_csi2_set_virtual_channel(tx, 0);
 
adv748x_subdev_init(>sd, state, _csi2_ops,
-   MEDIA_ENT_F_UNKNOWN,
+   MEDIA_ENT_F_VID_IF_BRIDGE,
is_txa(tx) ? "txa" : "txb");
 
/* Ensure that matching is based upon the endpoint fwnodes */
-- 
2.7.4



[GIT PULL FOR v4.19] Convert last soc-camera users, rcar fixes, subdev std support

2018-06-08 Thread Hans Verkuil
Hi Mauro,

This pull requests converts the last users of soc-camera (thanks, Jacopo!),
has a few rcar fixes and adds support for SDTV to v4l2-subdev (HDTV was
supported, but not SDTV).

Regards,

Hans

The following changes since commit f2809d20b9250c675fca8268a0f6274277cca7ff:

  media: omap2: fix compile-testing with FB_OMAP2=m (2018-06-05 09:56:56 -0400)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git for-v4.19b

for you to fetch changes up to eae1c8802533c940e2a6ca55db4b3aa0a3d0759f:

  v4l: Add support for STD ioctls on subdev nodes (2018-06-08 16:38:43 +0200)


Jacopo Mondi (5):
  media: i2c: Copy rj54n1cb0c soc_camera sensor driver
  media: i2c: rj54n1: Remove soc_camera dependencies
  arch: sh: kfr2r09: Use new renesas-ceu camera driver
  arch: sh: ms7724se: Use new renesas-ceu camera driver
  arch: sh: ap325rxa: Use new renesas-ceu camera driver

Niklas Söderlund (6):
  media: dt-bindings: media: rcar_vin: add support for r8a77965
  dt-bindings: media: rcar_vin: fix style for ports and endpoints
  rcar-vin: sync which hardware buffer to start capture from
  media: rcar-vin: enable support for r8a77965
  v4l2-ioctl: create helper to fill in v4l2_standard for ENUMSTD
  v4l: Add support for STD ioctls on subdev nodes

 Documentation/devicetree/bindings/media/rcar_vin.txt |   21 +-
 Documentation/media/uapi/v4l/vidioc-enumstd.rst  |   11 +-
 Documentation/media/uapi/v4l/vidioc-g-std.rst|   14 +-
 Documentation/media/uapi/v4l/vidioc-querystd.rst |   11 +-
 MAINTAINERS  |8 +
 arch/sh/boards/mach-ap325rxa/setup.c |  282 +++--
 arch/sh/boards/mach-kfr2r09/setup.c  |  217 ---
 arch/sh/boards/mach-se/7724/setup.c  |  120 ++--
 arch/sh/kernel/cpu/sh4a/clock-sh7723.c   |2 +-
 drivers/media/i2c/Kconfig|   11 +
 drivers/media/i2c/Makefile   |1 +
 drivers/media/i2c/rj54n1cb0c.c   | 1437 
++
 drivers/media/platform/rcar-vin/rcar-core.c  |   48 ++
 drivers/media/platform/rcar-vin/rcar-dma.c   |   16 +-
 drivers/media/platform/rcar-vin/rcar-vin.h   |2 +
 drivers/media/v4l2-core/v4l2-ioctl.c |   66 ++-
 drivers/media/v4l2-core/v4l2-subdev.c|   22 +
 include/media/v4l2-ioctl.h   |   15 +-
 include/uapi/linux/v4l2-subdev.h |4 +
 19 files changed, 1895 insertions(+), 413 deletions(-)
 create mode 100644 drivers/media/i2c/rj54n1cb0c.c


Re: [PATCH v3 00/20] v4l2 core: push ioctl lock down to ioctl handler

2018-06-08 Thread Ezequiel Garcia
On Fri, 2018-06-08 at 14:11 +0200, Hans Verkuil wrote:
> Hi Ezequiel,
> 
> On 05/24/2018 10:35 PM, Ezequiel Garcia wrote:
> > Third spin of the series posted by Hans:
> > 
> > https://www.mail-archive.com/linux-media@vger.kernel.org/msg131363.html
> 
> Can you rebase this? Several patches have already been merged, so it would
> be nice to have a new clean v4. Can you also move patch 11/20 (dvb-core) to
> after patch 16/20? It makes it a bit easier for me to apply the patches before
> that since the dvb patch needs an Ack from Mauro at the very least.
> 
> But I can take the v4l patches, that should be no problem.
> 
> 

No problem.

Thanks,
Eze


Re: Bug: media device controller node not removed when uvc device is unplugged

2018-06-08 Thread Laurent Pinchart
Hi Torleiv,

On Thursday, 7 June 2018 15:07:24 EEST Torleiv Sundre wrote:
> Hi,
> 
> Every time I plug in a UVC camera, a media controller node is created at
> /dev/media.
> 
> In Ubuntu 17.10, running kernel 4.13.0-43, the media controller device
> node is removed when the UVC camera is unplugged.
> 
> In Ubuntu 18.10, running kernel 4.15.0-22, the media controller device
> node is not removed. For every time I plug the device, a new device node
> with incremented minor number is created, leaving me with a growing list
> of media controller device nodes. If I repeat for long enough, I get the
> following error:
> "media: could not get a free minor"
> I also tried building a kernel from mainline, with the same result.

Thank you for the report. I'm sorry about that :-S It's a known issue, and a 
fix is already present in Linus' tree, on its way to v4.18-rc1.

commit f9ffcb0a21e1fa8e64d09ed613d884e054ae8191
Author: Philipp Zabel 
Date:   Mon May 21 06:24:58 2018 -0400

media: uvcvideo: Fix driver reference counting

It should then get backported to stable releases.

If you have time, could you try to apply that commit on top of mainline and 
see if it fixes the problem for you ?

> I'm running on x86_64.

-- 
Regards,

Laurent Pinchart





Re: [PATCH v3 00/20] v4l2 core: push ioctl lock down to ioctl handler

2018-06-08 Thread Hans Verkuil
Hi Ezequiel,

On 05/24/2018 10:35 PM, Ezequiel Garcia wrote:
> Third spin of the series posted by Hans:
> 
> https://www.mail-archive.com/linux-media@vger.kernel.org/msg131363.html

Can you rebase this? Several patches have already been merged, so it would
be nice to have a new clean v4. Can you also move patch 11/20 (dvb-core) to
after patch 16/20? It makes it a bit easier for me to apply the patches before
that since the dvb patch needs an Ack from Mauro at the very least.

But I can take the v4l patches, that should be no problem.

Regards,

Hans

> 
> Changelog
> -
> 
> v3:
> Reduce changes in patches 6 and 7 for omap3isp and omap4iss
> drivers, as suggested by Hans.
> 
> v2:
> Add the required driver modifications, fixing all
> drivers so they define a proper vb2_queue lock.
> 
> A only exception to this is netup_unidvb. It isn't really obvious
> to me how this driver should lock its vb2_queue. Neither it is
> clear how its vb2_queue is used by the driver in the first place.
> 
> Abylay, perhaps you can take a look at it?
> 
> Why?
> 
> 
> While working on the DMA fence API (aka expliciy sync framework)
> and the Request API it became clear that the core ioctl scheme
> was done at a too-high level.
> 
> Being able to actually look at the struct passed as the ioctl
> argument would help a lot in decide what lock(s) to take.
> 
> This patch series pushes the lock down into v4l2-ioctl.c, after
> video_usercopy() was called.
> 
> This series seems to improve overall quality of drivers:
> in practice, drivers choosing to do their own locking, end up
> introducing races and/or not setting wait_prepare/wait_finish
> despite being possible to do so.
> 
> Patch journal
> -
> 
> The first patch is for the only driver that does not set
> unlocked_ioctl to video_ioctl2: pvrusb2. It actually does
> call it in its own unlocked_ioctl function.
> 
> The second patch pushes the lock down.
> 
> The third patch adds support for mem2mem devices by selecting
> the correct queue lock (capture vs output): this was never
> possible before.
> 
> Patches 4 to 16 add the now mandatory vb2_queue lock and then
> sets wait_prepare and wait_finish hooks.
> 
> Patches 17 to 19 require that queue->lock is always set. This
> means wait_prepare and wait_finish is now unused.
> 
> The last patch removes the now unused wait_prepare and wait_finish.
> 
> This patchset is still based on top of the gspca vb2
> conversion series, hoping it would get merged sooner than this.
> 
> For those wanting to test this (again, on top of gspca vb2 rework),
> there's a branch here:
> 
>   http://git.infradead.org/users/ezequielg/linux gspca-queue-lock-v4.17-rc6
> 
> Ezequiel Garcia (14):
>   usbtv: Implement wait_prepare and wait_finish
>   sta2x11: Add video_device and vb2_queue locks
>   omap4iss: Add vb2_queue lock
>   omap3isp: Add vb2_queue lock
>   mtk-mdp: Add locks for capture and output vb2_queues
>   s5p-g2d: Implement wait_prepare and wait_finish
>   staging: bcm2835-camera: Provide lock for vb2_queue
>   dvb-core: Provide lock for vb2_queue
>   venus: Add video_device and vb2_queue locks
>   davinci_vpfe: Add video_device and vb2_queue locks
>   mx_emmaprp: Implement wait_prepare and wait_finish
>   m2m-deinterlace: Implement wait_prepare and wait_finish
>   stk1160: Set the vb2_queue lock before calling vb2_queue_init
>   media: Remove wait_{prepare, finish}
> 
> Hans Verkuil (6):
>   pvrusb2: replace pvr2_v4l2_ioctl by video_ioctl2
>   v4l2-core: push taking ioctl mutex down to ioctl handler.
>   v4l2-ioctl.c: use correct vb2_queue lock for m2m devices
>   videobuf2-core: require q->lock
>   videobuf2: assume q->lock is always set
>   v4l2-ioctl.c: assume queue->lock is always set
> 
>  Documentation/media/kapi/v4l2-dev.rst  |  7 +-
>  drivers/input/rmi4/rmi_f54.c   |  2 -
>  drivers/input/touchscreen/atmel_mxt_ts.c   |  2 -
>  drivers/input/touchscreen/sur40.c  |  2 -
>  drivers/media/common/videobuf2/videobuf2-core.c| 22 +++---
>  drivers/media/common/videobuf2/videobuf2-v4l2.c| 41 ++-
>  drivers/media/dvb-core/dvb_vb2.c   | 20 +-
>  drivers/media/dvb-frontends/rtl2832_sdr.c  |  2 -
>  drivers/media/pci/cobalt/cobalt-v4l2.c |  2 -
>  drivers/media/pci/cx23885/cx23885-417.c|  2 -
>  drivers/media/pci/cx23885/cx23885-dvb.c|  2 -
>  drivers/media/pci/cx23885/cx23885-vbi.c|  2 -
>  drivers/media/pci/cx23885/cx23885-video.c  |  2 -
>  drivers/media/pci/cx25821/cx25821-video.c  |  2 -
>  drivers/media/pci/cx88/cx88-blackbird.c|  2 -
>  drivers/media/pci/cx88/cx88-dvb.c  |  2 -
>  drivers/media/pci/cx88/cx88-vbi.c  |  2 -
>  drivers/media/pci/cx88/cx88-video.c|  2 -
>  drivers/media/pci/dt3155/dt3155.c  |  2 -
>  drivers/media/pci/intel/ipu3/ipu3-cio2.c   |  2 -
>  

[GIT PULL FOR v4.19] Various fixes

2018-06-08 Thread Hans Verkuil
The following changes since commit f2809d20b9250c675fca8268a0f6274277cca7ff:

  media: omap2: fix compile-testing with FB_OMAP2=m (2018-06-05 09:56:56 -0400)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git for-v4.19a

for you to fetch changes up to 5286c2c46ef67346079b86a700d90faec0756d33:

  media: rcar-vin: Drop unnecessary register properties from example vin port 
(2018-06-08 11:34:27 +0200)


Alexey Khoroshilov (1):
  media: tc358743: release device_node in tc358743_probe_of()

Arnd Bergmann (4):
  media: v4l: cadence: include linux/slab.h
  media: cx231xx: fix RC_CORE dependency
  media: v4l: cadence: add VIDEO_V4L2 dependency
  media: v4l: omap: add VIDEO_V4L2 dependency

Colin Ian King (1):
  media: mtk-vpu: fix spelling mistake: "Prosessor" -> "Processor"

Dmitry Osipenko (1):
  media: dt: bindings: tegra-vde: Document new optional Memory Client reset 
property

Gabriel Fanelli (1):
  staging: media: bcm2048: match alignment with open parenthesis

Geert Uytterhoeven (1):
  v4l: rcar_fdp1: Change platform dependency to ARCH_RENESAS

Jacopo Mondi (1):
  media: renesas-ceu: Add support for YUYV permutations

Janani Sankara Babu (1):
  Staging:media:imx Fix multiple assignments in a line

Nicholas Mc Guire (3):
  media: adv7604: simplify of_node_put()
  media: atmel-isi: drop unnecessary while loop
  media: atmel-isi: move of_node_put() to cover success branch as well

Pavel Machek (1):
  media: i2c: lm3560: add support for lm3559 chip

Simon Horman (1):
  media: rcar-vin: Drop unnecessary register properties from example vin 
port

 Documentation/devicetree/bindings/media/nvidia,tegra-vde.txt | 11 -
 Documentation/devicetree/bindings/media/rcar_vin.txt |  3 --
 drivers/media/i2c/adv7604.c  |  7 +---
 drivers/media/i2c/lm3560.c   |  3 +-
 drivers/media/i2c/tc358743.c |  5 ++-
 drivers/media/platform/Kconfig   |  2 +-
 drivers/media/platform/atmel/atmel-isi.c | 27 ++--
 drivers/media/platform/cadence/Kconfig   |  2 +
 drivers/media/platform/cadence/cdns-csi2rx.c |  1 +
 drivers/media/platform/cadence/cdns-csi2tx.c |  1 +
 drivers/media/platform/mtk-vpu/mtk_vpu.c |  2 +-
 drivers/media/platform/omap/Kconfig  |  1 +
 drivers/media/platform/renesas-ceu.c | 91 
++--
 drivers/media/usb/cx231xx/Kconfig|  2 +-
 drivers/staging/media/bcm2048/radio-bcm2048.c|  2 +-
 drivers/staging/media/imx/imx-media-csi.c|  6 ++-
 include/media/i2c/lm3560.h   |  1 +
 17 files changed, 121 insertions(+), 46 deletions(-)