Re: [PATCH RFC v2 2/6] drm/i2c: tda998x: Move tda998x to a couple encoder/connector

2014-03-27 Thread Jean-Francois Moine
Hi Laurent, On Wed, 26 Mar 2014 18:33:09 +0100 Laurent Pinchart wrote: > That could work in your case, but I don't really like that. > > We need to describe the hardware topology, that might be the only point we > all > agree on. There are various hardware topologies we need to support with

Re: [PATCH RFC v2 2/6] drm/i2c: tda998x: Move tda998x to a couple encoder/connector

2014-03-25 Thread Jean-Francois Moine
On Mon, 24 Mar 2014 23:39:01 +0100 Laurent Pinchart wrote: > Hi Jean-François, Hi Laurent, > Thank you for the patch. > > On Friday 21 March 2014 09:17:32 Jean-Francois Moine wrote: > > The 'slave encoder' structure of the tda998x driver asks for glue > &

[PATCH v3] drm/i2c: tda998x: Deprecate "nxp,tda998x" in favour of "nxp,tda9989"

2014-03-23 Thread Jean-Francois Moine
f-by: Jean-Francois Moine --- v3: - fix the I2C ID (the OF compatible is not used for such drivers) - define only one compatible (Sebastian Hesselbarth) - change the subject (Sebastian Hesselbarth) v2: - change the subject to drm/i2c This patch applies after drm/i2c: tda998x: F

[PATCH RFC v2 6/6] ARM: AM33XX: dts: Change the tda998x description

2014-03-21 Thread Jean-Francois Moine
The tda998x being moved from a 'slave encoder' to a normal DRM encoder/connector and the tilcdc_slave glue being removed, the declaration of the HDMI transmitter description must be changed in the associated DTs. Signed-off-by: Jean-Francois Moine --- arch/arm/boot/dts/am335x-base0033

[PATCH RFC v2 4/6] drm/tilcdc: Change the interface with the tda998x driver

2014-03-21 Thread Jean-Francois Moine
f-by: Jean-Francois Moine --- drivers/gpu/drm/tilcdc/Makefile | 1 - drivers/gpu/drm/tilcdc/tilcdc_drv.c | 85 +-- drivers/gpu/drm/tilcdc/tilcdc_slave.c | 406 -- drivers/gpu/drm/tilcdc/tilcdc_slave.h | 26 --- 4 files changed, 68 insertions(+), 450

[PATCH RFC v2 2/6] drm/i2c: tda998x: Move tda998x to a couple encoder/connector

2014-03-21 Thread Jean-Francois Moine
The 'slave encoder' structure of the tda998x driver asks for glue between the DRM driver and the encoder/connector structures. This patch changes the driver to a normal DRM encoder/connector thanks to the infrastructure for componentised subsystems. Signed-off-by: Jean-Fran

[PATCH RFC v2 5/6] drm/tilcd: dts: Remove unused slave description

2014-03-21 Thread Jean-Francois Moine
The tda998x being converted to a normal DRM encoder/connector, there is no slave notion anymore. Signed-off-by: Jean-Francois Moine --- Documentation/devicetree/bindings/drm/tilcdc/slave.txt | 18 -- 1 file changed, 18 deletions(-) delete mode 100644 Documentation/devicetree

[PATCH RFC v2 3/6] drm/tilcd: dts: Add the video output port

2014-03-21 Thread Jean-Francois Moine
The connection between the video source and sink must follow the media video interface. Signed-off-by: Jean-Francois Moine --- Documentation/devicetree/bindings/drm/tilcdc/tilcdc.txt | 14 ++ 1 file changed, 14 insertions(+) diff --git a/Documentation/devicetree/bindings/drm/tilcdc

[PATCH RFC v2 1/6] drm/i2c: tda998x: Add the required port property

2014-03-21 Thread Jean-Francois Moine
According to the media video interface, the video source and sink ports must be identified by mutual phandles. This patch adds the 'port' property of the tda998x (sink side). Signed-off-by: Jean-Francois Moine --- Documentation/devicetree/bindings/drm/i2c/tda998x.txt | 11 +

[PATCH RFC v2 0/6] drm/i2c: Move tda998x to a couple encoder/connector

2014-03-21 Thread Jean-Francois Moine
tion for non-DT systems Jean-Francois Moine (6): drm/i2c: tda998x: Add the required port property drm/i2c: tda998x: Move tda998x to a couple encoder/connector drm/tilcd: dts: Add the video output port drm/tilcdc: Change the interface with the tda998x driver drm/tilcd: dts: Remove unused slave de

Re: [PATCH v3 1/2] drivers/base: permit base components to omit the bind/unbind ops

2014-02-10 Thread Jean-Francois Moine
On Mon, 10 Feb 2014 13:12:33 + Russell King - ARM Linux wrote: > I've NAK'd these patches already - I believe they're based on a > mis-understanding of how this should be used. I believe Jean-Francois > has only looked at the core, rather than looking at the imx-drm example > it was posted w

Re: [PATCH RFC 0/2] drivers/base: simplify simple DT-based components

2014-02-09 Thread Jean-Francois Moine
On Fri, 7 Feb 2014 20:23:51 + Russell King - ARM Linux wrote: > Here's my changes to the TDA998x driver to add support for the component > helper. The TDA998x driver retains support for the old way so that > drivers can be transitioned. For any one DRM "card" the transition to I rewrote th

Re: [PATCH RFC 0/2] drivers/base: simplify simple DT-based components

2014-02-07 Thread Jean-Francois Moine
On Fri, 7 Feb 2014 17:33:26 + Russell King - ARM Linux wrote: > On Fri, Feb 07, 2014 at 06:11:08PM +0100, Jean-Francois Moine wrote: > > This patch series tries to simplify the code of simple devices in case > > they are part of componentised subsystems, are declared in a DT,

[PATCH v3 1/2] drivers/base: permit base components to omit the bind/unbind ops

2014-02-07 Thread Jean-Francois Moine
Some simple components don't need to do any specific action on bind to / unbind from a master component. This patch permits such components to omit the bind/unbind operations. Signed-off-by: Jean-Francois Moine --- drivers/base/component.c | 9 +++-- 1 file changed, 7 insertions(

[PATCH v3 0/2] *** SUBJECT HERE ***

2014-02-07 Thread Jean-Francois Moine
*** BLURB HERE *** Jean-Francois Moine (2): drivers/base: permit base components to omit the bind/unbind ops drivers/base: declare phandle DT nodes as components drivers/base/component.c | 21 +++-- drivers/base/core.c | 18 ++ include/linux/of.h

[PATCH v3 2/2] drivers/base: declare phandle DT nodes as components

2014-02-07 Thread Jean-Francois Moine
time, the devices which are pointed to by a phandle are declared as components, except when they declared themselves as such in their probe function. Signed-off-by: Jean-Francois Moine --- drivers/base/component.c | 12 drivers/base/core.c | 18 ++ include/linux

[PATCH RFC 2/2] drivers/base: declare phandle DT nodes as components

2014-02-07 Thread Jean-Francois Moine
time, the devices which are pointed to by a phandle are declared as components, except when they declared themselves as such in their probe function. Signed-off-by: Jean-Francois Moine --- drivers/base/component.c | 12 drivers/base/core.c | 18 ++ include/linux

[PATCH RFC 1/2] drivers/base: permit base components to omit the bind/unbind ops

2014-02-07 Thread Jean-Francois Moine
Some simple components don't need to do any specific action on bind to / unbind from a master component. This patch permits such components to omit the bind/unbind operations. Signed-off-by: Jean-Francois Moine --- drivers/base/component.c | 9 +++-- 1 file changed, 7 insertions(

[PATCH RFC 0/2] drivers/base: simplify simple DT-based components

2014-02-07 Thread Jean-Francois Moine
This patch series tries to simplify the code of simple devices in case they are part of componentised subsystems, are declared in a DT, and are not using the component bin/unbind functions. Jean-Francois Moine (2): drivers/base: permit base components to omit the bind/unbind ops drivers/base

Re: [PATCH RFC 26/46] drivers/base: provide an infrastructure for componentised subsystems

2014-02-07 Thread Jean-Francois Moine
On Fri, 7 Feb 2014 09:46:56 + Russell King - ARM Linux wrote: > On Fri, Feb 07, 2014 at 10:04:30AM +0100, Daniel Vetter wrote: > > I've chatted a bit with Hans Verkuil about this topic at fosdem and > > apparently both v4l and alsa have something like this already in their > > helper librarie

Re: [media] gspca - topro: New subdriver for Topro webcams

2014-01-30 Thread Jean-Francois Moine
On Thu, 30 Jan 2014 15:14:09 +0300 Dan Carpenter wrote: > Hello Jean-François Moine, > > The patch 8f12b1ab2fac: "[media] gspca - topro: New subdriver for > Topro webcams" from Sep 22, 2011, leads to the following > static checker warning: > drivers/media/usb/gspca/topro.c:4642 > sd_

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

2012-11-26 Thread Jean-Francois Moine
On Mon, 26 Nov 2012 18:12:41 +0100 Antonio Ospite wrote: > BTW the documentation might also be wrong or inaccurate. The ov7670 documentation has exactly the same description of the register 0x2b, and I don't think that the manufacturer would greatly change the meaning of such low registers in so

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

2012-11-26 Thread Jean-Francois Moine
On Mon, 26 Nov 2012 14:08:06 +0100 Antonio Ospite 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 for > 60Hz either. > > Should I remove it from patchwork as well? > > As I have the webcam and can perfor

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

2012-11-23 Thread Jean-Francois Moine
On Fri, 23 Nov 2012 18:09:09 +0100 Antonio Ospite wrote: > On Thu, 22 Nov 2012 12:46:52 +0100 [snip] > Jean-Francois Moine wrote: > > This patch was done thanks to the documentation of the right > > OmniVision sensors. > > In the datasheet I have for ov772x, bi

[PATCH] gspca - stv06xx: Fix a regression with the bridge/sensor vv6410

2012-11-22 Thread Jean-Francois Moine
From: Jean-François Moine Setting the H and V flip controls at webcam connection time prevents the webcam to work correctly. This patch checks if the webcam is streaming before setting the flips. It does not set the flips (nor other controls) at webcam start time. Tested-by: Philippe ROUBACH S

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

2012-11-22 Thread Jean-Francois Moine
From: Jean-François Moine The exchanges relative to the light frequency filter were adapted from a description found in a ms-windows driver. It seems that the registers were the ones of some other sensor. This patch was done thanks to the documentation of the right OmniVision sensors. Note: The

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

2012-11-22 Thread Jean-Francois Moine
(fix lack of signature) From: Jean-François Moine The exchanges relative to the light frequency filter were adapted from a description found in a ms-windows driver. It seems that the registers were the ones of some other sensor. This patch was done thanks to the documentation of the right OmniVi

Re: video: USB webcam fails since kernel 3.2

2012-10-09 Thread Jean-Francois Moine
On Tue, 9 Oct 2012 15:54:46 -0700 Jonathan Nieder wrote: > Hi, > > In June, Martin-Éric Racine wrote: > > > Since recent kernels, this ASUS W5F's built-in webcam fails to be > > detected. Gstreamer-based applications (Cheese, > > gstreamer-properties) immediately crash whenever trying to access

Re: video: USB webcam fails since kernel 3.2

2012-07-11 Thread Jean-Francois Moine
On Wed, 11 Jul 2012 16:43:47 +0300 Martin-Éric Racine wrote: > > Jean-Francois, can you perhaps make a patch against my latest tree for > > the po / PO3130 changes in your tarbal? > > Noted. Hopefully, the Debian kernel team can contribute to the > backporting part, since it's needed for

Re: video: USB webcam fails since kernel 3.2

2012-07-11 Thread Jean-Francois Moine
On Wed, 11 Jul 2012 14:14:24 +0300 Martin-Éric Racine wrote: > CC [M] /home/perkelix/gspca-2.15.18/build/ov534_9.o > /home/perkelix/gspca-2.15.18/build/ov534_9.c: In function ‘sd_init’: > /home/perkelix/gspca-2.15.18/build/ov534_9.c:1353:3: error: implicit > declaration of function ‘err’ [-Wer

Re: video: USB webcam fails since kernel 3.2

2012-07-11 Thread Jean-Francois Moine
On Wed, 11 Jul 2012 13:21:55 +0300 Martin-Éric Racine wrote: > I installed them. That still doesn't fix it: > > $ LC_ALL=C make > make -C /lib/modules/3.5.0-rc6+/build > M=/home/perkelix/gspca-2.15.18/build modules > make[1]: Entering directory `/usr/src/linux-headers-3.5.0-rc6+' > /usr/src/linu

Re: video: USB webcam fails since kernel 3.2

2012-07-11 Thread Jean-Francois Moine
On Wed, 11 Jul 2012 10:02:27 +0300 Martin-Éric Racine wrote: [snip] > > wget http://moinejf.free.fr/gspca-2.15.18.tar.gz > > tar -zxf gspca-2.15.18.tar.gz > > cd gspca-2.15.18 > > make > > $ LC_ALL=C make > make -C /lib/modules/3.5.0-rc6+/build > M=/home/pe

Re: video: USB webcam fails since kernel 3.2

2012-07-10 Thread Jean-Francois Moine
On Tue, 10 Jul 2012 15:56:08 +0300 Martin-Éric Racine wrote: [snip] > I hope that the above already provides some usable answers. Not a lot :( Well, I already saw these errors -71. One case was a cable problem. An other one occurred with skype only, while vlc worked correctly. So, it wo

Re: video: USB webcam fails since kernel 3.2

2012-07-08 Thread Jean-Francois Moine
On Sun, 08 Jul 2012 19:58:08 +0200 Hans de Goede wrote: > Hmm, this is then likely caused by the new isoc bandwidth negotiation code > in 3.2, unfortunately the vc032x driver is one of the few gspca drivers > for which I don't have a cam to test with. Can you try to build your own > kernel from s

[PATCH] gspca - sonixj: Fix bad values of webcam 0458:7025

2012-05-28 Thread Jean-Francois Moine
The webcam 0458:7025 (Eye911Q) has: - an inverted power pin, - a sensor mi0360b which cannot be probed. Signed-off-by: Jean-François Moine --- drivers/media/video/gspca/sonixj.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/video/gspca/sonixj.c b/drivers/m

[PATCH] gspca - ov534/ov534_9: Fix sccd_read/write errors

2012-05-28 Thread Jean-Francois Moine
The ov534 bridge is too slow to handle the sensor accesses requested by fast hosts giving 'sccb_reg_write failed'. A small delay fixes the problem. Signed-off-by: Jean-François Moine --- drivers/media/video/gspca/ov534.c |1 + drivers/media/video/gspca/ov534_9.c |1 + 2 files changed,

[PATCH] gspca: Maintainer change

2012-05-24 Thread Jean-Francois Moine
:Maintained F: drivers/media/video/gspca/t613.c GSPCA USB WEBCAM DRIVER -M: Jean-Francois Moine -W: http://moinejf.free.fr +M: Hans de Goede L: linux-media@vger.kernel.org T: git git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media.git S

gspca maintenance handover

2012-05-24 Thread Jean-Francois Moine
Hi Mauro, I was glad and proud to work for 4 years in the Linux media team, but, now, I am a bit weary of webcams! I proposed to Hans de Goede to take the maintenance of gspca. He perfectly knows the code and does a good work. The driver will be in safe hands. Anyway, I will not fully give up. I

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

2012-05-18 Thread Jean-Francois Moine
On Wed, 16 May 2012 23:42:43 +0200 Antonio Ospite wrote: > The second patch removes the dependency between auto gain and auto white > balance, I'd like to hear Jean-Francois on this, the webcam (the ov772x > sensor) is able to set the two parameters independently and the user can > see the differ

Re: How fix driver for this USB camera (MT9T031 sensor and Cypress FX2LP USB bridge)

2012-05-16 Thread Jean-Francois Moine
On Wed, 16 May 2012 22:14:48 + (UTC) Simon Gustafsson wrote: > 2) Where should I begin? My gut feeling is to go for > media/video/gspca/ov519.c, > since it has the code for talking to the USB bridge chip (BRIDGE_OVFX2), and Hi Simon, The FX2 is a processor, and, in ov519, used as a bridge,

Re: gspca zc3xx - JPEG quality / frame overflow

2012-05-06 Thread Jean-Francois Moine
On Sun, 06 May 2012 20:17:08 +0200 Hans de Goede wrote: > > - as it is (read the register 11 every 100 ms), the work queue is > >usefull when there is no polling of the snapshot button, because the > >frame overflow is reported as the bit 0 in the forth byte (data[3]) > >of the interr

gspca zc3xx - JPEG quality / frame overflow

2012-05-05 Thread Jean-Francois Moine
Hi Hans, I quickly looked at your patches about the changes for the JPEG quality, and I have some remarks. Indeed, as I don't have any zc3xx webcam nor a lot of documentation about the zc3xx bridge, my information come only from USB trace analysis, and I am not sure there are fully valid. - the

Re: [RFCv1 PATCH 1/7] gspca: allow subdrivers to use the control framework.

2012-05-05 Thread Jean-Francois Moine
On Sat, 05 May 2012 17:05:28 +0200 Hans de Goede wrote: > > Now I see that we are doing exactly that in for example vidioc_g_jpegcomp > > in gspca.c, so > > we should stop doing that. We can make vidioc_g/s_jpegcomp only do the usb > > locking if > > gspca_dev->vdev.ctrl_handler == NULL, and on

Re: [RFCv1 PATCH 1/7] gspca: allow subdrivers to use the control framework.

2012-05-05 Thread Jean-Francois Moine
On Sat, 05 May 2012 16:59:30 +0200 Hans de Goede wrote: > > Unless there is another good reason for doing the probing in sd_init I > > prefer > > to move it to sd_config. > > Sensor probing does more then just sensor probing, it also configures > things like the i2c clockrate, and if the bus

[GIT PATCHES FOR 3.4] gspca for_v3.4

2012-05-02 Thread Jean-Francois Moine
The following changes since commit 976a87b9ce3172065e21f0d136353a01df06d0d6: [media] gspca - sn9c20x: Change the exposure setting of Omnivision sensors (2012-04-09 14:22:52 -0300) are available in the git repository at: git://linuxtv.org/jfrancois/gspca.git for_v3.4 for you to fetch change

Re: [patch] [media] gspca: passing wrong length parameter to reg_w()

2012-05-01 Thread Jean-Francois Moine
0x0010, reg10, 2); > reg_w_val(gspca_dev, 0x0098, 0x41); > for (i = 0; i < 11; i++) { > if (i == 3 || i == 5 || i == 8) Hi Dan, Thanks for the patch. The bug is very very old (6 years, at least - neither have I such a webcam). Maybe the fix could have been r

Re: [RFCv1 PATCH 0/7] gspca: allow use of control framework and other fixes

2012-05-01 Thread Jean-Francois Moine
On Mon, 30 Apr 2012 13:13:59 +0200 Hans de Goede wrote: > I'll review this and add these to my tree. Jean-Francois, is it ok for these > changes > to go upstream through my tree? The reason I'm asking is that I plan to > convert > more subdrivers to the control framework for 3.5 and its easiest

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

2012-04-27 Thread Jean-Francois Moine
On Fri, 27 Apr 2012 10:20:23 +0200 Hans Verkuil wrote: > I might have some time (no guarantees yet) to help with this. It would > certainly be interesting to add support for the control framework in the > gspca core. Hmm, perhaps that's a job for the weekend... Hi Hans, I hope you will not do i

Re: gspca V4L2_CID_EXPOSURE_AUTO and VIDIOC_G/S/TRY_EXT_CTRLS

2012-04-27 Thread Jean-Francois Moine
On Wed, 18 Apr 2012 15:37:20 +0200 Antonio Ospite wrote: > I noticed that AEC (Automatic Exposure Control, or > V4L2_CID_EXPOSURE_AUTO) does not work in the ov534 gspca driver, either > from guvcview or qv4l2. [snip] > So in ov534, but I think in m5602 too, V4L2_CID_EXPOSURE_AUTO does not

Re: [PATCH v2] tinyjpeg: Dynamic luminance quantization table for Pixart JPEG

2012-04-25 Thread Jean-Francois Moine
Hi Hans, On Wed, 25 Apr 2012 16:19:57 +0200 Hans de Goede wrote: > > You say that the marker cannot be in the range 0..31 (index 0..7), but > > I have never seen a value lower than 68 (index 17). > > If you change register 0x80 in bank/page 1 to > 42 on pac7311 or larger then > circa 100 on p

Re: [PATCH v2] tinyjpeg: Dynamic luminance quantization table for Pixart JPEG

2012-04-24 Thread Jean-Francois Moine
On Mon, 23 Apr 2012 23:34:05 +0200 Hans de Goede wrote: > Thanks for your work on this! I've just spend almost 4 days wrestling > which the Pixart JPEG decompression code to try to better understand > these cams, and I have learned quite a bit and eventually came up > with a different approach. >

[PATCH v2] tinyjpeg: Dynamic luminance quantization table for Pixart JPEG

2012-04-12 Thread Jean-Francois Moine
In PJPG blocks, a marker gives the quantization tables to use for image decoding. This patch dynamically updates the luminance table when the marker changes. Note that the values of this table have been guessed from a small number of images and that they may not work fine in some situations, but,

[PATCH] tinyjpeg: Dynamic luminance quantization table for Pixart JPEG

2012-04-11 Thread Jean-Francois Moine
In PJPG blocks, a marker gives the quantization tables to use for image decoding. This patch dynamically updates the luminance table when the marker changes. Note that the values of this table have been guessed from a small number of images and that they may not work fine in some situations, but,

Re: Startup delay needed for a Sonix camera

2012-03-29 Thread Jean-Francois Moine
On Thu, 29 Mar 2012 09:16:59 +0200 Rafał Rzepecki wrote: > Actually after some more research I've found that b4b01071379 has fixed it. > (Jean-François says in the commit message "This problem was introduced > by commit 0e4d413af1a9d and its exact effects are unknown." So I guess > they are known

[GIT PATCHES FOR 3.4] gspca for_v3.4

2012-03-24 Thread Jean-Francois Moine
The following changes since commit f92c97c8bd77992ff8bd6ef29a23dc82dca799cb: [media] update CARDLIST.em28xx (2012-03-19 23:12:02 -0300) are available in the git repository at: git://linuxtv.org/jfrancois/gspca.git for_v3.4 for you to fetch changes up to 9aadae9dfed054929f80c9f2a7e8b35b195f0

[PATCH] tinyjpeg: Better luminance quantization table for Pixart JPEG

2012-03-23 Thread Jean-Francois Moine
An other luminance quantization table gives a better quality to the Pixart images created by the webcams handled by the gspca drivers pac7302 and pac7311 (pixel format 'PJPG'). Tests have been done with 5 different pac7302 webcams. The marker was always 0x44. Signed-off-by: Jean-François Moine

Re: [Q] v4l buffer format inside isoc

2012-03-21 Thread Jean-Francois Moine
On Tue, 20 Mar 2012 21:05:13 -0300 Ezequiel García wrote: > I'm a little lost while writing a driver for an easycap device > (saa7113 capture device). > I have my isoc handler, and the isoc urb flying OK. > I also have the videobuf2 queue setup (or at least I think so), and I > understand > I ne

[GIT PATCHES FOR 3.4] gspca for_v3.4

2012-03-19 Thread Jean-Francois Moine
Hi Mauro, This set includes the patch http://patchwork.linuxtv.org/patch/9494. The following changes since commit 632fba4d012458fd5fedc678fb9b0f8bc59ceda2: [media] cx25821: Add a card definition for "No brand" cards that have: subvendor = 0x subdevice = 0x (2012-03-08 12:42:28 -0300)

Re: My Microdia (SN9C201) webcam succumbs to glare in Linux

2012-03-16 Thread Jean-Francois Moine
On Fri, 16 Mar 2012 08:53:51 +1100 Xavion wrote: > As you can probably gather from the attached screenshots, I'm > attempting to use my SN9C201 webcam for home security.  The problem is > that it succumbs to external glare during the middle hours of sunny > days when used in Linux. > > The same

Re: [GIT PATCHES FOR 3.4] gspca for_v3.4

2012-03-10 Thread Jean-Francois Moine
On Tue, 28 Feb 2012 12:15:43 +0100 Sylwester Nawrocki wrote: > > I checked the changes in zc3xx.c, and I have made many commits. So, it > > would be simpler if you would remove your patch. I could give you a > > merged one once the media tree would be updated. > > OK, if it's easier please car

Re: My Microdia (SN9C201) webcam doesn't work properly in Linux anymore

2012-03-07 Thread Jean-Francois Moine
On Wed, 7 Mar 2012 09:59:28 +1100 Xavion wrote: > >>     root@Desktop /etc/motion # tail /var/log/kernel.log > >>     Mar  6 08:34:17 Desktop kernel: [ 7240.125167] gspca_main: ISOC > >> data error: [0] len=0, status=-18 > >>... > > > > Hmm, error -18 is EXDEV, which according to > > Docume

Re: My Microdia (SN9C201) webcam doesn't work properly in Linux anymore

2012-03-05 Thread Jean-Francois Moine
On Mon, 5 Mar 2012 08:58:30 +1100 Xavion wrote: > I can confirm that GSPCA v2.15.1 removes the bad pixels when I use > Cheese or VLC. However, I'm sorry to report that the Motion problems > unfortunately still remain. Is there something else I must do to > overcome the below errors? I'm happy

Re: My Microdia (SN9C201) webcam doesn't work properly in Linux anymore

2012-03-05 Thread Jean-Francois Moine
On Mon, 05 Mar 2012 09:33:18 +0100 Hans de Goede wrote: > I guess that motion is using the JPG compressed frames rather then > the i420 like special format these cameras also support, and it looks > like we don't reserve enoug space to buffer these frames. To fix this > we need to enlarge the siz

Re: [bug?] ov519 fails to handle Hercules Deluxe webcam

2012-03-05 Thread Jean-Francois Moine
On Mon, 5 Mar 2012 03:34:30 -0600 Jonathan Nieder wrote: > > To be sure, try the gspca test version from my web site. > > Skippy, assuming that works (and I expect it would), could you try the > attached patch against 2.6.32.y? It works like this: > > 0. Prerequisites: > apt-get insta

Re: [bug?] ov519 fails to handle Hercules Deluxe webcam

2012-03-05 Thread Jean-Francois Moine
On Sun, 4 Mar 2012 18:38:01 -0600 Jonathan Nieder wrote: > Hi, > > Skippy le Grand Gourou wrote[1]: > > > Hercules Deluxe USB webcam won't work, see the end of the kernel > > log. > [...] > > [521041.808976] gspca: probing 05a9:4519 > > [521042.469094] ov519: I2C synced in 3 attempt(s) > > [521

Re: My Microdia (SN9C201) webcam doesn't work properly in Linux anymore

2012-03-03 Thread Jean-Francois Moine
On Sun, 4 Mar 2012 11:25:19 +1100 Xavion wrote: > Thanks for letting me know that this problem will be fixed in Linux > v3.3. It could be several weeks before my distribution releases that > kernel. Dropping back to Linux v3.1 isn't an option, as my NVIDIA > driver requires Linux v3.2. Can the

Re: [GIT PATCHES FOR 3.4] gspca for_v3.4

2012-02-28 Thread Jean-Francois Moine
On Mon, 27 Feb 2012 21:01:21 +0100 Sylwester Nawrocki wrote: > This patch will conflict with patch: > > gspca: zc3xx: Add V4L2_CID_JPEG_COMPRESSION_QUALITY control support > > from my recent pull request http://patchwork.linuxtv.org/patch/10022/ > > How should we proceed with that ? Do you wa

[GIT PATCHES FOR 3.4] gspca for_v3.4

2012-02-27 Thread Jean-Francois Moine
The following changes since commit a3db60bcf7671cc011ab4f848cbc40ff7ab52c1e: [media] xc5000: declare firmware configuration structures as static const (2012-02-14 17:22:46 -0200) are available in the git repository at: git://linuxtv.org/jfrancois/gspca.git for_v3.4 for you to fetch changes

[PATCH] libv4lconvert: Add new rotated_90 webcam 06f8:301b

2012-02-18 Thread Jean-Francois Moine
The webcam Hercules Link is handled by the driver gspca pac7302. Signed-off-by: Jean-François Moine --- lib/libv4lconvert/control/libv4lcontrol.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/lib/libv4lconvert/control/libv4lcontrol.c b/lib/libv4lconvert/control/libv4

Re: [PATCH/RFC v4 3/3] gspca: zc3xx: Add V4L2_CID_JPEG_COMPRESSION_QUALITY control support

2012-01-25 Thread Jean-Francois Moine
iver. > Please let me know if there is anything that could be changed/improved. Hi Sylwester, The patch is OK for me. Acked-by: Jean-Francois Moine Thanks. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ -- To unsu

Re: [PATCH/RFC v2 3/4] gspca: sonixj: Add V4L2_CID_JPEG_COMPRESSION_QUALITY control support

2012-01-14 Thread Jean-Francois Moine
On Sat, 14 Jan 2012 18:42:20 +0100 Sylwester Nawrocki wrote: > Thank you for reviewing the patches. I wasn't sure I fully understood > the locking, hence I left the 'quality' field in 'struct sd' not removed. > I've modified both subdrivers according to your suggestions. I would have > one ques

Re: [PATCH/RFC v2 3/4] gspca: sonixj: Add V4L2_CID_JPEG_COMPRESSION_QUALITY control support

2012-01-14 Thread Jean-Francois Moine
On Fri, 6 Jan 2012 19:14:41 +0100 Sylwester Nawrocki wrote: > The JPEG compression quality value can currently be read using the > VIDIOC_G_JPEGCOMP ioctl. As the quality field of struct v4l2_jpgecomp > is being deprecated, we add the V4L2_CID_JPEG_COMPRESSION_QUALITY > control, so after the dep

Re: [PATCH/RFC v2 4/4] gspca: zc3xx: Add V4L2_CID_JPEG_COMPRESSION_QUALITY control support

2012-01-14 Thread Jean-Francois Moine
On Fri, 6 Jan 2012 19:14:42 +0100 Sylwester Nawrocki wrote: > The JPEG compression quality control is currently done by means of the > VIDIOC_S/G_JPEGCOMP ioctls. As the quality field of struct v4l2_jpgecomp > is being deprecated, we add the V4L2_CID_JPEG_COMPRESSION_QUALITY control, > so after

[GIT PATCHES FOR 3.3] gspca for_v3.3

2012-01-05 Thread Jean-Francois Moine
Hi Mauro, This set includes the patch http://patchwork.linuxtv.org/patch/8858. Most of these patches concern regression fixes and should be backported to the kernel 3.2. The following changes since commit 1e73fa5d56333230854ae9460579eb2fcee8af02: [media] stb6100: Properly retrieve symbol rate

Re: [GIT PATCHES FOR 3.3] gspca patches and new jl2005bcd driver

2011-12-30 Thread Jean-Francois Moine
On Fri, 30 Dec 2011 11:56:38 +0100 Hans de Goede wrote: > I took it as is from Theodore, I guess we should do a separate cleanup > patch on top to preserve the history / authorship. Since I'm busy testing > the new isoc bandwidth stuff today, could you perhaps do a cleanup patch for > this? Yes

Re: [GIT PATCHES FOR 3.3] gspca patches and new jl2005bcd driver

2011-12-30 Thread Jean-Francois Moine
On Fri, 30 Dec 2011 10:29:56 +0100 Hans de Goede wrote: [snip] > The following changes since commit 1a5cd29631a6b75e49e6ad8a770ab9d69cda0fa2: > >[media] tda10021: Add support for DVB-C Annex C (2011-12-20 14:01:08 -0200) > > are available in the git repository at: >git://linuxtv.

Re: [PATCH for 3.2 URGENT] gspca: Fix bulk mode cameras no longer working (regression fix)

2011-12-30 Thread Jean-Francois Moine
On Thu, 29 Dec 2011 21:36:42 +0100 Hans de Goede wrote: > The new iso bandwidth calculation code accidentally has broken support > for bulk mode cameras. This has broken the following drivers: > finepix, jeilinj, ovfx2, ov534, ov534_9, se401, sq905, sq905c, sq930x, > stv0680, vicam. > > Thix pat

Re: zc3xx webcam crashes on in-focus pictures

2011-12-11 Thread Jean-Francois Moine
On Sun, 11 Dec 2011 12:59:39 +0100 Johannes Bauer wrote: [snip] > I have a Logitech Labtec webcam: > > Bus 001 Device 012: ID 046d:08a2 Logitech, Inc. Labtec Webcam Pro > > Which I use with the zc3xx driver on a 2.6.34 Linux: > > usb 1-3.2: new full speed USB device using ehci_hcd and a

[GIT PATCHES FOR 3.3] gspca for_v3.3

2011-12-05 Thread Jean-Francois Moine
The following changes since commit 36d36b884c745c507d9b3f60eb42925749f7d758: [media] tm6000: Warning cleanup (2011-11-28 21:58:54 -0200) are available in the git repository at: git://linuxtv.org/jfrancois/gspca.git for_v3.3 Jean-François Moine (6): gspca: Remove the useless variable 'r

Re: [RFC] JPEG encoders control class

2011-11-29 Thread Jean-Francois Moine
On Mon, 28 Nov 2011 19:48:28 +0100 Luca Risolia wrote: > > Note that et61x251 and sn9c102 are going to be deprecated and removed at > > some > > time in the future (gspca will support these devices). > > Has this been discussed yet? Also, last time I tried the gspca driver > with a number of

Re: Cleanup proposal for media/gspca

2011-11-19 Thread Jean-Francois Moine
On Sat, 19 Nov 2011 15:59:50 -0300 Ezequiel wrote: > Hi Jef, > > I just sent a patch to linux-media for this little issue. > > I realize it is only a very minor patch, > so I am not sure If I am helping or just annoying the developers ;) > > Anyway, if you could check the patch I would appre

Re: Cleanup proposal for media/gspca

2011-11-17 Thread Jean-Francois Moine
On Wed, 16 Nov 2011 15:19:04 -0300 Ezequiel García wrote: > In 'media/video/gspca/gspca.c' I really hated this cast (maybe because > I am too dumb to understand it): > > gspca_dev = (struct gspca_dev *) video_devdata(file); > > wich is only legal because a struct video_device is the first mem

[GIT PATCHES FOR 3.2] gspca for_v3.2

2011-09-23 Thread Jean-Francois Moine
Hi Mauro, This set includes the patches: http://patchwork.linuxtv.org/patch/7358 http://patchwork.linuxtv.org/patch/114 Cheers. The following changes since commit e553000a14ead0e265a8aa4d241c7b3221e233e3: [media] sr030pc30: Remove empty s_stream op (2011-09-21 12:48:45 -0300)

Re: Question about USB interface index restriction in gspca

2011-09-21 Thread Jean-Francois Moine
On Mon, 19 Sep 2011 22:13:12 +0200 Frank Schäfer wrote: > > So, if your webcam is in the 99.99% which use the interface 0, use > > gspca_dev_probe(), otherwise, YOU know the right interface, so, call > > gspca_dev_probe2(). > > Isn't it also possible that we don't know the right interface and

Re: Question about USB interface index restriction in gspca

2011-09-15 Thread Jean-Francois Moine
On Thu, 15 Sep 2011 23:46:57 +0200 Frank Schäfer wrote: > > For webcam devices, the interface class is meaningful only when set to > > USB_CLASS_VIDEO (UVC). Otherwise, I saw many different values. > > Does that mean that there are devices out in the wild that report for > example USB_CLASS_WIR

Re: Question about USB interface index restriction in gspca

2011-09-13 Thread Jean-Francois Moine
On Tue, 13 Sep 2011 21:14:28 +0200 Frank Schäfer wrote: > I have a question about the following code in gspca.c: > > in function gspca_dev_probe(...): > ... > /* the USB video interface must be the first one */ > if (dev->config->desc.bNumInterfaces != 1 > && intf->cur_altsetting-

Re: [PATCH V2] gspca/zc3xx: add usb_id for HP Premium Starter Cam

2011-09-09 Thread Jean-Francois Moine
On Fri, 9 Sep 2011 11:33:15 +0200 Wolfram Sang wrote: > On Sat, Jul 02, 2011 at 02:02:00PM +0200, Wolfram Sang wrote: > > Signed-off-by: Wolfram Sang > > Cc: Jean-Francois Moine > > --- > > Ping. I can't find it in the git-repos on linuxtv.org? Hi Wolfram

Re: spca1528 device (Device 015: ID 04fc:1528 Sunplus Technology)..libv4l2: error turning on stream: Timer expired issue

2011-09-05 Thread Jean-Francois Moine
On Mon, 05 Sep 2011 11:33:49 -0400 Mauricio Henriquez wrote: > Thanks Jean, yeap I apply the patch, but still the same kind of messages > about timeout sing xawtv or svv: OK Mauricio. So, I need more information. May you set the gspca debug level to 0x0f echo 0x0f > /sys/module/gspca_m

Re: spca1528 device (Device 015: ID 04fc:1528 Sunplus Technology)..libv4l2: error turning on stream: Timer expired issue

2011-09-05 Thread Jean-Francois Moine
On Sun, 04 Sep 2011 15:39:30 -0400 Mauricio Henriquez wrote: > Recently I'm trying to make work a Sunplus crappy mini HD USB camera, lsusb > list this info related to the device: > > Picture Transfer Protocol (PIMA 15470) > Bus 001 Device 015: ID 04fc:1528 Sunplus Technology Co., Ltd > > idVe

Re: [git:v4l-dvb/for_v3.2] [media] Fix wrong register mask in gspca/sonixj.c

2011-08-29 Thread Jean-Francois Moine
On Mon, 29 Aug 2011 09:57:26 -0700 (PDT) Luiz Ramos wrote: > Mauro: > > To be fair, this patch itself isn't sufficient to solve the problem described > in the text provided. One other patch is necessary to get this goal > accomplished, named, one published in this same thread in 2011-07-18. >

Re: [PATCH 06/14] [media] cx18: Use current logging styles

2011-08-27 Thread Jean-Francois Moine
On Sat, 27 Aug 2011 09:42:32 -0700 Joe Perches wrote: > Andy, I fully understand how this stuff works. > You apparently don't (yet). > > Look at include/linux/printk.h > > #ifndef pr_fmt > #define pr_fmt(fmt) fmt > #endif > > A default empty define is used when one > is not specified before pr

Re: [PATCH 14/14] [media] gspca: Use current logging styles

2011-08-22 Thread Jean-Francois Moine
Hi Joe, OK, I did not see that you started such changes a long time ago! Thanks and good luck! Acked-by: Jean-Francois Moine -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ -- To unsubscribe from this list: send the

Re: [PATCH 14/14] [media] gspca: Use current logging styles

2011-08-22 Thread Jean-Francois Moine
On Sun, 21 Aug 2011 15:56:57 -0700 Joe Perches wrote: > Add pr_fmt. > Convert usb style logging macros to pr_. > Remove now unused old usb style logging macros. Hi Joe, Sorry, but I do not see the advantages of your patch. For gspca, the source files are bigger, and the only visible change is

Re: Image and webcam freeze on Ubuntu with Logitech QuickCam Communicate STX

2011-08-19 Thread Jean-Francois Moine
On Thu, 18 Aug 2011 12:31:51 +0200 Damien Cassou wrote: > my Logitech QuickCam Communicate STX works perfectly well for a few > minutes when I use it. However, after that the image is frozen and I > can't make it work again until I reboot the system (before rebooting, > the webcam has a blue colo

[GIT PATCHES FOR 3.2] gspca for_v3.2

2011-08-12 Thread Jean-Francois Moine
The following changes since commit 9bed77ee2fb46b74782d0d9d14b92e9d07f3df6e: [media] tuner_xc2028: Allow selection of the frequency adjustment code for XC3028 (2011-08-06 09:52:47 -0300) are available in the git repository at: git://linuxtv.org/jfrancois/gspca.git for_v3.2 Jean-François Moi

[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

Re: [Workshop-2011] Media Subsystem Workshop 2011

2011-08-04 Thread Jean-Francois Moine
On Thu, 04 Aug 2011 09:40:18 -0300 Mauro Carvalho Chehab wrote: > > What we need for this is a simple API (new v4l ioctl's I guess) for the > > stillcam mode of these dual mode cameras (stillcam + webcam). So that the > > webcam drivers can grow code to also allow access to the stored pictures, >

Re: [PATCH] Fix wrong register mask in gspca/sonixj.c

2011-07-21 Thread Jean-Francois Moine
On Thu, 21 Jul 2011 03:43:13 -0700 (PDT) Luiz Ramos wrote: > Now my doubts. Unless I misunderstood something, it seems these are > the our assumptions regarding reg01 and reg17: > > - reg01 bit 6 is set when bridge runs at 48 MHz; if reset, 24 MHz > - reg17 bits 0..4 is a mask for dividing a

Re: [PATCH] Fix wrong register mask in gspca/sonixj.c

2011-07-20 Thread Jean-Francois Moine
On Mon, 18 Jul 2011 18:39:14 -0700 (PDT) Luiz Ramos wrote: [snip] > I noticed that in 640x480 the device worked fine, but in 320x240 and > 160x120 it didn't (I mean: the image is dark). Or'ing reg17 with 0x04 > in line 2535 (as it's currently done for VGA) is sufficient to make > the webca

Re: [PATCH] Fix wrong register mask in gspca/sonixj.c

2011-07-15 Thread Jean-Francois Moine
On Fri, 15 Jul 2011 02:57:43 -0700 (PDT) Luiz Ramos wrote: > Ok, I'm now grabbing this tarball: > http://moinejf.free.fr/gspca-2.13.2.tar.gz. > > The site also features a (some) git repository(ies) but I understood > you mean the test version, is it right? Yes. The tarball is simpler to compile

  1   2   3   4   5   >