17 Jan
2017
17 Jan
'17
9:34 p.m.
On Wed, Jan 11, 2017 at 04:03:13PM +0100, Nicholas Mc Guire wrote:
As the overall delay is in range of seconds and the wait granularity is 10ms msleep() seems more reasonable than to put load on the highres timer subsystem.
Thia also contains a whole bunch of other refactoring that's not mentioned in the changelog...
This does throw a checkpatch warning with: WARNING: msleep < 20ms can sleep for up to 20ms;" but since this is in a loop and the intended granularity is 10ms with up to 2 seconds total delay this seems ok.
That does depend on how quickly the operation is expected to complete, the total delay is likely to be a massive overestimate.