Re: [PATCH] [media] ov2640: add support for async device registration

2014-03-26 Thread Josh Wu
Hi, Sylwester On 3/20/2014 10:44 PM, Sylwester Nawrocki wrote: Hi Josh, On 19/03/14 10:17, Josh Wu wrote: On 3/15/2014 5:17 AM, Sylwester Nawrocki wrote: On 03/14/2014 11:12 AM, Josh Wu wrote: +clk = v4l2_clk_get(client-dev, mclk); +if (IS_ERR(clk)) +return -EPROBE_DEFER;

Re: [PATCH v2] [media] ov2640: add support for async device registration

2014-03-26 Thread Josh Wu
Hi, all since v4l2_clk_get() WON'T return EPROBE_DEFER. So this version of patch is invalid. Please drop this version of the patch. Sorry for the noise. Best Regards, Josh Wu On 3/20/2014 5:01 PM, Josh Wu wrote: Since the the v4l2_clk_get() may return a EPROBE_DEFER during async probing. So

Re: [PATCH 11/11] libdvbv5: fix PMT parser

2014-03-26 Thread Bjørn Mork
André Roth neol...@gmail.com writes: On Tue, 25 Mar 2014 21:51:49 +0100 Bjørn Mork bj...@mork.no wrote: - * Copyright (c) 2011-2012 - Mauro Carvalho Chehab - * Copyright (c) 2012 - Andre Roth neol...@gmail.com + * Copyright (c) 2013 - Andre Roth neol...@gmail.com * * This program

Re: [PATCH 1/5] rc-main: add generic scancode filtering

2014-03-26 Thread Antti Seppälä
On 26 March 2014 01:21, David Härdeman da...@hardeman.nu wrote: On Tue, Mar 25, 2014 at 09:12:11AM +, James Hogan wrote: On Tuesday 25 March 2014 00:51:46 David Härdeman wrote: On Fri, Feb 28, 2014 at 11:17:02PM +, James Hogan wrote: Add generic scancode filtering of RC input events,

Re: why frameformat instead pixelformat?

2014-03-26 Thread Tom
Tom Bassai_Dai at gmx.net writes: Hello, ... For that I found the negotiation rfc of that topic, but I don't really get the relevance of a frameformat. http://www.spinics.net/lists/linux-media/msg10006.html Can anyone explain why the media-api uses the frameformat instead of the

Re: why frameformat instead pixelformat?

2014-03-26 Thread Hans Verkuil
On 03/24/14 17:52, Tom wrote: Hello, while reading into the media-api issue I found out that for configuring the entity pads a frameformat is used. For that I found the negotiation rfc of that topic, but I don't really get the relevance of a frameformat.

Re: [PATCH] adv7611: Set HPD GPIO direction to output

2014-03-26 Thread Hans Verkuil
Hi Laurent, Stupid question perhaps, but why is gpiod_set_value_cansleep() removed? Does setting the output direction force the value to 0 as well? Regards, Hans On 03/26/14 03:28, Laurent Pinchart wrote: The HPD GPIO is used as an output but its direction is never set. Fix it.

Re: [PATCH] adv7611: Set HPD GPIO direction to output

2014-03-26 Thread Laurent Pinchart
Hi Hans, On Wednesday 26 March 2014 10:52:30 Hans Verkuil wrote: Hi Laurent, Stupid question perhaps, but why is gpiod_set_value_cansleep() removed? Does setting the output direction force the value to 0 as well? The last argument to gpiod_direction_output() sets the initial output level,

Re: [PATCH v4 00/10] media: rc: ImgTec IR decoder driver

2014-03-26 Thread James Hogan
On 25/03/14 23:53, David Härdeman wrote: On Fri, Feb 28, 2014 at 11:28:50PM +, James Hogan wrote: Add a driver for the ImgTec Infrared decoder block. Two separate rc input devices are exposed depending on kernel configuration. One uses the hardware decoder which is set up with timings for

Re: [PATCH] adv7611: Set HPD GPIO direction to output

2014-03-26 Thread Hans Verkuil
In that case: Acked-by: Hans Verkuil hans.verk...@cisco.com Regards, Hans On 03/26/14 11:03, Laurent Pinchart wrote: Hi Hans, On Wednesday 26 March 2014 10:52:30 Hans Verkuil wrote: Hi Laurent, Stupid question perhaps, but why is gpiod_set_value_cansleep() removed? Does setting

Re: [PATCH 11/11] libdvbv5: fix PMT parser

2014-03-26 Thread Mauro Carvalho Chehab
Em Wed, 26 Mar 2014 07:38:15 +0100 Bjørn Mork bj...@mork.no escreveu: André Roth neol...@gmail.com writes: On Tue, 25 Mar 2014 21:51:49 +0100 Bjørn Mork bj...@mork.no wrote: - * Copyright (c) 2011-2012 - Mauro Carvalho Chehab - * Copyright (c) 2012 - Andre Roth neol...@gmail.com

Re: [PATCH v4 00/10] media: rc: ImgTec IR decoder driver

2014-03-26 Thread David Härdeman
On 2014-03-26 10:28, James Hogan wrote: On 25/03/14 23:53, David Härdeman wrote: One thing I just noticed...your copyright headers throughout the driver seems a bit...sparse? :) True, I can add the basic: ... to each of the files if you think it's necessary. I think doing so for each *.c

Re: [PATCH 1/5] rc-main: add generic scancode filtering

2014-03-26 Thread David Härdeman
On 2014-03-26 08:08, Antti Seppälä wrote: On 26 March 2014 01:21, David Härdeman da...@hardeman.nu wrote: On Tue, Mar 25, 2014 at 09:12:11AM +, James Hogan wrote: On Tuesday 25 March 2014 00:51:46 David Härdeman wrote: What's the purpose of providing the sw scancode filtering in the case

Re: [PATCH 1/5] rc-main: add generic scancode filtering

2014-03-26 Thread James Hogan
On 26/03/14 13:44, David Härdeman wrote: On 2014-03-26 08:08, Antti Seppälä wrote: On 26 March 2014 01:21, David Härdeman da...@hardeman.nu wrote: On Tue, Mar 25, 2014 at 09:12:11AM +, James Hogan wrote: On Tuesday 25 March 2014 00:51:46 David Härdeman wrote: What's the purpose of

Re: [PATCH v2 3/3] [media] atmel-isi: add primary DT support

2014-03-26 Thread Laurent Pinchart
Hi Josh, Thank you for the patch. On Tuesday 25 March 2014 18:45:20 Josh Wu wrote: This patch add the DT support for Atmel ISI driver. It use the same v4l2 DT interface that defined in video-interfaces.txt. Signed-off-by: Josh Wu josh...@atmel.com Cc: devicet...@vger.kernel.org

Re: [PATCH RFC v2 2/6] drm/i2c: tda998x: Move tda998x to a couple encoder/connector

2014-03-26 Thread Laurent Pinchart
Hi Jean-François, On Tuesday 25 March 2014 16:55:48 Jean-Francois Moine wrote: On Mon, 24 Mar 2014 23:39:01 +0100 Laurent Pinchart wrote: On Friday 21 March 2014 09:17:32 Jean-Francois Moine wrote: The 'slave encoder' structure of the tda998x driver asks for glue between the DRM driver

Re: [PATCH] [media] uvcvideo: Fix marking buffer erroneous in case of FID toggling

2014-03-26 Thread Laurent Pinchart
Hi Anton, Thank you for the patch. On Tuesday 25 March 2014 08:40:57 Anton Leontiev wrote: Set error bit for incomplete buffers when end of buffer is detected by FID toggling (for example when last transaction with EOF is lost). This prevents passing incomplete buffers to the userspace. But

[PATCH 2/3] TBS USB drivers (DVB-S/S2) - add lock led hooks to frontends

2014-03-26 Thread Andreas Steinmetz
[Please CC me on replies, I'm not subscribed] Based on GPLv2 code taken from: https://bitbucket.org/CrazyCat/linux-tbs-drivers/ The patch adds lock led hooks to the stv090x, stv0288, cx24116 and tda10071 frontends. Similar code already exists in the stv0900 frontend. -- Andreas Steinmetz

[PATCH 1/3] TBS USB drivers (DVB-S/S2) - basic driver

2014-03-26 Thread Andreas Steinmetz
[Please CC me on replies, I'm not subscribed] The patch adds a driver for TBS USB DVB-S/S2 devices for which complete GPLv2 code exists. Code was taken from: http://www.tbsdtv.com/download/ https://bitbucket.org/CrazyCat/linux-tbs-drivers/ https://bitbucket.org/updatelee/v4l-updatelee Supported

[PATCH 0/3] rc: Misc fixes for v3.15

2014-03-26 Thread James Hogan
A few misc fixes for v3.15, all relating to my previous patches. James Hogan (3): rc-main: Revert generic scancode filtering support rc-main: Limit to a single wakeup protocol group rc: img-ir: Expand copyright headers with GPL notices drivers/media/rc/img-ir/img-ir-core.c | 5 +

[PATCH 1/3] rc-main: Revert generic scancode filtering support

2014-03-26 Thread James Hogan
This reverts commit b8c7d915087c ([media] rc-main: add generic scancode filtering), and removes certain parts of commit 6bea25af147f ([media] rc-main: automatically refresh filter on protocol change) where generic filtering is taken into account when refreshing filters on a protocol change, but

[PATCH 3/3] rc: img-ir: Expand copyright headers with GPL notices

2014-03-26 Thread James Hogan
Add the basic GPLv2+ license notice to the copyright headers at the top of all the source files in the img-ir driver. Reported-by: David Härdeman da...@hardeman.nu Signed-off-by: James Hogan james.ho...@imgtec.com Cc: Mauro Carvalho Chehab m.che...@samsung.com Cc: David Härdeman da...@hardeman.nu

[PATCH 2/3] rc-main: Limit to a single wakeup protocol group

2014-03-26 Thread James Hogan
Limit the enabled wakeup protocols to be within a protocol group, as defined by the proto_names array. For example this prevents the selection of both rc-5 and nec, while allowing rc-5 alone (which encompasses both normal rc-5 and rc-5x). It doesn't usually make sense to enable more than one

[PATCH] rtl28xxu: remove duplicate ID 0458:707f Genius TVGo DVB-T03

2014-03-26 Thread Antti Palosaari
That ID was added twice mistakenly. 1st commit ac298ccdde4fe9b0a966e548a232ff4e8a6b8a31 2nd commit 1c1b8734094551eb4075cf68cf76892498c0da61 Revert commit 1c1b8734094551eb4075cf68cf76892498c0da61 Reported-by: Jan Vcelak j...@fcelda.cz Signed-off-by: Antti Palosaari cr...@iki.fi ---

Re: [PATCH 3/5] s5p-fimc: Align imagesize to row size for tiled formats

2014-03-26 Thread Nicolas Dufresne
Le mardi 25 mars 2014 à 16:51 -0400, Nicolas Dufresne a écrit : For tiled format, we need to allocated a multiple of the row size. A good example is for 1280x720, wich get adjusted to 1280x736. In tiles, this mean Y plane is 20x23 and UV plane 20x12. Because of the rounding, the previous code

[PATCH 3/5] s5p-fimc: Roundup imagesize to row size for tiled formats

2014-03-26 Thread Nicolas Dufresne
For tiled format, we need to allocated a multiple of the row size. A good example is for 1280x720, wich get adjusted to 1280x736. In tiles, this mean Y plane is 20x23 and UV plane 20x12. Because of the rounding, the previous code would only have enough space to fit half of the last row.

cron job: media_tree daily build: OK

2014-03-26 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: Thu Mar 27 04:00:17 CET 2014 git branch: test git hash: 8432164ddf7bfe40748ac49995356ab4dfda43b7 gcc