Re: [PATCH 1/2] USB: turn off padding of resume time delays

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Todd Brandt <todd.e.brandt@xxxxxxxxxxxxxxx> writes:
> Add a module parameter that replaces the USB_RESUME_TIMEOUT
> and other hardcoded delay numbers with the USB spec minimums.
> By default the patch retains the current values.
>
> The USB subsystem currently uses heavily padded values for TDRSMDN
> and TRSTRCY. This patch keeps the current values by default, but if

there's a reason for that. We've found many broken devices over the
years and a lot of inconsistencies were also found while with host
controller A or B or C, because they all used different timings.

Using spec minimum is, actually, a recipe for disaster :-) There's no
way we can guarantee all devices currently in use are respecting that
part of the spec.

This is a very fragile part of the USB spec which different folks have
different understanding and that's why so many devices are "broken".

I'd be careful with such a patchset. It's a nice idea, but I'd be
careful :-)

-- 
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux