Re: [PATCH v2 1/2] ASoC: sma1303: Add driver for Iron Device SMA1303 Amp

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

 



On Thu, Jan 26, 2023 at 01:28:32AM +0000, Ki-Seok Jo wrote:

> > This at the very least needs a comment since it's not clear how someone
> > would work that out from the code, there's no connection here to the mute
> > operation so someone might just remove the delay.  It may even be that this
> > is happening well over 55ms after the mute happened.  The ideal thing would
> > be to track the time from the last mute, the regulator core does something
> > along those lines with off_on_delay.

> Currently mute control is registered in mute_stream function in
> dai operations. After that, amp shutdown is performed in DAPM
> PRE_PMD. Since a 55 ms delay was required in the meantime, it
> seems correct to change the position of the delay like you
> said. I will also include a comment about delay. Please reply
> if there is any problem.

I think that sounds fine.

> > Please fix your mail client to word wrap within paragraphs at something
> > substantially less than 80 columns.  Doing this makes your messages much
> > easier to read and reply to.

> Thanks, I fixed the mail setting.

Seems not to have worked (at least for the text version) :/

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux