[alsa-devel] [v2 1/3] ASoC: Intel: Fix cracking noise in rt5663 headphones for kabylake platform
Vinod Koul
vinod.koul at intel.com
Thu Jul 27 04:40:57 CEST 2017
On Wed, Jul 26, 2017 at 01:47:43PM -0700, Harsha Priya wrote:
> This patch fixes the cracking noise in rt5663 headphones for kabylake platform
> by calling the rt5663_sel_asrc_clk_src() for RT5663_AD_STEREO_FILTER as well.
and it sets ASRC..?
>
> The ASRC function is for asynchronous MCLK and LRCK. For RT5663 ASRC should be
> enabled to support special i2s clock format like Intel's 100fs. ASRC function
^^^^
double space
> will track i2s clock and generate a corresponding system clock for codec.
> Calling this function helps select the clock source for both
> RT5663_AD_STEREO_FILTER and RT5663_DA_STEREO_FILTER filters which fixes the
> crackling sound.
>
> Signed-off-by: Harsha Priya <harshapriya.n at intel.com>
> ---
> sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c b/sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c
> index 3fe4a08..cfde894 100644
> --- a/sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c
> +++ b/sound/soc/intel/boards/kbl_rt5663_rt5514_max98927.c
> @@ -319,7 +319,9 @@ static int kabylake_rt5663_hw_params(struct snd_pcm_substream *substream,
> int ret;
>
> /* use ASRC for internal clocks, as PLL rate isn't multiple of BCLK */
> - rt5663_sel_asrc_clk_src(codec_dai->codec, RT5663_DA_STEREO_FILTER, 1);
> + rt5663_sel_asrc_clk_src(codec_dai->codec,
> + RT5663_DA_STEREO_FILTER | RT5663_AD_STEREO_FILTER,
> + RT5663_CLK_SEL_I2S1_ASRC);
>
> ret = snd_soc_dai_set_sysclk(codec_dai,
> RT5663_SCLK_S_MCLK, 24576000, SND_SOC_CLOCK_IN);
> --
> 1.9.1
>
--
~Vinod
More information about the Alsa-devel
mailing list