[v4l-utils] Add options to v4l2-ctrl to save/load settings to/from a file

2018-11-24 Thread Antonio Ospite
//git.alsa-project.org/?p=alsa-utils.git;a=tree;f=alsactl;hb=HEAD [1] https://superuser.com/questions/471597/linux-v4l-webcam-make-settings-stick [2] https://git.ao2.it/v4l2-persistent-settings.git/ [3] https://sourceforge.net/projects/v4l2ucp/ -- Antonio Ospite https://ao2.it https://twitter.com/

Re: [RFC] v4l2 support for thermopile devices

2016-11-03 Thread Antonio Ospite
it/lib/libv4lconvert?id=6daa2b1ce8674bda66b0f3bb5cf08089e42579fd After that any v4l2 program using libv4l2 will at least be able to show _an_ image. You can play with "false color" representations too in libv4lconvert, however I don't know if such representations are generic enough to be main

Re: gstreamer and v4l2

2016-03-15 Thread Antonio Ospite
launch-0.10 v4l2src device="/dev/video0" ! > video/x-raw,width=640,height=180,framerate=30 ! autovideosink > > But it keeps giving me auto negotiation error -4. > Trying to give other values did not help neither. BTW, the need for videoconvert is more likely because of the

Re: [PATCH 6/7] [media] gspca: fix a v4l2-compliance failure during VIDIOC_REQBUFS

2016-03-14 Thread Antonio Ospite
On Thu, 10 Mar 2016 15:54:37 +0100 Hans de Goede <hdego...@redhat.com> wrote: > Hi, > > On 09-03-16 17:03, Antonio Ospite wrote: > > When calling VIDIOC_REQBUFS v4l2-compliance fails with this message: > > > >fail: v4l2-test-buffers.cpp(476): q.reqbufs(no

[PATCH 6/7] [media] gspca: fix a v4l2-compliance failure during VIDIOC_REQBUFS

2016-03-09 Thread Antonio Ospite
/videobuf2-core.c::vb2_core_reqbufs() seems to confirm this interpretation; however gspca is just returning -EBUSY in this case. Removing the special handling for the case of a different memory value fixes the compliance failure. Signed-off-by: Antonio Ospite <a...@ao2.it> --- This should b

[PATCH 4/7] [media] gspca: fix a v4l2-compliance failure about VIDIOC_ENUM_FRAMEINTERVALS

2016-03-09 Thread Antonio Ospite
. Signed-off-by: Antonio Ospite <a...@ao2.it> --- drivers/media/usb/gspca/gspca.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/drivers/media/usb/gspca/gspca.c b/drivers/media/usb/gspca/gspca.c index 1bb7901..61cb16d 100644 --- a/drivers/media/usb

[PATCH 0/7] gspca: pass all v4l2-compliance tests + minor fixes

2016-03-09 Thread Antonio Ospite
, but it's also a preliminary change for patch 4. - Patches from 4 to 7 are the actual compliance fixes. More details are in the commit messages. Thanks, Antonio Antonio Ospite (7): [media] gspca: ov534/topro: use a define for the default framerate [media] gspca: fix setting frame interval

[PATCH 2/7] [media] gspca: fix setting frame interval type in vidioc_enum_frameintervals()

2016-03-09 Thread Antonio Ospite
Set the frame _interval_ type to V4L2_FRMIVAL_TYPE_DISCRETE instead of using V4L2_FRMSIZE_TYPE_DISCRETE which is meant for frame _size_. The old and new values happen to be the same so there is no functional change. Signed-off-by: Antonio Ospite <a...@ao2.it> --- drivers/media/usb/gspca/g

[PATCH 3/7] [media] gspca: rename wxh_to_mode() to wxh_to_nearest_mode()

2016-03-09 Thread Antonio Ospite
The name wxh_to_nearest_mode() reflects better what the function does. Signed-off-by: Antonio Ospite <a...@ao2.it> --- drivers/media/usb/gspca/gspca.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/usb/gspca/gspca.c b/drivers/media/usb/gspca/g

[PATCH 5/7] [media] gspca: fix a v4l2-compliance failure about buffer timestamp

2016-03-09 Thread Antonio Ospite
pca uses v4l2_get_timestamp() which uses ktime_get_ts() which uses ktime_get_ts64() which returns a monotonic timestamp, so it's safe to initialize the buffer flags to V4L2_BUF_FLAG_TIMESTAMP_MONOTONIC to fix the failure. Signed-off-by: Antonio Ospite <a...@ao2.it> --- drivers/media/usb/gspca/gspca.c

[PATCH 1/7] [media] gspca: ov534/topro: use a define for the default framerate

2016-03-09 Thread Antonio Ospite
te has its own define to avoid risking using different values in different places, and for readability. While at it also remove some trivial comments about the framerates which don't add much to the code anymore. Signed-off-by: Antonio Ospite <a...@ao2.it> --- drivers/media/usb/g

[PATCH 7/7] [media] gspca: fix a v4l2-compliance failure during read()

2016-03-09 Thread Antonio Ospite
calls frame_ready_nolock() but the latter returns -EINVAL in a case when v4l2-compliance expects -EBUSY. Fix the failure by changing the return value in frame_ready_nolock(). Signed-off-by: Antonio Ospite <a...@ao2.it> --- drivers/media/usb/gspca/gspca.c | 2 +- 1 file changed, 1 insertion

Re: [PATCH] [media] gspca: ov534/topro: prevent a division by 0

2015-10-23 Thread Antonio Ospite
On Sat, 3 Oct 2015 10:14:17 +0200 Hans de Goede <hdego...@redhat.com> wrote: > Hi, > Hi HdG, > On 02-10-15 22:33, Antonio Ospite wrote: [...] > > Signed-off-by: Antonio Ospite <a...@ao2.it> > > Cc: sta...@vger.kernel.org > > Good catch: > > Re

Re: v4l2-ctrl is unable to set autogain to 0 with gspca/ov534

2015-10-14 Thread Antonio Ospite
On Mon, 12 Oct 2015 09:56:08 +0200 Hans Verkuil <hverk...@xs4all.nl> wrote: > Hi Antonio, > > On 10/07/2015 10:05 AM, Antonio Ospite wrote: [...] > > After a little investigation I figured out some more details: in my use > > case the autogain is a master

[PATCH] [media] media/v4l2-ctrls: fix setting autocluster to manual with VIDIOC_S_CTRL

2015-10-14 Thread Antonio Ospite
the original bug by making update_from_auto_cluster() not override the new master control value. Signed-off-by: Antonio Ospite <a...@ao2.it> Cc: <sta...@vger.kernel.org> # for v3.17 and up --- Hi, I did check the patch with scripts/checkpatch.pl but it gives an error, I think it's a fa

v4l2-ctrl is unable to set autogain to 0 with gspca/ov534

2015-10-07 Thread Antonio Ospite
his case when it's being changed from auto to manual. I may be missing some details tho, so I am asking if my reasoning is correct before sending a proper patch. And should I CC stable on it as the change fixes a regression? Thanks, Antonio -- Antonio Ospite http://ao2.it A: Because it mes

gspca/ov534 gets two failures with v4l2-compliance

2015-10-03 Thread Antonio Ospite
_pixfmts does not get populated because of the first failure. 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? -- To unsubscribe f

[PATCH] [media] gspca: ov534/topro: prevent a division by 0

2015-10-02 Thread Antonio Ospite
the issue by resetting the frame rate to a default value in case of an unusable timeperframe. The fix is done in the subdrivers instead of gspca.c because only the subdrivers have notion of a default frame rate to reset the camera to. Signed-off-by: Antonio Ospite <a...@ao2.it> Cc: sta...@vger.kern

[PATCH 07/12] trivial: [media] cx25821: cx25821-medusa-reg.h: fix 0x0x prefix

2015-04-28 Thread Antonio Ospite
Fix the 0x0x prefix in integer constants. In this case a padding 0 must also be inserted to make the constants look like all the other 16 bits ones. Signed-off-by: Antonio Ospite a...@ao2.it Cc: Mauro Carvalho Chehab mche...@osg.samsung.com Cc: linux-media@vger.kernel.org --- drivers/media/pci

Re: input_polldev interval (was Re: [sur40] Debugging a race condition)?

2015-03-26 Thread Antonio Ospite
— on a more complex system. Ciao, 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? -- To unsubscribe from this list: send the line unsubscribe linux

Re: [PATCH] gspca_stv06xx: enable button found on some Quickcam Express variant

2015-01-06 Thread Antonio Ospite
On Tue, 28 Oct 2014 15:39:41 +0100 Antonio Ospite a...@ao2.it wrote: On Mon, 14 Jul 2014 12:27:57 +0200 Hans de Goede hdego...@redhat.com wrote: Hi, On 07/11/2014 02:56 PM, Antonio Ospite wrote: Signed-off-by: Antonio Ospite a...@ao2.it Thanks, I've added this to my tree

Re: [PATCH 1/1] gspca: Add high-speed modes for PS3 Eye camera

2015-01-04 Thread Antonio Ospite
price range supports such high frame rates. Correct a QVGA mode that was listed as 40 FPS. It is really 37 FPS (half of 75 FPS). Tests confirm that the nominal frame rates are achieved. Signed-off-by: Joe Howse josephho...@nummist.com Tested-by: Antonio Ospite a...@ao2.it Thanks Joe. I

Re: [PATCH] gspca_stv06xx: enable button found on some Quickcam Express variant

2014-10-28 Thread Antonio Ospite
On Mon, 14 Jul 2014 12:27:57 +0200 Hans de Goede hdego...@redhat.com wrote: Hi, On 07/11/2014 02:56 PM, Antonio Ospite wrote: Signed-off-by: Antonio Ospite a...@ao2.it Thanks, I've added this to my tree and send a pull-req for it to Mauro. Hi Hans, I still don't see the change in 3.18

Re: [PATCH/RFC 1/1] Add a libv4l plugin for Exynos4 camera

2014-10-08 Thread Antonio Ospite
wanted to completely reset the build environment you could even use git clean, FWIW I have this git distclean alias in ~/.gitconfig: [alias] distclean = clean -f -d -X You'll need to rerun autoreconf -i after such a cleanup. Ciao, Antonio -- Antonio Ospite http://ao2.it A: Because

Re: [PATCH 01/12] trivial: drivers/media/usb/gspca/gspca.c: fix the indentation of a comment

2014-09-01 Thread Antonio Ospite
On Wed, 4 Jun 2014 14:03:39 +0200 Antonio Ospite a...@ao2.it wrote: Fix indentation of a comment, put it on the same level of the code it refers to. Signed-off-by: Antonio Ospite a...@ao2.it Cc: Hans de Goede hdego...@redhat.com Cc: linux-media@vger.kernel.org Ping, I cannot see

Re: [PATCH 02/12] trivial: drivers/media/usb/gspca/gspca.h: indent with TABs, not spaces

2014-09-01 Thread Antonio Ospite
On Wed, 4 Jun 2014 14:03:40 +0200 Antonio Ospite a...@ao2.it wrote: Signed-off-by: Antonio Ospite a...@ao2.it Cc: Hans de Goede hdego...@redhat.com Cc: linux-media@vger.kernel.org Ping. linux-media patchwork link: https://patchwork.linuxtv.org/patch/24156/ Thanks, Antonio --- drivers

Re: [PATCH 12/29] kinect: fix sparse warnings

2014-08-21 Thread Antonio Ospite
-by: Antonio Ospite a...@ao2.it --- drivers/media/usb/gspca/kinect.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/media/usb/gspca/kinect.c b/drivers/media/usb/gspca/kinect.c index 45bc1f5..3cb30a3 100644 --- a/drivers/media/usb/gspca/kinect.c

Re: [PATCHv1 02/12] vivid.txt: add documentation for the vivid driver.

2014-08-01 Thread Antonio Ospite
but JFTR there is already a simplistic implementation of fixed point operations in include/linux/fixp-arith.h I used them in drivers/media/usb/gspca/ov534.c for some hue calculation. Ciao, Antonio -- Antonio Ospite http://ao2.it A: Because it messes up the order in which people normally read text

Re: [PATCH 1/8] get_dvb_firmware: Add firmware extractor for si2165

2014-07-29 Thread Antonio Ospite
On Tue, 29 Jul 2014 07:37:44 +0200 Matthias Schwarzott z...@gentoo.org wrote: On 23.07.2014 22:10, Antonio Ospite wrote: On Wed, 23 Jul 2014 21:03:14 +0200 Matthias Schwarzott z...@gentoo.org wrote: [...] The crc value: It protects the content of the file until it is in the demod

Re: [PATCH 1/8] get_dvb_firmware: Add firmware extractor for si2165

2014-07-23 Thread Antonio Ospite
#ALGONAME{i} = #CHECKSUM{i} payload.bin Ciao, Antonio [1] http://www.jonelo.de/java/jacksum/ -- 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

[PATCH] gspca_stv06xx: enable button found on some Quickcam Express variant

2014-07-11 Thread Antonio Ospite
Signed-off-by: Antonio Ospite a...@ao2.it --- drivers/media/usb/gspca/stv06xx/stv06xx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/gspca/stv06xx/stv06xx.c b/drivers/media/usb/gspca/stv06xx/stv06xx.c index 49d209b..6ac93d8 100644 --- a/drivers/media

Re: [RFC 2/2] gspca_kinect: add support for the depth stream

2014-06-25 Thread Antonio Ospite
On Thu, 19 Jun 2014 16:35:17 +0200 Hans de Goede hdego...@redhat.com wrote: Hi, On 06/04/2014 10:24 PM, Antonio Ospite wrote: Add support for the depth mode at 10bpp, use a command line parameter to switch mode. NOTE: this is just a proof-of-concept, the final implementation

[PATCH 1/2] gspca: provide a mechanism to select a specific transfer endpoint

2014-06-25 Thread Antonio Ospite
field to struct gspca_dev, and change alt_xfer() so that it accepts a parameter which represents a specific endpoint address to look for. If a subdriver wants to specify a value for gspca_dev-xfer_ep it can do that in its sd_config() callback. Signed-off-by: Antonio Ospite a...@ao2.it --- drivers

[PATCH 0/2] gspca, gspca_kinect: add support for the depth stream

2014-06-25 Thread Antonio Ospite
Hi, here are the patches to make gspca able to deal with the Kinect depth stream at 10bpp. If anyone is really interested in the 11bpp data too, ping me. Alexander, please let us know if you can test these anytime soon. Thanks, Antonio Antonio Ospite (2): gspca: provide a mechanism

[PATCH 2/2] gspca_kinect: add support for the depth stream

2014-06-25 Thread Antonio Ospite
Add support for the depth stream at 10bpp, for now use a 'depth_mode' command line parameter to switch between video and depth mode. Signed-off-by: Alexander Sosna alexan...@xxor.de Signed-off-by: Antonio Ospite a...@ao2.it --- drivers/media/usb/gspca/kinect.c | 98

Re: [RFC 1/2] gspca: provide a mechanism to select a specific transfer endpoint

2014-06-24 Thread Antonio Ospite
On Thu, 19 Jun 2014 16:27:59 +0200 Hans de Goede hdego...@redhat.com wrote: Hi Antonio, Thanks for working on this. On 06/04/2014 10:24 PM, Antonio Ospite wrote: Add a xfer_ep_index field to struct gspca_dev, and change alt_xfer() so that it accepts a parameter which represents

Re: [PATCH RESEND] libv4lconvert: Fix a regression when converting from Y10B

2014-06-18 Thread Antonio Ospite
On Wed, 18 Jun 2014 13:46:10 +0200 Hans de Goede hdego...@redhat.com wrote: Hi, On 06/18/2014 01:43 PM, Hans de Goede wrote: Hi, On 06/16/2014 05:00 PM, Antonio Ospite wrote: Fix a regression introduced in commit efc29f1764a30808ebf7b3e1d9bfa27b909bf641 (libv4lconvert: Reject too

Re: [PATCH RESEND] libv4lconvert: Fix a regression when converting from Y10B

2014-06-18 Thread Antonio Ospite
On Wed, 18 Jun 2014 15:59:13 +0200 Hans de Goede hdego...@redhat.com wrote: Hi, On 06/18/2014 03:23 PM, Antonio Ospite wrote: On Wed, 18 Jun 2014 13:46:10 +0200 Hans de Goede hdego...@redhat.com wrote: Hi, On 06/18/2014 01:43 PM, Hans de Goede wrote: Hi, On 06/16/2014 05:00

[PATCH RESEND] libv4lconvert: Fix a regression when converting from Y10B

2014-06-16 Thread Antonio Ospite
was misunderstood and v4lconvert_convert_pixfmt() was made to return an error even in the case of a successful conversion from Y10B. Fix the check, and now print only the message letting the errno and the result from the conversion routines to be propagated to the caller. Signed-off-by: Antonio Ospite

[PATCH 01/12] trivial: drivers/media/usb/gspca/gspca.c: fix the indentation of a comment

2014-06-04 Thread Antonio Ospite
Fix indentation of a comment, put it on the same level of the code it refers to. Signed-off-by: Antonio Ospite a...@ao2.it Cc: Hans de Goede hdego...@redhat.com Cc: linux-media@vger.kernel.org --- drivers/media/usb/gspca/gspca.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff

[PATCH 02/12] trivial: drivers/media/usb/gspca/gspca.h: indent with TABs, not spaces

2014-06-04 Thread Antonio Ospite
Signed-off-by: Antonio Ospite a...@ao2.it Cc: Hans de Goede hdego...@redhat.com Cc: linux-media@vger.kernel.org --- drivers/media/usb/gspca/gspca.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/gspca/gspca.h b/drivers/media/usb/gspca/gspca.h index 300642d

[RFC 2/2] gspca_kinect: add support for the depth stream

2014-06-04 Thread Antonio Ospite
Signed-off-by: Antonio Ospite a...@ao2.it --- For now a command line parameter called depth_mode is used to select which mode to activate when loading the driver, this is necessary because gspca is not quite ready to have a subdriver call gspca_dev_probe() multiple times. The problem seems

[RFC 1/2] gspca: provide a mechanism to select a specific transfer endpoint

2014-06-04 Thread Antonio Ospite
Add a xfer_ep_index field to struct gspca_dev, and change alt_xfer() so that it accepts a parameter which represents a specific endpoint to look for. If a subdriver wants to specify a value for gspca_dev-xfer_ep_index it can do that in its sd_config() callback. Signed-off-by: Antonio Ospite

[RFC 0/2] gspca_kinect: add support for the depth stream

2014-06-04 Thread Antonio Ospite
stream in the mainline driver. The first patch is about supporting data streams on endpoints other than the first one in gspca. The second patch adds support for the depth data to the kinect subdriver. Some more specific comments are annotated per-patch. Ciao ciao, Antonio Antonio Ospite (2

[PATCH] libv4lconvert: Fix a regression when converting from Y10B

2014-06-03 Thread Antonio Ospite
was misunderstood and the v4lconvert_convert_pixfmt() was made to return an error even in the case of a successful conversion from Y10B. Fix the check, and now print only the message letting the errno and the result from the conversion routines to be propagated to the caller. Signed-off-by: Antonio Ospite

Re: [PATCH] libv4lconvert: Fix a regression when converting from Y10B

2014-06-03 Thread Antonio Ospite
On Tue, 3 Jun 2014 15:48:46 +0200 Antonio Ospite a...@ao2.it wrote: Fix a regression introduced in commit efc29f1764a30808ebf7b3e1d9bfa27b909bf641 (libv4lconvert: Reject too short source buffer before accessing it). The old code: case V4L2_PIX_FMT_Y10BPACK: ... if (result

Re: GSPCA ov534 payload error

2014-02-17 Thread Antonio Ospite
: packet l:12ov534 2-2.3:1.0: payload error ... Is this likely to be a problem with the gspca ov534 driver or with the USB subsystem? Oscillating packet sizes (l:12, l:8768, l:564), looks more like an USB issue. Ciao, Antonio -- Antonio Ospite http://ao2.it A: Because it messes up the order

Re: question about drivers/media/usb/gspca/kinect.c

2013-12-30 Thread Antonio Ospite
+v3.12.6/drivers/media/usb/dvb-usb-v2/dvb_usb_urb.c#L37 [2] http://lxr.linux.no/#linux+v3.12.6/Documentation/usb/error-codes.txt#L134 [3] https://github.com/OpenKinect/libfreenect/blob/master/src/flags.c#L87 -- Antonio Ospite http://ao2.it A: Because it messes up the order in which people normally

[PATCH 1/2] gspca_kinect: fix kinect_read() error path

2013-12-30 Thread Antonio Ospite
with -EREMOTEIO the case of a partial transfer. Reported-by: Julia Lawall julia.law...@lip6.fr Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/gspca/kinect.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/gspca/kinect.c b/drivers/media

[PATCH 0/2] Documentation/DocBook/media/v4l: typos

2013-12-16 Thread Antonio Ospite
Hi, here are a couple of fixes for typos in Documentation/DocBook/media/v4l/subdev-formats.xml Ciao, Antonio Antonio Ospite (2): [media] Documentation/DocBook/media/v4l/subdev-formats.xml: fix a typo [media] Documentation/DocBook/media/v4l: fix typo, s/packet/packed/ Documentation

[PATCH 2/2] [media] Documentation/DocBook/media/v4l: fix typo, s/packet/packed/

2013-12-16 Thread Antonio Ospite
Change packet to packed where the doc is talking about packed data formats. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Documentation/DocBook/media/v4l/subdev-formats.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/DocBook/media/v4l

[PATCH 1/2] [media] Documentation/DocBook/media/v4l/subdev-formats.xml: fix a typo

2013-12-16 Thread Antonio Ospite
The xref to the v4l2-mbus-pixelcode-yuv8 table gets rendered as Table 4.22, “YUV Formats”, so use the verb in the third person singular because it refers to Table: s/list/lists/ Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Documentation/DocBook/media/v4l/subdev-formats.xml | 2

Re: [PATCH] [media] gspca-ov534: don't call sd_start() from sd_init()

2013-08-20 Thread Antonio Ospite
for inclusion in stable releases, I was just wondering why you didn't consider it as a fix for 3.11, the patch fixes an actual crash experienced by a user. Regards, Antonio -- Antonio Ospite http://ao2.it A: Because it messes up the order in which people normally read text. See http

[PATCH] [media] gspca-ov534: don't call sd_start() from sd_init()

2013-08-15 Thread Antonio Ospite
---[ end trace 6786f15abfd2ac90 ]--- Original bug report from: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1173723/ Signed-off-by: Antonio Ospite osp...@studenti.unina.it Tested-by: Yaroslav Zakharuk slav...@gmail.com Cc: sta...@vger.kernel.org # 3.6+ --- drivers/media/usb/gspca/ov534.c | 3

[PATCH RFC] [media] gspca-ov534: don't call sd_start() from sd_init()

2013-08-04 Thread Antonio Ospite
--- Hi Yaroslav, the patch below should fix the Oops caused by sd_start() called too early, but I am not sure about why sd_start() was called from sd_init() for Hercules webcams in the first place, maybe the snippet marked with: /* (from ms-win trace) */ in sd_start() must be moved to

Re: [Regression 3.5-3.6, bisected] gspca_ov534: kernel oops when connecting Hercules Blog Webcam

2013-07-31 Thread Antonio Ospite
-end. [...] Additional info can be found here: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1173723/ I saw you also tested with 3.11-rc2 kernel, and the issue is still there: https://launchpadlibrarian.net/145608306/kern.log Thanks, Antonio -- Antonio Ospite http://ao2

Re: [PATCH] m920x: silence compiler warning

2013-03-11 Thread Antonio Ospite
: note: ‘ret’ was declared here Signed-off-by: Antti Palosaari cr...@iki.fi Acked-by: Antonio Ospite osp...@studenti.unina.it And thanks. BTW Antti, there was another patch for this warning: http://thread.gmane.org/gmane.linux.kernel/1450717 but your change is easier to validate. --- drivers

Re: On MIPI-CSI2 YUV420 formats and V4L2 Media Bus formats

2013-02-06 Thread Antonio Ospite
On Wed, 30 Jan 2013 01:23:48 +0100 Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Antonio, Sorry for the delay Laurent and thanks for your reply, some more questions below. On Monday 28 January 2013 13:22:10 Antonio Ospite wrote: Hi, looking at the MIPI Alliance

Re: [PATCH 0/2] A couple documentation fixes for media stuff

2013-02-06 Thread Antonio Ospite
On Mon, 28 Jan 2013 22:45:30 +0100 Antonio Ospite osp...@studenti.unina.it wrote: From: Antonio Ospite a...@amarulasolutions.com Take a look at the per-patch annotations too, please. Ping on this series. Thanks, Antonio Antonio Ospite (2): [media] Documentation/media

[PATCH 1/2] [media] Documentation/media-framework.txt: fix a sentence

2013-01-28 Thread Antonio Ospite
From: Antonio Ospite a...@amarulasolutions.com Signed-off-by: Antonio Ospite a...@amarulasolutions.com --- Hi, Actually I am not 100% sure whether the old form was correct English already or not but the new one sounds better to me. Thanks, Antonio Documentation/media-framework.txt |2

[PATCH 0/2] v4l-utils: some fixes for contrib/m920x/m920x_parse.pl

2012-12-29 Thread Antonio Ospite
and the firmware can be extracted with no problems. Patch 02 works around a warning. Antonio Ospite (2): contrib/m920x/m920x_parse.pl: stricter check when extracting firmware contrib/m920x/m920x_parse.pl: silence a warning contrib/m920x/m920x_parse.pl |5 +++-- 1 file changed, 3

[PATCH 2/2] contrib/m920x/m920x_parse.pl: silence a warning

2012-12-29 Thread Antonio Ospite
Silence a warning due to the way get_line() is supposed to be called: Use of uninitialized value $cmd in split at m920x_parse.pl line 118 Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- contrib/m920x/m920x_parse.pl |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 1/2] contrib/m920x/m920x_parse.pl: stricter check when extracting firmware

2012-12-29 Thread Antonio Ospite
Extract firmware only from the right messages, skip the other messages. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- contrib/m920x/m920x_parse.pl |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) mode change 100644 = 100755 contrib/m920x/m920x_parse.pl diff --git

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

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

Re: cannot make this Asus my cinema-u3100miniplusv2 work under linux

2012-12-16 Thread Antonio Ospite
://img.alibaba.com/wsphoto/v0/456087010/Free-shipping-50pcs-lot-fashion-style-wedding-box-candy-tin-box-jewellery-tin-box-06.jpg I verified that reception does improve a little bit, don't expect miracles tho :) Regards, Antonio -- Antonio Ospite http://ao2.it A: Because it messes up the order

[PATCHv2 1/9] [media] dvb-usb: fix indentation of a for loop

2012-12-10 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/dvb-usb-init.c | 60 +++--- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/media/usb/dvb-usb/dvb-usb-init.c b/drivers/media/usb/dvb-usb/dvb-usb-init.c index

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

2012-12-10 Thread Antonio Ospite
me to do otherwise. Thanks, Antonio Antonio Ospite (9): [media] dvb-usb: fix indentation of a for loop [media] m920x: fix a typo in a comment [media] m920x: factor out a m920x_write_seq() function [media] m920x: factor out a m920x_parse_rc_state() function [media] m920x: avoid

[PATCHv2 3/9] [media] m920x: factor out a m920x_write_seq() function

2012-12-10 Thread Antonio Ospite
This is in preparation for the vp7049 frontend attach function which is going to set a sequence of registers as well. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/m920x.c | 28 +++- 1 file changed, 19 insertions(+), 9 deletions

[PATCHv2 2/9] [media] m920x: fix a typo in a comment

2012-12-10 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/m920x.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/m920x.c b/drivers/media/usb/dvb-usb/m920x.c index 661bb75..433696d 100644 --- a/drivers/media/usb/dvb-usb

[PATCHv2 4/9] [media] m920x: factor out a m920x_parse_rc_state() function

2012-12-10 Thread Antonio Ospite
This is in preparation to using RC core infrastructure for some devices, the RC button state parsing logic can be shared berween rc.legacy and rc.core callbacks as it is independent from the mechanism used for RC handling. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media

[PATCHv2 9/9] [media] m920x: add support for the VP-7049 Twinhan DVB-T USB Stick

2012-12-10 Thread Antonio Ospite
-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/dvb-core/dvb-usb-ids.h |1 + drivers/media/usb/dvb-usb/m920x.c| 123 ++ 2 files changed, 124 insertions(+) diff --git a/drivers/media/dvb-core/dvb-usb-ids.h b/drivers/media/dvb-core/dvb-usb-ids.h

[PATCHv2 8/9] [media] get_dvb_firmware: add entry for the vp7049 firmware

2012-12-10 Thread Antonio Ospite
Add an entry to download the dvb-usb-vp7049-0.95.fw firmware for the Twinhan vp7049 and similar devices. Known devices of this kind are: Twinhan/Azurewave DTV-DVB UDTT7049 Digicom Digitune-S Think Xtra Hollywood DVB-T USB2.0 Signed-off-by: Antonio Ospite osp...@studenti.unina.it

[PATCHv2 7/9] [media] m920x: send the RC init sequence also when rc.core is used

2012-12-10 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/m920x.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/m920x.c b/drivers/media/usb/dvb-usb/m920x.c index bddd763..531a681 100644 --- a/drivers/media/usb/dvb-usb

[PATCHv2 6/9] [media] m920x: introduce m920x_rc_core_query()

2012-12-10 Thread Antonio Ospite
Add an m920x_rc_core_query() function for drivers which want to use the linux RC core infrastructure. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/m920x.c | 32 1 file changed, 32 insertions(+) diff --git a/drivers/media

[PATCHv2 5/9] [media] m920x: avoid repeating RC state parsing at each keycode

2012-12-10 Thread Antonio Ospite
Parsing the RC press state is invariant wrt. the keycode, take it out of the keycode scanning loop. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/m920x.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb

[PATCH] [media] gspca_kinect: add Kinect for Windows USB id

2012-12-09 Thread Antonio Ospite
From: Jacob Schloss jacob.schl...@unlimitedautomata.com Add the USB ID for the Kinect for Windows RGB camera so it can be used with the gspca_kinect driver. Signed-off-by: Jacob Schloss jacob.schl...@unlimitedautomata.com Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- Thanks Jacob

Re: [PATCH] gspca - ov534: Fix the light frequency filter

2012-11-29 Thread Antonio Ospite
On Thu, 29 Nov 2012 10:25:19 +0100 Hans de Goede hdego...@redhat.com wrote: Hi Jean-Francois, Antonio Ospite, Could it be that you're both right, and that the register Jean-Francois suggest is used (0x13) and uses in his patch is for enabling / disabling the light-freq filter, where

Re: [PATCH] gspca - ov534: Fix the light frequency filter

2012-11-26 Thread Antonio Ospite
On Fri, 23 Nov 2012 19:12:32 +0100 Jean-Francois Moine moin...@free.fr wrote: On Fri, 23 Nov 2012 18:09:09 +0100 Antonio Ospite osp...@studenti.unina.it wrote: [...] In the datasheet I have for ov772x, bit[6] of register 0x13 is described as: Bit[6]: AEC - Step size limit 0

Re: [PATCH] gspca - ov534: Fix the light frequency filter

2012-11-26 Thread Antonio Ospite
On Mon, 26 Nov 2012 16:23:18 +0100 Jean-Francois Moine moin...@free.fr wrote: On Mon, 26 Nov 2012 14:08:06 +0100 Antonio Ospite osp...@studenti.unina.it wrote: For now I'd NAK the patch since it is a regression for users with 50Hz power sources and it looks like it does not _always_ work

Re: [PATCH] gspca - ov534: Fix the light frequency filter

2012-11-23 Thread Antonio Ospite
, 0x13) | 0x20); } 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? -- To unsubscribe from this list: send the line unsubscribe linux

Re: [PATCH 1/5] [media] dvb-usb: add a pre_init hook to struct dvb_usb_device_properties

2012-11-13 Thread Antonio Ospite
On Tue, 06 Nov 2012 02:07:19 +0200 Antti Palosaari cr...@iki.fi wrote: On 11/06/2012 01:28 AM, Antonio Ospite wrote: Some devices need to issue a pre-initialization command sequence via i2c in order to enable the communication with some adapter components. This happens for instance

[PATCH 0/5] dvb-usb: support VP-7049 Twinhan DVB-T USB Stick and other fixes

2012-11-05 Thread Antonio Ospite
arbitrarily ignored checkpatch.pl warnings and errors because I preferred to stick with the code style in use in the dvb-usb files, let me know if you want me to do otherwise. Thanks, Antonio Antonio Ospite (5): [media] dvb-usb: add a pre_init hook to struct dvb_usb_device_properties [media

[PATCH 1/5] [media] dvb-usb: add a pre_init hook to struct dvb_usb_device_properties

2012-11-05 Thread Antonio Ospite
. Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- If this approach is OK I can send a similar patch for dvb-usb-v2. Are all the dvb-usb drivers going to be ported to dvb-usb-v2 eventually? drivers/media/usb/dvb-usb/dvb-usb.h |5 + drivers/media/usb/dvb-usb/dvb-usb

[PATCH 4/5] [media] dvb-usb: fix indentation of a for loop

2012-11-05 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/dvb-usb-init.c | 60 +++--- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/media/usb/dvb-usb/dvb-usb-init.c b/drivers/media/usb/dvb-usb/dvb-usb-init.c index

[PATCH 5/5] [media] m920x: fix a typo in a comment

2012-11-05 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- drivers/media/usb/dvb-usb/m920x.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/m920x.c b/drivers/media/usb/dvb-usb/m920x.c index ec820ec..c66c78c 100644 --- a/drivers/media/usb/dvb-usb

[PATCH 3/5] [media] m920x: Add support for the VP-7049 Twinhan DVB-T USB Stick

2012-11-05 Thread Antonio Ospite
be downloaded with: $ ./Documentation/dvb/get_dvb_firmware vp7049 [1] http://www.azurewave.com/Support_Utility_Driver.asp [2] http://www.digicom.it/digisit/driver_link.nsf/driverprodotto?openformprodotto=DigiTuneS [3] http://www.txitalia.it/prodotto.asp?prodotto=txhollywooddvttv Signed-off-by: Antonio

[PATCH 2/5] [media] get_dvb_firmware: add dvb-usb-vp7049-0.95.fw

2012-11-05 Thread Antonio Ospite
Firmware for vp7049 based design, known actual devices are: Twinhan/Azurewave DTV-DVB UDTT7049 Digicom Digitune-S Think Xtra Hollywood DVB-T USB2.0 Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- I tried to contact the original manufacturer (Twinhan) but it does not exists

[PATCH 3/5] m920x_parse.pl: use string comparison operators

2012-10-10 Thread Antonio Ospite
--- contrib/m920x/m920x_parse.pl | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/contrib/m920x/m920x_parse.pl b/contrib/m920x/m920x_parse.pl index 135ed5a..9093e16 100755 --- a/contrib/m920x/m920x_parse.pl +++ b/contrib/m920x/m920x_parse.pl

[PATCH 1/5] contrib: add some scripts to extract m920x firmwares from USB dumps

2012-10-10 Thread Antonio Ospite
From: Aapo Tahkola a...@rasterburn.org Signed-off-by: Aapo Tahkola a...@rasterburn.org --- contrib/m920x/m920x_parse.pl | 277 contrib/m920x/m920x_sp_firmware.pl | 115 +++ 2 files changed, 392 insertions(+) create mode 100755

[PATCH 5/5] m920x_parse.pl: add support for consuming the output of parse-sniffusb2.pl

2012-10-10 Thread Antonio Ospite
--- contrib/m920x/m920x_parse.pl | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/contrib/m920x/m920x_parse.pl b/contrib/m920x/m920x_parse.pl index b315400..b309250 100755 --- a/contrib/m920x/m920x_parse.pl +++ b/contrib/m920x/m920x_parse.pl @@ -103,6 +103,13

[PATCH 0/5] v4l-utils: add some scripts from the wiki.

2012-10-10 Thread Antonio Ospite
firmwares from USB dumps contrib: add a script to convert usbmon captures to usbsnoop Antonio Ospite (3): m920x_parse.pl: use string comparison operators m920x_parse.pl: fix strict and warnings checks m920x_parse.pl: add support for consuming the output of parse-sniffusb2.pl contrib/m920x

[PATCH 2/5] contrib: add a script to convert usbmon captures to usbsnoop

2012-10-10 Thread Antonio Ospite
From: Aapo Tahkola a...@rasterburn.org This makes it possible to reuse tools written for usbsnoop with captures done using a virtual machine and usbmon. Signed-off-by: Aapo Tahkola a...@rasterburn.org --- contrib/usbmon2usbsnoop.pl | 53 1 file

[PATCH 4/5] m920x_parse.pl: fix strict and warnings checks

2012-10-10 Thread Antonio Ospite
Signed-off-by: Antonio Ospite osp...@studenti.unina.it --- contrib/m920x/m920x_parse.pl | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/contrib/m920x/m920x_parse.pl b/contrib/m920x/m920x_parse.pl index 9093e16..b315400 100755 --- a/contrib

Re: [PATCH 0/5] v4l-utils: add some scripts from the wiki.

2012-10-10 Thread Antonio Ospite
On Wed, 10 Oct 2012 17:22:53 +0300 Antti Palosaari cr...@iki.fi wrote: On 10/10/2012 04:39 PM, Antonio Ospite wrote: Hi, I recently used some scripts I found on the linuxtv.org wiki to extract a firmware for a m920x device from USB dumps made with UsbSniff2.0 on WIndows XP. I

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

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

[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

  1   2   3   >