From 2cad48e88fd3bbd9cd8c73b034559441360ebb72 Mon Sep 17 00:00:00 2001 From: Kirill Marinushkin Date: Mon, 16 Apr 2018 20:26:38 +0200 Subject: [PATCH] ASoC: topology: Fix bclk and fsync inversion in set_link_hw_format() The values of bclk and fsync are inverted WRT the codec. But the existing solution already works for Broadwell, see the alsa-lib config: `alsa-lib/src/conf/topology/broadwell/broadwell.conf` This commit provides the backwards-compatible solution to fix this misuse. This commit goes in pair with the corresponding patch for linux. Signed-off-by: Kirill Marinushkin Reviewed-by: Pierre-Louis Bossart Tested-by: Pan Xiuli Tested-by: Pierre-Louis Bossart Cc: Jaroslav Kysela Cc: Takashi Iwai Cc: Mark Brown Cc: Liam Girdwood Cc: linux-kernel@vger.kernel.org Cc: alsa-devel@alsa-project.org Signed-off-by: Takashi Iwai --- topology/broadwell/broadwell.conf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/topology/broadwell/broadwell.conf b/topology/broadwell/broadwell.conf index b8405d9..09fc4da 100644 --- a/topology/broadwell/broadwell.conf +++ b/topology/broadwell/broadwell.conf @@ -393,8 +393,8 @@ SectionGraph."dsp" { SectionHWConfig."CodecHWConfig" { id "1" format "I2S" # physical audio format. - bclk "master" # Platform is master of bit clock - fsync "master" # platform is master of fsync + bclk "codec_slave" # platform is master of bit clock (codec is slave) + fsync "codec_slave" # platform is master of fsync (codec is slave) } SectionLink."Codec" { -- 2.47.3