Re: [PATCH] media: add support to gspca/pac7302.c for 093a:2627 (Genius FaceCam 300)

2012-05-15 Thread Németh Márton
Acked-by: Márton Németh nm...@freemail.hu Jozsef Marton wrote: From: Jozsef Marton jmar...@users.sourceforge.net gspca_pac7302 module supports the webcam with usb id: 093a:2627. It is a Genius FaceCam 300. The module does not need any changes but listing the usb id along with a vertical

Re: [PATCH] v4l2: handle multiplication overflow

2012-01-23 Thread Németh Márton
Mauro Carvalho Chehab írta: Em 22-12-2011 07:42, Németh Márton escreveu: From: Márton Németh nm...@freemail.hu When a V4L2 ioctl is executed the memory provided by the caller in user space is copied to the kernel space in video_usercopy() function. To find out how many bytes has to be copied

[PATCH] v4l2: handle multiplication overflow

2011-12-22 Thread Németh Márton
From: Márton Németh nm...@freemail.hu When a V4L2 ioctl is executed the memory provided by the caller in user space is copied to the kernel space in video_usercopy() function. To find out how many bytes has to be copied the check_array_args() helper function is used. This patch adds a check for

Re: pac7311

2011-10-20 Thread Németh Márton
Lars Noschinski wrote: * Németh Márton nm...@freemail.hu [11-10-18 22:14]: Hi Lars, Lars Noschinski wrote: I'm using a webcam (Philipps SPC500NC) which identifies itself as 093a:2603 Pixart Imaging, Inc. PAC7312 Camera and is sort-of supported by the gspca_pac7311 module. sort

Re: pac7311

2011-10-18 Thread Németh Márton
Hi Lars, Lars Noschinski wrote: I'm using a webcam (Philipps SPC500NC) which identifies itself as 093a:2603 Pixart Imaging, Inc. PAC7312 Camera and is sort-of supported by the gspca_pac7311 module. sort-of because the image alternates quickly between having a red tint or a green tint

gspca: video0 becomes video1 after ISOC data error

2011-07-04 Thread Németh Márton
Hi, I'm running Debian with Linux 2.6.38-2-486 on a computer. I have a hama AC-150 webcam attached to this computer. The webcam works continouosly and I use mencoder to do some cropping and transformation and to encode the video stream to a file. When I plug the device the following appears in

Re: [PATCH RFC] gspca_sonixj: handle return values from USB subsystem

2010-10-18 Thread Németh Márton
Hi, Jean-Francois Moine írta: On Sun, 17 Oct 2010 16:45:03 +0200 Németh Márton nm...@freemail.hu wrote: The usb_control_msg() may return error at any time so it is necessary to handle it. The error handling mechanism is taken from the pac7302. The resulting driver was tested with hama AC

Re: [PATCH 1/2] gspca_sonixj: add hardware vertical flip support for hama AC-150

2010-10-18 Thread Németh Márton
Hi Jean-Francois, Jean-Francois Moine wrote: On Sun, 17 Oct 2010 13:08:01 +0200 Németh Márton nm...@freemail.hu wrote: The PO2030N sensor chip found in hama AC-150 webcam supports vertical flipping the image by hardware. Add support for this in the gspca_sonixj driver also. [snip

[PATCH 2/2] gspca_sonixj: add hardware horizontal flip support for hama AC-150

2010-10-17 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The PO2030N sensor chip found in hama AC-150 webcam supports horizontal flipping the image by hardware. Add support for this in the gspca_sonixj driver also. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -upr c/drivers/media/video/gspca/sonixj.c

[PATCH 1/2] gspca_sonixj: add hardware vertical flip support for hama AC-150

2010-10-17 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The PO2030N sensor chip found in hama AC-150 webcam supports vertical flipping the image by hardware. Add support for this in the gspca_sonixj driver also. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -upr b/drivers/media/video/gspca/sonixj.c

[PATCH] gspca_sonixj: remove magic numbers for delay

2010-10-17 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The number 0xdd is used for marking delay init sequence steps. Replace 0xdd values only if the meaning is delay. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -upr a/drivers/media/video/gspca/sonixj.c b/drivers/media/video/gspca/sonixj.c ---

Re: em28xx in v4l-dvb destroyed my USB TV card

2010-10-17 Thread Németh Márton
Hi, Marius Bjørnstad wrote: A problem with the em28xx driver was brought up in June by Thorsten Hirsch: http://www.spinics.net/lists/linux-media/msg20588.html . I also have a TerraTec Cinergy Hybrid T USB XS. When I used my device with Linux, it would take a long time to be recognised by the

[PATCH RFC] gspca_sonixj: handle return values from USB subsystem

2010-10-17 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The usb_control_msg() may return error at any time so it is necessary to handle it. The error handling mechanism is taken from the pac7302. The resulting driver was tested with hama AC-150 webcam (USB ID 0c45:6142). Signed-off-by: Márton Németh

Re: Old patches sent via the Mailing list

2010-10-17 Thread Németh Márton
Mauro Carvalho Chehab wrote: == Gspca patches - Waiting Hans de Goede hdego...@redhat.com review == Jan,29 2010: [gspca_jf,tree] gspca zc3xx: signal when unknown packet received http://patchwork.kernel.org/patch/75837 Németh Márton nm...@freemail.hu This was

mem2mem_testdev: BUG: unable to handle kernel NULL pointer dereference

2010-06-21 Thread Németh Márton
Hi, I tried the mem2mem_testdev driver from Linux kernel 2.6.35-rc3 together with the test environment from http://v4l-test.sourceforge.net . I found that the VIDIOC_G_FMT ioctl may result in a NULL pointer dereference in kernel. I derived a simple test case which I also attached. Steps to

Re: genius islim 310 webcam test

2010-06-14 Thread Németh Márton
Hi Sergei, thanks for the report. Hi Jean-Francois, I got this report about a working Genius iSlim 310 webcam. Maybe it would be a good idea to add the device 0x093a:0x2625 in pac7302.c. Should I send a patch for you? Regards, Márton Németh Krivchikov Sergei wrote: Hi! All works.

[PATCH] gspca_pac7302: add Genius iSlim 310

2010-06-14 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Add Genius iSlim 310 webcam to the supported list of the PAC7302 driver. For more information see http://linuxtv.org/wiki/index.php/PixArt_PAC7301/PAC7302 . Signed-off-by: Márton Németh nm...@freemail.hu --- diff --git a/Documentation/video4linux/gspca.txt

Re: genius islim 310 webcam test

2010-03-31 Thread Németh Márton
Jean-Francois Moine írta: On Wed, 31 Mar 2010 07:56:59 +0200 Németh Márton nm...@freemail.hu wrote: The next thing is that you need to learn how to compile the Linux kernel from source code. There is a description for Ubuntu at https://help.ubuntu.com/community/Kernel/Compile . After you

Re: genius islim 310 webcam test

2010-03-30 Thread Németh Márton
Hello Sergei, Krivchikov Sergei wrote: Hi! I have a Genius iSlim 310 webcam and linux ubuntu 9.10 kernel 2.6.31. My developer level under linux - beginner (very beginner) :) but i'm ready to test this webcam undr linux. But I need the instructions:) How was your success in the launch of this

Sony DCR-HC23 + USB (0540:00c0) + Linux?

2010-03-17 Thread Németh Márton
Hi, does anybody have experience connecting Sony DCR-HC23 Handycam to Linux through USB? I send the USB descriptor below. The device appears only after selecting the FN/MENU/SETUP MENU/USB STREAM menu item of the handycam to ON. When I select OFF and press EXEC button then the device disconnects

Re: problem compiling modoule mt9t031 in current v4l-dvb-hg

2010-03-07 Thread Németh Márton
Hi, Halim Sahin wrote: Hi Folks, I was not able to build v4l-dvb from hg (checked out today). /root/work/v4l-dvb/v4l/mt9t031.c:729: error: unknown field 'runtime_suspend' specified in initializer /root/work/v4l-dvb/v4l/mt9t031.c:730: error: unknown field 'runtime_resume' specified in

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

2010-03-07 Thread Németh Márton
Hans Verkuil írta: On Sunday 07 March 2010 13:20:16 Németh Márton wrote: Hello Hans, 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. Would you mind adding a new test case for your test environment

Re: [PATCH 1/2] bttv: Move I2C IR initialization

2010-03-06 Thread Németh Márton
Hi, Jean Delvare wrote: Move I2C IR initialization from just after I2C bus setup to right before non-I2C IR initialization. This avoids the case where an I2C IR device is blocking audio support (at least the PV951 suffers from this). It is also more logical to group IR support together,

[PATCH] bttv: fix compiler warning before kernel 2.6.30

2010-03-06 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Fix the following compiler warnings when compiling before Linux kernel version 2.6.30: bttv-i2c.c: In function 'init_bttv_i2c': bttv-i2c.c:440: warning: control reaches end of non-void function Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r

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

2010-03-06 Thread Németh Márton
Hello Hans, 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. Results of the daily build of v4l-dvb: date:Sat Mar 6 19:00:21 CET 2010 path:http://www.linuxtv.org/hg/v4l-dvb

[PATCH] gspca cpia1: make local functions static

2010-03-06 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Make the local functions static. Note that the function command_setlights() is currently not called from anywhere. This will remove the following sparse warnings (see make C=1): * symbol 'command_setformat' was not declared. Should it be static? * symbol

[RFC, PATCH 1/3] gspca: add LEDs subsystem connection

2010-03-02 Thread Németh Márton
From: Márton Németh nm...@freemail.hu On some webcams one or more LEDs can be found. One type of these LEDs are feedback LEDs: they usually shows the state of streaming mode. The LED can be programmed to constantly switched off state (e.g. for power saving reasons, preview mode) or on state (e.g.

[RFC, PATCH 2/3] gspca pac7302: separate LED handling

2010-03-02 Thread Németh Márton
From: Márton Németh nm...@freemail.hu On Labtec Webcam 2200 there is a feedback LED which can be controlled independent from the streaming. The feedback LED can be used from user space application to show for example detected motion or to distinguish between the preview and on-air state of the

[RFC, PATCH 3/3] gspca pac7302: remove LED blinking when switching stream on and off

2010-03-02 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The init sequences for PAC7302 contained register settings affecting the LED state which can result blinking of the LED when it is set to always on or always off. The blinking happened when the stream was switched on or off. Remove the register changes from

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 int instead

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

2010-02-28 Thread Németh Márton
From: Márton Németh nm...@freemail.hu 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' *

[PATCH] nGene: use NULL when pointer is needed

2010-02-28 Thread Németh Márton
From: Márton Németh nm...@freemail.hu 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:

[PATCH] tlg2300: make local variables and functions static

2010-02-28 Thread Németh Márton
From: Márton Németh nm...@freemail.hu 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

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 nm...@freemail.hu 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

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

[PATCH] az6027: remove redundant condition check

2010-02-28 Thread Németh Márton
From: Márton Németh nm...@freemail.hu 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

Re: [PATCH 1/2] gspca pac7302: allow controlling LED separately

2010-02-27 Thread Németh Márton
Hi, I missed the CONFIG_LEDS_CLASS_MODULE configuration option in the previous version of this patch, now it is added. Regards, Márton Németh --- From: Márton Németh nm...@freemail.hu On Labtec Webcam 2200 there is a feedback LED which can be controlled independent from the streaming.

Re: [PATCH 1/2] gspca pac7302: allow controlling LED separately

2010-02-27 Thread Németh Márton
Hi, Hans de Goede wrote: Hi, On 02/27/2010 01:20 AM, Németh Márton wrote: From: Márton Némethnm...@freemail.hu On Labtec Webcam 2200 there is a feedback LED which can be controlled independent from the streaming. This is true for a lot of cameras, so if we are going to add a way

Re: [PATCH 1/2] gspca pac7302: allow controlling LED separately

2010-02-27 Thread Németh Márton
Hi, Jean-Francois Moine wrote: On Sat, 27 Feb 2010 08:53:16 +0100 Hans de Goede hdego...@redhat.com wrote: This is true for a lot of cameras, so if we are going to add a way to support control of the LED separate of the streaming state, we should do that at the gspca_main level, and let sub

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

2010-02-27 Thread Németh Márton
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 int instead. While at it keycode being signed does not make much sense either. Are the scan codes and

[PATCH 1/3] add feedback LED control

2010-02-27 Thread Németh Márton
From: Márton Németh nm...@freemail.hu 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 state (e.g. the application

[PATCH 2/3] gspca pac7302: add LED control

2010-02-27 Thread Németh Márton
From: Márton Németh nm...@freemail.hu On Labtec Webcam 2200 there is a feedback LED which can be controlled independent from the streaming. The feedback LED can be used from user space application to show for example detected motion or to distinguish between the preview and on-air state of the

[PATCH 3/3] gspca pac7302: remove LED blinking when switching stream on and off

2010-02-27 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The init sequences for PAC7302 contained register settings affecting the LED state which can result blinking of the LED when it is set to always on or always off. The blinking happened when the stream was switched on or off. Remove the register changes from

[PATCH 1/2] gspca pac7302: allow controlling LED separately

2010-02-26 Thread Németh Márton
From: Márton Németh nm...@freemail.hu On Labtec Webcam 2200 there is a feedback LED which can be controlled independent from the streaming. The feedback LED can be used from user space application to show for example detected motion or to distinguish between the preview and on-air state of the

[PATCH 2/2] gspca pac7302: remove LED blinking when switching stream on and off

2010-02-26 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The init sequences for PAC7302 contained register settings affecting the LED state which can result blinking of the LED when it is set to always on or always off. The blinking happened when the stream was switched on or off. Remove the register changes from

Linux testers wanted for Genius iSlim 310

2010-02-24 Thread Németh Márton
Hi, based on the information from installing the Windows driver the Genius iSlim 310 is a potential candidate that the gspca_pac7302 driver under Linux may handle, see http://linuxtv.org/wiki/index.php/PixArt_PAC7301/PAC7302#Identification . If you have access to Genius iSlim 310 and would like

Re: [PATCH] soc_camera: match signedness of soc_camera_limit_side()

2010-02-23 Thread Németh Márton
Hi Guennadi, Guennadi Liakhovetski wrote: Hi Németh On Tue, 9 Feb 2010, Guennadi Liakhovetski wrote: Ok, I modified your patch a bit, how about the below version? If you agree, I'll commit it in that form (after converting to utf-8): From: Márton Németh nm...@freemail.hu The first two

[PATCH] gspca pac7302: add USB PID range based on heuristics

2010-02-23 Thread Németh Márton
From: Márton Németh nm...@freemail.hu On the schematics in PixArt PAC7301/PAC7302 datasheet (http://www.pixart.com.tw/upload/PAC7301_7302%20%20Spec%20V1_20091228174030.pdf) pages 19, 20, 21 and 22 there is a note titled PID IO_TRAP which describes the possible product ID range 0x2620..0x262f. In

Re: [PATCH libv4l tree, RFC] libv4l: skip false Pixart markers with buffer copy

2010-02-04 Thread Németh Márton
Hi, This is a proof-of-concept patch to try to decode the JPEG with PixArt markers. Please check whether it is working at your side. My experience is that the number of frames with glitch are reduced. Regards, Márton Németh --- From: Márton Németh nm...@freemail.hu Before trying to

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

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

libv4l: possible problem found in PAC7302 JPEG decoding

2010-02-01 Thread Németh Márton
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, 0xff, 0x96 is found (pac_find_sof()). Before

[PATCH ] libv4l: skip false Pixart markers

2010-02-01 Thread Németh Márton
From: Márton Németh nm...@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 to pass the first 0xff byte as an image data.

Re: PAC7302 short datasheet from PixArt

2010-01-31 Thread Németh Márton
Hi, Theodore Kilgore wrote: On Sat, 30 Jan 2010, Németh Márton wrote: Hi, if anyone interested there is a brief overview datasheet about PixArt PAC7301/PAC7302 at http://www.pixart.com.tw/upload/PAC7301_7302%20%20Spec%20V1_20091228174030.pdf [...] Now, as to the substance of the mail

[PATCH, RFC] gspca pac7302: add USB PID range based on heuristics

2010-01-31 Thread Németh Márton
Hi, as I was reading the PixArt PAC7301/PAC7302 datasheet ( http://www.pixart.com.tw/upload/PAC7301_7302%20%20Spec%20V1_20091228174030.pdf ) I recognised a little description on the schematics. This is about how to set up the USB Product ID from range 0x2620..0x262f via hardware wires. I had

pac7302 sporadic plug-in problem fix for 2.6.33?

2010-01-30 Thread Németh Márton
Dear Jean-Francois and Mauro, I just compiled the vanilla 2.6.33-rc6 kernel and tested together with Labtec Webcam 2200. I realized that this version still has the plug-in problem which was fixed at http://linuxtv.org/hg/~jfrancois/gspca/rev/ea88b3abee04 What do you think, is it possible

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

2010-01-29 Thread Németh Márton
Mauro Carvalho Chehab wrote: 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 . Patch didn't apply. Had you generated against our -git tree? http://git.linuxtv.org/v4l

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

2010-01-29 Thread Németh Márton
Mauro Carvalho Chehab wrote: Németh Márton wrote: Mauro Carvalho Chehab wrote: 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 . Patch didn't apply. Had you generated against our

Re: Make failed - standard ubuntu 9.10

2010-01-29 Thread Németh Márton
David Henig wrote: Thanks, this is sounding promising, only thing is I'm not seeing a .config in the v4l directory although it shows up with the locate command, am I missing something very obvious. Sorry if I'm telling evidence, but just a hint: every file starts with . is a hidden file. So

[PATCH gspca_jf tree] gspca zc3xx: signal when unknown packet received

2010-01-29 Thread Németh Márton
Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 95d3956ea3e5 linux/drivers/media/video/gspca/zc3xx.c --- a/linux/drivers/media/video/gspca/zc3xx.c Fri Jan 29 15:05:25 2010 +0100 +++ b/linux/drivers/media/video/gspca/zc3xx.c Fri Jan 29 21:01:52 2010 +0100 @@ -7213,14 +7213,17 @@

[PATCH] hdpvr-core: make module parameters local

2010-01-29 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The default_video_input and default_audio_input module parameters are only used inside the hdpvr-core.c file so make them static. This will remove the following sparse warnings (see make C=1): * warning: symbol 'default_video_input' was not declared. Should

[PATCH] smsir: make local variables static

2010-01-29 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Make the file local parameters static. This will remove the following sparse warnings (see make C=1): * warning: symbol 'ir_pos' was not declared. Should it be static? * warning: symbol 'ir_word' was not declared. Should it be static? * warning: symbol

Re: [PATCH gspca_jf tree] gspca zc3xx: signal when unknown packet received

2010-01-29 Thread Németh Márton
TAS5130-VF250 Philips SPC 200NC 0471:0325 zc3xx PAS106 Creative WebCam NX Pro 041e:401e zc3xx HV7131B No brand0ac8:307b zc3xx ADCM2700 Regards, Hans On 01/29/2010 09:07 PM, Németh Márton wrote: Signed-off-by: Márton

PAC7302 short datasheet from PixArt

2010-01-29 Thread Németh Márton
Hi, if anyone interested there is a brief overview datasheet about PixArt PAC7301/PAC7302 at http://www.pixart.com.tw/upload/PAC7301_7302%20%20Spec%20V1_20091228174030.pdf Regards, Márton Németh -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a

[PATCH] gspca pac7302: add support for camera button

2010-01-28 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Add support for snapshot button found on Labtec Webcam 2200. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 875c200a19dc linux/drivers/media/video/gspca/pac7302.c --- a/linux/drivers/media/video/gspca/pac7302.c Sun Jan 17 07:58:51 2010 +0100 +++

Re: [PATCH] gspca pac7302: add support for camera button

2010-01-28 Thread Németh Márton
Hello Jean-Francois, thank you for accepting this patch and for the previous suggestions which made it possible to reach the current state of this patchset. Also thanks for correcting the character encoding in my name. Regards, Márton Németh Németh Márton írta: From: Márton Németh

Re: [PATCH] soc_camera: match signedness of soc_camera_limit_side()

2010-01-28 Thread Németh Márton
Guennadi Liakhovetski wrote: On Wed, 27 Jan 2010, Németh Márton wrote: Guennadi Liakhovetski wrote: You didn't reply to my most important objection: On Wed, 27 Jan 2010, Németh Márton wrote: diff -r 31eaa9423f98 linux/include/media/soc_camera.h --- a/linux/include/media/soc_camera.h

Re: Make failed - standard ubuntu 9.10

2010-01-28 Thread Németh Márton
David Henig wrote: Please can someone assist, not sure what the cause of the below is? This is my second attempt to get linux tv to work, I suspect it's a basic level error - sorry I'm fairly new to Linux... output below, I'm running a fairly standard ubuntu 9.10 setup. make[1]: Entering

Re: [PATCH] soc_camera: match signedness of soc_camera_limit_side()

2010-01-27 Thread Németh Márton
Guennadi Liakhovetski wrote: On Sat, 23 Jan 2010, Németh Márton wrote: From: Márton Németh nm...@freemail.hu The parameters of soc_camera_limit_side() are either a pointer to a structure element from v4l2_rect, or constants. The structure elements of the v4l2_rect are signed (see linux

Re: [PATCH] soc_camera: match signedness of soc_camera_limit_side()

2010-01-27 Thread Németh Márton
Guennadi Liakhovetski wrote: You didn't reply to my most important objection: On Wed, 27 Jan 2010, Németh Márton wrote: diff -r 31eaa9423f98 linux/include/media/soc_camera.h --- a/linux/include/media/soc_camera.h Mon Jan 25 15:04:15 2010 -0200 +++ b/linux/include/media/soc_camera.h

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

2010-01-27 Thread Németh Márton
Hans Verkuil wrote: On Wednesday 27 January 2010 22:48:24 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. It's up and running again. Note that I upgraded to the latest gcc 4.4.3. I also updated all

Re: git problem with uvcvideo

2010-01-25 Thread Németh Márton
Laurent Pinchart wrote: Hi Márton, On Sunday 24 January 2010 22:31:29 Németh Márton wrote: Hi, I'm trying to fetch the uvcvideo from http://linuxtv.org/git/?p=pinchartl/uvcvideo.git;a=summary . I tryied to follow the instructions but at the third step I get fatal error messages

[RFC, PATCH] gspca pac7302: propagate footer to userspace

2010-01-24 Thread Németh Márton
Hi, I'm dealing with Labtec Webcam 2200 and I found that the pac7302 driver does not forward the image footer information to userspace. This footer contains some information which might be interesting to the userspace. What exactly this footer means is not clear as of this writing, but it is

git problem with uvcvideo

2010-01-24 Thread Németh Márton
Hi, I'm trying to fetch the uvcvideo from http://linuxtv.org/git/?p=pinchartl/uvcvideo.git;a=summary . I tryied to follow the instructions but at the third step I get fatal error messages: $ git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git v4l-dvb Initialized

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

2010-01-23 Thread Németh Márton
Jean-Francois Moine írta: On Fri, 22 Jan 2010 18:54:46 -0200 Mauro Carvalho Chehab mche...@infradead.org wrote: Huh? -static int reg_w_buf(struct gspca_dev *gspca_dev, +static void reg_w_buf(struct gspca_dev *gspca_dev, __u8 index, const char *buffer,

[PATCH] soc_camera: match signedness of soc_camera_limit_side()

2010-01-23 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The parameters of soc_camera_limit_side() are either a pointer to a structure element from v4l2_rect, or constants. The structure elements of the v4l2_rect are signed (see linux/videodev2.h) so do the computations also with signed values. This will remove

[PATCH] zoran: remove variable shadowing

2010-01-23 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The loop counter j is declared twice in function error_handler(). Remove the redundant declaration. This will remove the following sparse warning (see make C=1): * symbol 'j' shadows an earlier one Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r

[PATCH] hdpvr-video: cleanup signedness

2010-01-23 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The fifth parameter of usb_bulk_msg() is a pointer to signed (see linux/usb.h) so also call this function with pointer to signed. This will remove the following sparse warning (see make C=1): * warning: incorrect type in argument 5 (different signedness)

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

2010-01-23 Thread Németh Márton
Mauro Carvalho Chehab írta: Jean-Francois Moine wrote: On Sat, 23 Jan 2010 09:10:44 +0100 Németh Márton nm...@freemail.hu wrote: is more readable, smaller and quicker (less MMU switches) than: What do you mean under MMU switches? The MMU has an associative memory which is used in the first

[PATCH] zoran: match parameter signedness of g_input_status

2010-01-22 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The second parameter of g_input_status operation in media/v4l2-subdev.h is unsigned so also call it with unsigned paramter. This will remove the following sparse warning (see make C=1): * incorrect type in argument 2 (different signedness) expected

Re: [PATCH 1/2, RFC] gspca: add input support for interrupt endpoints

2010-01-18 Thread Németh Márton
increases during this review phase. On 01/17/2010 02:08 PM, Németh Márton wrote: [snip] +#ifdef CONFIG_INPUT +#if LINUX_VERSION_CODE KERNEL_VERSION(2, 6, 19) +static void int_irq(struct urb *urb, struct pt_regs *regs) +#else +static void int_irq(struct urb *urb) +#endif +{ +struct

Re: [PATCH 1/2, RFC] gspca: add input support for interrupt endpoints

2010-01-18 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Add support functions for interrupt endpoint based input handling. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 875c200a19dc linux/drivers/media/video/gspca/gspca.c --- a/linux/drivers/media/video/gspca/gspca.c Sun Jan 17 07:58:51 2010 +0100

[PATCH 1/2, RFC] gspca: add input support for interrupt endpoints

2010-01-17 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Add support functions for interrupt endpoint based input handling. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 875c200a19dc linux/drivers/media/video/gspca/gspca.c --- a/linux/drivers/media/video/gspca/gspca.c Sun Jan 17 07:58:51 2010 +0100

[PATCH 2/2, RFC] gspca pac7302: add support for camera button

2010-01-17 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Add support for snapshot button found on Labtec Webcam 2200. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 875c200a19dc linux/drivers/media/video/gspca/pac7302.c --- a/linux/drivers/media/video/gspca/pac7302.c Sun Jan 17 07:58:51 2010 +0100 +++

[PATCH] zoran: cleanup pointer condition

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Remove the following sparse warning (see make C=1): * warning: Using plain integer as NULL pointer Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 5bcdcc072b6d linux/drivers/media/video/zoran/zoran_driver.c ---

[PATCH] disable building cx23885 before 2.6.33

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The cx23885 driver does not compile before Linux kernel 2.6.33 because of incompatible fifo API changes. Disable this driver being built before 2.6.33. Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 5bcdcc072b6d v4l/versions.txt ---

[PATCH] cx231xx: cleanup dvb_attach() return value handling

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Remove the following sparse error (see make C=1): * error: incompatible types for operation () left side has type struct dvb_frontend * right side has type int Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 5bcdcc072b6d

[PATCH] stv0900: make local functions static

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The functions stv0900_sw_algo() and stv0900_set_dvbs1_track_car_loop() are only used locally so mark them static. This will remove the following sparse warnings (see make C=1): * symbol 'stv0900_sw_algo' was not declared. Should it be static? * symbol

[PATCH] stv0900: make more local functions static

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu Some functions are only used locally so mark them static. This will remove the following sparse warnings (see make C=1): * symbol 'extract_mask_pos' was not declared. Should it be static? * symbol 'stv0900_initialize' was not declared. Should it be static?

[PATCH] bt819: cleanup v4l2_subdev_notify() parameters

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu The 3rd parameter v4l2_subdev_notify() is passed to the notify() callback which is a pointer, see media/v4l2-subdev.h and media/v4l2-device.h. This will remove the following sparse warning (see make C=1): * Using plain integer as NULL pointer

[PATCH] dib0090: cleanup dib0090_dcc_freq()

2010-01-16 Thread Németh Márton
From: Márton Németh nm...@freemail.hu 'extern' is not needed at function definition. This will remove the following sparse warning (see make C=1): * function 'dib0090_dcc_freq' with external linkage has definition Signed-off-by: Márton Németh nm...@freemail.hu --- diff -r 5bcdcc072b6d

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

2010-01-14 Thread Németh Márton
Hans Verkuil wrote: Detailed results are available here: http://www.xs4all.nl/~hverkuil/logs/Thursday.log linux-2.6.32-i686: ERRORS /marune/build/v4l-dvb-master/v4l/cx23888-ir.c: In function 'cx23888_ir_irq_handler': CC [M] /marune/build/v4l-dvb-master/v4l/cx23885-f300.o

Re: gspca_pac7302: sporatdic problem when plugging the device

2010-01-11 Thread Németh Márton
Jean-Francois Moine írta: Sorry, I forgot to attach the patch. In some usbsnoop files I have, the index 0x48 is not loaded. May you try the attached patch (skip the index 0x48 and remove the delay). 100 plug-ins out of 100 OK, no errors: I say go for this patch. I applied the patch on top

Re: [PATCH 2/2] gspca pac7302: add support for camera button

2010-01-10 Thread Németh Márton
Mauro Carvalho Chehab worte: Németh Márton wrote: From: Márton Németh nm...@freemail.hu Add support for snapshot button found on Labtec Webcam 2200 (USB ID 093a:2626 This patch breaks compilation against current -tip: There was several versions of adding support to pac7302. The last

Re: [PATCH 2/2] gspca pac7302: add support for camera button

2010-01-10 Thread Németh Márton
Mauro Carvalho Chehab írta: Németh Márton wrote: Mauro Carvalho Chehab worte: Németh Márton wrote: From: Márton Németh nm...@freemail.hu Add support for snapshot button found on Labtec Webcam 2200 (USB ID 093a:2626 This patch breaks compilation against current -tip: There was several

gspca_sunplus problem: more than one device is created

2010-01-10 Thread Németh Márton
Hi, I tried the gspca_sunplus driver from http://linuxtv.org/hg/~jfrancois/gspca/ rev 13915 on top of Linux kernel 2.6.32. When I plug the Trust 610 LCD pow...@m Zoom device in webcam mode (0x06d6:0x0031) then two devices are created: /dev/video0 and /dev/video1: [31636.528184] usb 3-2: new

gspca_pac7302: sporatdic problem when plugging the device

2010-01-10 Thread Németh Márton
Hi, I have sporadic problem with Labtec Webcam 2200 (0x093a:0x2626). I'm using gspca_pac7302 driver from http://linuxtv.org/hg/~jfrancois/gspca/ rev 13915 on top of Linux kernel 2.6.32. I executed the following command in an xterm window: $ while true; do ./svv; done Then I plugged and

Re: gspca_sunplus problem: more than one device is created

2010-01-10 Thread Németh Márton
2010/1/10 Németh Márton nm...@freemail.hu: Hi, I tried the gspca_sunplus driver from http://linuxtv.org/hg/~jfrancois/gspca/ rev 13915 on top of Linux kernel 2.6.32. When I plug the Trust 610 LCD pow...@m Zoom device in webcam mode (0x06d6:0x0031) then two devices are created: /dev/video0

Re: gspca_pac7302: sporatdic problem when plugging the device

2010-01-10 Thread Németh Márton
Jean-Francois Moine wrote: On Sun, 10 Jan 2010 17:58:58 +0100 Németh Márton nm...@freemail.hu wrote: Then I plugged and unplugged the device 16 times. When I last plugged the device I get the following error in the dmesg: [32393.421313] gspca: probing 093a:2626 [32393.426193] gspca

  1   2   3   >