Re: [PATCH RFC] ASoC: rt5663: use msleep() for long delay loop

2017-01-19 Thread Nicholas Mc Guire
On Tue, Jan 17, 2017 at 07:34:34PM +, Mark Brown wrote: > 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. >

Re: [PATCH RFC] ASoC: rt5663: use msleep() for long delay loop

2017-01-17 Thread Mark Brown
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 men

[PATCH RFC] ASoC: rt5663: use msleep() for long delay loop

2017-01-11 Thread Nicholas Mc Guire
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. Fixes: commit df7c52168ee1 ("ASoC: add rt5663 codec driver") Signed-off-by: Nicholas Mc Guire --- Problem was found by cocinelle script. Th