Re: Videotext application crashes the kernel due to DVB-demux patch

2010-02-02 Thread Chicken Shack
Am Montag, den 01.02.2010, 21:00 -0500 schrieb Andy Walls: On Mon, 2010-02-01 at 07:41 -0500, Andy Walls wrote: On Mon, 2010-02-01 at 10:56 +0100, Chicken Shack wrote: Hi, here is a link to a patch which breaks backwards compatibility for a teletext software called alevt-dvb.

Re: [PATCH ] libv4l: skip false Pixart markers

2010-02-02 Thread Hans de Goede
Hi, On 02/01/2010 11:13 PM, Németh Márton wrote: From: Márton Némethnm...@freemail.hu The byte sequence 0xff, 0xff, 0xff 0xff is not a real marker to skip, instead it is one byte from the image and the following three 0xff bytes might belong to a real marker. Modify pixart_fill_nbits() macro

Re: libv4l: possible problem found in PAC7302 JPEG decoding

2010-02-02 Thread Thomas Kaiser
On 02/01/2010 10:23 PM, Németh Márton wrote: Hello Hans, while I was dealing with Labtec Webcam 2200 and with gspca_pac7302 driver I recognised the following behaviour. The stream received from the webcam is splitted by the gspca_pac7302 subdriver when the byte sequence 0xff, 0xff, 0x00,

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-02 Thread Mauro Carvalho Chehab
Jean Delvare wrote: Hi Hermann, On Tue, 02 Feb 2010 02:47:53 +0100, hermann pitton wrote: Hi Jean, Am Montag, den 01.02.2010, 10:56 +0100 schrieb Jean Delvare: Hi Hermann, On Mon, 01 Feb 2010 02:16:35 +0100, hermann pitton wrote: For now, I only faked a P7131 Dual with a broken IR

Re: Lost remote after kernel/v4l update cx23885 chipset

2010-02-02 Thread Andy Walls
On Mon, 2010-02-01 at 12:03 -0700, Timothy D. Lenz wrote: On 1/24/2010 1:07 PM, Timothy D. Lenz wrote: After updating from kernel 2.6.26.8 to 2.6.32.2 and from v4l of 05/19/2009 to 01/18/2010 I lost remote function with Dvico FusionHDTV7 Dual Express. The driver is loading, but not

[PATCH] dvb: return -ENOMEM if kzalloc failed in dvb_usb_device_init()

2010-02-02 Thread Roel Kluin
If in a cold state and the download succeeded ret is zero, but we should return -ENOMEM. Signed-off-by: Roel Kluin roel.kl...@gmail.com --- Or shouldn't we? diff --git a/drivers/media/dvb/dvb-usb/dvb-usb-init.c b/drivers/media/dvb/dvb-usb/dvb-usb-init.c index e331db8..5d91f70 100644 ---

Re: [PATCH] cx23885: fix support for TBS 6920 card

2010-02-02 Thread Mauro Carvalho Chehab
Hi Steven, Steven Toth wrote: On 8/19/09 7:20 PM, Konstantin Dimitrov wrote: fix: GPIO initialization for TBS 6920 fix: wrong I2C address for demod on TBS 6920 fix: wrong I2C bus number for demod on TBS 6920 fix: wrong gen_ctrl_val value for TS1 port on TBS 6920 (and some other cards)

Re: [PATCH 3/8] drivers/media/video/hdpvr: introduce missing kfree

2010-02-02 Thread Mauro Carvalho Chehab
Hi Janne, Janne Grunau wrote: On Fri, Sep 11, 2009 at 06:21:35PM +0200, Julia Lawall wrote: Error handling code following a kzalloc should free the allocated data. Thanks for the report. I'll commit a different patch which adds the buffer to the buffer list as soon it is allocated. The

Re: DM1105: could not attach frontend 195d:1105

2010-02-02 Thread Nameer Kazzaz
Hi Igor, What do you think ? if I can help you solve this, let me know what I can do. Thanks Nameer Nameer Kazzaz wrote: Hi there, dmesg output with patched dm1105.c against current v4l-dvb 'modprob dm1105 card=4' dm1105 :05:0f.0: PCI INT A - GSI 16 (level, low) - IRQ 16 DVB:

Re: [PATCH] V4L/DVB: lgs8gxx: remove firmware for lgs8g75

2010-02-02 Thread Mauro Carvalho Chehab
Mauro Carvalho Chehab wrote: Ben Hutchings wrote: The recently added support for lgs8g75 included some 8051 machine code without accompanying source code. Replace this with use of the firmware loader. Compile-tested only. Signed-off-by: Ben Hutchings b...@decadent.org.uk --- This

RE: [PATCH v3 1/6] V4L - vpfe capture - header files for ISIF driver

2010-02-02 Thread Karicheri, Muralidharan
Mauro, If you scan the patch, you would see the status of this patch series. --- Applies to linux-next tree of v4l-dvb - rebasing to latest for merge (v3) Not sure if there is a better way to include this information. The arch part of this series requires sign-off from Kevin who is copied

Re: dvb-usb-remote woes [was: HID: ignore afatech 9016]

2010-02-02 Thread Pekka Sarnila
Ok, that was it, I had the old firmware. Now it works with the patch. I can now see that the afateck driver does not use the HID interface/endpoint (1/83) at all, but vendor specific bulk endpoint (0/81). The fact that manufacturers have started to more more use the usb vendor class instead

Re: dvb-usb-remote woes [was: HID: ignore afatech 9016]

2010-02-02 Thread Pekka Sarnila
Please, no mail to this address, use only sarn...@adit.fi. Pekka Antti Palosaari wrote: On 02/01/2010 09:42 PM, Jiri Slaby wrote: On 02/01/2010 07:28 PM, Jiri Kosina wrote: On Mon, 1 Feb 2010, Pekka Sarnila wrote: I pulled few days ago latest

Re: dvb-usb-remote woes [was: HID: ignore afatech 9016]

2010-02-02 Thread Pekka Sarnila
In the old kernel (2.6.24) it did not work if it was plugged while booting. If I recall right then it was due to the order different layers are initialized (modules loaded). Pekka Jiri Slaby wrote: On 02/01/2010 10:23 PM, Antti Palosaari wrote: On 02/01/2010 09:42 PM, Jiri Slaby wrote:

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Stefan Ringel
Am 02.02.2010 00:05, schrieb Mauro Carvalho Chehab: Stefan Ringel wrote: Am 01.02.2010 22:44, schrieb Devin Heitmueller: On Mon, Feb 1, 2010 at 4:23 PM, Stefan Ringel stefan.rin...@arcor.de wrote: You should start by breaking it down into a patch series, so that the

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Mauro Carvalho Chehab
Stefan Ringel wrote: So, it will basically preserve bits 8,7,6,4 and 1 of register 8, and will clear bit 4 (EM_GPIO_4 is 1 4 - e. g. bit 4). After that, it will sleep for 10 miliseconds, and will then do a reset on bit 3 of Register 4 (writing 0, then 1 to the bit). reset example :

Re: DM1105: could not attach frontend 195d:1105

2010-02-02 Thread Igor M. Liplianin
On 2 февраля 2010 17:21:46 Nameer Kazzaz wrote: Hi Igor, What do you think ? if I can help you solve this, let me know what I can do. Thanks Nameer Nameer Kazzaz wrote: Hi there, dmesg output with patched dm1105.c against current v4l-dvb 'modprob dm1105 card=4' dm1105

Re: [PATCH v3 1/6] V4L - vpfe capture - header files for ISIF driver

2010-02-02 Thread Mauro Carvalho Chehab
Karicheri, Muralidharan wrote: Mauro, If you scan the patch, you would see the status of this patch series. Ok, but it is not script-able. So, I need to have an easy way to know when a patch is still under discussion at the ML or if it is ready for merging. I need something that I can write

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Stefan Ringel
Am 01.02.2010 23:52, schrieb Mauro Carvalho Chehab: Stefan Ringel wrote: add Terratec Cinergy Hybrid XE bugfix i2c transfer add frontend callback add init for tm6010 add digital-init for tm6010 add callback for analog/digital switch bugfix usb transfer in DVB-mode signed-off-by:

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Stefan Ringel
Am 02.02.2010 17:44, schrieb Mauro Carvalho Chehab: Stefan Ringel wrote: So, it will basically preserve bits 8,7,6,4 and 1 of register 8, and will clear bit 4 (EM_GPIO_4 is 1 4 - e. g. bit 4). After that, it will sleep for 10 miliseconds, and will then do a reset on bit 3 of Register 4

Re: [PATCH v3 1/6] V4L - vpfe capture - header files for ISIF driver

2010-02-02 Thread Mauro Carvalho Chehab
Karicheri, Muralidharan wrote: Mauro, How to request sign-off? Do I only send patches to the person, not to the list? I think you have not answered this. For the patches exchanged via the ML, I need some scriptable way to mark them as RFC at the Patchwork. So, adding [RFC PATCH] or

Re: [PATCH ] libv4l: skip false Pixart markers

2010-02-02 Thread Németh Márton
Hans de Goede wrote: Hi, On 02/01/2010 11:13 PM, Németh Márton wrote: From: Márton Némethnm...@freemail.hu The byte sequence 0xff, 0xff, 0xff 0xff is not a real marker to skip, instead it is one byte from the image and the following three 0xff bytes might belong to a real marker. Modify

Re: libv4l: possible problem found in PAC7302 JPEG decoding

2010-02-02 Thread Németh Márton
Hi Thomas, Thomas Kaiser wrote: On 02/01/2010 10:23 PM, Németh Márton wrote: Hello Hans, while I was dealing with Labtec Webcam 2200 and with gspca_pac7302 driver I recognised the following behaviour. The stream received from the webcam is splitted by the gspca_pac7302 subdriver when the

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-02 Thread Mauro Carvalho Chehab
Hi Jean, From: Jean Delvare kh...@linux-fr.org Subject: saa7134: Fix IR support of some ASUS TV-FM 7135 variants Some variants of the ASUS TV-FM 7135 are handled as the ASUSTeK P7131 Analog (card=146). However, by the time we find out, some card-specific initialization is missed. In

[cron job] v4l-dvb daily build 2.6.22 and up: ERRORS, 2.6.16-2.6.21: ERRORS

2010-02-02 Thread Hans Verkuil
This message is generated daily by a cron job that builds v4l-dvb for the kernels and architectures in the list below. Results of the daily build of v4l-dvb: date:Tue Feb 2 19:00:06 CET 2010 path:http://www.linuxtv.org/hg/v4l-dvb changeset: 14096:860d29bfa3a7 gcc version:

Re: libv4l: possible problem found in PAC7302 JPEG decoding

2010-02-02 Thread Thomas Kaiser
On 02/02/2010 07:59 PM, Németh Márton wrote: Hi Thomas, Thomas Kaiser wrote: On 02/01/2010 10:23 PM, Németh Márton wrote: Hello Hans, while I was dealing with Labtec Webcam 2200 and with gspca_pac7302 driver I recognised the following behaviour. The stream received from the webcam is

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Mauro Carvalho Chehab
Stefan Ringel wrote: Am 01.02.2010 23:52, schrieb Mauro Carvalho Chehab: This struct is meant to pass static parameters to the driver. the analog/digital mode is dynamic, so, this is not the right place for doing it. Can you tell me how that work? Where would call it? Switch it after

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Mauro Carvalho Chehab
Stefan Ringel wrote: Ok, but maybe you missed my point: at the long term, we should get rid of hack.c, and be sure that all needed initializations are done by zl10353 driver or by tm6010-dvb. I think I all are done by zl10353 driver. I thinbk all config param is usefull and .tm6000

[PATCH] Updated procedures for patch submission

2010-02-02 Thread Mauro Carvalho Chehab
Updates the procedures to reflect the -git tree support on LinuxTV. The previous version of the document were already a little outdated, since it assumed that all patches were sent via -hg. So, email submission weren't mentioned. Also it used to mix mercurial procedures with best practices. With

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Stefan Ringel
Am 02.02.2010 21:03, schrieb Mauro Carvalho Chehab: Those tuner callback initializations are board-specific. So, it is better to test for your board model, if you need something different than what's currently done. This tuner reset works with my stick, but I think that can

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Mauro Carvalho Chehab
Stefan Ringel wrote: Am 02.02.2010 21:03, schrieb Mauro Carvalho Chehab: Those tuner callback initializations are board-specific. So, it is better to test for your board model, if you need something different than what's currently done. This tuner reset works with my stick,

Re: [cron job] v4l-dvb daily build 2.6.22 and up: ERRORS, 2.6.16-2.6.21: ERRORS

2010-02-02 Thread Németh Márton
Hi, Hans Verkuil wrote: This message is generated daily by a cron job that builds v4l-dvb for the kernels and architectures in the list below. [snip] Detailed results are available here: http://www.xs4all.nl/~hverkuil/logs/Tuesday.log linux-2.6.16.62-i686: ERRORS

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Stefan Ringel
Am 02.02.2010 21:03, schrieb Mauro Carvalho Chehab: @@ -404,6 +432,7 @@ int tm6000_init (struct tm6000_core *dev) { int board, rc=0, i, size; struct reg_init *tab; +u8 buf[40]; Why 40 ? Please avoid using magic numbers here, especially if you're not checking

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Mauro Carvalho Chehab
Stefan Ringel wrote: Am 02.02.2010 21:03, schrieb Mauro Carvalho Chehab: @@ -404,6 +432,7 @@ int tm6000_init (struct tm6000_core *dev) { int board, rc=0, i, size; struct reg_init *tab; +u8 buf[40]; Why 40 ? Please avoid using magic numbers here, especially if

Re: [PATCH] - tm6000 DVB support

2010-02-02 Thread Stefan Ringel
Am 02.02.2010 21:52, schrieb Mauro Carvalho Chehab: Stefan Ringel wrote: Am 02.02.2010 21:03, schrieb Mauro Carvalho Chehab: @@ -404,6 +432,7 @@ int tm6000_init (struct tm6000_core *dev) { int board, rc=0, i, size; struct reg_init *tab; +u8 buf[40];

[PATCH] Add support for Twinhan 1027 DVB-S card (once again)

2010-02-02 Thread Sergey Ivanov
This patch is the modified version of sfstudio (Denis Romanenko) patch, adapted for current mercurial revision. Previous patch seems to be broken. So submit once again. -- WBR Sergey Kash Ivanov. vp1027.patch Description: Binary data

Re: DM1105: could not attach frontend 195d:1105

2010-02-02 Thread Nameer Kazzaz
Nameer Kazzaz wrote: Igor M. Liplianin wrote: On 2 февраля 2010 17:21:46 Nameer Kazzaz wrote: Hi Igor, What do you think ? if I can help you solve this, let me know what I can do. Thanks Nameer Nameer Kazzaz wrote: Hi there, dmesg output with patched dm1105.c against current

[patch 3/7] drivers/media/video/Kconfig: add VIDEO_DEV dependency as needed in drivers/media/video/Kconfig

2010-02-02 Thread akpm
From: Mike Pagano mpag...@gentoo.org Add VIDEO_DEV as dependency of VIDEO_CAPTURE_DRIVERS and all of the devices listed under this setting. Signed-off-by: Mike Pagano mpag...@gentoo.org Cc: Mauro Carvalho Chehab mche...@infradead.org Signed-off-by: Andrew Morton a...@linux-foundation.org ---

[patch 2/7] drivers/media/video/pms.c needs version.h

2010-02-02 Thread akpm
From: Andrew Morton a...@linux-foundation.org i386 allmodconfig: drivers/media/video/pms.c: In function 'pms_querycap': drivers/media/video/pms.c:682: error: implicit declaration of function 'KERNEL_VERSION' Cc: Mauro Carvalho Chehab mche...@infradead.org Cc: Hans Verkuil hverk...@xs4all.nl

[patch 5/7] drivers/media/video/cx18/cx18-alsa-pcm.c: fix printk warning

2010-02-02 Thread akpm
From: Andrew Morton a...@linux-foundation.org drivers/media/video/cx18/cx18-alsa-pcm.c: In function 'cx18_alsa_announce_pcm_data': drivers/media/video/cx18/cx18-alsa-pcm.c:82: warning: format '%d' expects type 'int', but argument 5 has type 'size_t' Cc: Andy Walls awa...@radix.net Cc: Mauro

[patch 4/7] radio: add support for SAA7706H Car Radio DSP

2010-02-02 Thread akpm
From: Richard Röjfors richard.rojf...@pelagicore.com Initial support for the SAA7706H Car Radio DSP. It is a I2C device and currently the mute control is supported. When the device is unmuted it is brought out of reset and initiated using the proposed intialisation sequence. When muted the DSP

[patch 7/7] drivers/media/dvb/frontends/stv090x.c: fix use-uninitlalised

2010-02-02 Thread akpm
From: Andrew Morton a...@linux-foundation.org Mad guess. Cc: Manu Abraham m...@linuxtv.org Cc: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Andrew Morton a...@linux-foundation.org --- drivers/media/dvb/frontends/stv090x.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [patch 7/7] drivers/media/dvb/frontends/stv090x.c: fix use-uninitlalised

2010-02-02 Thread Manu Abraham
Hi Andrew, On Wed, Feb 3, 2010 at 2:40 AM, a...@linux-foundation.org wrote: From: Andrew Morton a...@linux-foundation.org Mad guess. Cc: Manu Abraham m...@linuxtv.org Cc: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Andrew Morton a...@linux-foundation.org ---  

Any saa711x users out there?

2010-02-02 Thread Devin Heitmueller
Hello all, I am doing some quality improvements for a couple of the em28xx/saa7113 designs, and I found a pretty serious problem which appears to have been there for some time. In fact, the regression was introduced when the saa7115 support was added in 2005 (hg revision 2750). This change

Re: [patch 6/7] drivers/media/dvb/frontends/stv090x.c: fix use-uninitialised

2010-02-02 Thread Manu Abraham
On Wed, Feb 3, 2010 at 2:40 AM, a...@linux-foundation.org wrote: From: Andrew Morton a...@linux-foundation.org drivers/media/dvb/frontends/stv090x.c: In function 'stv090x_blind_search': drivers/media/dvb/frontends/stv090x.c:1967: warning: 'coarse_fail' may be used uninitialized in this

Re: [PATCH] saa7134: Fix IR support of some ASUS TV-FM 7135 variants

2010-02-02 Thread hermann pitton
Hi Jean, Mauro and all, Am Dienstag, den 02.02.2010, 08:54 +0100 schrieb Jean Delvare: Hi Hermann, On Tue, 02 Feb 2010 02:47:53 +0100, hermann pitton wrote: Hi Jean, Am Montag, den 01.02.2010, 10:56 +0100 schrieb Jean Delvare: Hi Hermann, On Mon, 01 Feb 2010 02:16:35 +0100,

Re: [PATCH] uvcvideo: check minimum border of control

2010-02-02 Thread Laurent Pinchart
Hi Márton, On Tuesday 26 January 2010 22:37:52 Németh Márton wrote: Check also the minimum border of a value before setting it to a control value. See also http://bugzilla.kernel.org/show_bug.cgi?id=12824 . Signed-off-by: Márton Németh nm...@freemail.hu I've updated the previous patch

Re: [PATCH] Video : pwc : Fix regression in pwc_set_shutter_speed caused by bad constant = sizeof conversion.

2010-02-02 Thread Laurent Pinchart
On Saturday 30 January 2010 17:26:51 Martin Fuzzey wrote: Regression was caused by my commit 6b35ca0d3d586b8ecb8396821af21186e20afaf0 which determined message size using sizeof rather than hardcoded constants. Unfortunately pwc_set_shutter_speed reuses a 2 byte buffer for a one byte message

Re: Any saa711x users out there?

2010-02-02 Thread Mauro Carvalho Chehab
Devin Heitmueller wrote: Hello all, I am doing some quality improvements for a couple of the em28xx/saa7113 designs, and I found a pretty serious problem which appears to have been there for some time. In fact, the regression was introduced when the saa7115 support was added in 2005 (hg

Re: Videotext application crashes the kernel due to DVB-demux patch

2010-02-02 Thread hermann pitton
Am Dienstag, den 02.02.2010, 07:52 -0500 schrieb Andy Walls: On Tue, 2010-02-02 at 10:11 +0100, Chicken Shack wrote: Am Montag, den 01.02.2010, 21:00 -0500 schrieb Andy Walls: On Mon, 2010-02-01 at 07:41 -0500, Andy Walls wrote: On Mon, 2010-02-01 at 10:56 +0100, Chicken Shack wrote:

Re: [PATCH v2 00/10] add linux driver for chip TLG2300

2010-02-02 Thread Huang Shijie
I'm assuming that you're referring to the analog part, right? right. The country code only effects the Analog TV and radio, it has no effect on DVB-T. Instead of a country code, the driver should use the V4L2_STD_ macros to If we are in the radio mode, I do not have any video

Re: [PATCH v2 00/10] add linux driver for chip TLG2300

2010-02-02 Thread Mauro Carvalho Chehab
Huang Shijie wrote: I'm assuming that you're referring to the analog part, right? right. The country code only effects the Analog TV and radio, it has no effect on DVB-T. Instead of a country code, the driver should use the V4L2_STD_ macros to If we are in the radio mode, I do

Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb

2010-02-02 Thread Hans Verkuil
Hi Mauro, On Fri, 29 Jan 2010, Hans Verkuil wrote: (resent: forgot to mail it to the linux-media list) Hi Mauro, Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb for the following: - saa7134: remove stray unlock_kernel This is a high-prio bugfix that must go to 2.6.33! Dmitri,

Re: [patch 3/7] drivers/media/video/Kconfig: add VIDEO_DEV dependency as needed in drivers/media/video/Kconfig

2010-02-02 Thread Mauro Carvalho Chehab
a...@linux-foundation.org wrote: From: Mike Pagano mpag...@gentoo.org Add VIDEO_DEV as dependency of VIDEO_CAPTURE_DRIVERS and all of the devices listed under this setting. Signed-off-by: Mike Pagano mpag...@gentoo.org Cc: Mauro Carvalho Chehab mche...@infradead.org Signed-off-by: Andrew