From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 66981C433DF for ; Fri, 31 Jul 2020 16:43:39 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id E19F621744 for ; Fri, 31 Jul 2020 16:43:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="E4dbNkyO"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="BZ/IQ65k" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E19F621744 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org 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 652A91614; Fri, 31 Jul 2020 18:42:47 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 652A91614 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1596213817; bh=IHs1Gy2eQqx3hUYseGXeXPwN9kh746iSmm5M2Zufh3I=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=E4dbNkyOjOsS37RywjkhstQ5Z9Of6yW2epH5Imsb5o3kTXarspQr9/eWYRra4LOTH 8szmY7SOYMsRdPTFTp9sAk+y3v0eGg2W/8lDijdzMajX1gqVFnm2/wpYXVss2/Kg4+ wpjdQV6DIF0Qtx26aqTQsoW8YGwvBKQt929s4bns= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id DF44FF80161; Fri, 31 Jul 2020 18:42:46 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 98F82F80171; Fri, 31 Jul 2020 18:42:44 +0200 (CEST) Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id B83B0F80111 for ; Fri, 31 Jul 2020 18:42:34 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz B83B0F80111 Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="BZ/IQ65k" Received: from localhost (fw-tnat.cambridge.arm.com [217.140.96.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id B60C821744; Fri, 31 Jul 2020 16:42:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1596213752; bh=IHs1Gy2eQqx3hUYseGXeXPwN9kh746iSmm5M2Zufh3I=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=BZ/IQ65konpkGC1wadZ8q0V+e/CYsWHgatFvRtmrlSxbixuqouTkwZsnvI9OsdMGD 1qHq4Zmb2LNqSj0FEFIS5Bq89wG/0Gn2KOCxdbEyfa4TPKowf91bibs1HgZsCJDtiZ 0hAgIUTpGl05KUYFQflvzxcQgymhxUWa82YJVDl0= Date: Fri, 31 Jul 2020 17:42:11 +0100 From: Mark Brown To: Lars-Peter Clausen Subject: Re: [PATCH 0/3] ASoC: core: Two step component registration Message-ID: <20200731164211.GB4510@sirena.org.uk> References: <20200731144146.6678-1-cezary.rojewski@intel.com> <1828f3a2-f198-e6cd-ca6c-c66dc1ef82d1@linux.intel.com> <61faef6e-70e9-5574-5f3a-e333d0a71617@intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="Yylu36WmvOXNoKYn" Content-Disposition: inline In-Reply-To: X-Cookie: Some optional equipment shown. User-Agent: Mutt/1.10.1 (2018-07-13) Cc: Pierre-Louis Bossart , Cezary Rojewski , olivier.moysan@st.com, alexandre.torgue@st.com, alsa-devel@alsa-project.org, tiwai@suse.com, arnaud.pouliquen@st.com, lgirdwood@gmail.com, mcoquelin.stm32@gmail.com 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" --Yylu36WmvOXNoKYn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Jul 31, 2020 at 05:58:00PM +0200, Lars-Peter Clausen wrote: > On 7/31/20 5:47 PM, Cezary Rojewski wrote: > > Patchset targets entire ASoC framework, not Intel catalog. As > > _initialize and _add are already in place, having a two-step > > registration is cohesive with other "frameworks" e.g. device one. > For what it is worth, I think this is a sensible change for the outlined > reasons. It's something I've always had in the back of my mind, but there > was never enough of a need to actually do it. Yeah, it's a common pattern in the kernel and someone might want it so there's no great reason not to do it. > One change I'd like to see is the addition of snd_soc_component_alloc(), > which combines the step of kzalloc() and snd_soc_component_init() as these > will be done pretty much always in lockstep. And it also matches the APIs > that other frameworks offer. That'd be good. --Yylu36WmvOXNoKYn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl8kSeIACgkQJNaLcl1U h9C63Qf/WuZoonGanky6ar3rcnyaw8zYmZ9Dq+tV0cmr6kKic1pI+6e46a60bsxd p1xbgS6b27gSruRomGid+PSULgNm/lNAaG7UptgcOFWOBGfzAvDqeTP9vFI+eWDM hNBE8t0OtCP/du2zU/Gzg5rWuBtfbUI6T8Ekm/TQb/aYGDfqna0w4FcFJ355Tr3S omRMXry/wil/JTVMCrS0nMrf8SO86NDjrrQ8yxMPJig9ezgdi+AvYAOX7vJ80OlI 7asU7DJGB24iMenvgXQjEwQ8h4lTIElm3fZ6lJGZ7hjig0j9GUgu02id2h2oHz8z q/kNz1j5oh/wbmG1cKFNXcXIeWekrw== =F6E+ -----END PGP SIGNATURE----- --Yylu36WmvOXNoKYn--