Re: uvcvideo Logitech patch

Re: uvcvideo Logitech patch

[Ondrej Zary]

On Tuesday 15 December 2009 20:03:36 Mitar wrote:
> Hi!
>
> I have Logitech QuickCam Pro 9000 webcam and I had the same problems
> described here:
>
> http://patchwork.kernel.org/patch/52261/
>
> I have applied the patch and it did not help. But it helped when I
> increased UVC_CTRL_CONTROL_TIMEOUT to 1000 and
> UVC_CTRL_STREAMING_TIMEOUT 5000. So 300 and 3000 values were not enough.
> I do not know if it was really necessary to increase
> UVC_CTRL_CONTROL_TIMEOUT or if it would be enough something between 3000
> and 5000 for UVC_CTRL_STREAMING_TIMEOUT as I did not have more time to
> test it.
>
> So maybe 5000 would be a good default for UVC_CTRL_STREAMING_TIMEOUT?
>
> I have been doing this on 2.6.30 amd64 system.
>
> Just to let you know. And thanks for the patch.
>
>
> Mitar

[Added UVC mailing lists to CC]


-- 
Ondrej Zary

Re: uvcvideo Logitech patch

[Laurent Pinchart]

Hi Mitar,

On Tuesday 15 December 2009 22:15:19 Ondrej Zary wrote:
> On Tuesday 15 December 2009 20:03:36 Mitar wrote:
> > Hi!
> >
> > I have Logitech QuickCam Pro 9000 webcam and I had the same problems
> > described here:
> >
> > http://patchwork.kernel.org/patch/52261/
> >
> > I have applied the patch and it did not help. But it helped when I
> > increased UVC_CTRL_CONTROL_TIMEOUT to 1000 and UVC_CTRL_STREAMING_TIMEOUT
> > 5000. So 300 and 3000 values were not enough.
> > I do not know if it was really necessary to increase
> > UVC_CTRL_CONTROL_TIMEOUT or if it would be enough something between 3000
> > and 5000 for UVC_CTRL_STREAMING_TIMEOUT as I did not have more time to
> > test it.
> >
> > So maybe 5000 would be a good default for UVC_CTRL_STREAMING_TIMEOUT?

Could be, but I'd like to know if increasing the control streaming timeout is 
required as well.

-- 
Regards,

Laurent Pinchart

Re: uvcvideo Logitech patch

[Mitar]

Hi!

> Could be, but I'd like to know if increasing the control streaming
> timeout is required as well.

I had some time now and have tested it and it is enough just to increase
UVC_CTRL_STREAMING_TIMEOUT to 5000, I left UVC_CTRL_CONTROL_TIMEOUT at
300. And everything seems to work.


Mitar

Re: uvcvideo Logitech patch

[Laurent Pinchart]

Hi Mitar,

On Tuesday 29 December 2009 17:51:14 Mitar wrote:
> Hi!
> 
> > Could be, but I'd like to know if increasing the control streaming
> > timeout is required as well.
> 
> I had some time now and have tested it and it is enough just to increase
> UVC_CTRL_STREAMING_TIMEOUT to 5000, I left UVC_CTRL_CONTROL_TIMEOUT at
> 300. And everything seems to work.

Thanks. I'll increase the streaming timeout value to 5000.

-- 
Regards,

Laurent Pinchart