Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-11 Thread Peter Chen
On Mon, Mar 11, 2013 at 10:58:05AM -0400, Alan Stern wrote: > On Mon, 11 Mar 2013, Alexander Shishkin wrote: > > > > For USB 3.0 host CV test, the host sends GET_DESCRIPTOR very quick > > > (<500us) > > > once it accept the status of SET_ADDRESS > > > > The USB 2.0 spec says the recovery period

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-11 Thread Alan Stern
On Mon, 11 Mar 2013, Alexander Shishkin wrote: > > For USB 3.0 host CV test, the host sends GET_DESCRIPTOR very quick (<500us) > > once it accept the status of SET_ADDRESS > > The USB 2.0 spec says the recovery period after the status phase of > SetAddress is 2ms. That should be enough to process

RE: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-11 Thread Chen Peter-B29397
> > > > For USB 3.0 host CV test, the host sends GET_DESCRIPTOR very quick > (<500us) > > once it accept the status of SET_ADDRESS > > The USB 2.0 spec says the recovery period after the status phase of > SetAddress is 2ms. That should be enough to process the transfer > completion interrupt, sh

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-11 Thread Alexander Shishkin
Peter Chen writes: > On Fri, Mar 08, 2013 at 04:06:30PM +0200, Alexander Shishkin wrote: >> Peter Chen writes: >> >> > On Fri, Mar 08, 2013 at 12:39:04PM +0200, Alexander Shishkin wrote: >> >> Peter Chen writes: >> >> >> >> > Hi David >> >> >> >> Hi, >> >> >> >> > I notice at your code for

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-10 Thread Peter Chen
On Fri, Mar 08, 2013 at 04:06:30PM +0200, Alexander Shishkin wrote: > Peter Chen writes: > > > On Fri, Mar 08, 2013 at 12:39:04PM +0200, Alexander Shishkin wrote: > >> Peter Chen writes: > >> > >> > Hi David > >> > >> Hi, > >> > >> > I notice at your code for hw_usb_set_address, there is a co

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-08 Thread Alexander Shishkin
Peter Chen writes: > On Fri, Mar 08, 2013 at 12:39:04PM +0200, Alexander Shishkin wrote: >> Peter Chen writes: >> >> > Hi David >> >> Hi, >> >> > I notice at your code for hw_usb_set_address, there is a comment for it >> > /** >> > * This function explicitly sets the address, without the "US

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-08 Thread Matthieu CASTET
Alexander Shishkin a écrit : > Peter Chen writes: > >> Hi David > > Hi, > >> I notice at your code for hw_usb_set_address, there is a comment for it >> /** >> * This function explicitly sets the address, without the "USBADRA" (advance) >> * feature, which is not supported by older versions of

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-08 Thread Peter Chen
On Fri, Mar 08, 2013 at 12:39:04PM +0200, Alexander Shishkin wrote: > Peter Chen writes: > > > Hi David > > Hi, > > > I notice at your code for hw_usb_set_address, there is a comment for it > > /** > > * This function explicitly sets the address, without the "USBADRA" > > (advance) > > * fea

Re: About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-08 Thread Alexander Shishkin
Peter Chen writes: > Hi David Hi, > I notice at your code for hw_usb_set_address, there is a comment for it > /** > * This function explicitly sets the address, without the "USBADRA" (advance) > * feature, which is not supported by older versions of the controller. > */ That's mine. David's

About USBADRA bit at DEVICEADDR for chipidea driver

2013-03-08 Thread Peter Chen
Hi David I notice at your code for hw_usb_set_address, there is a comment for it /** * This function explicitly sets the address, without the "USBADRA" (advance) * feature, which is not supported by older versions of the controller. */ If we use USB3.0 host for CV test, we must use this bit, o