On 2020/11/6 上午 12:21, Pierre-Louis Bossart wrote:
On 11/5/20 9:02 AM, Mark Brown wrote:
On Thu, Nov 05, 2020 at 10:20:33AM +0800, Tzung-Bi Shih wrote:
If nau8315 doesn't share I2S with other components for now, it could be better to not introduce the software mute control.
The mute callback is there because there's some controllers that don't start up cleanly and end up outputing glitches (for example due to not being able to flush their FIFOs) - keeping the CODEC muted until after the I2S is running covers those glitches.
Thanks for explaining those dependencies. The code looks good to me:
Reviewed-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com
Thanks for your comments. However, I also agree the opinion from Tzung-Bi Shih @ Google. May I confirm whether I should keep this patch, and remove dapm widget of EN_PIN next patch? ________________________________ ________________________________ The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.