Re: [PATCH v3 4/4] uvcvideo: add a metadata device node

2016-12-30 Thread Laurent Pinchart
Hi Guennadi, On Friday 30 Dec 2016 11:43:02 Guennadi Liakhovetski wrote: > Hi Laurent, > > I'd like to discuss extending this patch a bit, preferably as an > incremental patch. > > First let me confirm my current understanding of the way the UVC driver > creates its media device topology. Do I

Re: [PATCH v3 1/4] uvcvideo: (cosmetic) add and use an inline function

2016-12-30 Thread Laurent Pinchart
Hi Guennadi, Thank you for the patch. On Monday 12 Dec 2016 12:16:49 Guennadi Liakhovetski wrote: > From: Guennadi Liakhovetski > > Add an inline function to obtain a struct uvc_buffer pointer from a > struct vb2_v4l2_buffer one. > > Signed-off-by: Guennadi Liakhovetski

Re: [PATCH 1/5] [media] ir-rx51: port to rc-core

2016-12-30 Thread Sean Young
Hi Ivo,, On Fri, Dec 30, 2016 at 01:30:01PM +0200, Ivaylo Dimitrov wrote: > On 20.12.2016 19:50, Sean Young wrote: > >This driver was written using lirc since rc-core did not support > >transmitter-only hardware at that time. Now that it does, port > >this driver. > > > >Compile tested only. > >

Re: [PATCH 1/5] [media] ir-rx51: port to rc-core

2016-12-30 Thread Sean Young
On Fri, Dec 30, 2016 at 01:07:52PM +, Sean Young wrote: > Hi Ivo,, > > On Fri, Dec 30, 2016 at 01:30:01PM +0200, Ivaylo Dimitrov wrote: > > On 20.12.2016 19:50, Sean Young wrote: > > >This driver was written using lirc since rc-core did not support > > >transmitter-only hardware at that

Re: [PATCH v3 2/4] uvcvideo: (cosmetic) remove a superfluous assignment

2016-12-30 Thread Laurent Pinchart
Hi Guennadi, Thank you for the patch. On Monday 12 Dec 2016 12:16:50 Guennadi Liakhovetski wrote: > From: Guennadi Liakhovetski > > Remove a superfluous assignment to a local variable at the end of a > function. > > Signed-off-by: Guennadi Liakhovetski

[PATCH][V3] [media] gp8psk: fix spelling mistake: "firmare" -> "firmware"

2016-12-30 Thread Colin King
From: Colin Ian King trivial fix to spelling mistake in err message Signed-off-by: Colin Ian King --- drivers/media/usb/dvb-usb/gp8psk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/gp8psk.c

Re: [PATCH 1/5] [media] ir-rx51: port to rc-core

2016-12-30 Thread Sean Young
Hi Ivo, On Fri, Dec 30, 2016 at 03:50:42PM +0200, Ivaylo Dimitrov wrote: > On 30.12.2016 15:30, Sean Young wrote: > >On Fri, Dec 30, 2016 at 01:07:52PM +, Sean Young wrote: > >Speaking of which, if you would please test this, that would be great. My > >N900 died many years ago. > > Will do,

Re: [PATCH v3 4/4] uvcvideo: add a metadata device node

2016-12-30 Thread Laurent Pinchart
Hi Guennadi, On Friday 30 Dec 2016 14:04:34 Guennadi Liakhovetski wrote: > On Fri, 30 Dec 2016, Laurent Pinchart wrote: > > On Friday 30 Dec 2016 11:43:02 Guennadi Liakhovetski wrote: > >> Hi Laurent, > >> > >> I'd like to discuss extending this patch a bit, preferably as an > >> incremental

Re: [PATCH 1/5] [media] ir-rx51: port to rc-core

2016-12-30 Thread Ivaylo Dimitrov
On 30.12.2016 15:30, Sean Young wrote: On Fri, Dec 30, 2016 at 01:07:52PM +, Sean Young wrote: Hi Ivo,, On Fri, Dec 30, 2016 at 01:30:01PM +0200, Ivaylo Dimitrov wrote: On 20.12.2016 19:50, Sean Young wrote: This driver was written using lirc since rc-core did not support

Re: [PATCH v3 1/4] uvcvideo: (cosmetic) add and use an inline function

2016-12-30 Thread Guennadi Liakhovetski
Hi Laurent, On Fri, 30 Dec 2016, Laurent Pinchart wrote: > Hi Guennadi, > > Thank you for the patch. > > On Monday 12 Dec 2016 12:16:49 Guennadi Liakhovetski wrote: > > From: Guennadi Liakhovetski > > > > Add an inline function to obtain a struct uvc_buffer pointer

Re: [PATCH 1/5] [media] ir-rx51: port to rc-core

2016-12-30 Thread Ivaylo Dimitrov
Hi, On 20.12.2016 19:50, Sean Young wrote: This driver was written using lirc since rc-core did not support transmitter-only hardware at that time. Now that it does, port this driver. Compile tested only. I guess after that change, there will be no more /dev/lircN device, right? Neither

Re: [PATCH 10/20] gpio: pca953x: Add optional reset gpio control

2016-12-30 Thread Linus Walleij
On Thu, Dec 29, 2016 at 11:27 PM, Steve Longerbeam wrote: > Add optional reset-gpios pin control. If present, de-assert the > specified reset gpio pin to bring the chip out of reset. > > Signed-off-by: Steve Longerbeam > Cc: Linus Walleij

Re: [PATCH v3 3/4] uvcvideo: fix a wrong macro

2016-12-30 Thread Laurent Pinchart
Hi Guennadi, Thank you for the patch. On Monday 12 Dec 2016 12:16:51 Guennadi Liakhovetski wrote: > From: Guennadi Liakhovetski > > Don't mix up UVC_BUF_STATE_* and VB2_BUF_STATE_* codes. > > Signed-off-by: Guennadi Liakhovetski > Cc:

Re: [PATCH][V2] [media] gp8psk: fix spelling mistake: "firmare" -> "firmware"

2016-12-30 Thread Kalle Valo
Colin King writes: > From: Colin Ian King > > Trivial fix to spelling mistake in err message. Also change "don't" to > "does not". > > Signed-off-by: Colin Ian King > --- > drivers/media/usb/dvb-usb/gp8psk.c

Re: [PATCH 10/20] gpio: pca953x: Add optional reset gpio control

2016-12-30 Thread Steve Longerbeam
Hi Linus, Lothar, On 12/30/2016 05:17 AM, Linus Walleij wrote: On Thu, Dec 29, 2016 at 11:27 PM, Steve Longerbeam wrote: Add optional reset-gpios pin control. If present, de-assert the specified reset gpio pin to bring the chip out of reset. Signed-off-by: Steve

Re: [PATCH v3 4/4] uvcvideo: add a metadata device node

2016-12-30 Thread Guennadi Liakhovetski
On Fri, 30 Dec 2016, Laurent Pinchart wrote: > Hi Guennadi, > > On Friday 30 Dec 2016 11:43:02 Guennadi Liakhovetski wrote: > > Hi Laurent, > > > > I'd like to discuss extending this patch a bit, preferably as an > > incremental patch. > > > > First let me confirm my current understanding of

Re: Problem with uvcvideo timestamps

2016-12-30 Thread Laurent Pinchart
Hi Niels, On Monday 31 Oct 2016 14:42:54 Niels Möller wrote: > Hi, > > I'm tracking down a problem in Chrome, where video streams captured > from a Logitech c930e camera get bogus timestamps. Chrome started > using camera timestamps on linux a few months ago. I've noted commit > > >

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-12-30 Thread Steve Longerbeam
On 12/30/2016 11:06 AM, Marek Vasut wrote: On 12/29/2016 09:51 PM, Robert Schwebel wrote: Hi Jean-Michel, Hi, On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote: What is the status of this work? Philipp's patches have been reworked with the review feedback from the last

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-12-30 Thread Marek Vasut
On 12/29/2016 09:51 PM, Robert Schwebel wrote: > Hi Jean-Michel, Hi, > On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote: >> What is the status of this work? > > Philipp's patches have been reworked with the review feedback from the > last round and a new version will be

cron job: media_tree daily build: ERRORS

2016-12-30 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 Dec 31 05:00:08 CET 2016 media-tree git hash:40eca140c404505c09773d1c6685d818cb55ab1a media_build

The update of RockChip media framework plan

2016-12-30 Thread Randy Li
To those guy who cares Rockchip There is a update about Rockchip media framework, I decided not to continue of developing the VA-API driver for produces purpose(both official and private plan). As there is really to much problem with VA-API with V4L2. But I may develop the VA-API driver to

Re: [PATCH v3 4/4] uvcvideo: add a metadata device node

2016-12-30 Thread Guennadi Liakhovetski
Hi Laurent, I'd like to discuss extending this patch a bit, preferably as an incremental patch. First let me confirm my current understanding of the way the UVC driver creates its media device topology. Do I understand it correctly, that the driver allocates UVC entities (not media controller