Re: [PATCH 1/6 v4] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-09 Thread Hans Verkuil
On Tuesday, August 09, 2011 00:06:10 Laurent Pinchart wrote: > Hi Hans, > > On Monday 08 August 2011 14:40:27 Hans Verkuil wrote: > > On Monday, August 08, 2011 13:40:23 Laurent Pinchart wrote: > > > On Monday 08 August 2011 11:16:41 Hans Verkuil wrote: > > > > On Friday, August 05, 2011 09:47:13

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Hans de Goede
Hi, On 08/08/2011 07:39 PM, Theodore Kilgore wrote: On Mon, 8 Aug 2011, Mauro Carvalho Chehab wrote: Mauro, In fact none of the currently known and supported cameras are using PTP. All of them are proprietary. They have a rather intimidating set of differences in functionality, too. Nam

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Hans de Goede
Hi, OK, another example. The cameras supported in camlibs/jl2005c do not have webcam ability, but someone could at any time design and market a dualmode which has in stillcam mode the same severe limitation. What limitation? Well, the entire memory of the camera must be dumped, or else the cam

Re: [PATCH v2] [media] omap3isp: queue: fail QBUF if user buffer is too small

2011-08-09 Thread Laurent Pinchart
Hi Michael, On Tuesday 09 August 2011 08:42:20 Michael Jones wrote: > Add buffer length check to sanity checks for USERPTR QBUF > > Signed-off-by: Michael Jones Acked-by: Laurent Pinchart I'll push the patch to v3.2. > --- > Changes for v2: > - only check when V4L2_MEMORY_USERPTR > > driv

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Hans de Goede
Hi, On 08/08/2011 07:58 PM, Sarah Sharp wrote: On Fri, Aug 05, 2011 at 09:45:56AM +0200, Hans de Goede wrote: Hi, On 08/05/2011 12:56 AM, Greg KH wrote: On Thu, Aug 04, 2011 at 07:21:47PM -0300, Mauro Carvalho Chehab wrote: I think it is important to separate oranges from apples here, there

Re: [RFC] The clock dependencies between sensor subdevs and the host interface drivers

2011-08-09 Thread Sylwester Nawrocki
Hi Laurent, On 08/08/2011 05:44 PM, Laurent Pinchart wrote: > Hi Sylwester, > > On Monday 08 August 2011 17:36:32 Sylwester Nawrocki wrote: >> Hi everyone, >> >> Nowadays many of the V4L2 camera device drivers heavily depend on the board >> code to set up voltage supplies, clocks, and some contro

RE: Possible issue in videobuf2 with buffer length check at QBUF time

2011-08-09 Thread Marek Szyprowski
Hello, On Monday, August 08, 2011 12:11 PM Laurent Pinchart wrote: > On Friday 05 August 2011 17:01:09 Pawel Osciak wrote: > > On Fri, Aug 5, 2011 at 01:55, Laurent Pinchart wrote: > > > Hi Marek and Pawel, > > > > > > While reviewing an OMAP3 ISP patch, I noticed that videobuf2 doesn't > > > ver

Re: [ANN] Meeting minutes of the Cambourne meeting

2011-08-09 Thread Subash Patel
Hi Sakari, I have a point with the pixel clock. During discussion we found that pixel clock get/set is required for user space to do fine control over the frame-rate etc. What if the user sets the pixel array clock which is above the system/if bus clock? Suppose we are setting the pixel clock

Re: [RFCv1 PATCH for v3.1] v4l2-ioctl: fix ENOTTY handling.

2011-08-09 Thread Hans Verkuil
On Sunday, July 31, 2011 14:43:44 Mauro Carvalho Chehab wrote: > Em 29-07-2011 09:10, Hans Verkuil escreveu: > > Hi all, > > > > While converting v4l2-compliance to correctly handle ENOTTY errors I found > > several regressions in v4l2-ioctl.c: > > > > 1) VIDIOC_ENUM/G/S/TRY_FMT would return -ENO

[PATCH v2] CXD2820R: Replace i2c message translation with repeater gate control

2011-08-09 Thread Steve Kerrison
This patch implements an i2c_gate_ctrl op for the cxd2820r. Thanks to Robert Schlabbach for identifying the register address and field to set. The old i2c intercept code that prefixed messages with a passthrough byte has been removed and the PCTV nanoStick T2 290e entry in em28xx-dvb has been upda

Anyone tested the DVB-T2 dual tuner TBS6280?

2011-08-09 Thread Harald Gustafsson
Hi, I searched for a dual tuner PCI-e DVB-T2 card with Linux support and found this TBS6280 card: http://tbsdvb.blog.com/2011/07/22/tbs-6280-freeview-hd-twin-tuner-card/ http://www.buydvb.net/tbs6280-pcie-dvbt2t-dual-tuner-card_p38.html http://www.tbsdtv.com/english/Download.html Previously I hav

Re: [PATCH v2] CXD2820R: Replace i2c message translation with repeater gate control

2011-08-09 Thread Antti Palosaari
Acked-by: Antti Palosaari Tested-by: Antti Palosaari On 08/09/2011 01:16 PM, Steve Kerrison wrote: > This patch implements an i2c_gate_ctrl op for the cxd2820r. Thanks to Robert > Schlabbach for identifying the register address and field to set. > > The old i2c intercept code that prefixed mes

Re: Anyone tested the DVB-T2 dual tuner TBS6280?

2011-08-09 Thread Steve Kerrison
Hi Harald, Currently the controller chip on the BlackGold card is unsupported - I don't know if there's any development work going on for that in this arena or if perhaps BlackGold are working on something themselves. Perhaps somebody else here knows the full story. This TBS card has only just be

Re: Anyone tested the DVB-T2 dual tuner TBS6280?

2011-08-09 Thread Antti Palosaari
On 08/09/2011 01:57 PM, Steve Kerrison wrote: > This TBS card has only just been brought to my attention. I cannot tell > what PCIe chip it uses and if it's supported. The alleged Linux driver > download for it doesn't have the cxd2820r code in it, so I can't see > that having much chance of workin

Re: Anyone tested the DVB-T2 dual tuner TBS6280?

2011-08-09 Thread Bjørn Mork
Steve Kerrison writes: > This TBS card has only just been brought to my attention. I cannot tell > what PCIe chip it uses and if it's supported. The alleged Linux driver > download for it doesn't have the cxd2820r code in it, so I can't see > that having much chance of working. There's a binary-

Re: [RFCv1 PATCH for v3.1] v4l2-ioctl: fix ENOTTY handling.

2011-08-09 Thread Mauro Carvalho Chehab
Em 09-08-2011 07:10, Hans Verkuil escreveu: > On Sunday, July 31, 2011 14:43:44 Mauro Carvalho Chehab wrote: >> Em 29-07-2011 09:10, Hans Verkuil escreveu: >>> Hi all, >>> >>> While converting v4l2-compliance to correctly handle ENOTTY errors I found >>> several regressions in v4l2-ioctl.c: >>> >>>

Re: [RFCv1 PATCH for v3.1] v4l2-ioctl: fix ENOTTY handling.

2011-08-09 Thread Hans Verkuil
On Tuesday, August 09, 2011 13:22:55 Mauro Carvalho Chehab wrote: > Em 09-08-2011 07:10, Hans Verkuil escreveu: > > On Sunday, July 31, 2011 14:43:44 Mauro Carvalho Chehab wrote: > >> Em 29-07-2011 09:10, Hans Verkuil escreveu: > >>> Hi all, > >>> > >>> While converting v4l2-compliance to correctly

Re: [RFC] The clock dependencies between sensor subdevs and the host interface drivers

2011-08-09 Thread Laurent Pinchart
Hi Sylwester, On Tuesday 09 August 2011 10:10:40 Sylwester Nawrocki wrote: > On 08/08/2011 05:44 PM, Laurent Pinchart wrote: > > On Monday 08 August 2011 17:36:32 Sylwester Nawrocki wrote: > >> Hi everyone, > >> > >> Nowadays many of the V4L2 camera device drivers heavily depend on the > >> board

Re: Anyone tested the DVB-T2 dual tuner TBS6280?

2011-08-09 Thread Harald Gustafsson
Thanks all for your quick answers. On Tue, Aug 9, 2011 at 1:10 PM, Bjørn Mork wrote: > There's a binary-only tbs62x0fe_driver for x86_{32,64} in the archive, > so it _might_ work.  But I don't think anyone here will recommend > something like that... Yes, this is what I was afraid of. That it wa

Re: Possible issue in videobuf2 with buffer length check at QBUF time

2011-08-09 Thread Laurent Pinchart
Hi Marek, On Tuesday 09 August 2011 11:14:43 Marek Szyprowski wrote: > On Monday, August 08, 2011 12:11 PM Laurent Pinchart wrote: > > On Friday 05 August 2011 17:01:09 Pawel Osciak wrote: > > > On Fri, Aug 5, 2011 at 01:55, Laurent Pinchart wrote: > > > > Hi Marek and Pawel, > > > > > > > > Whil

RE: Possible issue in videobuf2 with buffer length check at QBUF time

2011-08-09 Thread Marek Szyprowski
Hello, On Tuesday, August 09, 2011 1:52 PM Laurent Pinchart wrote: > On Tuesday 09 August 2011 11:14:43 Marek Szyprowski wrote: > > On Monday, August 08, 2011 12:11 PM Laurent Pinchart wrote: > > > On Friday 05 August 2011 17:01:09 Pawel Osciak wrote: > > > > On Fri, Aug 5, 2011 at 01:55, Laurent

Re: Possible issue in videobuf2 with buffer length check at QBUF time

2011-08-09 Thread Laurent Pinchart
Hi Marek, On Tuesday 09 August 2011 14:07:12 Marek Szyprowski wrote: > On Tuesday, August 09, 2011 1:52 PM Laurent Pinchart wrote: > > On Tuesday 09 August 2011 11:14:43 Marek Szyprowski wrote: > > > On Monday, August 08, 2011 12:11 PM Laurent Pinchart wrote: > > > > On Friday 05 August 2011 17:01

[RFC PATCH] More ENOTTY fixes for v3.1

2011-08-09 Thread Hans Verkuil
This patch fixes the remaining -ENOTTY fallout: - The ENUM_FMT ops form a subset of the ops used with G/S/TRY_FMT, so list those explicitly instead of using a macro. - Extend the G/S/TRY_FMT ops test to the full set. - Move all prio tests down to the correct switch cases. - Fix some 'returns ENO

Re: [RFC] The clock dependencies between sensor subdevs and the host interface drivers

2011-08-09 Thread Sylwester Nawrocki
Hi Laurent, On 08/09/2011 01:49 PM, Laurent Pinchart wrote: ... The following is usually handled in the board files: 1) sensor/frontend power supply 2) sensor's master clock (provided by the host device) ... For 2) I'd like to propose adding a callback to struct v4l2_devic

Re: RTL2831U driver updates

2011-08-09 Thread Alistair Buxton
On 8 August 2011 20:50, Antti Palosaari wrote: > On 08/06/2011 08:13 AM, Alistair Buxton wrote: >> On 6 August 2011 04:56, Alistair Buxton wrote: >>> With the latest driver my card never gets a signal lock, not even >>> once. As before there are no error messages. It does always probe >>> correct

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Alan Stern
On Tue, 9 Aug 2011, Hans de Goede wrote: > I would really like to see the dual mode camera and TV tuner discussion > separated. They are 2 different issues AFAIK. > > 1) Dual mode cameras: > > In the case of the dual mode camera we have 1 single device (both at > the hardware level and at the lo

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Marko Ristola
Hi I've been thinking about the Kernel driver side. Mauro and others emailed requirements on Jun or July. I'm sorry for this spam: maybe you have thought this already. A linked list of read/write locks as the solution for these protections could be a base for the general solution. Locks could b

Re: [ANN] Meeting minutes of the Cambourne meeting

2011-08-09 Thread Sakari Ailus
Subash Patel wrote: > Hi Sakari, Hi Subash, > I have a point with the pixel clock. During discussion we found that > pixel clock get/set is required for user space to do fine control over > the frame-rate etc. What if the user sets the pixel array clock which is > above the system/if bus clock? S

[patch] [media] ddbridge: fix ddb_ioctl()

2011-08-09 Thread Dan Carpenter
There were a several problems in this function: 1) Potential integer overflow in the comparison: if (fio.write_len + fio.read_len > 1028) { 2) If the user gave bogus values for write_len and read_len then returning -EINVAL is more appropriate than returning -ENOMEM. 3) wbuf was set t

Re: [ANN] Meeting minutes of the Cambourne meeting

2011-08-09 Thread Sakari Ailus
nitesh moundekar wrote: > Hi all, Hi Nitesh, > I am worried about direction v4l2 is taking. It looks against the basic > principle of driver i.e. hardware abstraction. So i think giving out pixel > clock, binning, skipping, bayer pattern, etc device varying features to user > space questionable.

[RFC PATCH] Modify volatile auto cluster handling as per earlier discussions

2011-08-09 Thread Hans Verkuil
This patch modifies the way autoclusters work when the 'foo' controls are volatile if autofoo is on. E.g.: if autogain is true, then gain returns the gain set by the autogain circuitry. This patch makes the following changes: 1) The V4L2_CTRL_FLAG_VOLATILE flag is added to let userspace know tha

omap3isp driver

2011-08-09 Thread Sriram V
Hi, Does the omap3isp driver implementation support rgb888 format? Regards, sriram -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Theodore Kilgore
On Tue, 9 Aug 2011, Hans de Goede wrote: > Hi, > > On 08/08/2011 07:39 PM, Theodore Kilgore wrote: > > > > > > On Mon, 8 Aug 2011, Mauro Carvalho Chehab wrote: > > > > > > > Mauro, > > > > In fact none of the currently known and supported cameras are using PTP. > > All of them are propri

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Sarah Sharp
On Tue, Aug 09, 2011 at 09:52:44AM +0200, Hans de Goede wrote: > Hi, > > On 08/08/2011 07:58 PM, Sarah Sharp wrote: > >On Fri, Aug 05, 2011 at 09:45:56AM +0200, Hans de Goede wrote: > >>Hi, > >> > >>On 08/05/2011 12:56 AM, Greg KH wrote: > >>>On Thu, Aug 04, 2011 at 07:21:47PM -0300, Mauro Carvalh

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Theodore Kilgore
On Tue, 9 Aug 2011, Hans de Goede wrote: > Hi, > > > > > OK, another example. The cameras supported in camlibs/jl2005c do not have > > webcam ability, but someone could at any time design and market a dualmode > > which has in stillcam mode the same severe limitation. What limitation? > > Wel

[cron job] v4l-dvb daily build: ERRORS

2011-08-09 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:Tue Aug 9 19:00:33 CEST 2011 git hash:9bed77ee2fb46b74782d0d9d14b92e9d07f3df6e gcc version: i686-linux-gcc (GCC) 4.5

Re: [PATCH] add support for the dvb-t part of CT-3650 v3

2011-08-09 Thread Jose Alberto Reguero
On Lunes, 8 de Agosto de 2011 23:44:43 Antti Palosaari escribió: > Reviewed-by: Antti Palosaari > > It looks just fine. > > regards > Antti > Forgot the Signed-off-by Signed-off-by: Jose Alberto Reguero Jose Alberto > On 08/08/2011 01:35 PM, Jose Alberto Reguero wrote: > > On Martes, 2 de

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Hans de Goede
Hi, On 08/09/2011 04:19 PM, Alan Stern wrote: On Tue, 9 Aug 2011, Hans de Goede wrote: I would really like to see the dual mode camera and TV tuner discussion separated. They are 2 different issues AFAIK. 1) Dual mode cameras: In the case of the dual mode camera we have 1 single device (both

Re: [GIT PATCHES FOR 3.1] s5p-fimc and noon010pc30 driver updates

2011-08-09 Thread Mauro Carvalho Chehab
Em 29-07-2011 05:36, Laurent Pinchart escreveu: > Hi Mauro, > > On Friday 29 July 2011 06:02:54 Mauro Carvalho Chehab wrote: >> Em 28-07-2011 19:57, Sylwester Nawrocki escreveu: >>> On 07/28/2011 03:20 PM, Mauro Carvalho Chehab wrote: Accumulating sub-dev controls at the video node is the rig

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Hans de Goede
Hi, On 08/09/2011 07:10 PM, Theodore Kilgore wrote: On Tue, 9 Aug 2011, Hans de Goede wrote: No, but both Adam and I realized, approximately at the same time yesterday afternoon, something which is rather important here. Gphoto is not developed exclusively for Linux. Furthermore, it has a

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Adam Baker
On Tuesday 09 August 2011, Hans de Goede wrote: > Hi, > > On 08/09/2011 04:19 PM, Alan Stern wrote: > > Does it really make sense to combine 5 drivers into one? > > Right, that is not the plan. The plan is to simply stop having 2 drivers > for 1 logical (and physical) block. So we go from 10 dr

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Hans de Goede
Hi, On 08/09/2011 10:31 PM, Adam Baker wrote: On Tuesday 09 August 2011, Hans de Goede wrote: It has also just occured to me that it might be possible to solve the issues we are facing just in the kernel. At the moment when the kernel performs a USBDEVFS_DISCONNECT it keeps the kernel drive

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Theodore Kilgore
On Tue, 9 Aug 2011, Hans de Goede wrote: > Hi, > > On 08/09/2011 04:19 PM, Alan Stern wrote: > > On Tue, 9 Aug 2011, Hans de Goede wrote: > > > > > I would really like to see the dual mode camera and TV tuner discussion > > > separated. They are 2 different issues AFAIK. > > > > > > 1) Dual m

Re: [GIT PATCHES FOR 3.1] s5p-fimc and noon010pc30 driver updates

2011-08-09 Thread Sakari Ailus
Hi Mauro, On Tue, Aug 09, 2011 at 05:05:47PM -0300, Mauro Carvalho Chehab wrote: > Em 29-07-2011 05:36, Laurent Pinchart escreveu: > > Hi Mauro, > > > > On Friday 29 July 2011 06:02:54 Mauro Carvalho Chehab wrote: > >> Em 28-07-2011 19:57, Sylwester Nawrocki escreveu: > >>> On 07/28/2011 03:20 PM

Re: [PATCH 1/6 v4] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-09 Thread Sakari Ailus
On Tue, Aug 09, 2011 at 09:26:30AM +0200, Hans Verkuil wrote: ... > > Wouldn't that be a security issue ? Any application with permissions to > > access > > the video device could DoS the system. > > How is this any different from an application that tries to use more memory > then there is avai

Re: [GIT PATCHES FOR 3.1] s5p-fimc and noon010pc30 driver updates

2011-08-09 Thread Mauro Carvalho Chehab
Em 09-08-2011 20:18, Sakari Ailus escreveu: > Hi Mauro, > > On Tue, Aug 09, 2011 at 05:05:47PM -0300, Mauro Carvalho Chehab wrote: >> Em 29-07-2011 05:36, Laurent Pinchart escreveu: >>> Hi Mauro, >>> >>> On Friday 29 July 2011 06:02:54 Mauro Carvalho Chehab wrote: Em 28-07-2011 19:57, Sylwest

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-09 Thread Theodore Kilgore
On Tue, 9 Aug 2011, Hans de Goede wrote: > Hi, > > On 08/09/2011 07:10 PM, Theodore Kilgore wrote: > > > > > > On Tue, 9 Aug 2011, Hans de Goede wrote: > > > > > No, but both Adam and I realized, approximately at the same time > > yesterday afternoon, something which is rather important he

Re: USB mini-summit at LinuxCon Vancouver

2011-08-09 Thread Xiaofan Chen
On Wed, Aug 10, 2011 at 4:57 AM, Hans de Goede wrote: > This has been discussed over and over and over again, playing clever > tricks with USBDEVFS_[DIS]CONNECT like adding a new > USBDEVFS_TRYDISCONNECT > which the v4l2 driver could intercept won't cut it. We need some central > manager of the de

Re: [PATCH 1/6 v4] V4L: add two new ioctl()s for multi-size videobuffer management

2011-08-09 Thread Hans Verkuil
On Wednesday, August 10, 2011 01:37:27 Sakari Ailus wrote: > On Tue, Aug 09, 2011 at 09:26:30AM +0200, Hans Verkuil wrote: > ... > > > Wouldn't that be a security issue ? Any application with permissions to > > > access > > > the video device could DoS the system. > > > > How is this any differe

[GIT PATCHES FOR 3.1] gspca for_v3.1

2011-08-09 Thread Jean-Francois Moine
The following changes since commit 46540f7ac646ada7f22912ea7ea9b761ff5c4718: [media] ir-mce_kbd-decoder: include module.h for its facilities (2011-07-29 12:52:23 -0300) are available in the git repository at: git://linuxtv.org/jfrancois/gspca.git for_v3.1 Jean-François Moine (2): gspc