Messages in this thread | | | From | Ryan Lee <> | Subject | RE: [PATCH] ASoC: max98373: Added 10ms delay after amp software reset | Date | Fri, 24 Aug 2018 01:00:42 +0000 |
| |
>-----Original Message----- >From: Dmitry Torokhov <dmitry.torokhov@gmail.com> >Sent: Thursday, August 23, 2018 5:08 PM >To: Mark Brown <broonie@kernel.org> >Cc: Ryan Lee <RyanS.Lee@maximintegrated.com>; Liam Girdwood ><lgirdwood@gmail.com>; Jaroslav Kysela <perex@perex.cz>; Takashi Iwai ><tiwai@suse.com>; Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>; >alsa-devel@alsa-project.org; lkml <linux-kernel@vger.kernel.org>; >ryan.lee.maxim@gmail.com >Subject: Re: [PATCH] ASoC: max98373: Added 10ms delay after amp software >reset > >EXTERNAL EMAIL > > > >On Thu, Aug 23, 2018 at 10:51:07AM +0100, Mark Brown wrote: >> On Wed, Aug 22, 2018 at 05:31:04PM -0700, Dmitry Torokhov wrote: >> > On Wed, Aug 22, 2018 at 5:21 PM Ryan Lee ><ryans.lee@maximintegrated.com> wrote: >> > > + mdelay(10); >> >> > Is it really necessary for the CPU to spin for 10msec here? >> > usleep_range() would be better solution. >> >> Or msleep() for that matter, it's not going ot be the end of the world >> if we delay a bit too long. > >Frankly, I'd rather we did not introduce extra delays. I do not know if we use >this part, but on ChromeOS we are interested in shaving as much from the boot >time as possible and 10 msecs here and 10 msecs there do add up.
Thanks for all your feedback. Let me apply usleep_range() or msleep() instead of using mdelay().
> >Thanks. > >-- >Dmitry
| |