From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f53.google.com (mail-pj1-f53.google.com [209.85.216.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E70A13D1703 for ; Fri, 8 May 2026 10:39:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778236749; cv=none; b=t/qajrIE3Nf9QtRctcW6+LDBtIx0YRStopyG5pVQvCRXopaG6OXfwYVwdYWaeOcvTCf8//KruCn+IPGlO/7VQY2EcBldQtEViD1JkKGno7sSyEJniEFsudY80UX/yFAjwr9Dq9Vsl8MfHP714wMmI6gX5XdQTvXTxOBYgM5RjQ8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778236749; c=relaxed/simple; bh=+Cat/hrzlP9TQa7q7VLyIEo3Rx45NIuQaGGmUTbrMPU=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=LhPbUHJYmMtF75OjiA7rfw1ZwRTcyIuyHjdocjhfaMGWk/QDUE9JQtO3I9A7UAarJpvVoOGgjwPJAiKiq0dS+7nXf25AVciukfZ3d/se6ILtKanSvpsZ+46nObw14XWNgaEJv4DBzEH8fmR1LrI42i9YGMSlAhy/ZcrDHdHbYwA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=IFtEicd6; arc=none smtp.client-ip=209.85.216.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="IFtEicd6" Received: by mail-pj1-f53.google.com with SMTP id 98e67ed59e1d1-36643b96b99so736115a91.0 for ; Fri, 08 May 2026 03:39:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778236746; x=1778841546; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=N1fF2g2fGUgug/smobxh1GYiC42PEqoDgJD4+eh4r10=; b=IFtEicd64JDkJNoYlJd6jHifz3oo6kVZqVS16EvYhL5af1t8Dmv4e32Gz8m/+I9Ewu eJTsJ9fEdBVqTPppqgUKQFM+l7yHk+2OQZwn7GpNPQLX3dKODCpNLQnPjgReVOtHZ9W9 sGKfUS7eInuPJMuNVijeb3XDfksgc3tGDTk/V0sCa2Dli8+Crl/hrvrw5PU6fL5rM4ti kZyRKBjd7DTYgxM0ndfgTy+WIi2pXSQMgVdQf3gjhYWHcJWmJgMqutA3onmfptZRFLd1 mNKDnnY7z7Vxvt1IbML51QNazfME5GAh3qfk2TrH1z7eSEQ9IyFayJXWV0WV+54s9DnF OSPg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778236746; x=1778841546; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=N1fF2g2fGUgug/smobxh1GYiC42PEqoDgJD4+eh4r10=; b=WkVydeGBXUTgtqAiyvptewVozFaxnnn3TVcXq6Sy3LO+XVhdwgg9Uyzjns/mnB4FSh 46GvHEQS4dNqNrAB3BXJu8E5cxeSvtMzgNOBdR15qm6rob1triLLUfIV7bVkq2jchxL5 pzsVI7NJ5LHBcbYxnaQb7pPsR19vFcZp8BbSTYHxL5PulWmxyoahCSdBFBUlM1/+qYBi vuV+SS/ifmThjqgLHEw6WoTFYu9r0hevn68NTThehPlQEDl1TFZsp3nEsUPFVMydEwPL kZ81I7q9hpX23nYMU+weyaKzsbnoamd+p4AXQP8dtncE+ttdfDwC09pEoLBhMRuNXBs4 N3fA== X-Forwarded-Encrypted: i=1; AFNElJ+shUZURb2WK2ha21MDVzjqZTfzCeYRR+uV/OwGa8y+mnXk5RBpthHbII9s6pl5is393ohT57uMPfJD/4k=@vger.kernel.org X-Gm-Message-State: AOJu0Yy+ioioTvOOuJilXtXZuyI9MlUAnmCd7/rR/K5wtFSueOurLnHS kciZLEz2F7AdQi5ewvXxDnVKwicxjBNMNdnx014V2uezCtrji0t/Xg45 X-Gm-Gg: Acq92OEsI98hdhWhexYCCQv+S5qXKCxjRt9wAwGW4SZnnmqf6nlpVGtbUqn0bwAjQ00 giy1MR8K8816DaSJR1mu0u5nbesftPM7QM9Q+8e6M+xb2RjFK7mruNsD4LSiTnwFlDSflz3H4Qx AdwoQTV7c5GQCzi/StvDpZjvSM1aebpWCZO1Dw4ncuN8fb2P5xZ3XV/TQqcUbNrqoKDtO361ieW MyQ7g5GaYg0/RIg53PiupTILfeXMqENS4sFk+UBTo4qGmuWbJPs0pYpzVgN/4QfAISlumH54LQ2 ms/Xs4hBmNemUFUhG3L4Y+jwYBh5K4r6fCkC2a6+gQMQwF3kcAnxNSOXQ4wpiq0Ju+1xlx3HfOI uNa0ru5p1AnM0DK65c5FF4jwlI19Fnx/SJ5hDPtNK6HJslCM8UfbunbhqX9u7TW9PC2cTFa9sBe wYeAAbAIbp5uSHSceEtt3Kl/7RVA8PHe98+iKtIKHsT83vspSFcLAQVLu6Og== X-Received: by 2002:a17:903:1b50:b0:2b2:42da:25c4 with SMTP id d9443c01a7336-2ba794bf96emr121392205ad.14.1778236746201; Fri, 08 May 2026 03:39:06 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2baf1d27109sm18309345ad.1.2026.05.08.03.39.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 May 2026 03:39:05 -0700 (PDT) From: phucduc.bui@gmail.com To: peter.ujfalusi@gmail.com, broonie@kernel.org Cc: lgirdwood@gmail.com, perex@perex.cz, tiwai@suse.com, jarkko.nikula@bitmer.com, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, bui duc phuc Subject: [PATCH 2/7] ASoC: ti: omap-dmic: Use guard() for mutex locks Date: Fri, 8 May 2026 17:38:32 +0700 Message-ID: <20260508103837.138142-3-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260508103837.138142-1-phucduc.bui@gmail.com> References: <20260508103837.138142-1-phucduc.bui@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: bui duc phuc Replace open-coded mutex_lock()/mutex_unlock() pairs with guard(mutex)() and scoped_guard() helpers. This also simplifies the control flow by removing temporary return variables and unnecessary goto-based cleanup paths. No functional change intended. Signed-off-by: bui duc phuc --- sound/soc/ti/omap-dmic.c | 44 ++++++++++++++++------------------------ 1 file changed, 18 insertions(+), 26 deletions(-) diff --git a/sound/soc/ti/omap-dmic.c b/sound/soc/ti/omap-dmic.c index fb92bb88eb5c..dc92fdb89a0f 100644 --- a/sound/soc/ti/omap-dmic.c +++ b/sound/soc/ti/omap-dmic.c @@ -91,18 +91,14 @@ static int omap_dmic_dai_startup(struct snd_pcm_substream *substream, struct snd_soc_dai *dai) { struct omap_dmic *dmic = snd_soc_dai_get_drvdata(dai); - int ret = 0; - - mutex_lock(&dmic->mutex); - if (!snd_soc_dai_active(dai)) - dmic->active = 1; - else - ret = -EBUSY; + guard(mutex)(&dmic->mutex); - mutex_unlock(&dmic->mutex); + if (snd_soc_dai_active(dai)) + return -EBUSY; - return ret; + dmic->active = 1; + return 0; } static void omap_dmic_dai_shutdown(struct snd_pcm_substream *substream, @@ -110,14 +106,12 @@ static void omap_dmic_dai_shutdown(struct snd_pcm_substream *substream, { struct omap_dmic *dmic = snd_soc_dai_get_drvdata(dai); - mutex_lock(&dmic->mutex); + guard(mutex)(&dmic->mutex); cpu_latency_qos_remove_request(&dmic->pm_qos_req); if (!snd_soc_dai_active(dai)) dmic->active = 0; - - mutex_unlock(&dmic->mutex); } static int omap_dmic_select_divider(struct omap_dmic *dmic, int sample_rate) @@ -334,26 +328,24 @@ static int omap_dmic_select_fclk(struct omap_dmic *dmic, int clk_id, return -ENODEV; } - mutex_lock(&dmic->mutex); - if (dmic->active) { - /* disable clock while reparenting */ - pm_runtime_put_sync(dmic->dev); - ret = clk_set_parent(mux, parent_clk); - pm_runtime_get_sync(dmic->dev); - } else { - ret = clk_set_parent(mux, parent_clk); + scoped_guard(mutex, &dmic->mutex) { + if (dmic->active) { + /* disable clock while reparenting */ + pm_runtime_put_sync(dmic->dev); + ret = clk_set_parent(mux, parent_clk); + pm_runtime_get_sync(dmic->dev); + } else { + ret = clk_set_parent(mux, parent_clk); + } } - mutex_unlock(&dmic->mutex); if (ret < 0) { dev_err(dmic->dev, "re-parent failed\n"); - goto err_busy; + } else { + dmic->sysclk = clk_id; + dmic->fclk_freq = freq; } - dmic->sysclk = clk_id; - dmic->fclk_freq = freq; - -err_busy: clk_put(mux); clk_put(parent_clk); -- 2.43.0