[PATCH] ASoC: rt715: Add module parameter to fix dmic pop sound issue.

Pierre-Louis Bossart pierre-louis.bossart at linux.intel.com
Thu Sep 17 15:00:39 CEST 2020



On 9/17/20 6:25 AM, Mark Brown wrote:
> On Wed, Sep 16, 2020 at 03:47:58PM -0500, Pierre-Louis Bossart wrote:
>> From: Jack Yu <jack.yu at realtek.com>
>>
>> Add module parameter "power_up_delay" to fix pop noise on capture. The
>> power_up_delay value is set with a default value of 400ms, smaller
>> values are not recommended.
> 
> Normally we would just add a delay in the driver unconditionally, why
> make this a module paramter?  If there are board variations then we
> should be getting them from board data, not forcing individual users to
> bodge things with a module parameter.

that wasn't the intent. 400ms is the recommended value, but the 
parameter provides a way to experiment without having to recompile 
during integration/debug stages.

It was my recommendation to add this parameter, I don't mind removing it 
if you prefer it that way. Or I can respin the commit message and 
comments to make it clearer what the intended use was.



More information about the Alsa-devel mailing list