Re: [PATCH] tlg2300: cleanups when power management is not configured

2010-02-28 Thread Huang Shijie
The function poseidon_fe_release is redundent in fact, I will remove it in future. But now I still acked this patch. thanks a lot. Acked-by: Huang Shijie From: Márton Németh When power management is not configured (CONFIG_PM) then some code is no longer necessary. This patch will remove the

Re: Announcing v4l-utils-0.7.90 (which includes libv4l-0.7.90)

2010-02-28 Thread Hans Verkuil
On Friday 26 February 2010 20:43:19 Hans de Goede wrote: > Hi, > > I'm happy to announce the first (test / beta) release of v4l-utils, > v4l-utils is the combination of various v4l and dvb utilities which > used to be part of the v4l-dvb mercurial kernel tree and libv4l. Is it correct that I only

Re: [PATCH] tlg2300: make local variables and functions static

2010-02-28 Thread Huang Shijie
thanks. Acked-by: Huang Shijie From: Márton Németh Make the local variables and functions static. Some of them are not exported by their symbol name but used trough other means. For example a pointer of the operation structure is passed through a function call. This will remove the followin

[PATCH] az6027: remove redundant condition check

2010-02-28 Thread Németh Márton
From: Márton Németh The condition (msg[i].addr == 0xd0) is checked twice the second one is not necessary. This will remove the following compiler warning: az6027.c: In function 'az6027_i2c_xfer': az6027.c:942: warning: 'index' may be used uninitialized in this function az6027.c:943: war

az6027: variables may be used uninitialized in az6027_i2c_xfer()

2010-02-28 Thread Németh Márton
Hi Adams, 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. > [...] > > Detailed results are available here: > > http://www.xs4all.nl/~hverkuil/logs/Sunday.log > linux-2.6.29.1-i686: WARNINGS > > /home/

Re: [PATCH 1/2] New driver for MI2020 sensor with GL860 bridge

2010-02-28 Thread Jean-Francois Moine
On Sun, 28 Feb 2010 21:19:25 +0100 Olivier Lorin wrote: > - General changes for all drivers because of new MI2020 sensor > driver : > - add the light source control > - control value changes only applied after an end of image > - replace msleep with duration less than 5 ms by a busy loop >

Re: [PATCH] tlg2300: cleanups when power management is not configured

2010-02-28 Thread Németh Márton
Hi, Huang Shijie wrote: > hi Marton, thanks a lot. > >> From: Márton Németh >> >> When power management is not configured (CONFIG_PM) then some code is no >> longer >> necessary. >> >> This patch will remove the following compiler warnings: >> * pd-dvb.c: In function 'poseidon_fe_release': >>

[IR RC, REGRESSION] Didn't work IR RC

2010-02-28 Thread Dmitri Belimov
Hi All After rework of the IR subsystem, IR RC no more work in our TV cards. As I see call saa7134_probe_i2c_ir, configure i2c call i2c_new_device New i2c device not registred. The module kbd-i2c-ir loaded after i2c_new_device. I try to found what happens. With my best regards, Dmitry. --

[PATCH] tlg2300: make local variables and functions static

2010-02-28 Thread Németh Márton
From: Márton Németh Make the local variables and functions static. Some of them are not exported by their symbol name but used trough other means. For example a pointer of the operation structure is passed through a function call. This will remove the following sparse warnings (see "make C=1"):

Re: Problems with Pinnacle 310i (saa7134) and recent kernels

2010-02-28 Thread hermann pitton
Hi Avl, Am Samstag, den 20.02.2010, 11:49 + schrieb Avl Jawrowski: > Hi, sorry for the late. > > hermann pitton arcor.de> writes: > > > > But I can't see any video. > > > With Kaffeine I can see the same channels as well. > > > > My guess is, kaffeine has a bit more trust in error correcti

Re: [PATCH] tlg2300: cleanups when power management is not configured

2010-02-28 Thread Huang Shijie
hi Marton, thanks a lot. From: Márton Németh When power management is not configured (CONFIG_PM) then some code is no longer necessary. This patch will remove the following compiler warnings: * pd-dvb.c: In function 'poseidon_fe_release': * pd-dvb.c:101: warning: unused variable 'pd' * p

Re: [PATCH] tlg2300: make local variables and functions static

2010-02-28 Thread Huang Shijie
From: Márton Németh Make the local variables and functions static. Some of them are not exported by their symbol name but used trough other means. For example a pointer of the operation structure is passed through a function call. This will remove the following sparse warnings (see "make C=1"

How do private controls actually work?

2010-02-28 Thread Devin Heitmueller
This might seem like a bit of a silly question, but I've been banging my head on the wall for a while on this. I need to add a single private control to the saa7115 driver. However, it's not clear to me exactly how this is supposed to work. The v4l2-ctl program will always use the extended contro

Re: Mantis not in modules.pcimap

2010-02-28 Thread Manu Abraham
On Mon, Mar 1, 2010 at 4:58 AM, Andy Walls wrote: > On Mon, 2010-03-01 at 00:07 +0100, Helmut Auer wrote: >> Hello, >> >> The mantis module is build and working fine with the Skystar2 HD, but it I >> cannot autodetect it, >> because modules.pcimap is not filled with the vendor id of the card usin

Re: Mantis not in modules.pcimap

2010-02-28 Thread Andy Walls
On Mon, 2010-03-01 at 00:07 +0100, Helmut Auer wrote: > Hello, > > The mantis module is build and working fine with the Skystar2 HD, but it I > cannot autodetect it, > because modules.pcimap is not filled with the vendor id of the card using > this module. > What's to do to get these ID's ? > I

Re: Mantis not in modules.pcimap

2010-02-28 Thread Manu Abraham
On Mon, Mar 1, 2010 at 3:07 AM, Helmut Auer wrote: > Hello, > > The mantis module is build and working fine with the Skystar2 HD, but it I > cannot autodetect it, > because modules.pcimap is not filled with the vendor id of the card using > this module. > What's to do  to get these ID's ? > In m

Mantis not in modules.pcimap

2010-02-28 Thread Helmut Auer
Hello, The mantis module is build and working fine with the Skystar2 HD, but it I cannot autodetect it, because modules.pcimap is not filled with the vendor id of the card using this module. What's to do to get these ID's ? In my case its a: 01:08.0 0480: 1822:4e35 (rev 01) Subsystem:

[PATCH 2/2] New driver for MI2020 sensor with GL860 bridge

2010-02-28 Thread Olivier Lorin
gspca - gl860: new driver for MI2020 sensor From: Olivier Lorin - new MI2020 driver version made from a webcam given to me - remove all previous flavors of this driver Priority: normal Signed-off-by: Olivier Lorin diff -rupN ../i/linux/drivers/media/video/gspca/gl860/gl860.c ./linux/drivers

[PATCH 1/2] New driver for MI2020 sensor with GL860 bridge

2010-02-28 Thread Olivier Lorin
gspca - gl860: updates to prepare the new driver for MI2020 sensor From: Olivier Lorin - General changes for all drivers because of new MI2020 sensor driver : - add the light source control - control value changes only applied after an end of image - replace msleep with duration less than

Re: [PATCH 10/11] ov534: Add Powerline Frequency control

2010-02-28 Thread Jean-Francois Moine
On Sun, 28 Feb 2010 20:18:50 +0100 Antonio Ospite wrote: > Maybe we could just use > V4L2_CID_POWER_LINE_FREQUENCY_DISABLED = 0, > V4L2_CID_POWER_LINE_FREQUENCY_50HZ = 1, > > It looks like the code matches the DISABLED state (writing 0 to the > register). Mosalam? I don't know

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

2010-02-28 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:Sun Feb 28 19:00:21 CET 2010 path:http://www.linuxtv.org/hg/v4l-dvb changeset: 14319:37581bb7e6f1 gcc version: i686-

Re: [PATCH 05/11] ov534: Fix setting manual exposure

2010-02-28 Thread Jean-Francois Moine
On Sun, 28 Feb 2010 19:54:25 +0100 Antonio Ospite wrote: > JF, the intent here is to cover all the range of values available in > Auto Exposure mode too, doesn't this make sense to you? > > I could set .maximum to 253 to limit the "UI" control precision but > then I should use 2*value when setti

Re: [PATCH 1/3] add feedback LED control

2010-02-28 Thread Németh Márton
Jean-Francois Moine wrote: > On Sun, 28 Feb 2010 08:55:04 +0100 > Németh Márton wrote: > >> On some webcams a feedback LED can be found. This LED usually shows >> the state of streaming mode: this is the "Auto" mode. The LED can >> be programmed to constantly switched off state (e.g. for power sa

Re: [PATCH 1/3] add feedback LED control

2010-02-28 Thread Jean-Francois Moine
On Sun, 28 Feb 2010 08:55:04 +0100 Németh Márton wrote: > On some webcams a feedback LED can be found. This LED usually shows > the state of streaming mode: this is the "Auto" mode. The LED can > be programmed to constantly switched off state (e.g. for power saving > reasons, preview mode) or on

Re: [PATCH 10/11] ov534: Add Powerline Frequency control

2010-02-28 Thread Antonio Ospite
On Sun, 28 Feb 2010 19:49:51 +0100 Jean-Francois Moine wrote: > On Sat, 27 Feb 2010 21:20:27 +0100 > Antonio Ospite wrote: > > > +static int sd_querymenu(struct gspca_dev *gspca_dev, > > + struct v4l2_querymenu *menu) > > +{ > > + switch (menu->id) { > > + case V4L2_CID_POWER_LINE

Re: [PATCH 09/11] ov534: Cosmetics: fix indentation and hex digits

2010-02-28 Thread Antonio Ospite
On Sun, 28 Feb 2010 19:46:36 +0100 Jean-Francois Moine wrote: > On Sat, 27 Feb 2010 21:20:26 +0100 > Antonio Ospite wrote: > > > * Indent with tabs, not with spaces. > > * Less indentation for controls index comments. > [snip] > > -}, > > +}, > > }; > > I had preferred one more

Re: [PATCH 05/11] ov534: Fix setting manual exposure

2010-02-28 Thread Antonio Ospite
On Sun, 28 Feb 2010 19:38:14 +0100 Jean-Francois Moine wrote: > On Sat, 27 Feb 2010 21:20:22 +0100 > Antonio Ospite wrote: > > > Exposure is now a u16 value, both MSB and LSB are set, but values in > > the v4l2 control are limited to the interval [0,506] as 0x01fa (506) > > is the maximum obser

Re: [PATCH 10/11] ov534: Add Powerline Frequency control

2010-02-28 Thread Jean-Francois Moine
On Sat, 27 Feb 2010 21:20:27 +0100 Antonio Ospite wrote: > +static int sd_querymenu(struct gspca_dev *gspca_dev, > + struct v4l2_querymenu *menu) > +{ > + switch (menu->id) { > + case V4L2_CID_POWER_LINE_FREQUENCY: > + switch (menu->index) { > + case 0:

Re: [PATCH 09/11] ov534: Cosmetics: fix indentation and hex digits

2010-02-28 Thread Jean-Francois Moine
On Sat, 27 Feb 2010 21:20:26 +0100 Antonio Ospite wrote: > * Indent with tabs, not with spaces. > * Less indentation for controls index comments. [snip] > -}, > +}, > }; I had preferred one more TAB for all controls. -- Ken ar c'hentañ | ** Breizh ha Linux atav! **

Re: [PATCH 05/11] ov534: Fix setting manual exposure

2010-02-28 Thread Jean-Francois Moine
On Sat, 27 Feb 2010 21:20:22 +0100 Antonio Ospite wrote: > Exposure is now a u16 value, both MSB and LSB are set, but values in > the v4l2 control are limited to the interval [0,506] as 0x01fa (506) > is the maximum observed value with AEC enabled. [snip] > .type= V4L2_CTRL_

[PATCH] tlg2300: make local variables and functions static

2010-02-28 Thread Németh Márton
From: Márton Németh Make the local variables and functions static. Some of them are not exported by their symbol name but used trough other means. For example a pointer of the operation structure is passed through a function call. This will remove the following sparse warnings (see "make C=1"):

Analog TV issue with Empire Dual TV (probably wrong GPIO analogue setting)

2010-02-28 Thread andrea.amoros...@gmail.com
Hi to all, I've tried my Empire Dual TV usb device which is supported and correctly recognized by the kernel. Digital TV works, but analogue tv is slow and with no audio. I suppose that there is something wrong in the GPIO analogue setting. I would like to know what I can do to solve the issue.

[PATCH] nGene: use NULL when pointer is needed

2010-02-28 Thread Németh Márton
From: Márton Németh Use NULL when calling a function with pointer parameter, initializing a pointer and returning a pointer. This will remove the following sparse warning at different locations (see "make C=1"): * warning: Using plain integer as NULL pointer Signed-off-by: Márton Németh --- di

[PATCH] tlg2300: cleanups when power management is not configured

2010-02-28 Thread Németh Márton
From: Márton Németh When power management is not configured (CONFIG_PM) then some code is no longer necessary. This patch will remove the following compiler warnings: * pd-dvb.c: In function 'poseidon_fe_release': * pd-dvb.c:101: warning: unused variable 'pd' * pd-video.c:14: warning: 'pm_vid

Re: cx18: where do the transport stream PIDs come from?

2010-02-28 Thread Andy Walls
On Fri, 2010-02-26 at 17:23 +0100, Lars Hanisch wrote: > Hi, > > while working on a small test app which repacks the ivtv-PS into a TS, I > received a sample from a cx18-based card. The > TS contains the video PID 301, audio PID 300 and PCR pid 101. > > Where do these PIDs come from, are th

Re: [PATCH] Input: scancode in get/set_keycodes should be unsigned

2010-02-28 Thread Németh Márton
Dmitry Torokhov wrote: > On Sun, Feb 28, 2010 at 07:44:32AM +0100, Németh Márton wrote: >> Hi, >> Dmitry Torokhov wrote: >>> The HID layer has some scan codes of the form 0xffbc for logitech >>> devices which do not work if scancode is typed as signed int, so we need >>> to switch to unsigned i