Re: Patch update notification: 5 patches updated

2012-12-27 Thread Tushar Behera
On 12/26/2012 06:03 PM, Patchwork wrote: > Hello, > > The following patches (submitted by you) have been updated in patchwork: > > * [05/14,media] atmel-isi: Update error check for unsigned variables > - http://patchwork.linuxtv.org/patch/15475/ > was: New > now: Under Review > >

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Vikas Sajjan
On 27 December 2012 20:13, Tomasz Figa wrote: > Hi Laurent, > > On Monday 24 of December 2012 15:12:28 Laurent Pinchart wrote: >> Hi Tomasz, >> >> On Friday 21 December 2012 11:00:52 Tomasz Figa wrote: >> > On Tuesday 18 of December 2012 08:31:30 Vikas Sajjan wrote: >> > > On 17 December 2012 20:5

Re: [PATCH 01/23] uvc: Replace memcpy with struct assignment

2012-12-27 Thread Mauro Carvalho Chehab
Em Thu, 27 Dec 2012 18:12:46 -0300 Ezequiel Garcia escreveu: > Mauro, > > On Tue, Oct 23, 2012 at 4:57 PM, Ezequiel Garcia > wrote: > > This kind of memcpy() is error-prone. Its replacement with a struct > > assignment is prefered because it's type-safe and much easier to read. > > > > Found b

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Sascha Hauer
On Thu, Dec 27, 2012 at 01:57:56PM -0600, Rob Clark wrote: > On Thu, Dec 27, 2012 at 1:18 PM, Sascha Hauer wrote: > > On Thu, Dec 27, 2012 at 09:54:55AM -0600, Rob Clark wrote: > >> On Mon, Dec 24, 2012 at 7:37 AM, Laurent Pinchart > > > > This implies that the master driver knows all potential su

[PATCH] [media] blackfin Kconfig: select is evil; use, instead depends on

2012-12-27 Thread Mauro Carvalho Chehab
Select is evil as it has issues with dependencies. Better to convert it to use depends on. That fixes a breakage with out-of-tree compilation of the media tree. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/blackfin/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(

[PATCH 5/6] em28xx: IR RC: move assignment of get_key functions from *_change_protocol() functions to em28xx_ir_init()

2012-12-27 Thread Frank Schäfer
The get_key functions are independent from the selected protocol, so assign them once only at device initialization. Signed-off-by: Frank Schäfer --- drivers/media/usb/em28xx/em28xx-input.c |6 ++ 1 Datei geändert, 2 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-) diff --git a/drivers/media

[PATCH 6/6] ir-kbd-i2c: fix get_key_knc1()

2012-12-27 Thread Frank Schäfer
- return valid key code when button is hold - debug: print key code only when a button is pressed Tested with device "Terratec Cinergy 200 USB" (em28xx). Signed-off-by: Frank Schäfer --- drivers/media/i2c/ir-kbd-i2c.c | 15 +-- 1 Datei geändert, 5 Zeilen hinzugefügt(+), 10 Zeilen

[PATCH 4/6] em28xx: IR RC: get rid of function em28xx_get_key_terratec()

2012-12-27 Thread Frank Schäfer
Module "ir-kbd-i2c" already provides this function as IR_KBD_GET_KEY_KNC1. Signed-off-by: Frank Schäfer --- drivers/media/usb/em28xx/em28xx-input.c | 30 +- 1 Datei geändert, 1 Zeile hinzugefügt(+), 29 Zeilen entfernt(-) diff --git a/drivers/media/usb/em28xx/em28xx

[PATCH 3/6] em28xx: make remote controls of devices with external IR IC working again

2012-12-27 Thread Frank Schäfer
Tested with device "Terratec Cinergy 200 USB". Signed-off-by: Frank Schäfer --- drivers/media/usb/em28xx/em28xx-cards.c |9 +- drivers/media/usb/em28xx/em28xx-i2c.c |1 + drivers/media/usb/em28xx/em28xx-input.c | 142 +-- 3 Dateien geändert, 83 Zeilen hinzu

[PATCH 1/6] em28xx: simplify device state tracking

2012-12-27 Thread Frank Schäfer
DEV_INITIALIZED of enum em28xx_dev_state state is used nowhere and there is no need for DEV_MISCONFIGURED, so remove this enum and use a boolean field 'disconnected' in the device struct instead. Signed-off-by: Frank Schäfer --- drivers/media/usb/em28xx/em28xx-cards.c |5 ++--- drivers/media

[PATCH 2/6] em28xx: refactor the code in em28xx_usb_disconnect()

2012-12-27 Thread Frank Schäfer
The main purpose of this patch is to move the call of em28xx_release_resources() after the call of em28xx_close_extension(). This is necessary, because some resources might be needed/used by the extensions fini() functions when they get closed. Also mark the device as disconnected earlier in this f

[PATCH 0/6] em28xx: make remote controls of devices with external IR IC working again

2012-12-27 Thread Frank Schäfer
Support for remote controls of em28xx devices with external IR receiver/decoder IC are broken since a long time. This patch series makes them working again and fixes+improves several parts of the code. Frank Schäfer (6): em28xx: simplify device state tracking em28xx: refactor the code in em

Re: [PATCH] [media] coda: Fix build due to iram.h rename

2012-12-27 Thread Mauro Carvalho Chehab
Em Mon, 17 Dec 2012 10:37:14 +0100 Sascha Hauer escreveu: > On Wed, Nov 14, 2012 at 11:04:42AM -0200, Fabio Estevam wrote: > > commit c045e3f13 (ARM: imx: include iram.h rather than mach/iram.h) changed > > the > > location of iram.h, which causes the following build error when building > > the

Re: [GIT PULL FOR v3.9] separate Montage ts2020 from ds3000 and rs2000, support for new TeVii cards

2012-12-27 Thread Mauro Carvalho Chehab
Hi Igor, Em Mon, 24 Dec 2012 11:23:56 +0300 "Igor M. Liplianin" escreveu: > The following changes since commit 8b2aea7878f64814544d0527c659011949d52358: > > [media] em28xx: prefer bulk mode on webcams (2012-12-23 17:24:30 -0200) > > are available in the git repository at: > > git://git.li

Re: [PATCH 01/23] uvc: Replace memcpy with struct assignment

2012-12-27 Thread Ezequiel Garcia
Mauro, On Tue, Oct 23, 2012 at 4:57 PM, Ezequiel Garcia wrote: > This kind of memcpy() is error-prone. Its replacement with a struct > assignment is prefered because it's type-safe and much easier to read. > > Found by coccinelle. Hand patched and reviewed. > Tested by compilation only. > > A sim

Re: [PATCH] omap3isp: Add support for interlaced input data

2012-12-27 Thread Mauro Carvalho Chehab
Em Tue, 25 Dec 2012 22:05:48 +0100 Laurent Pinchart escreveu: > Hi William, > > Thanks for the patch. Btw, you missed to add a Signed-off-by: line on it. > > On Monday 17 December 2012 18:12:19 William Swanson wrote: > > If the remote video sensor reports an interlaced video mode, the CCDC bl

Re: [PATCH V3 01/15] [media] marvell-ccic: use internal variable replace global frame stats variable

2012-12-27 Thread Mauro Carvalho Chehab
Em Sun, 16 Dec 2012 08:36:59 -0700 Jonathan Corbet escreveu: > On Sat, 15 Dec 2012 17:57:50 +0800 > Albert Wang wrote: > > > This patch replaces the global frame stats variables by using > > internal variables in mcam_camera structure. > > This one seems fine. Someday it might be nice to have

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Rob Clark
On Thu, Dec 27, 2012 at 1:18 PM, Sascha Hauer wrote: > On Thu, Dec 27, 2012 at 09:54:55AM -0600, Rob Clark wrote: >> On Mon, Dec 24, 2012 at 7:37 AM, Laurent Pinchart >> wrote: >> > Hi Rob, >> > >> > On Tuesday 18 December 2012 00:21:32 Rob Clark wrote: >> >> On Mon, Dec 17, 2012 at 11:04 PM, Dav

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Sascha Hauer
On Thu, Dec 27, 2012 at 10:04:22AM -0600, Rob Clark wrote: > On Mon, Dec 24, 2012 at 11:27 AM, Laurent Pinchart > wrote: > > On Wednesday 19 December 2012 16:57:56 Jani Nikula wrote: > >> It just seems to me that, at least from a DRM/KMS perspective, adding > >> another layer (=CDF) for HDMI or DP

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Sascha Hauer
On Thu, Dec 27, 2012 at 09:54:55AM -0600, Rob Clark wrote: > On Mon, Dec 24, 2012 at 7:37 AM, Laurent Pinchart > wrote: > > Hi Rob, > > > > On Tuesday 18 December 2012 00:21:32 Rob Clark wrote: > >> On Mon, Dec 17, 2012 at 11:04 PM, Dave Airlie wrote: > >> >> Many developers showed interest in th

[PATCH] [media] m920x: Fix CodingStyle issues

2012-12-27 Thread Mauro Carvalho Chehab
Fix CodingStyle issues introduced by the last patch Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/dvb-usb/m920x.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/dvb-usb/m920x.c b/drivers/media/usb/dvb-usb/m920x.c index bddd763..a70c5ea

cron job: media_tree daily build: ERRORS

2012-12-27 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 Dec 27 19:00:30 CET 2012 git hash:30ad64b8ac539459f8975aa186421ef3db0bb5cb gcc version: i686-linux-gcc (GCC

[PATCH 1/2] [media] ttpci: Fix a missing Kconfig dependency

2012-12-27 Thread Mauro Carvalho Chehab
Fix a Kconfig warning that appears with allmodconfig on arm: warning: (DVB_USB_PCTV452E) selects TTPCI_EEPROM which has unmet direct dependencies (MEDIA_SUPPORT && MEDIA_PCI_SUPPORT && MEDIA_DIGITAL_TV_SUPPORT && I2C) Signed-off-by: Mauro Carvalho Chehab --- drivers/media/Kconfig

[PATCH 2/2] [media] omap: Fix Kconfig dependencies on OMAP2

2012-12-27 Thread Mauro Carvalho Chehab
Resolves the following warning that appears with allmodconfig on -arm: warning: (VIDEO_OMAP2_VOUT && DRM_OMAP) selects OMAP2_DSS which has unmet direct dependencies (HAS_IOMEM && ARCH_OMAP2PLUS) Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/omap/Kconfig | 2 +- 1 file

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Rob Clark
On Mon, Dec 24, 2012 at 11:35 AM, Laurent Pinchart wrote: > On Wednesday 19 December 2012 09:26:40 Rob Clark wrote: >> And, there are also external HDMI encoders (for example connected over >> i2c) that can also be shared between boards. So I think there will be >> a number of cases where CDF is

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Rob Clark
On Mon, Dec 24, 2012 at 11:27 AM, Laurent Pinchart wrote: > On Wednesday 19 December 2012 16:57:56 Jani Nikula wrote: >> It just seems to me that, at least from a DRM/KMS perspective, adding >> another layer (=CDF) for HDMI or DP (or legacy outputs) would be >> overengineering it. They are pretty

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Rob Clark
On Mon, Dec 24, 2012 at 11:09 AM, Laurent Pinchart wrote: > On the topic of discussions, would anyone be interested in a > BoF/brainstorming/whatever session during the FOSDEM ? I will be at FOSDEM.. and from http://wiki.x.org/wiki/fosdem2013 it looks like at least Daniel will be there. If enoug

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Rob Clark
On Mon, Dec 24, 2012 at 7:37 AM, Laurent Pinchart wrote: > Hi Rob, > > On Tuesday 18 December 2012 00:21:32 Rob Clark wrote: >> On Mon, Dec 17, 2012 at 11:04 PM, Dave Airlie wrote: >> >> Many developers showed interest in the first RFC, and I've had the >> >> opportunity to discuss it with most o

Re: [RFC v2 0/5] Common Display Framework

2012-12-27 Thread Tomasz Figa
Hi Laurent, On Monday 24 of December 2012 15:12:28 Laurent Pinchart wrote: > Hi Tomasz, > > On Friday 21 December 2012 11:00:52 Tomasz Figa wrote: > > On Tuesday 18 of December 2012 08:31:30 Vikas Sajjan wrote: > > > On 17 December 2012 20:55, Laurent Pinchart wrote: > > > > Hi Vikas, > > > > >

Re: HD-PVR fails consistently on Linux, works on Windows

2012-12-27 Thread David Röthlisberger
On 17 Oct 2012, at 18:54, Keith Pyle wrote: > On 09/26/12 21:42, Keith Pyle wrote: > > I recently purchased a Hauppauge HD-PVR (the 1212 version, label on bottom > > 49001LF, Rev F2). I have consistent capture failures on Linux where data > > from > > the device simply stops, generally within a fe

Re: [PATCHv2 0/9] dvb-usb/m920x: support VP-7049 DVB-T USB Stick and other fixes

2012-12-27 Thread Antonio Ospite
On Mon, 10 Dec 2012 22:37:08 +0100 Antonio Ospite wrote: > Hi, > Ping. > Here is a second iteration of the patchset to add support for the > Twinhan VP7049 DVB-T USB Stick, v1 is at: > http://thread.gmane.org/gmane.linux.drivers.video-input-infrastructure/56714 > > Patches from 1 to 7 are smal

Re: [PATCH 1/6] uvcvideo: Set error_idx properly for extended controls API failures

2012-12-27 Thread Laurent Pinchart
Hi Hans, On Thursday 27 December 2012 12:59:15 Hans Verkuil wrote: > On Wed December 26 2012 12:33:58 Laurent Pinchart wrote: > > On Tuesday 25 December 2012 12:50:51 Hans Verkuil wrote: > > > On Tue December 25 2012 12:23:00 Laurent Pinchart wrote: > > > > On Tuesday 25 December 2012 12:15:25 Han

Re: [PATCH 1/6] uvcvideo: Set error_idx properly for extended controls API failures

2012-12-27 Thread Hans Verkuil
On Wed December 26 2012 12:33:58 Laurent Pinchart wrote: > Hi Hans, > > On Tuesday 25 December 2012 12:50:51 Hans Verkuil wrote: > > On Tue December 25 2012 12:23:00 Laurent Pinchart wrote: > > > On Tuesday 25 December 2012 12:15:25 Hans Verkuil wrote: > > > > On Mon December 24 2012 13:27:08 Laur

Re: [PATCH] [media] vivi: Constify structures

2012-12-27 Thread Hans Verkuil
On Wed December 26 2012 16:29:43 Kirill Smelkov wrote: > Most of *_ops and other structures in vivi.c were already declared const > but some have not. Constify and code/data will take less space: > > $ size drivers/media/platform/vivi.o > textdata bss dec hex file

Re: DVB-T Scan File Update Required for au-Darwin

2012-12-27 Thread Ben Manfield
Just a minor modification... # Darwin Digital Television (UHF 33) T 564625 7Mhz 3/4 NONE QAM64 8k 1/16 NONE Cheers Ben On Thu, Dec 27, 2012 at 7:47 PM, Ben Manfield wrote: > Could you please include the following channel in your scan file for > au-Darwin? > > # Darwin Digital Television (UHF 3