Received: by 2002:ac0:e350:0:0:0:0:0 with SMTP id g16csp324358imn; Thu, 4 Aug 2022 07:23:47 -0700 (PDT) X-Google-Smtp-Source: AA6agR6MsQoM/alXSaX5Lge8Nft1rqaswT5dNRY6kkun9fJxPKV7jghdu9mnPvgQ3SinMf5pCZZM X-Received: by 2002:a17:90b:4a86:b0:1f3:b701:87d6 with SMTP id lp6-20020a17090b4a8600b001f3b70187d6mr2353492pjb.213.1659623026634; Thu, 04 Aug 2022 07:23:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1659623026; cv=none; d=google.com; s=arc-20160816; b=UrsU1Sl5mAOzr9MonQLRBIPrwgC8CH39HYmubl2Sfp+3MLVg9uHT83RIz27qhwhwmx pkdLCVXCmgcRLNFWg+aFq/4vWt72bwhHi224h62tLKkqQeNrHwWjvwI8nrLm55H7aDve u9M4JdUKL9L2iEK18RZqTZECFFe1U0DMLIYTwVJkgp113wJOTNc/bZOK/TaR+16jJ9WW Xhwo5RY2fKlaUoAiZONlK1SNUjuE7srZ+jvJuu+Lv6hDthNJsQskzpQnkN86Zm1vbz9U ROJ5MindCyCUnFKf9em22cIeotgdzHeisDz0VeF6zYcbXQs5elSQ9iXH/VjGwFBUbDRj R2TQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=0TMTthrybuHSw38BcPdvVtrHCMWIDAeNmf9go1g+GtA=; b=1G4WtMRw5E0AxXCv5nt/6z0k0NNPrXwM85Bb3js35jkPWMKhkFaRR3h1UKdEWzxHvM v4j8jsSmmgdCItZxaV5ja9uIfgIXxrjVwPO9GuV/1/z7Gq7ovOKcBrZBebrkHNvT9I7Y /mdGmzEJw0kjR+XlhyYb4IRpE3A0p/EiwNCPTTy3i6ReFBCZEBnbLsDB1H2Bx8IA1tPC aMep8NaSnFnWbtSgf28JwTz4qvssIvrqky6fM8zqg3SWDEbIDJ2esPJ+WbouHYZCYCqq YjW9Gw/KYN6t/Xi64Um91iL3Up5fPtDNiZ5fEj/mxxvA3JCX42ztn+Ld2XGp4W4wDwzz yczQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=m6stWst9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f2-20020a637542000000b0041cd02e2de4si1628706pgn.421.2022.08.04.07.23.32; Thu, 04 Aug 2022 07:23:46 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=m6stWst9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239637AbiHDNRe (ORCPT + 99 others); Thu, 4 Aug 2022 09:17:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53698 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231807AbiHDNRc (ORCPT ); Thu, 4 Aug 2022 09:17:32 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2D6BD1DA4A for ; Thu, 4 Aug 2022 06:17:31 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 72104B82529 for ; Thu, 4 Aug 2022 13:17:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 214E2C433C1; Thu, 4 Aug 2022 13:17:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1659619048; bh=pGAubC3p5cu8E3aqPdqw/fxyUjG67zf8Tpf1JKEF55w=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=m6stWst9e41BreGAtGPuz0nAu0TXJwPKwlYdFFr82zphG4M14QXdbrLumphztxMuJ YDRDikw+/QifdRAghI887qlMaly1LeHw8FJsuERxDrwM3ejq344jv/6P8SI4qYYdJ6 5Of9c4WpCS92Rl1VVFiCsroAA0eleXumCjmYnyo8IFRFv4y/qUerR/K5kbtnJ5LAIg UDbeeAdrYpjf3kwUPHCL5Z0qhffTosxBEGEg81cPe0P6vOxWBysZgF7P1CoL7ba0tt Us8sPo+r1mwRof5YQzLUBmLtUbQcZ4hxS5HMbkd31WBWTjRPaXzWomF7kb8cwcDSmD P4nOg8r8+XqwQ== Date: Thu, 4 Aug 2022 14:17:21 +0100 From: Mark Brown To: Chunxu Li Cc: pierre-louis.bossart@linux.intel.com, peter.ujfalusi@linux.intel.com, lgirdwood@gmail.com, angelogioacchino.delregno@collabora.com, daniel.baluta@nxp.com, matthias.bgg@gmail.com, yc.hung@mediatek.com, tinghan.shen@mediatek.com, linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org, sound-open-firmware@alsa-project.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, project_global_chrome_upstream_group@mediatek.com Subject: Re: [PATCH 1/2] ASoC: SOF: Introduce optional callback of_machine_select Message-ID: References: <20220804091359.31449-1-chunxu.li@mediatek.com> <20220804091359.31449-2-chunxu.li@mediatek.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="3rmzYVoKrLLLmuXO" Content-Disposition: inline In-Reply-To: <20220804091359.31449-2-chunxu.li@mediatek.com> X-Cookie: Did I say 2? I lied. X-Spam-Status: No, score=-7.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --3rmzYVoKrLLLmuXO Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Aug 04, 2022 at 05:13:58PM +0800, Chunxu Li wrote: > @@ -284,6 +284,7 @@ struct snd_sof_dsp_ops { > void (*machine_unregister)(struct snd_sof_dev *sdev, > void *pdata); /* optional */ > struct snd_soc_acpi_mach * (*machine_select)(struct snd_sof_dev *sdev); /* > optional */ > + struct snd_sof_of_mach * (*of_machine_select)(struct snd_sof_dev *sdev); I don't understand why we pass this in as a function when as far as I can see it should always be the standard operation provided by the core - why not just always call the function? We can tell at runtime if the system is using DT so there's no issue there and there shouldn't be any concerns with ACPI or other firmware interfaces. --3rmzYVoKrLLLmuXO Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmLrxuEACgkQJNaLcl1U h9C1FQf/QjsEtLV9fR8Iz6XmIP9rAebV+D6AS3pDxlY44Q3n5kTg2SprGpkW0N3X PZVh1GJ4B4MWkO2ODQKF2X322Hr78Np7AzYSQQv0QrcRbRj7/IJHnn4zCs26DlaB IOk/MGuxUw0kHzBERUoreV9XF2MjBJXTBURBc198Uw1xcb/WarVtk6OPRfmn8blp 5DQZ640pif50r6aQaRaOq9/c7RnUy+hAjsT78GNnIsbEinE1w5JOb2Xf9BTQAXZ5 slxV69bTH2+yDkqU/G92cqXI8RhuD6Sz6KP6gKnAF8n1DNUVTk+pAZeHrRIJ2cQT pw8nMfmjoexKvLFc+FTsGrdNuJ2zTA== =PLcm -----END PGP SIGNATURE----- --3rmzYVoKrLLLmuXO--