cron job: media_tree daily build: ERRORS

2016-06-09 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Fri Jun 10 04:00:31 CEST 2016 git branch: test git hash: cc650b65bea5613f04a0523c3ee2b91df371e175 gcc versi

Re: [PATCH 2/2] [media] media-device: dynamically allocate struct media_devnode

2016-06-09 Thread Sakari Ailus
Hi Mauro, On Tue, Jun 07, 2016 at 10:11:33AM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 6 Jun 2016 17:13:12 -0600 > Shuah Khan escreveu: > > > > A few general comments on the patch --- I agree we've had the problem from > > > the day one, but it's really started showing up recently. I agree w

Re: [PATCH] [media]: Driver for Toshiba et8ek8 5MP sensor

2016-06-09 Thread Sakari Ailus
Hi Ivaylo, On Sat, Jun 04, 2016 at 10:54:58PM +0300, Ivaylo Dimitrov wrote: > Hi, > > On 26.05.2016 00:45, Sakari Ailus wrote: > >Hi Ivaylo, > > > >I've got some comments here but I haven't reviewed everything yet. What's > >missing is > > > >- the user space interface for selecting the sensor co

Re: [PATCHv2] device tree description for AD5820 camera auto-focus coil

2016-06-09 Thread Sakari Ailus
On Tue, Jun 07, 2016 at 09:10:04AM +0200, Pavel Machek wrote: > > Add documentation for ad5820 device tree binding. > > Signed-off-by: Pavel Machek > Acked-by: Sakari Ailus > Acked-by: Rob Herring Thanks! I've replaced the patch in my tree, will send a pull req later. -- Cheers, Sakari Ai

Re: Kernel 4.7.0-rc2 warnings with Facetime HD camera on Macbook Pro 8,1

2016-06-09 Thread Sakari Ailus
HI Augusto, On Wed, Jun 08, 2016 at 02:00:46AM -0300, Augusto Mecking Caringi wrote: > Hi, > > I have a Macbook Pro 8,1 running Linux (Fedora) and until now I > was running the kernel provided by the distro. Today I decided to > compile a new kernel from Linus HEAD > (3613a6245b9fb5091724961e

[PATCH] [media] s5p-mfc: use vb2_is_streaming() to check vb2 queue status

2016-06-09 Thread Javier Martinez Canillas
The streaming field in struct vb2_queue is meant to be private and should not be used by drivers directly, instead the vb2_is_streaming() function should be used to check the videobuf2 queue streaming status. Signed-off-by: Javier Martinez Canillas --- drivers/media/platform/s5p-mfc/s5p_mfc_dec

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Antti Palosaari
On 06/09/2016 10:35 PM, Mauro Carvalho Chehab wrote: The above code is racy, as some other request to the frontend may arrive between the if() statement and kfree(). A kref would likely be safer. Here is working proof-of-concept hack. It is for si2157 tuner module. It prevents module unbind an

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Mauro Carvalho Chehab
Antti, Em Thu, 09 Jun 2016 22:14:12 +0300 Antti Palosaari escreveu: > On 06/09/2016 09:30 PM, Mauro Carvalho Chehab wrote: > > Em Thu, 9 Jun 2016 19:38:04 +0300 > > Antti Palosaari escreveu: > > >>> The V4L2 core handles everything that it is needed for it to work, and > >>> no extra code is

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Antti Palosaari
On 06/09/2016 09:30 PM, Mauro Carvalho Chehab wrote: Em Thu, 9 Jun 2016 19:38:04 +0300 Antti Palosaari escreveu: The V4L2 core handles everything that it is needed for it to work, and no extra code is needed to do module_put() or i2c_unregister_device(). That example attachs 2 I2C drivers,

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Mauro Carvalho Chehab
Em Thu, 9 Jun 2016 19:38:04 +0300 Antti Palosaari escreveu: > On 06/09/2016 07:18 PM, Mauro Carvalho Chehab wrote: > > Em Thu, 09 Jun 2016 18:41:10 +0300 > > Antti Palosaari escreveu: > > > >> On 06/09/2016 06:24 PM, Mauro Carvalho Chehab wrote: > >>> Hi Akihiro, > >>> > >>> Em Thu, 09 Jun 2

[PATCH] [media] v4l: vsp1: Fix format-info documentation

2016-06-09 Thread Kieran Bingham
Minor tweaks to document the swap register and make the documentation match the struct ordering Signed-off-by: Kieran Bingham --- drivers/media/platform/vsp1/vsp1_pipe.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/vsp1/vsp1_pipe.h b/drivers/me

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Akihiro TSUKADA
thanks for the replys. so, the new i2c "attach" helper code should 1. be usable for V4L2-DVB multi-functional(hybrid) tuner devices (or at least co-exist well with them), 2. properly ref-count driver module to prevent (accidental) 'unload before use' by users. 3. a. block un-binding of the d

[PATCH RFC 2/2] MAINTAINERS: Add support for FDP driver

2016-06-09 Thread Kieran Bingham
Signed-off-by: Kieran Bingham --- MAINTAINERS | 9 + 1 file changed, 9 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 66de4da2d244..bc083b58e478 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7312,6 +7312,15 @@ S: Supported F: Documentation/devicetree/bindings/medi

[PATCH RFC 1/2] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-06-09 Thread Kieran Bingham
The FDP1 driver performs advanced de-interlacing on a memory 2 memory based video stream, and supports conversion from YCbCr/YUV to RGB pixel formats Signed-off-by: Kieran Bingham --- drivers/media/platform/Kconfig | 13 + drivers/media/platform/Makefile|1 + drivers/media/platform

[PATCH RFC 0/2] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-06-09 Thread Kieran Bingham
This early version of the driver, is submitted for review, and functions only as a pixel format converter. The FDP1, (Fine Display Processor) is a de-interlacer device, with capability to convert from various YCbCr/YUV formats to both YCbCr/YUV and RGB formats at the same time as converting interl

[PATCH] v4l: Extend FCP compatible list to support the FDP

2016-06-09 Thread Kieran Bingham
The FCP must be powered up for the FDP1 to function, even when the FDP1 does not make use of the FCNL features. Extend the compatible list to allow us to use the power domain and runtime-pm support. Signed-off-by: Kieran Bingham --- drivers/media/platform/rcar-fcp.c | 1 + 1 file changed, 1 inse

[PATCH] v4l: Extend FCP compatible list to support the FDP

2016-06-09 Thread Kieran Bingham
The following patch extends Laurents FCP driver to support the FCP for FDP (FCPF) which is used by the FDP driver currently in development This patch is of course dependant upon Laurents FCP driver series [0] which has not yet hit mainline [0] http://www.spinics.net/lists/linux-media/msg97302.htm

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Antti Palosaari
On 06/09/2016 07:18 PM, Mauro Carvalho Chehab wrote: Em Thu, 09 Jun 2016 18:41:10 +0300 Antti Palosaari escreveu: On 06/09/2016 06:24 PM, Mauro Carvalho Chehab wrote: Hi Akihiro, Em Thu, 09 Jun 2016 21:49:33 +0900 Akihiro TSUKADA escreveu: Hi, excuse me for taking up a very old post aga

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Mauro Carvalho Chehab
Em Thu, 09 Jun 2016 18:41:10 +0300 Antti Palosaari escreveu: > On 06/09/2016 06:24 PM, Mauro Carvalho Chehab wrote: > > Hi Akihiro, > > > > Em Thu, 09 Jun 2016 21:49:33 +0900 > > Akihiro TSUKADA escreveu: > > > >> Hi, > >> excuse me for taking up a very old post again, > >> but I'd like to kno

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Antti Palosaari
On 06/09/2016 06:24 PM, Mauro Carvalho Chehab wrote: Hi Akihiro, Em Thu, 09 Jun 2016 21:49:33 +0900 Akihiro TSUKADA escreveu: Hi, excuse me for taking up a very old post again, but I'd like to know the status of the patch: https://patchwork.linuxtv.org/patch/27922/ , which provides helper c

Re: dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Mauro Carvalho Chehab
Hi Akihiro, Em Thu, 09 Jun 2016 21:49:33 +0900 Akihiro TSUKADA escreveu: > Hi, > excuse me for taking up a very old post again, > but I'd like to know the status of the patch: > https://patchwork.linuxtv.org/patch/27922/ > , which provides helper code for defining/loading i2c DVB subdev driver

[PATCH 3/3] dt-bindings: Add Renesas R-Car FDP1 bindings

2016-06-09 Thread Kieran Bingham
The FDP1 is a de-interlacing module which converts interlaced video to progressive video. It is also capable of performing pixel format conversion between YCbCr/YUV formats and RGB formats. Signed-off-by: Kieran Bingham --- .../devicetree/bindings/media/renesas,fdp1.txt | 34

[PATCH 1/3] dt-bindings: Update Renesas R-Car FCP DT binding

2016-06-09 Thread Kieran Bingham
The FCP driver, can also support the FCPF variant for FDP1 compatible processing. Signed-off-by: Kieran Bingham --- Documentation/devicetree/bindings/media/renesas,fcp.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/media/renesas,fcp.t

[PATCH 2/3] dt-bindings: Document Renesas R-Car FCP power-domains usage

2016-06-09 Thread Kieran Bingham
The power domain must be specified to bring the device out of module standby. Document this in the example provided, so that new additions are not missed. Signed-off-by: Kieran Bingham --- Documentation/devicetree/bindings/media/renesas,fcp.txt | 1 + 1 file changed, 1 insertion(+) diff --git a

dvb-core: how should i2c subdev drivers be attached?

2016-06-09 Thread Akihiro TSUKADA
Hi, excuse me for taking up a very old post again, but I'd like to know the status of the patch: https://patchwork.linuxtv.org/patch/27922/ , which provides helper code for defining/loading i2c DVB subdev drivers. Was it rejected and each i2c demod/tuner drivers should provide its own version o

Re: [PATCH] [media] rc: Remove init_ir_raw_event and DEFINE_IR_RAW_EVENT

2016-06-09 Thread Mauro Carvalho Chehab
Em Wed, 13 Apr 2016 10:36:51 +0100 Sean Young escreveu: > On Wed, Apr 13, 2016 at 02:07:49PM +0800, kbuild test robot wrote: > > Hi Sean, > > > > [auto build test ERROR on linuxtv-media/master] > > [also build test ERROR on v4.6-rc3 next-20160412] > > [if your patch is applied to the wrong git t

[PATCH v4 2/2] media: Add a driver for the ov5645 camera sensor.

2016-06-09 Thread Todor Tomov
The ov5645 sensor from Omnivision supports up to 2592x1944 and CSI2 interface. The driver adds support for the following modes: - 1280x960 - 1920x1080 - 2592x1944 Output format is packed 8bit UYVY. Signed-off-by: Todor Tomov --- drivers/media/i2c/Kconfig | 12 + drivers/media/i2c/Makefile |

[PATCH v4 1/2] media: i2c/ov5645: add the device tree binding document

2016-06-09 Thread Todor Tomov
Add the document for ov5645 device tree binding. Signed-off-by: Todor Tomov Acked-by: Rob Herring --- .../devicetree/bindings/media/i2c/ov5645.txt | 50 ++ 1 file changed, 50 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5645.txt diff

[PATCH v4 0/2] OV5645 camera sensor driver

2016-06-09 Thread Todor Tomov
This is the fourth version of the OV5645 camera sensor driver patchset. Only one change since version 3: - build failure on kernel v4.7-rc1 fixed: s/media_entity_init/media_entity_pads_init/ Changes from version 2 include: - external camera clock configuration is moved from DT to driver; - pwdn

Re: [PATCH 5/6] drivers/media/media-device: add "release" callback

2016-06-09 Thread Mauro Carvalho Chehab
Em Mon, 21 Mar 2016 14:30:38 +0100 Max Kellermann escreveu: > Allow the client to free its data structures only after all files have > been closed (fixing use-after-free bugs). Hmm... Shuah is also working on fixing such issues at the media controller stuff, and I made a few fix patches myself.

Re: [PATCH 2/6] drivers/media/dvb-core/en50221: postpone release until file is closed

2016-06-09 Thread Mauro Carvalho Chehab
Em Mon, 21 Mar 2016 14:30:22 +0100 Max Kellermann escreveu: > Fixes use-after-free bug which occurs when I disconnect my DVB-S > received while VDR is running. I guess that the best way to fix it is to use a kref() that would be also incremented at open() and decremented at release(). This wor