[PATCH v2] usb: ehci-orion: add more constants for register values
Alan Stern
stern at rowland.harvard.edu
Thu Mar 19 13:30:09 PDT 2015
On Thu, 19 Mar 2015, Thomas Petazzoni wrote:
> These are indeed all valid concerns. However, as you can see, those
> concerns are completely orthogonal to the patch: the original code
> already has those issues.
Quite true. You may add
Acked-by: Alan Stern <stern at rowland.harvard.edu>
to the v2 patch.
> Regarding the addition of a timeout, I unfortunately have absolutely no
> idea what would be the a proper timeout value at this place. I quickly
> glanced through
> http://www.intel.com/content/dam/www/public/us/en/documents/technical-specifications/ehci-specification-for-usb.pdf
> for the documentation of this reset bit, but couldn't spot a maximum
> accepted duration for the operation.
ehci-hcd uses 250 ms. This seems to be an arbitrary value, but at
least it's better than hanging the system.
Alan Stern
More information about the linux-arm-kernel
mailing list