Re: Terratec Grabby hwrev 2

2013-04-01 Thread Timo Teras
On Mon, 01 Apr 2013 19:26:53 +0200 Frank Schäfer wrote: > Am 30.03.2013 10:54, schrieb Timo Teras: > > On Thu, 28 Mar 2013 12:22:52 -0300 > > Mauro Carvalho Chehab wrote: > > > >>> On the W7 driver, I don't get any of the above mentioned problems. > >>> > >>> I looked at the saa7113 register ini

Re: [PATCH] drm/exynos: change the method for getting the interrupt resource of FIMD

2013-04-01 Thread Vikas Sajjan
Hi Mr. Inki Dae, Can you please review this patch.? On Wed, Mar 13, 2013 at 4:22 PM, Vikas Sajjan wrote: > Replaces the "platform_get_resource() for IORESOURCE_IRQ" with > platform_get_resource_byname(). > Both in exynos4 and exynos5, FIMD IP has 3 interrupts in the order: "fifo", > "vsync", and

Re: Status of the patches under review at LMML (32 patches)

2013-04-01 Thread Manu Abraham
On Sun, Mar 24, 2013 at 11:41 PM, Mauro Carvalho Chehab wrote: > This is the summary of the patches that are currently under review at > Linux Media Mailing List . > > Manu, > > Yet another patch adding IR support for mantis. It seems that this is > is a long waited feature, as from time to time,

Re: [PATCH 0/3] em28xx: add support for two buses on em2874 and upper

2013-04-01 Thread Devin Heitmueller
On Mon, Apr 1, 2013 at 6:14 PM, Mauro Carvalho Chehab wrote: > In time, I meant to say: > "So, it seems very unlikely that any change here will keep it working > for > model 16009 while breaking it for other HVR-930 devices." As far as I know, there's only ever been one em28xx/d

Re: Status of the patches under review at LMML (32 patches)

2013-04-01 Thread Laurent Pinchart
Hi Guennadi, On Friday 29 March 2013 16:13:41 Guennadi Liakhovetski wrote: > Hi Mauro > > On Sun, 24 Mar 2013, Mauro Carvalho Chehab wrote: > > == Guennadi Liakhovetski == > > > > I suspect that some of the above may be obsoleted: > > > > Nov,13 2012: sh_vou: Move from videobuf to

Re: mb86a20s and cx23885

2013-04-01 Thread Alfredo Jesús Delaiti
Hi all After analyzing 15 samples (took me many days),I obtained the table underneath. { 0x70, 0x0f }, { 0x70, 0xff }, { 0x09, 0x3a }, { 0x50, 0xd1 }, { 0x51, 0x22 }, { 0x39, 0x00 }, { 0x28, 0x2a }, { 0x29, 0x00 }, { 0x2a, 0xfd }, { 0x2b, 0xc8 }, { 0x3b, 0x21 },

Re: [PATCH 0/3] em28xx: add support for two buses on em2874 and upper

2013-04-01 Thread Mauro Carvalho Chehab
Em Mon, 1 Apr 2013 19:12:24 -0300 Mauro Carvalho Chehab escreveu: > Em Mon, 01 Apr 2013 22:39:28 +0200 > Frank Schäfer escreveu: > > > Am 01.04.2013 21:22, schrieb Mauro Carvalho Chehab: > > > Em Mon, 01 Apr 2013 19:14:03 +0200 > > > Frank Schäfer escreveu: > > > > > >> Am 18.03.2013 22:22, sc

Re: [PATCH 0/3] em28xx: add support for two buses on em2874 and upper

2013-04-01 Thread Mauro Carvalho Chehab
Em Mon, 01 Apr 2013 22:39:28 +0200 Frank Schäfer escreveu: > Am 01.04.2013 21:22, schrieb Mauro Carvalho Chehab: > > Em Mon, 01 Apr 2013 19:14:03 +0200 > > Frank Schäfer escreveu: > > > >> Am 18.03.2013 22:22, schrieb Mauro Carvalho Chehab: > >>> Em Wed, 06 Mar 2013 18:44:07 +0100 > >>> Frank Sc

Re: [PATCH] Fix undefined reference to `au8522_attach'

2013-04-01 Thread Michael Krufky
Good catch! I don't know if this fix got merged yet or not, but it's correct. Reviewed-by: Michael Krufky On Tue, Mar 26, 2013 at 7:17 AM, Hans Verkuil wrote: > au8522_attach is dependent on CONFIG_DVB_AU8522_DTV, not CONFIG_DVB_AU8522. > > Signed-off-by: Hans Verkuil > > diff --git a/drivers

Re: [PATCH 0/3] em28xx: add support for two buses on em2874 and upper

2013-04-01 Thread Frank Schäfer
Am 01.04.2013 21:22, schrieb Mauro Carvalho Chehab: > Em Mon, 01 Apr 2013 19:14:03 +0200 > Frank Schäfer escreveu: > >> Am 18.03.2013 22:22, schrieb Mauro Carvalho Chehab: >>> Em Wed, 06 Mar 2013 18:44:07 +0100 >>> Frank Schäfer escreveu: >>> Am 05.03.2013 16:43, schrieb Devin Heitmueller: >

Re: [PATCH 0/3] em28xx: add support for two buses on em2874 and upper

2013-04-01 Thread Mauro Carvalho Chehab
Em Mon, 01 Apr 2013 19:14:03 +0200 Frank Schäfer escreveu: > Am 18.03.2013 22:22, schrieb Mauro Carvalho Chehab: > > Em Wed, 06 Mar 2013 18:44:07 +0100 > > Frank Schäfer escreveu: > > > >> Am 05.03.2013 16:43, schrieb Devin Heitmueller: > >>> 2013/3/5 Mauro Carvalho Chehab : > The em2874 ch

cron job: media_tree daily build: WARNINGS

2013-04-01 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: Mon Apr 1 19:30:52 CEST 2013 git branch: test git hash: 6bf7861fa2bb4be3cc70a6e9aed664ce65270027 gcc versi

Re: Status of the patches under review at LMML (32 patches)

2013-04-01 Thread Michael Krufky
On Sun, Mar 24, 2013 at 2:11 PM, Mauro Carvalho Chehab wrote: > > == Michael Krufky == > > Michael promised to review and apply them on his tree, sending me a pull > request, but this never happened. Mike, did you see any issues on those > patches? Apologies, the year 2013 has

Re: Terratec Grabby hwrev 2

2013-04-01 Thread Frank Schäfer
Am 30.03.2013 10:54, schrieb Timo Teras: > On Thu, 28 Mar 2013 12:22:52 -0300 > Mauro Carvalho Chehab wrote: > >>> On the W7 driver, I don't get any of the above mentioned problems. >>> >>> I looked at the saa7113 register init sequence, and copied that >>> over to linux saa7113 init, but that did

Re: [PATCH 0/3] em28xx: add support for two buses on em2874 and upper

2013-04-01 Thread Frank Schäfer
Am 18.03.2013 22:22, schrieb Mauro Carvalho Chehab: > Em Wed, 06 Mar 2013 18:44:07 +0100 > Frank Schäfer escreveu: > >> Am 05.03.2013 16:43, schrieb Devin Heitmueller: >>> 2013/3/5 Mauro Carvalho Chehab : The em2874 chips and upper have 2 buses. On all known devices, bus 0 is currently u

Re: [PATCH] [media] gspca_touptek: Add support for ToupTek UCMOS series USB cameras

2013-04-01 Thread Hans de Goede
Hi John, Thanks for the new driver! Unfortunately the driver is still using gspca's own/deprecated control mechanism rather then the new v4l2-control framework which all drivers now a days use. And we've just finished converting all the gspca sub-drivers to using this new control framework, and

[PATCH] [media] siano: Fix array boundary at smscore_translate_msg()

2013-04-01 Thread Mauro Carvalho Chehab
As reported by Dan Carpenter: FYI, there are new smatch warnings show up in tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next master head: da17d7bda957ae4697b6abc0793f74fb9b50b58f commit: 4c3bdb5e2f5612ceb99ac17dbbe673b59a94d105 [media] siano:

[PATCH 3/5] [media] mb86a20s: fix audio sub-channel check

2013-04-01 Thread Mauro Carvalho Chehab
As reported by Dan Carpenter FYI, there are new smatch warnings show up in: tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next master head: da17d7bda957ae4697b6abc0793f74fb9b50b58f commit: 04fa725e7b1c22c583dd71a8cd85b8d997edfce3 [media] mb86a20s: I

[PATCH 1/5] [media] mb86a20s: Use a macro for the number of layers

2013-04-01 Thread Mauro Carvalho Chehab
Instead of using the magic number "3", use NUM_LAYERS macro on all places that are related to the ISDB-T layers. This makes the source code a little more readable. No functional changes. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/mb86a20s.c | 32 +-

[PATCH 4/5] [media] mb86a20s: Use 'layer' instead of 'i' on all places

2013-04-01 Thread Mauro Carvalho Chehab
We're using the anonymous 'i' to indicate the layer number on several places on the driver. That's not good, as some cut-and-paste type of change might be doing the wrong thing. So, call it as "layer" everywhere. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/mb86a20s.c |

[PATCH 5/5] [media] mb86a20s: better name temp vars at mb86a20s_layer_bitrate()

2013-04-01 Thread Mauro Carvalho Chehab
Using 'i' for the guard interval temporary var is a bad idea, as 'i' is generally used by "anonymous" indexes. Let's rename modulation, fec and guard interval temp vars with a meaningful name, as that makes easier to understand the code and avoids cut-and-paste types of error. Signed-off-by: Maur

[PATCH 2/5] [media] mb86a20s: Fix estimate_rate setting

2013-04-01 Thread Mauro Carvalho Chehab
As reported by Dan Carpenter : Smatch warnings: drivers/media/dvb-frontends/mb86a20s.c:644 mb86a20s_layer_bitrate() error: buffer overflow 'state->estimated_rate' 3 <= 3 What happens there is that estimate_rate index should be the layer number, and not the guard interval. Report

[PATCH 0/5] mb86a20s: some smatch fixes

2013-04-01 Thread Mauro Carvalho Chehab
As pointed by Dan Carpenter, there are two smatch warnings on this driver: drivers/media/dvb-frontends/mb86a20s.c:1897 mb86a20s_set_frontend() error: buffer overflow 'mb86a20s_subchannel' 8 <= 8 drivers/media/dvb-frontends/mb86a20s.c:644 mb86a20s_layer_bitrate() error: buffer ove

Re: [PATCH] xawtv: release buffer if it can't be displayed

2013-04-01 Thread Hans Verkuil
On Mon April 1 2013 16:23:51 Hans de Goede wrote: > Hi, > > On 04/01/2013 12:19 PM, Hans Verkuil wrote: > > Hi Hans, > > > > On Sun March 31 2013 14:48:01 Hans de Goede wrote: > >> Hi, > >> > >> On 03/30/2013 10:47 AM, Hans Verkuil wrote: > >>> This patch for xawtv3 releases the buffer if it can't

Re: [PATCH] xawtv: release buffer if it can't be displayed

2013-04-01 Thread Hans de Goede
Hi, On 04/01/2013 12:19 PM, Hans Verkuil wrote: Hi Hans, On Sun March 31 2013 14:48:01 Hans de Goede wrote: Hi, On 03/30/2013 10:47 AM, Hans Verkuil wrote: This patch for xawtv3 releases the buffer if it can't be displayed because the resolution of the current format is larger than the size

Re: [PATCH] crystalhd gst: Port to GStreamer 1.0 API

2013-04-01 Thread Guido Günther
Hi, On Mon, Mar 11, 2013 at 09:40:22AM +0100, Guido Günther wrote: > --- > Tested with totem as well as gst-launch. Any chance this can be applied > to http://git.linuxtv.org/jarod/crystalhd.git? Any chance this gets applied. > Cheers, > -- Guido > > filters/gst/gst-plugin/configure.ac|

Re: [PATCH] xawtv: release buffer if it can't be displayed

2013-04-01 Thread Hans Verkuil
Hi Hans, On Sun March 31 2013 14:48:01 Hans de Goede wrote: > Hi, > > On 03/30/2013 10:47 AM, Hans Verkuil wrote: > > This patch for xawtv3 releases the buffer if it can't be displayed because > > the resolution of the current format is larger than the size of the window. > > > > This will happen

Re: ddbridge v0.8

2013-04-01 Thread Oliver Schinagl
Hey Ralph, On 02/15/13 14:47, Ralph Metzler wrote: Oliver Schinagl writes: > On 02/11/13 23:20, Martin Vidovic wrote: > > Hi, > > > > Is there any plan to include ddbridge driver version 0.8 in mainline kernel > > (currently it's 0.5). I really see no reason it's in the vacuum like no

Re: [PATCH RFC] [media] add Aptina mt9m114 HD digital image sensor driver

2013-04-01 Thread Scott Jiang
Hi Laurent, >> > >> >> +struct mt9m114_reg { >> >> + u16 reg; >> >> + u32 val; >> >> + int width; >> >> +}; >> >> + >> >> +enum { >> >> + MT9M114_QVGA, >> >> + MT9M114_VGA, >> >> + MT9M114_WVGA, >> >> + MT9M114_720P, >> >> +}; >> > >> > This is the part I don't like. In

Re: [PATCH v2] davinci: vpif: add pm_runtime support

2013-04-01 Thread Rajagopal Venkat
On 1 April 2013 13:53, Prabhakar Lad wrote: > On Mon, Apr 1, 2013 at 12:47 PM, Rajagopal Venkat > wrote: >> On 1 April 2013 12:06, Prabhakar lad wrote: >>> From: Lad, Prabhakar >>> >>> Add pm_runtime support to the TI Davinci VPIF driver. >>> >>> Signed-off-by: Lad, Prabhakar >>> Cc: Mauro Car

Re: [PATCH v3] drm/exynos: enable FIMD clocks

2013-04-01 Thread Viresh Kumar
On 1 April 2013 14:13, Vikas Sajjan wrote: > While migrating to common clock framework (CCF), found that the FIMD clocks s/found/we found/ > were pulled down by the CCF. > If CCF finds any clock(s) which has NOT been claimed by any of the > drivers, then such clock(s) are PULLed low by CCF. > >

[PATCH v3] drm/exynos: enable FIMD clocks

2013-04-01 Thread Vikas Sajjan
While migrating to common clock framework (CCF), found that the FIMD clocks were pulled down by the CCF. If CCF finds any clock(s) which has NOT been claimed by any of the drivers, then such clock(s) are PULLed low by CCF. By calling clk_prepare_enable() for FIMD clocks fixes the issue. this patc

Re: [PATCH v2] davinci: vpif: add pm_runtime support

2013-04-01 Thread Prabhakar Lad
On Mon, Apr 1, 2013 at 12:47 PM, Rajagopal Venkat wrote: > On 1 April 2013 12:06, Prabhakar lad wrote: >> From: Lad, Prabhakar >> >> Add pm_runtime support to the TI Davinci VPIF driver. >> >> Signed-off-by: Lad, Prabhakar >> Cc: Mauro Carvalho Chehab >> Cc: Hans Verkuil >> Cc: Laurent Pincha

Re: [PATCH v2] davinci: vpif: add pm_runtime support

2013-04-01 Thread Rajagopal Venkat
On 1 April 2013 12:06, Prabhakar lad wrote: > From: Lad, Prabhakar > > Add pm_runtime support to the TI Davinci VPIF driver. > > Signed-off-by: Lad, Prabhakar > Cc: Mauro Carvalho Chehab > Cc: Hans Verkuil > Cc: Laurent Pinchart > Cc: Sakari Ailus > Cc: Sekhar Nori > --- > Changes for v2: