Received: by 2002:a05:6a10:a841:0:0:0:0 with SMTP id d1csp4628968pxy; Tue, 27 Apr 2021 09:08:19 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxx9QiQF6QlsRHYYi3FDW3qcyFvPE+TP4XuRL6P3eGWS6wHPTd6KgdX3J2msV4v8kzcslVE X-Received: by 2002:a05:6402:3587:: with SMTP id y7mr5613460edc.54.1619539698818; Tue, 27 Apr 2021 09:08:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1619539698; cv=none; d=google.com; s=arc-20160816; b=qNTgpfQ/uW/0X98DHenV8QNzQEnm4WHFu/EGYA9dJd4sGtpzGTd3uLvZDiwJ0DFIO1 qctwLEFzEZo/WmdGP9ugC1+SHIsKvEA26AsD/dDGrAVF4eQN0wayssRqeHOVRgex+cMM pUMlyOJiK3nO2PfPXKL+g9AEFBplnIkzoHCDAEH4CUslQlzu/c+DuGGtt05gDYz2iWU7 4k9ye2Y9KTz62jSegL8155mhWVU4XVc9+u2m+NOqFJ1XFvD9CQ+R7Zcz5EY2WRbEXELd GR5pkjK4GI8wvoGuzhvZYx8YaC1C2hLaUYwKOJwdL3pO+vp/e3NtJe2cbT9rfaaratCu Vp/Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:user-agent:references:in-reply-to :subject:cc:to:from:date:content-transfer-encoding:mime-version :dkim-signature; bh=iEVFDUINoOm1rLf1JpF4YSna7leXoZim54DERO49TZg=; b=QEh+Rqag/n0vkAN+VYW0Zp2Z43PmjQ8T2S27fdhGrwi4hX0t/7bBd/1V3FgWPzv85f bBNm68mYz/x00thkfNfvsRLj34FmiGiOI3EsjQZeeiRi6rtLjxjfm0iuQuh47ZWMUAjI utyYZuZIFEBW8gbJWKBtFri4BMWZQrZaH72HNIDWe/+rUGZR5DKs2khsI3VmTf9pn6dA 3oHHrkn3EjKIaDr6UeK/tIMFg3sIvH8Ta/9dJRy5NR0gs7IwmsWva+Yjv21PGMwHX7u4 Rq3GZZUv1eXujmwA5QCyRSwIzEjvwWqkYTclst5VNB1owBVVPeYKnktOt+mtb1HhfQiW Vl2g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@walle.cc header.s=mail2016061301 header.b=L4N6lp0+; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b1si303438ejh.35.2021.04.27.09.07.52; Tue, 27 Apr 2021 09:08:18 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@walle.cc header.s=mail2016061301 header.b=L4N6lp0+; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236552AbhD0PaW (ORCPT + 99 others); Tue, 27 Apr 2021 11:30:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40038 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235974AbhD0PaW (ORCPT ); Tue, 27 Apr 2021 11:30:22 -0400 Received: from ssl.serverraum.org (ssl.serverraum.org [IPv6:2a01:4f8:151:8464::1:2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CEFB5C061574 for ; Tue, 27 Apr 2021 08:29:38 -0700 (PDT) Received: from ssl.serverraum.org (web.serverraum.org [172.16.0.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ssl.serverraum.org (Postfix) with ESMTPSA id 22EF422258; Tue, 27 Apr 2021 17:29:35 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=walle.cc; s=mail2016061301; t=1619537376; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=iEVFDUINoOm1rLf1JpF4YSna7leXoZim54DERO49TZg=; b=L4N6lp0+PC1eCnVBMbtTlBBkb+IzG0TBsOvw84xVahqO+Sh7B/0/M9CY15dE/d80zOyPbe xPQyAn33WuYdxKv0CcnDut2h1bdBkOToAcYb8hue/Muh7BERXKSAYvreEiwaIUTzbXMmlE I4JIhiY7P7Ly+JZ4CHEa9PAlxOeT6ts= MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Tue, 27 Apr 2021 17:29:35 +0200 From: Michael Walle To: Mark Brown Cc: Guillaume Tucker , kernelci-results@groups.io, Kuninori Morimoto , Heiko Thiery , alsa-devel@alsa-project.org, Jaroslav Kysela , Liam Girdwood , Takashi Iwai , Stephan Gerhold , Thierry Reding , Jon Hunter , linux-kernel@vger.kernel.org Subject: Re: broonie-sound/for-next bisection: baseline.bootrr.asoc-simple-card-probed on kontron-sl28-var3-ads2 In-Reply-To: References: <6080e82c.1c69fb81.cd60c.2a13@mx.google.com> <3ca62063-41b4-c25b-a7bc-8a8160e7b684@collabora.com> <877dkp5141.wl-kuninori.morimoto.gx@renesas.com> <20210426144242.GF4590@sirena.org.uk> <8735vc4r59.wl-kuninori.morimoto.gx@renesas.com> <20210427101926.GA4605@sirena.org.uk> <20210427135703.GH4605@sirena.org.uk> User-Agent: Roundcube Webmail/1.4.11 Message-ID: X-Sender: michael@walle.cc Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am 2021-04-27 16:16, schrieb Michael Walle: > Am 2021-04-27 15:57, schrieb Mark Brown: >> On Tue, Apr 27, 2021 at 03:43:42PM +0200, Michael Walle wrote: >>> Am 2021-04-27 14:51, schrieb Guillaume Tucker: >>> > On 27/04/2021 11:19, Mark Brown wrote: >>> > > On Tue, Apr 27, 2021 at 07:28:34AM +0900, Kuninori Morimoto wrote: >> >>> > > > > > If so, all sai1 - sai6 are using "fsl,vf610-sai", >>> > > > > > all saiX doesn't have .name. I think it should have different name. >>> > > > > > In your case, at least, sai5 / sai6 needs to have > > Where does that (null) come from? I've briefly tried the following > patch and I'd assume I get a duplicate for "abc-wm8904-hifi", but I > still > get the old (null)-wm8904-hifi. > > diff --git a/sound/soc/fsl/fsl_sai.c b/sound/soc/fsl/fsl_sai.c > index 407a45e48eee..36280008b5c6 100644 > --- a/sound/soc/fsl/fsl_sai.c > +++ b/sound/soc/fsl/fsl_sai.c > @@ -1081,6 +1081,7 @@ static int fsl_sai_probe(struct platform_device > *pdev) > sai->cpu_dai_drv.symmetric_rate = 1; > sai->cpu_dai_drv.symmetric_channels = 1; > sai->cpu_dai_drv.symmetric_sample_bits = 1; > + sai->cpu_dai_drv.name = "abc"; > > if (of_find_property(np, "fsl,sai-synchronous-rx", NULL) && > of_find_property(np, "fsl,sai-asynchronous", NULL)) { Ok here it is whats going on: We have a simple-audio-card. We use freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts. The codec name is parsed correctly and set to either f140000.audio-controller or f150000.audio-controller. Thus have different names. _But_ later asoc_simple_canonicalize_cpu() will reset the dai_name pointer to NULL in simple_parse_node() if "single" is 1 and then we end up having two links with the same name. Which begs the two questions: (1) What is "single" actually and when should it be 1? (2) If single is 1, then the sysfs file will be named "(null)-codec-name". Do we want that? I guess there is a reason for it to be set to NULL, see [1]. -michael [1] https://elixir.bootlin.com/linux/v5.12/source/sound/soc/generic/simple-card-utils.c#L420