Re: gspca - pac7302: Add a delay on loading the bridge registers.

Re: gspca - pac7302: Add a delay on loading the bridge registers.

[Németh Márton]

Hi,

At http://linuxtv.org/hg/~jfrancois/gspca/rev/b0a374674388 :
> From: Jean-Francois Moine <moinejf@free.fr>
>
> Without the delay, the usb_control_msg() may fail when loading the
> page 3 with error -71 or -62.
>
> Priority: normal
>
> Signed-off-by: Jean-Francois Moine <moinejf@free.fr>

include/asm-generic/errno.h:
> #define ETIME           62      /* Timer expired */
> #define EPROTO          71      /* Protocol error */

I'm interested in which device have you used for testing this?

Regards,

	Márton Németh

Re: gspca - pac7302: Add a delay on loading the bridge registers.

[Jean-Francois Moine]

On Fri, 08 Jan 2010 18:29:02 +0100
Németh Márton <nm127@freemail.hu> wrote:

> > Without the delay, the usb_control_msg() may fail when loading the
> > page 3 with error -71 or -62.
> >
> > Priority: normal
> >
> > Signed-off-by: Jean-Francois Moine <moinejf@free.fr>  
> 
> include/asm-generic/errno.h:
> > #define ETIME           62      /* Timer expired */
> > #define EPROTO          71      /* Protocol error */  
> 
> I'm interested in which device have you used for testing this?

Hi,

These errors occured with the webcam 06f8:3009, but I do not know
exactly how. I thought about a speed problem. May be, when the webcam
cannot treat quickly enough the requests, either it returns errors or
it crashes. An other cause could be a bug in the ohci_hcd, but I do not
looked at it. The delay was just a test and it fixed the problem. That
is all the user wanted...

Regards.

-- 
Ken ar c'hentañ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/