Re: v4l2-compliance utility

2011-01-21 Thread Andy Walls
On Fri, 2011-01-21 at 23:37 +0100, Hans Verkuil wrote: > Hi all, > > As you may have seen I have been adding a lot of tests to the v4l2-compliance > utility in v4l-utils lately. It is now getting to the state that is becomes > quite useful even though there is no full coverage yet. > > Currently

v4l2-compliance utility

2011-01-21 Thread Hans Verkuil
Hi all, As you may have seen I have been adding a lot of tests to the v4l2-compliance utility in v4l-utils lately. It is now getting to the state that is becomes quite useful even though there is no full coverage yet. Currently the following ioctls are being tested: General ioctls: VIDIOC_QUERY

[LONG TERM REGRESSION] Hauppauge WinTV HVR-1400 (XC3028L) not working since 2.6.33!

2011-01-21 Thread Alina Friedrichsen
Can you please give me all filenames which are involved with the driver, so that I can look for the differences between 2.6.33 and 2.6.34? The changes in the direct driver files are unremarkable. Alina -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a mes

[cron job] v4l-dvb daily build: OK

2011-01-21 Thread Hans Verkuil
This message is generated daily by a cron job that builds v4l-dvb for the kernels and architectures in the list below. Results of the daily build of v4l-dvb: date:Fri Jan 21 19:00:27 CET 2011 git master: 1b59be2a6cdcb5a12e18d8315c07c94a624de48f git media-master: gcc version: i6

BUG: double mutex lock in drivers/media/radio/si470x/radio-si470x-common.c

2011-01-21 Thread Alexander Strakh
KERNEL_VERSION: 2.6.37 SUBJECT: double mutex lock in drivers/media/radio/si470x/radio-si470x- common.c in function ssize_t si470x_fops_read. 1. First mutex_lock on &radio->lock in line 441 2. Second in line 462 I think that mutex in line 462 is not needed. 433static ssize_t si47

[PATCH 1/3 v2] hdpvr: fix up i2c device registration

2011-01-21 Thread Jarod Wilson
We have to actually call i2c_new_device() once for each of the rx and tx addresses. Also improve error-handling and device remove i2c cleanup. Reviewed-by: Andy Walls Signed-off-by: Jarod Wilson --- v2: address Andy's review comments from v1 - - wrap i2c_del_adapter with #if defined i2c conditi

Re: [PATCH 3/3] ir-kbd-i2c: improve remote behavior with z8 behind usb

2011-01-21 Thread Mike Isely
On Fri, 21 Jan 2011, Jarod Wilson wrote: > On Fri, Jan 21, 2011 at 10:31:42AM -0600, Mike Isely wrote: > > > > The pvrusb2 change is obviously trivial so I have no issue with it. > > > > Acked-By: Mike Isely > > > > Note the spelling of my last name "Isely" not "Isley". A good way to > > rem

Re: [PATCH 3/3] ir-kbd-i2c: improve remote behavior with z8 behind usb

2011-01-21 Thread Mike Isely
On Fri, 21 Jan 2011, Mike Isely wrote: > > Note the spelling of my last name "Isely" not "Isley". A good way to > remember is to think of the normal word "wisely" and just drop the > leading "w". (And yes, is...@isely.net and is...@pobox.com lead to the > same inbox.) And of course having

Re: [PATCH 3/3] ir-kbd-i2c: improve remote behavior with z8 behind usb

2011-01-21 Thread Jarod Wilson
On Fri, Jan 21, 2011 at 10:31:42AM -0600, Mike Isely wrote: > > The pvrusb2 change is obviously trivial so I have no issue with it. > > Acked-By: Mike Isely > > Note the spelling of my last name "Isely" not "Isley". A good way to > remember is to think of the normal word "wisely" and just dro

Re: [PATCH 3/3] ir-kbd-i2c: improve remote behavior with z8 behind usb

2011-01-21 Thread Mike Isely
The pvrusb2 change is obviously trivial so I have no issue with it. Acked-By: Mike Isely Note the spelling of my last name "Isely" not "Isley". A good way to remember is to think of the normal word "wisely" and just drop the leading "w". (And yes, is...@isely.net and is...@pobox.com lead to

Re: [cron job] v4l-dvb daily build: OK

2011-01-21 Thread Mauro Carvalho Chehab
Em 21-01-2011 13:24, Hans Verkuil escreveu: > On Friday, January 21, 2011 15:09:09 Mauro Carvalho Chehab wrote: >> Em 20-01-2011 19:26, Hans Verkuil escreveu: >>> On Thursday, January 20, 2011 21:32:43 Mauro Carvalho Chehab wrote: Em 20-01-2011 16:34, Hans Verkuil escreveu: > This message

Re: [cron job] v4l-dvb daily build: OK

2011-01-21 Thread Hans Verkuil
On Friday, January 21, 2011 15:09:09 Mauro Carvalho Chehab wrote: > Em 20-01-2011 19:26, Hans Verkuil escreveu: > > On Thursday, January 20, 2011 21:32:43 Mauro Carvalho Chehab wrote: > >> Em 20-01-2011 16:34, Hans Verkuil escreveu: > >>> This message is generated daily by a cron job that builds v4

Re: [PATCH 1/3] hdpvr: fix up i2c device registration

2011-01-21 Thread Jarod Wilson
On Fri, Jan 21, 2011 at 08:34:58AM -0500, Andy Walls wrote: > On Thu, 2011-01-20 at 23:30 -0500, Jarod Wilson wrote: > > We have to actually call i2c_new_device() once for each of the rx and tx > > addresses. Also improve error-handling and device remove i2c cleanup. > > > > Signed-off-by: Jarod W

Re: [cron job] v4l-dvb daily build: OK

2011-01-21 Thread Mauro Carvalho Chehab
Em 20-01-2011 19:26, Hans Verkuil escreveu: > On Thursday, January 20, 2011 21:32:43 Mauro Carvalho Chehab wrote: >> Em 20-01-2011 16:34, Hans Verkuil escreveu: >>> This message is generated daily by a cron job that builds v4l-dvb for >>> the kernels and architectures in the list below. >>> >>> Res

Re: [PATCH 3/3] ir-kbd-i2c: improve remote behavior with z8 behind usb

2011-01-21 Thread Andy Walls
On Thu, 2011-01-20 at 23:30 -0500, Jarod Wilson wrote: > Add the same "are you ready?" i2c_master_send() poll command to > get_key_haup_xvr found in lirc_zilog, which is apparently seen in > the Windows driver for the PVR-150 w/a z8. This stabilizes what is > received from both the HD-PVR and HVR-1

Re: [PATCH 2/3] lirc_zilog: z8 on usb doesn't like back-to-back i2c_master_send

2011-01-21 Thread Andy Walls
On Thu, 2011-01-20 at 23:30 -0500, Jarod Wilson wrote: > Both the HD-PVR and HVR-1950, driven by the hdpvr and pvrusb2 drivers > respectively, have a zilog z8 chip exposed via i2c. These are both > usb-connected devices, and on both of them, back-to-back i2c_master_send > calls that work fine with

Re: [PATCH 1/3] hdpvr: fix up i2c device registration

2011-01-21 Thread Andy Walls
On Thu, 2011-01-20 at 23:30 -0500, Jarod Wilson wrote: > We have to actually call i2c_new_device() once for each of the rx and tx > addresses. Also improve error-handling and device remove i2c cleanup. > > Signed-off-by: Jarod Wilson Reviewed-by: Andy Walls I do have some comments, but the rea

Re: [GIT PULL for 2.6.38-rc2] V4L/DVB patches

2011-01-21 Thread Mauro Carvalho Chehab
Linus, Em 20-01-2011 15:01, Mauro Carvalho Chehab escreveu: > Hi Linus, > > Those are some changes that I tried to prepare to send you during the > merge window, but, unfortunately, the new videobuf2 driver took me a longer > time to review/test than I originally expected. > > Please pull from:

RE: [PATCH] [media] v4l: soc-camera: add enum-frame-size ioctl

2011-01-21 Thread Qing Xu
Thanks! I am now prototyping Marvell camera driver to align with soc-camera architecture, may need some time to stabilize it. Our subdev driver now already follows those xxx_mbus_xxx operations. We hope we could finalize and submit our camera driver patch to open source this year. Thanks! -Qing

[patch] Re: media_build: build fails against (ubuntu) 2.6.32 on pvrusb2-debugifc.c

2011-01-21 Thread Vincent McIntyre
The problem was the new check function was not being called. Signed-off-by: vincent.mcint...@gmail.com diff --git a/v4l/scripts/make_config_compat.pl b/v4l/scripts/make_config_compat. index 438561a..f1dd577 100755 --- a/v4l/scripts/make_config_compat.pl +++ b/v4l/scripts/make_config_compat.pl @@ -

Re: [PATCH] [media] v4l: soc-camera: add enum-frame-size ioctl

2011-01-21 Thread Guennadi Liakhovetski
On Fri, 21 Jan 2011, Hans Verkuil wrote: > On Friday, January 21, 2011 09:05:07 Guennadi Liakhovetski wrote: > > On Thu, 20 Jan 2011, Qing Xu wrote: > > > > > Hi Guennadi, Hans, > > > > > > I update this patch, I use enum_framesizes instead of > > > enum_mbus_fsizes, which is already defined in

[PATCH] Fix dependencies for Technisat USB2.0 DVB-S/S2

2011-01-21 Thread Oleg Roitburd
Device is based on STV0903 demod and STV6110x tuner Signed-off-by: Oleg Roitburd diff -Nur linux-media-LATEST-orig/drivers/media/dvb/dvb-usb/Kconfig linux-media-LATEST-patched/drivers/media/dvb/dvb-usb/Kconfig --- linux-media-LATEST-orig/drivers/media/dvb/dvb-usb/Kconfig 2011-01-11 05:45:22.

Re: [PATCH] [media] v4l: soc-camera: add enum-frame-size ioctl

2011-01-21 Thread Hans Verkuil
On Friday, January 21, 2011 09:05:07 Guennadi Liakhovetski wrote: > On Thu, 20 Jan 2011, Qing Xu wrote: > > > Hi Guennadi, Hans, > > > > I update this patch, I use enum_framesizes instead of > > enum_mbus_fsizes, which is already defined in v4l2-subdev.h, > > so, do not need to modify v4l2-subdev

Fwd: Re: BUG: double mutex_unlock in drivers/media/video/tlg2300/pd-video.c

2011-01-21 Thread Alexander Strakh
--- Begin Message --- Hello, > Hi Strakh: > > Thanks for your patch. > > But I prefer to remove the mutex_unlock() in the pd_vidioc_s_fmt(), > since the pd_vidioc_s_fmt() is also called in restore_v4l2_context(). > > would you please change the patch? > I will ack it. > > Best Regards > Huang

[RFC] ISP lane shifter support

2011-01-21 Thread Michael Jones
Hi all, In the OMAP ISP driver, I'm interested in being able to choose between 8-bit and 12-bit formats when I have a 12-bit sensor attached. At the moment it looks like it's only possible to define this statically with data_lane_shift in the board definition. But with the ISP's lane shifter, it

RE: [PATCH] [media] v4l: soc-camera: add enum-frame-size ioctl

2011-01-21 Thread Guennadi Liakhovetski
On Thu, 20 Jan 2011, Qing Xu wrote: > Hi Guennadi, Hans, > > I update this patch, I use enum_framesizes instead of > enum_mbus_fsizes, which is already defined in v4l2-subdev.h, > so, do not need to modify v4l2-subdev.h now. > > Are you ok with it? Hm, you see, this would mean, hijacking a "wro