From patchwork Thu Jul 21 23:32:27 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Cristian Ciocaltea X-Patchwork-Id: 12925851 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2F556C43334 for ; Thu, 21 Jul 2022 23:34:09 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 2A41E1676; Fri, 22 Jul 2022 01:33:17 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 2A41E1676 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1658446447; bh=oo2+O9BB8Qmzpv1tcToj+qItiPJ98pNfn+AXvTBYbfU=; h=From:To:Subject:Date:Cc:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From; b=ZPkJbTbNp4ZiD+CzWrmp083nsKtYxDPz/91RzINRpaTLLpu5BU8udM+j8RdPtciZK LjnoppVqUtLlwB9L6Cw5s4sFivMM7Kw+Xqsz624P23vjUe9GNZ6O1yJ3vaz4/SChXA WuVP69JGh9SM5yRJvl1mCoXS8uY5MTleElbQ3yK4= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 89805F80169; Fri, 22 Jul 2022 01:33:16 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id DFE3CF8016D; Fri, 22 Jul 2022 01:33:13 +0200 (CEST) Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 00DB8F800BA for ; Fri, 22 Jul 2022 01:33:06 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 00DB8F800BA Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="aEJOje+U" Received: from localhost (unknown [188.24.146.105]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: cristicc) by madras.collabora.co.uk (Postfix) with ESMTPSA id BE2FF6601AB4; Fri, 22 Jul 2022 00:33:04 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1658446384; bh=oo2+O9BB8Qmzpv1tcToj+qItiPJ98pNfn+AXvTBYbfU=; h=From:To:Cc:Subject:Date:From; b=aEJOje+UkZlurTkF8NLKuv8m34YKwNmSxY38UiwaCKt0+vzcMQYvL0q6ujuKKrO0h pQq7u7y6bd6TQwkyIQClE2ys8fwWdd2q7jzeR/sPU1/0wf03bTCbKybmtu/DWQSv+i tuZFFxEWbo0iTa9Ed7hjF33iC1roqZy6BLN9Zm569eEQ/daf1MdX0bu77tYrkIx/tF EDMA5H2qQ04DmdMEPRzG32BD82XjvZdLtCQRjJylZLISC+diGcM35U3AyWv8EiLGLD 9bucarg5pEOdqrkThvtH6Kb1ooTJ8qqy64rr4BKs+h3S7QC1VH7kKju9p/CUgiRlZf WUF+LlEq/mBZw== From: Cristian Ciocaltea To: Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Charles Keepax Subject: [PATCH] ASoC: amd: vangogh: Use non-legacy DAI naming for cs35l41 Date: Fri, 22 Jul 2022 02:32:27 +0300 Message-Id: <20220721233227.1459374-1-cristian.ciocaltea@collabora.com> X-Mailer: git-send-email 2.37.1 MIME-Version: 1.0 Cc: alsa-devel@alsa-project.org, kernel@collabora.com, linux-kernel@vger.kernel.org X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" Unlike most CODEC drivers, the CS35L41 driver did not have the non_legacy_dai_naming set, meaning the corresponding DAI has been traditionally registered using the legacy naming: spi-VLV1776:0x The recent migration to the new legacy DAI naming style has implicitly corrected that behavior and DAI gets now registered via the non-legacy naming, i.e. cs35l41-pcm. The problem is the acp5x platform driver is now broken as it continues to refer to the above mentioned codec using the legacy DAI naming in function acp5x_cs35l41_hw_params() and, therefore, the related setup is not being executed anymore. Let's fix that by replacing the obsolete DAI name with the correct one. Fixes: bc949a3b4af3 ("ASoC: core: Switch core to new DAI naming flag") Signed-off-by: Cristian Ciocaltea Reviewed-by: Charles Keepax --- sound/soc/amd/vangogh/acp5x-mach.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/sound/soc/amd/vangogh/acp5x-mach.c b/sound/soc/amd/vangogh/acp5x-mach.c index 727de46860b1..af3737ef9707 100644 --- a/sound/soc/amd/vangogh/acp5x-mach.c +++ b/sound/soc/amd/vangogh/acp5x-mach.c @@ -178,8 +178,7 @@ static int acp5x_cs35l41_hw_params(struct snd_pcm_substream *substream, ret = 0; for (i = 0; i < num_codecs; i++) { codec_dai = asoc_rtd_to_codec(rtd, i); - if ((strcmp(codec_dai->name, "spi-VLV1776:00") == 0) || - (strcmp(codec_dai->name, "spi-VLV1776:01") == 0)) { + if (strcmp(codec_dai->name, "cs35l41-pcm") == 0) { switch (params_rate(params)) { case 48000: bclk_val = 1536000;