Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
From:   Chancel Liu <chancel.liu@nxp.com>
To:     lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz,
        tiwai@suse.com, linux-sound@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     Chancel Liu <chancel.liu@nxp.com>
Subject: [PATCH] ASoC: soc-pcm.c: Complete the active count for components without DAIs
Date:   Mon,  4 Dec 2023 19:15:32 +0800
Message-Id: <20231204111532.3165-1-chancel.liu@nxp.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?FcDpz8NkdOEmb1CWBSBGU7YmXgCo2Y3vyGu9jVGrO3qVoPnMP/fU2c0j3rj8?=
 =?us-ascii?Q?KrLzY3bLI8ADLrZNmDpWJBzrBOQTVuGriqdqQa8zbtO+/r3Sa2RM87jp4q5I?=
 =?us-ascii?Q?10KpOCK8dDeGaLUIO14rUlX9mHXpdh9XVvCMNTVBZ1mivV2poNPWJuZhUmSP?=
 =?us-ascii?Q?XZOZw6QF7gjii0cU4HOmjzJ6RxWyRzL0jqdLnfsqM+le9NICaaygMitpQz+6?=
 =?us-ascii?Q?bW3FAK45cG041CW3ZjZ+BGFZpEZKs0Dlw4aF+Gzb2LciZkhWgF0z6xLlhOCe?=
 =?us-ascii?Q?ArIb+61naB57TlKYjJ4KKZ6f0sf5ZWwsX0j7h8IR86CILmWQ4J3bbV8LEF/+?=
 =?us-ascii?Q?YS+Y9iRpLULPUR0WWodDIneU+Nacmx//ymZGQ1YKmO2Zzahcc6plHAqqH16t?=
 =?us-ascii?Q?v2BwZ0HAmxPapOGKvQL2yjMB9WBirX0ygDeGy9PaCdpN5beOVygFk93bDcQP?=
 =?us-ascii?Q?ZCtFMQhoHrMqGDGWaBfA3Ui86rJ4tLpzgLavzeqBub8+aFac6kZX7kZ5SXWd?=
 =?us-ascii?Q?ifE+/bVYSUR7N6Jn7M1HINWw+tiCrPmSFyFmlJWC9tPAdhdMgz1xhZMm162O?=
 =?us-ascii?Q?ANPdCed9qG5jIjMAF+q9jO83edxY8vuTWps7/6qvbWq5fPYeJ/MEf6iwEGlm?=
 =?us-ascii?Q?CQsp+DKjVLpIFNEPgfM1omUIPaX7DXZfvXaMah+l8nWCS0YwjMFuWE5BQnWH?=
 =?us-ascii?Q?P59mpFZwz8+jyGRGRxsMv3Z3LPA32Ky9pQmGgMffuQTnkDNPZ0qsQW87YoFC?=
 =?us-ascii?Q?3VALW7l/jlUg0Lm7na+H4SHBMcWmnuKudNFiKbs4g9JyVkgQC9v+q3dk4+XB?=
 =?us-ascii?Q?LJaIwHEb9DNdyKjVDV5QAm/g+tfV7YYEJjtTaiJrSaW92xVgf2PP3+InIyH+?=
 =?us-ascii?Q?XvTWXdYZHBoFZ9gKmS6pp+U8T8wAAo2kQSpL0RMOe8pp5cBkR8TLSJBvlBYw?=
 =?us-ascii?Q?93rhxBN6KgksTjXGlTk/b7dYYAZlfLg9jEG9QEzaH6Wur/QBbjbGcuHbp0k9?=
 =?us-ascii?Q?mUEl/VnyuzFtpuwMx7NMIl4iPgm2QyXFWH/LZl1WiTSSZc0sIqMv8AoOOS8L?=
 =?us-ascii?Q?XbhgNeVACYdUuMVxhpBbjFeIPgs46LYaa+n8L5SHu59BpJEGrlF+vj76thoo?=
 =?us-ascii?Q?x8WimvfM05ynugrbUyfskX2Y4K9TTZVF1TGlLCR/zWcubJtUaEjl3mBL4rhq?=
 =?us-ascii?Q?B+Q6gdlXrBY2BQW822CJyM2Nh9d44aFNgeJauJhztlC8AxJZCu8Mb/8FR5Dk?=
 =?us-ascii?Q?aZefL4LITqeUqOj8qTkQufaHcXsuZQDh4UGBcx/HjHKQX2947cW6BNcg2/xZ?=
 =?us-ascii?Q?AwOurXlh90WmXUidTCQ059+gTxGYa3FzEgWa6H+emegi+Ynbh8Dq7pDBfHm+?=
 =?us-ascii?Q?ZxxuEVLL6xPhg/X8N25Xqq9nkGSnrfDh9g5ojsmUZgJh0o+/iyH/2HFZbpI2?=
 =?us-ascii?Q?yhGpz3IIJyeojurVG9TQhXr4VNeEd9kUkv7MKTQz6B53eOTmHzPIBBpF8urd?=
 =?us-ascii?Q?jfLtoTTEsC6rBMDXChpnrpeZGAgP4rb4pJVoCXvDVww2t55pHw1j+x4iLO+j?=
 =?us-ascii?Q?LStPWrSWmw5BV2AZ3Ai1Sq8N8xvJJOIC4uc2uI5b?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4851a649-db44-4b1c-14d4-08dbf4bd6f0d
X-MS-Exchange-CrossTenant-AuthSource: DB9PR04MB9498.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Dec 2023 11:37:45.6722
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: oq6mKeRDkjHhR32SNRfVfcM+PeKF4Jo6BmzI8/wtV4njuOCu2r8sU7H6ru8o1PuMTHanosvy56H/ZF1Qm2bQeA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB8089
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Mon, 04 Dec 2023 03:38:01 -0800 (PST)

Some components like platforms don't have DAIs. If the active count of
these components is ignored pinctrl may be wrongly selected between
default and sleep state. So need to increment or decrement the active
count for components without DAIs to avoid it.

Signed-off-by: Chancel Liu <chancel.liu@nxp.com>
---
 sound/soc/soc-pcm.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c
index efde4203bbf7..7dd4d41f680a 100644
--- a/sound/soc/soc-pcm.c
+++ b/sound/soc/soc-pcm.c
@@ -292,6 +292,7 @@ static void dpcm_set_be_update_state(struct snd_soc_pcm_runtime *be,
 void snd_soc_runtime_action(struct snd_soc_pcm_runtime *rtd,
 			    int stream, int action)
 {
+	struct snd_soc_component *component;
 	struct snd_soc_dai *dai;
 	int i;
 
@@ -299,6 +300,13 @@ void snd_soc_runtime_action(struct snd_soc_pcm_runtime *rtd,
 
 	for_each_rtd_dais(rtd, i, dai)
 		snd_soc_dai_action(dai, stream, action);
+
+	/* Increments/Decrements the active count for components without DAIs */
+	for_each_rtd_components(rtd, i, component) {
+		if (component->num_dai)
+			continue;
+		component->active += action;
+	}
 }
 EXPORT_SYMBOL_GPL(snd_soc_runtime_action);
 
-- 
2.42.0