gspca, audio and ov534: regression.

2010-10-06 Thread Antonio Ospite
-- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing? pgpXAx1oGhpGx.pgp Description: PGP signature

Re: gspca, audio and ov534: regression.

2010-10-06 Thread Antonio Ospite
On Wed, 6 Oct 2010 13:48:55 +0200 Jean-Francois Moine moin...@free.fr wrote: On Wed, 6 Oct 2010 12:33:21 +0200 Antonio Ospite osp...@studenti.unina.it wrote: with 2.6.36-rc6 I can't use the ov534 gspca subdriver (with PS3 eye) anymore, when I try to capture video in dmesg I get: gspca

Re: gspca, audio and ov534: regression.

2010-10-06 Thread Antonio Ospite
On Wed, 6 Oct 2010 16:04:41 +0200 Antonio Ospite osp...@studenti.unina.it wrote: On Wed, 6 Oct 2010 13:48:55 +0200 Jean-Francois Moine moin...@free.fr wrote: On Wed, 6 Oct 2010 12:33:21 +0200 Antonio Ospite osp...@studenti.unina.it wrote: with 2.6.36-rc6 I can't use the ov534 gspca

Re: gspca, audio and ov534: regression.

2010-10-10 Thread Antonio Ospite
On Thu, 7 Oct 2010 19:44:01 +0200 Jean-Francois Moine moin...@free.fr wrote: On Wed, 6 Oct 2010 16:53:37 +0200 Antonio Ospite osp...@studenti.unina.it wrote: PS3 Eye audio is working with linux-2.6.33.7 it is broken in linux-2.6.35.7 already, I'll try to further narrow down

Re: gspca, audio and ov534: regression.

2010-10-10 Thread Antonio Ospite
On Wed, 6 Oct 2010 20:35:53 +0200 Jean-Francois Moine moin...@free.fr wrote: On Wed, 6 Oct 2010 16:04:41 +0200 Antonio Ospite osp...@studenti.unina.it wrote: Thanks, the following change fixes it, was this what you had in mind? diff --git a/drivers/media/video/gspca/gspca.c

Re: gspca, audio and ov534: regression.

2010-10-10 Thread Antonio Ospite
On Sun, 10 Oct 2010 12:21:29 +0200 Jean-Francois Moine moin...@free.fr wrote: On Sun, 10 Oct 2010 12:02:50 +0200 Antonio Ospite osp...@studenti.unina.it wrote: JF this change as is does not work for me, if I change the second check to if (gspca_dev-audio i 1) it does, but I

Re: [GIT PATCHES FOR 2.6.36] gspca for_2.6.36

2010-10-15 Thread Antonio Ospite
info at http://vger.kernel.org/majordomo-info.html -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing? pgpz9ZBF3yqXo.pgp

Re: gspca for_2.6.36 - maybe does not work properly for me (ov534)

2010-11-03 Thread Antonio Ospite
-media@vger.kernel.org/msg23039.html This is the patch: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=f43402fa55bf5e7e190c176343015122f694857c Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which

[PATCH] [media] gspca - main: Fix a regression with the PS3 Eye webcam

2010-11-05 Thread Antonio Ospite
or only alternate setting. Reported-by: Antonio Ospite osp...@studenti.unina.it Tested-by: Antonio Ospite osp...@studenti.unina.it Signed-off-by: Jean-Fran├žois Moine moin...@free.fr Signed-off-by: Mauro Carvalho Chehab mche...@redhat.com --- This is to be applied to 2.6.36 only, as the regression

Re: [PATCH 2/2] v4l: Remove module_name argument to the v4l2_i2c_new_subdev* functions

2010-11-09 Thread Antonio Ospite
On Tue, 9 Nov 2010 16:30:28 +0100 Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: The argument isn't used anymore by the functions, remote it. s/remote/remove/ Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order

Re: Bounty for the first Open Source driver for Kinect

2010-11-10 Thread Antonio Ospite
with a proper gspca driver (I helped with the PS3 Eye already). Are there other RGB-Depth cams supported in linux? Are they usually exposed just as two distinct cameras? Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which

Re: Bounty for the first Open Source driver for Kinect

2010-11-10 Thread Antonio Ospite
On Wed, 10 Nov 2010 11:11:59 -0500 Steven Toth st...@kernellabs.com wrote: On 11/10/10 10:54 AM, Antonio Ospite wrote: On Wed, 10 Nov 2010 15:20:40 +0100 Mohamed Ikbel Boulabiar boulab...@gmail.com wrote: [...] http://git.marcansoft.com/?p=libfreenect.git Good, if anyone is willing

Re: Bounty for the first Open Source driver for Kinect

2010-11-10 Thread Antonio Ospite
, about the motor I still don't know. Do those show up as usb HID devices somehow? About integrating the audio part (which has not even been guessed yet) is where I shamelessly show my ignorance :) Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because

Re: Bounty for the first Open Source driver for Kinect

2010-11-10 Thread Antonio Ospite
: the casual user who wants to just use kinect as a normal webcam doesn't have to care about installing some library If there are arguments against a kernel driver I can't see them yet. Ciao, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up

Re: Bounty for the first Open Source driver for Kinect

2010-11-10 Thread Antonio Ospite
On Thu, 11 Nov 2010 00:13:09 +0100 Markus Rechberger mrechber...@gmail.com wrote: On Wed, Nov 10, 2010 at 11:48 PM, Mohamed Ikbel Boulabiar boulab...@gmail.com wrote: On Wed, Nov 10, 2010 at 10:24 PM, Antonio Ospite osp...@studenti.unina.it wrote: If there are arguments against a kernel

gspca_ov534: Changing framerates, different behaviour in 2.6.36

2010-11-13 Thread Antonio Ospite
in this area. Thanks, Antonio [1] http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/24899 -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q

Re: gspca_ov534: Changing framerates, different behaviour in 2.6.36

2010-11-13 Thread Antonio Ospite
On Sat, 13 Nov 2010 16:12:05 +0100 Antonio Ospite osp...@studenti.unina.it wrote: guvcview: If I: 1. Go to the Video Files Tab 2. Change the Frame Rate value from the drop down menu [...] since 2.6.36 + the regression fix in [1] (please apply it): 3a. dmesg shows the message

Re: [PATCH] [media] gspca - main: Fix a regression with the PS3 Eye webcam

2010-11-22 Thread Antonio Ospite
On Fri, 5 Nov 2010 23:08:12 +0100 Antonio Ospite osp...@studenti.unina.it wrote: From: =?UTF-8?q?Jean-Fran=C3=A7ois=20Moine?= moin...@free.fr commit f43402fa55bf5e7e190c176343015122f694857c upstream. When audio is present, some alternate settings were skipped. This prevented some webcams

Re: gspca_ov534: Changing framerates, different behaviour in 2.6.36

2010-11-22 Thread Antonio Ospite
On Sat, 13 Nov 2010 16:45:37 +0100 Antonio Ospite osp...@studenti.unina.it wrote: On Sat, 13 Nov 2010 16:12:05 +0100 Antonio Ospite osp...@studenti.unina.it wrote: guvcview: If I: 1. Go to the Video Files Tab 2. Change the Frame Rate value from the drop down menu

Kinect sensor and Linux kernel driver.

2010-12-06 Thread Antonio Ospite
would be chosen, settle on a format usable also by future RGB-D devices. Comment and Critics more than welcomed. Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http

Re: Kinect sensor and Linux kernel driver.

2010-12-09 Thread Antonio Ospite
On Tue, 07 Dec 2010 12:29:31 +0100 Hans de Goede hdego...@redhat.com wrote: On 12/06/2010 10:18 PM, Antonio Ospite wrote: [...] Now the hard part begins, here's a loose TODO-list: - Discuss the fragmentation problem: * the webcam kernel driver and the libusb backend

Re: Kinect sensor and Linux kernel driver.

2010-12-09 Thread Antonio Ospite
On Mon, 6 Dec 2010 22:49:48 +0100 Hans Verkuil hverk...@xs4all.nl wrote: On Monday, December 06, 2010 22:18:47 Antonio Ospite wrote: [...] Now the hard part begins, here's a loose TODO-list: - Discuss the fragmentation problem: * the webcam kernel driver and the libusb backend

Question about libv4lconvert.

2010-12-15 Thread Antonio Ospite
; - } break; case V4L2_PIX_FMT_RGB565: switch (dest_pix_fmt) { Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why

Re: Question about libv4lconvert.

2010-12-15 Thread Antonio Ospite
On Wed, 15 Dec 2010 21:10:52 +0100 Hans de Goede hdego...@redhat.com wrote: Hi, Hi Hans, thanks for the quick reply. On 12/15/2010 05:11 PM, Antonio Ospite wrote: Hi, I am taking a look at libv4lconvert, and I have a question about the logic in v4lconvert_convert_pixfmt(), in some

[RFC, PATCH] Add 10 bit packed greyscale format.

2010-12-16 Thread Antonio Ospite
A 10 bits per pixel greyscale format in a packed array representation is supplied for instance by Kinect sensor device. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Hi, This is the first attempt to add v4l support for the 10bpp packed array format used by the Kinect sensor device

[PATCH] [media] videodev: fix a NULL pointer dereference in v4l2_device_release()

2011-10-12 Thread Antonio Ospite
-by: Antonio Ospite osp...@studenti.unina.it --- Hi, can anyone reproduce this? This is the complete trace, I left it out of the commit message, but feel free to include it if you think it is worth it. BUG: unable to handle kernel NULL pointer dereference at 0090 IP: [a10cc604

Re: [PATCH] [media] videodev: fix a NULL pointer dereference in v4l2_device_release()

2011-10-18 Thread Antonio Ospite
On Wed, 12 Oct 2011 23:59:26 +0200 Antonio Ospite osp...@studenti.unina.it wrote: The change in 8280b66 does not cover the case when v4l2_dev is already NULL, fix that. With a Kinect sensor, seen as an USB camera using GSPCA in this context, a NULL pointer dereference BUG can be triggered

Re: kernel OOPS when releasing usb webcam (random)

2011-10-18 Thread Antonio Ospite
last week: http://patchwork.linuxtv.org/patch/8124/ Tomas, can you test it fixes the problem for you too? Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki

Re: [RFC, PATCH] Add 10 bit packed greyscale format.

2010-12-28 Thread Antonio Ospite
On Thu, 16 Dec 2010 12:29:44 +0100 Antonio Ospite osp...@studenti.unina.it wrote: A 10 bits per pixel greyscale format in a packed array representation is supplied for instance by Kinect sensor device. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Ping. Hi

Re: [GIT PATCHES FOR 2.6.38] gspca for_2.6.38

2011-01-13 Thread Antonio Ospite
functions? I am going to test the changes and report back if there's anything more, I like the cleanup tho. Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki

Re: [GIT PATCHES FOR 2.6.38] gspca for_2.6.38

2011-01-14 Thread Antonio Ospite
On Thu, 13 Jan 2011 17:30:21 +0100 Jean-Francois Moine moin...@free.fr wrote: On Thu, 13 Jan 2011 12:38:04 +0100 Antonio Ospite osp...@studenti.unina.it wrote: Jean-Fran├žois Moine (9): [...] gspca - ov534: Use the new video control mechanism In this commit

[PATCH] Add a 10 bpp packed greyscale format.

2011-01-14 Thread Antonio Ospite
Add a 10 bits per pixel greyscale format in a packed array representation, naming it Y10P. Such pixel format is supplied for instance by the Kinect sensor device. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Hi, this version should look better than the previous one. It's not marked

[PATCH v2] Add a 10 bpp packed greyscale format.

2011-02-01 Thread Antonio Ospite
Add a 10 bits per pixel greyscale format in a packed array representation, naming it Y10P. Such pixel format is supplied for instance by the Kinect sensor device. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Hi, Changes since v1: * Fixed a trailing space, I forgot to run

Re: [PATCH v3 1/4] v4l: add V4L2_PIX_FMT_Y12 format

2011-03-11 Thread Antonio Ospite
changed, 80 insertions(+), 0 deletions(-) create mode 100644 Documentation/DocBook/v4l/pixfmt-y12.xml Hi Michael, are you going to release also Y12 conversion routines for libv4lconvert? Regards, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes

Re: [PATCH v3 1/4] v4l: add V4L2_PIX_FMT_Y12 format

2011-03-11 Thread Antonio Ospite
On Fri, 11 Mar 2011 10:38:08 +0100 Michael Jones michael.jo...@matrix-vision.de wrote: On 03/11/2011 10:21 AM, Antonio Ospite wrote: Hi Michael, are you going to release also Y12 conversion routines for libv4lconvert? Regards, Antonio Hi Antonio, As I am neither a user

[PATCH 1/2] Add Y10B, a 10 bpp bit-packed greyscale format.

2011-04-07 Thread Antonio Ospite
Add a 10 bits per pixel greyscale format in a bit-packed array representation, naming it Y10B. Such pixel format is supplied for instance by the Kinect sensor device. Cc: Steven Toth st...@kernellabs.com Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Documentation/DocBook/media

[PATCH 0/2] Add a GSPCA subdriver for the Microsoft Kinect sensor

2011-04-07 Thread Antonio Ospite
for the Kinect sensor. There was some positive feedback about calling the new format Y10B from Hans Verkuil, Mauro Carvalho Chehab and Guennadi Liakhovetski. Any comment is appreciated. Thanks, Antonio Antonio Ospite (2): Add Y10B, a 10 bpp bit-packed greyscale format. gspca: add a subdriver

[PATCH 2/2] gspca: add a subdriver for the Microsoft Kinect sensor device

2011-04-07 Thread Antonio Ospite
...@kernellabs.com Cc: OpenKinect openkin...@googlegroups.com Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/Kconfig |9 + drivers/media/video/gspca/Makefile |2 + drivers/media/video/gspca/kinect.c | 427 3 files

[PATCH] libv4lconvert-priv.h: indent with tabs, not spaces

2011-04-07 Thread Antonio Ospite
Indent wrapped lines with tabs, just like it is done for the other functions in the same file. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- lib/libv4lconvert/libv4lconvert-priv.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libv4lconvert

[RFC, PATCH] libv4lconvert: Add support for Y10B grey format (V4L2_PIX_FMT_Y10BPACK)

2011-04-07 Thread Antonio Ospite
Y10B is a 10 bits per pixel greyscale format in a bit-packed array representation. Such pixel format is supplied for instance by the Kinect sensor device. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Hi, this is a very first attempt about supporting Y10B in libv4lconvert

Re: [RFC, PATCH] libv4lconvert: Add support for Y10B grey format (V4L2_PIX_FMT_Y10BPACK)

2011-04-18 Thread Antonio Ospite
... and this control would be created by libv4l when in wrapped mode? Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing

Re: [GIT PATCHES FOR 2.6.40] gspca for_v2.6.40

2011-04-19 Thread Antonio Ospite
://linuxtv.org/jfrancois/gspca.git for_v2.6.40 Antonio Ospite (2): Add Y10B, a 10 bpp bit-packed greyscale format. gspca - kinect: New subdriver for Microsoft Kinect Ah glad to see that, so there was no major concern on the code, was there? Thanks Jean-Francois, Antonio

[PATCH 1/3] gspca - kinect: move communications buffers out of stack

2011-04-21 Thread Antonio Ospite
/media/video/gspca/kinect.c:202: warning: the frame size of 1548 bytes is larger than 1024 bytes Reported-by: Mauro Carvalho Chehab mche...@redhat.com Signed-off-by: Drew Fisher drew.m.fis...@gmail.com Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/kinect.c

[PATCH 3/3] gspca - kinect: fix comments referring to color camera

2011-04-21 Thread Antonio Ospite
Use the expression video stream instead of color camera which is more correct as the driver supports the RGB and IR image on the same endpoint. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/kinect.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions

[PATCH 0/3] gspca_kinect fixup

2011-04-21 Thread Antonio Ospite
Hi, some incremental fixes for the gspca kinect driver, the first patch in the series by Drew Fisher addresses the issue Mauro was pointing out. Thanks, Antonio Antonio Ospite (1): gspca - kinect: fix comments referring to color camera Drew Fisher (2): gspca - kinect: move

[PATCH 2/3] gspca - kinect: fix a typo s/steram/stream/

2011-04-21 Thread Antonio Ospite
From: Drew Fisher drew.m.fis...@gmail.com Signed-off-by: Drew Fisher drew.m.fis...@gmail.com Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/kinect.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/media/video/gspca/kinect.c

Re: [git:v4l-dvb/for_v2.6.40] [media] gspca - kinect: move communications buffers out of stack

2011-04-29 Thread Antonio Ospite
of stack Author: Antonio Ospite osp...@studenti.unina.it Date:Thu Apr 21 06:51:34 2011 -0300 Hi Mauro, actually this one is from Drew Fisher as well, git-am should have picked up the additional From header: http://www.spinics.net/lists/linux-media/msg31576.html Move large communications

Re: [git:v4l-dvb/for_v2.6.40] [media] gspca - kinect: move communications buffers out of stack

2011-04-29 Thread Antonio Ospite
On Fri, 29 Apr 2011 12:43:55 -0300 Mauro Carvalho Chehab mche...@redhat.com wrote: Em 29-04-2011 12:27, Antonio Ospite escreveu: On Fri, 29 Apr 2011 16:42:04 +0200 Mauro Carvalho Chehab mche...@redhat.com wrote: This is an automatic generated email to let you know that the following

Re: [GIT PATCHES FOR 2.6.40] gspca for_v2.6.40

2011-05-17 Thread Antonio Ospite
and the why and how go into the long commit message when they are not immediately trivial from the code; for instance the why of the USB trace changes in this series wasn't trivial to me. Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up

Re: [GIT PATCHES FOR 2.6.40] gspca for_v2.6.40

2011-05-21 Thread Antonio Ospite
messages? I am not sure about the name tho: D_COMM, D_DEV, D_REGS, D_IC, D_HW? Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top

Re: [PATCH] [media] gspca/kinect: wrap gspca_debug with GSPCA_DEBUG

2011-05-26 Thread Antonio Ospite
/video/gspca/kinect.c +++ b/drivers/media/video/gspca/kinect.c @@ -34,7 +34,7 @@ MODULE_AUTHOR(Antonio Ospite osp...@studenti.unina.it); MODULE_DESCRIPTION(GSPCA/Kinect Sensor Device USB Camera Driver); MODULE_LICENSE(GPL); -#ifdef DEBUG +#ifdef GSPCA_DEBUG int gspca_debug

Re: [REVIEW] v4l2 loopback

2009-04-14 Thread Antonio Ospite
On Tue, 14 Apr 2009 15:53:00 +0300 vas...@gmail.com wrote: On Tue, Apr 14, 2009 at 3:12 PM, Mauro Carvalho Chehab mche...@infradead.org wrote: The issue I see is that the V4L drivers are meant to support real devices. This driver that is a loopback for some userspace driver. I don't

Re: [REVIEW] v4l2 loopback

2009-04-27 Thread Antonio Ospite
at http://vger.kernel.org/majordomo-info.html Regards, Antonio Ospite -- A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? Web site: http://www.studenti.unina.it

UVC gadget driver for linux.

2009-05-06 Thread Antonio Ospite
/media/video/camera4uvc.c ./drivers/media/video/camera4uvc.h ./drivers/usbd/UVC_fd ./drivers/usbd/UVC_fd/uvc.c ./drivers/usbd/UVC_fd/uvc.h Regards, Antonio Ospite [1] http://www.motorola.com/motoinfo/product/details.jsp?globalObjectId=175 [1a] http://lists.berlios.de/pipermail/linux-uvc

Re: [REVIEW] v4l2 loopback

2009-05-07 Thread Antonio Ospite
On Thu, 7 May 2009 02:54:00 +0300 Vasily vas...@gmail.com wrote: This patch introduces v4l2 loopback module Hi Vasily, next time it would be useful to summarize what you changed from the previous version, and put a revision number in the Subject, like [PATCH v2] [PATCH v3], etc. Also, the

Re: pxa_camera + mt9m1111: Failed to configure for format 50323234

2009-10-04 Thread Antonio Ospite
On Sun, 4 Oct 2009 00:31:24 +0200 (CEST) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Sat, 3 Oct 2009, Antonio Ospite wrote: [...] Anyways your patch works, but the picture is now shifted, see: http://people.openezx.org/ao2/a780-pxa-camera-mt9m111-shifted.jpg Is this because

Re: pxa_camera + mt9m1111: image shifted (was: Failed to configure for format 50323234)

2009-11-03 Thread Antonio Ospite
On Mon, 05 Oct 2009 08:32:10 +0200 Stefan Herbrechtsmeier hbme...@hni.uni-paderborn.de wrote: Antonio Ospite schrieb: On Sun, 4 Oct 2009 00:31:24 +0200 (CEST) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Anyways your patch works, but the picture is now shifted, see: http

Re: [PATCH 1/3 v2] ezx: Add camera support for A780 and A910 EZX phones

2009-11-05 Thread Antonio Ospite
On Thu, 5 Nov 2009 00:53:46 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Wed, 4 Nov 2009, Antonio Ospite wrote: Signed-off-by: Antonio Ospite osp...@studenti.unina.it Signed-off-by: Bart Visscher ba...@thisnet.nl Is this patch going via Bart? Or should

Re: [PATCH 1/3 v2] ezx: Add camera support for A780 and A910 EZX phones

2009-11-10 Thread Antonio Ospite
Ospite osp...@studenti.unina.it wrote: On Fri, 6 Nov 2009 15:11:55 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Thu, 5 Nov 2009, Antonio Ospite wrote: See? It's power(), reset(), init(). Maybe the problem is in soc_camera_probe()? Sorry, you'd have

Re: [PATCH 1/3] ezx: Add camera support for A780 and A910 EZX phones

2009-11-10 Thread Antonio Ospite
,/* CAM_FLASH */ }; #endif Is that right? I am putting also this into the next version I am going to send for submission, if you don't object. Thanks, Antonio -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text

Re: [PATCH 1/3 v3] Add camera support for A780 and A910 EZX phones

2009-11-12 Thread Antonio Ospite
On Wed, 11 Nov 2009 19:02:11 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: Hi Antonio Still one more nitpick: Comments below. On Wed, 11 Nov 2009, Antonio Ospite wrote: [...] +/* camera */ +static int a780_camera_init(void) This function returns an error

[PATCH 1/3 v4] Add camera support for A780 and A910 EZX phones

2009-11-12 Thread Antonio Ospite
Signed-off-by: Bart Visscher ba...@thisnet.nl Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Changes since v3: - Check {a780,a910}_camera_init() return value, and register camera conditionally. arch/arm/mach-pxa/ezx.c | 174 +- 1 files

[RFC, PATCH] gspca: implement vidioc_enum_frameintervals

2009-11-17 Thread Antonio Ospite
-bulk_size = 16384; -- Antonio Ospite http://ao2.it PGP public key ID: 0x4553B001 A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail

[PATCH 0/3] pxa_camera: remove init() callback

2009-11-17 Thread Antonio Ospite
were really exposing some dynamic and generic initialization, this could be done in soc-camera itself, not in pxa_camera anyways. Thanks, Antonio Antonio Ospite (3): em-x270: don't use pxa_camera init() callback pcm990-baseboard: don't use pxa_camera init() callback pxa_camera: remove

[PATCH 1/3] em-x270: don't use pxa_camera init() callback

2009-11-17 Thread Antonio Ospite
pxa_camera init() is going to be removed. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- arch/arm/mach-pxa/em-x270.c |9 + 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-pxa/em-x270.c b/arch/arm/mach-pxa/em-x270.c index aec7f42..f71f34c

[PATCH 2/3] pcm990-baseboard: don't use pxa_camera init() callback

2009-11-17 Thread Antonio Ospite
pxa_camera init() is going to be removed. Configure PXA CIF pins directly in machine init function. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- arch/arm/mach-pxa/pcm990-baseboard.c |8 +--- 1 files changed, 1 insertions(+), 7 deletions(-) diff --git a/arch/arm/mach-pxa

[PATCH 3/3] pxa_camera: remove init() callback

2009-11-17 Thread Antonio Ospite
in pxa_camera_activate(), hence at device node open, but its users use it like a generic initialization to be done at module init time (configure MFP, request GPIOs for *sensor* control). Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- arch/arm/mach-pxa/include/mach/camera.h |2

Re: [PATCH 1/3] em-x270: don't use pxa_camera init() callback

2009-11-18 Thread Antonio Ospite
On Wed, 18 Nov 2009 11:10:06 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Tue, 17 Nov 2009, Antonio Ospite wrote: pxa_camera init() is going to be removed. My nitpick here would be - I would put it the other way round. We do not remove .init() in platforms, because

Re: [RFC, PATCH] gspca: implement vidioc_enum_frameintervals

2009-11-19 Thread Antonio Ospite
On Thu, 19 Nov 2009 09:08:22 +0100 Hans de Goede hdego...@redhat.com wrote: Hi, Hi, thanks for commenting on this. On 11/17/2009 11:41 AM, Antonio Ospite wrote: Hi, gspca does not implement vidioc_enum_frameintervals yet, so even if a camera can support multiple frame rates (or frame

Re: [PATCH 3/3] pxa_camera: remove init() callback

2009-11-27 Thread Antonio Ospite
On Fri, 27 Nov 2009 15:06:53 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Tue, 17 Nov 2009, Antonio Ospite wrote: pxa_camera init() callback is sometimes abused to setup MFP for PXA CIF, or even to request GPIOs to be used by the camera *sensor

Re: [PATCH 3/3] pxa_camera: remove init() callback

2009-11-27 Thread Antonio Ospite
On Fri, 27 Nov 2009 15:37:19 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Fri, 27 Nov 2009, Antonio Ospite wrote: On Fri, 27 Nov 2009 15:06:53 +0100 (CET) Guennadi Liakhovetski g.liakhovet...@gmx.de wrote: On Tue, 17 Nov 2009, Antonio Ospite wrote

[PATCH 1/3 v2] em-x270: don't use pxa_camera init() callback

2009-11-27 Thread Antonio Ospite
pxa_camera init() is ambiguous, it's better to statically configure the sensor. Signed-off-by: Antonio Ospite osp...@studenti.unina.it Acked-by: Mike Rapoport m...@compulab.co.il --- The only change from previous version is the commit message, we don't want to mention .init() removal yet. Since

[PATCH 2/3 v2] pcm990-baseboard: don't use pxa_camera init() callback

2009-11-27 Thread Antonio Ospite
pxa_camera init() is ambiguous, it's better to configure PXA CIF pins statically in machine init function. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- The only change from previous version is the commit message, we don't want to mention .init() removal yet. arch/arm/mach-pxa

Re: [PATCH 0/3] pxa_camera: remove init() callback

2009-11-27 Thread Antonio Ospite
On Tue, 17 Nov 2009 23:04:20 +0100 Antonio Ospite osp...@studenti.unina.it wrote: Hi, this series removes the init() callback from pxa_camera_platform_data, and fixes its users to do initialization statically at machine init time. [...] Antonio Ospite (3): em-x270: don't use pxa_camera

[PATCH] gspca: implement vidioc_enum_frameintervals

2009-12-01 Thread Antonio Ospite
Some drivers support multiple frameintervals (framerates), make gspca able to enumerate them. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Changes since the initial RFC version: - 'i' is now a __u32 and the variable 'index' has been dropped. - some more comments in gspca.h Thanks

Re: [PATCH] pxa_camera: move fifo reset direct before dma start

2010-04-21 Thread Antonio Ospite
good feeling about it. I have not tested it, but it looks good to me. I'll assume Stefan has tested it, and if you want it, please take my : Acked-by: Robert Jarzmik robert.jarz...@free.fr FWIW, Tested-by: Antonio Ospite osp...@studenti.unina.it It works on Motorola A780: pxa_camera + mt9m111

[PATCH] mt9m111: Call icl-reset() on mt9m111_reset().

2009-02-17 Thread Antonio Ospite
Call icl-reset() on mt9m111_reset(). Signed-off-by: Antonio Ospite osp...@studenti.unina.it diff --git a/drivers/media/video/mt9m111.c b/drivers/media/video/mt9m111.c index c043f62..92dd7f3 100644 --- a/drivers/media/video/mt9m111.c +++ b/drivers/media/video/mt9m111.c @@ -393,6 +393,8 @@ static

gspca V4L2_CID_EXPOSURE_AUTO and VIDIOC_G/S/TRY_EXT_CTRLS

2012-04-18 Thread Antonio Ospite
, so if anyone wants to anticipate me I'd be glad to test :) Unrelated, but maybe worth mentioning is that V4L2_CID_EXPOSURE_AUTO is of type MENU, while some drivers are treating it as a boolean, I think I can fix this one if needed. Thanks, Antonio -- Antonio Ospite http://ao2.it A: Because

[RFC PATCH 0/3] gspca: Implement VIDIOC_G_EXT_CTRLS and VIDIOC_S_EXT_CTRLS

2012-04-20 Thread Antonio Ospite
but which can't be controlled from v4l2 userspace apps right now. Let me know what do you think and if you have any suggestion. If needed I will add VIDIOC_TRY_EXT_CTRLS too. Thanks, Antonio Antonio Ospite (3): [media] gspca - main: rename get_ctrl to get_ctrl_index [media] gspca - main

[RFC PATCH 1/3] [media] gspca - main: rename get_ctrl to get_ctrl_index

2012-04-20 Thread Antonio Ospite
This reflects better what the function does and is also in preparation of a refactoring of setting and getting controls. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/gspca.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[RFC PATCH 2/3] [media] gspca - main: factor out the logic to set and get controls

2012-04-20 Thread Antonio Ospite
Factor out the logic to set and get controls from vidioc_s_ctrl() and vidioc_g_ctrl() so that the code can be reused in the coming implementation of vidioc_s_ext_ctrls() and vidioc_g_ext_ctrls(). Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/gspca.c | 148

[RFC PATCH 3/3] [media] gspca - main: implement vidioc_g_ext_ctrls and vidioc_s_ext_ctrls

2012-04-20 Thread Antonio Ospite
-infrastructure/47010 for an example of a problem fixed by this change. NOTE: gspca currently won't handle control types like V4L2_CTRL_TYPE_INTEGER64 or V4L2_CTRL_TYPE_STRING, so just the __s32 field 'value' of 'struct v4l2_ext_control' is handled for now. Signed-off-by: Antonio Ospite osp

[PATCH 1/3] [media] gspca - ov534: Add Saturation control

2012-04-23 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/ov534.c | 31 ++- 1 file changed, 30 insertions(+), 1 deletion(-) diff --git a/drivers/media/video/gspca/ov534.c b/drivers/media/video/gspca/ov534.c index 0475339..45139b5 100644

[PATCH 2/3] Input: move drivers/input/fixp-arith.h to include/linux

2012-04-23 Thread Antonio Ospite
Move drivers/input/fixp-arith.h to include/linux so that the functions defined there can be used by other subsystems, for instance some video devices ISPs can control the output HUE value by setting registers for sin(HUE) and cos(HUE). Signed-off-by: Antonio Ospite osp...@studenti.unina.it

[PATCH 3/3] [media] gspca - ov534: Add Hue control

2012-04-23 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/ov534.c | 61 - 1 file changed, 60 insertions(+), 1 deletion(-) diff --git a/drivers/media/video/gspca/ov534.c b/drivers/media/video/gspca/ov534.c index 45139b5..1a437b5

Re: [PATCH 0/3] gspca - ov534: saturation and hue (using fixp-arith.h)

2012-04-23 Thread Antonio Ospite
If you reply to all please fix Johann Deneux's address, I mis-wrote it, sorry. Thanks, Antonio -- Antonio Ospite http://ao2.it A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why is top-posting such a bad thing

Re: gspca V4L2_CID_EXPOSURE_AUTO and VIDIOC_G/S/TRY_EXT_CTRLS

2012-04-27 Thread Antonio Ospite
On Fri, 27 Apr 2012 09:53:09 +0200 Jean-Francois Moine moin...@free.fr wrote: On Wed, 18 Apr 2012 15:37:20 +0200 Antonio Ospite osp...@studenti.unina.it wrote: I noticed that AEC (Automatic Exposure Control, or V4L2_CID_EXPOSURE_AUTO) does not work in the ov534 gspca driver, either from

Re: [PATCH 0/3] gspca - ov534: saturation and hue (using fixp-arith.h)

2012-04-30 Thread Antonio Ospite
On Mon, 23 Apr 2012 14:16:25 -0600 Jonathan Corbet cor...@lwn.net wrote: On Mon, 23 Apr 2012 15:21:04 +0200 Antonio Ospite osp...@studenti.unina.it wrote: Jonathan, maybe fixp_sin() and fixp_cos() can be used in drivers/media/video/ov7670.c too where currently ov7670_sine

Re: [PATCH 0/3] gspca - ov534: saturation and hue (using fixp-arith.h)

2012-05-05 Thread Antonio Ospite
On Mon, 30 Apr 2012 15:51:01 +0200 Antonio Ospite osp...@studenti.unina.it wrote: On Mon, 23 Apr 2012 14:16:25 -0600 Jonathan Corbet cor...@lwn.net wrote: On Mon, 23 Apr 2012 15:21:04 +0200 Antonio Ospite osp...@studenti.unina.it wrote: Jonathan, maybe fixp_sin() and fixp_cos() can

[PATCH v2 1/3] gspca - ov534: Add Saturation control

2012-05-06 Thread Antonio Ospite
Also merge the COLORS control into it as it was V4L2_CID_SATURATION anyway. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Changes since version 1: - Merged SATURATION and COLORS controls drivers/media/video/gspca/ov534.c | 83 - 1 file

[PATCH v2 3/3] gspca - ov534: Add Hue control

2012-05-06 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Changes since version 1: - Disabled the HUE control for SENSOR_OV767x drivers/media/video/gspca/ov534.c | 65 +++-- 1 file changed, 63 insertions(+), 2 deletions(-) diff --git a/drivers/media/video

Re: [PATCH v2 0/3] gspca - ov534: saturation and hue (using fixp-arith.h)

2012-05-11 Thread Antonio Ospite
On Sun, 6 May 2012 12:14:55 +0200 Antonio Ospite osp...@studenti.unina.it wrote: Hi, I am sending a second version of this patchset, changes since version 1 are annotated per-patch. Just FYI I intend to work on porting ov534 to v4l2 framework too once these ones go in and the gspca core

Re: [PATCH v2 0/3] gspca - ov534: saturation and hue (using fixp-arith.h)

2012-05-14 Thread Antonio Ospite
On Sun, 6 May 2012 12:14:55 +0200 Antonio Ospite osp...@studenti.unina.it wrote: Hi, [...] Antonio Ospite (3): gspca - ov534: Add Saturation control Input: move drivers/input/fixp-arith.h to include/linux gspca - ov534: Add Hue control These changes are now in http

Re: [PATCH] media_build: add fixp-arith.h in linux/include/linux

2012-05-15 Thread Antonio Ospite
/ov534.c). I don't know yet if I'll do it or when it will be done but the file _might_ change not too far in the future. Regards, Antonio -- Antonio Ospite http://ao2.it A: Because it messes up the order in which people normally read text. See http://en.wikipedia.org/wiki/Posting_style Q: Why

[PATCH 1/3] gspca_kinect: remove traces of the gspca control mechanism

2012-05-16 Thread Antonio Ospite
This driver has no controls, so there is no need to convert it to the control framework. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/kinect.c |9 - 1 file changed, 9 deletions(-) diff --git a/drivers/media/video/gspca/kinect.c b/drivers

[PATCH 2/3] gspca_ov534: make AGC and AWB controls independent

2012-05-16 Thread Antonio Ospite
more control to the user. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/ov534.c | 31 ++- 1 file changed, 2 insertions(+), 29 deletions(-) diff --git a/drivers/media/video/gspca/ov534.c b/drivers/media/video/gspca/ov534.c index

[PATCH 3/3] gspca_ov534: Convert to the control framework

2012-05-16 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/video/gspca/ov534.c | 569 + 1 file changed, 267 insertions(+), 302 deletions(-) diff --git a/drivers/media/video/gspca/ov534.c b/drivers/media/video/gspca/ov534.c index c16bd1b

[PATCH 0/3] gspca: kinect cleanup, ov534 port to control framework

2012-05-16 Thread Antonio Ospite
, and disabled controls with associated automatic settings can show the value calculated by the hardware on every update, very instructive if not super-useful. Thanks, Antonio Antonio Ospite (3): gspca - kinect: remove traces of gspca control handling gspca - ov534: make AGC and AWB

Re: [PATCH 0/3] gspca: kinect cleanup, ov534 port to control framework

2012-05-18 Thread Antonio Ospite
On Fri, 18 May 2012 09:08:29 +0200 Jean-Francois Moine moin...@free.fr wrote: On Wed, 16 May 2012 23:42:43 +0200 Antonio Ospite osp...@studenti.unina.it wrote: The second patch removes the dependency between auto gain and auto white balance, I'd like to hear Jean-Francois

Re: Problems with the gspca_ov519 driver

2012-05-22 Thread Antonio Ospite
the exact cause, sorry. For the record the guvcview error messages were: VIDIOC_QBUF - Unable to queue buffer: Invalid argument Could not grab image (select timeout): Resource temporarily unavailable I feel I can add a: Tested-by: Antonio Ospite osp...@studenti.unina.it I can backport the change

  1   2   3   >