From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) (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 B178E3EDACB for ; Thu, 23 Apr 2026 12:52:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776948730; cv=none; b=n/+Ss7JMisRfJFDUwOe1WNoKBuU06FTQ2CsxVfA6LWp9PudCD2r6/iPuXuyevhr96//TrYZ1IXJGgCow3chEL+uY8hNYfoZTcFBEdcfZIDh8wvevnj3hd6admSlF7HMMX8D4w0LW9i6xzhpo81QEyINC8KyI/5UR75FwSfKh4Jk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776948730; c=relaxed/simple; bh=Jg5SR7o1VcWMJNB5wdQqpvyhrEnCSnyB6mn3nEk4jeM=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=JjnXp/7yybNH3MpeIaGItlN4ZBlGL2/HrcSXmkyUiIYMxqWuAJYNVxVfPJ6RjPKteIfMd5DaUhep3EE3YuaehjIJ3Bk1GptSTN2dfZug9X9fRSMHH1bjaCp0YbQkRQVVPR8/ZNzMPXHxXqHwq3RggRA0KxAMYpPmQd9NU7U1hA0= 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=V9WO+nV2; arc=none smtp.client-ip=209.85.214.176 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="V9WO+nV2" Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-2a871daa98fso43668945ad.1 for ; Thu, 23 Apr 2026 05:52:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1776948729; x=1777553529; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=kvrsh829buHJ1xDv00U40C6X+HtmBnUZPS87U4o6y9M=; b=V9WO+nV243wyzVLxMrD5RsD0T5GSU8TxGAmuZE6+K5NrKvT/x3JkGzkXhAd9u82L+F J8DrVrtBw+0RqoDk3ezx9Qx51ZpjlruYSM3iGZjEFOrb+ZyWEbyCivXiQ2EOYxOgFw79 YYsfiE2yjKXoHUq+OjEZ8CUKh2UEgHyA3ZGhztuWAXd8L6vgt13MG2lJinbC9T8udvRt t+mDvO7GvDw08CbgHMJ56Po8ulg1pvGZjoab7/SwOui8HSDExlkPqTZikKtGzKkEODSA VmjsO7Kzou6QPaWyXB6ZkA0GF3lQC+yWjSQbrX6Y2lkyGeR1YMGjDdgzCh6rXUeHK4ir 2HBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776948729; x=1777553529; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=kvrsh829buHJ1xDv00U40C6X+HtmBnUZPS87U4o6y9M=; b=Wx+K36SLImetpSZ078vtT1j9hQREaYhiX4tbUmCKfxpV8hQh3IDJY0CgBpRwFO/lfH a1Uvwu1sTk0gKo0rIokpkBKPUQCeeDEQhma15tDD89taHmzpRARBb6ReniAt/sXbzNRi kbokAjxpDiAkJIO0IXCxxhOHWUYPOfrL+CYxUxqWDEeugcqq3CXtf3FUzwYYT3gDq/Pi gJZ8cCm0bmGtDwMScCGFR92Td6Opm5o2zle6P/4hL1J3iQ5tulAqZAJkzy+SGwPCOiMn R6iiIda1J2ZJPmb0sNB13RgtEMFRuyFHS4LSZij1kmLIU8tNkLbTqRpZ9ajNCDrzOlrn +AWA== X-Forwarded-Encrypted: i=1; AFNElJ/7EQkduX8xcKGjKKsQJxL2YRSo1xawACdP1jeuAXciNdAORUUVB3xfRmroJiXiPvLJRcN9ESeMHACZeX4=@vger.kernel.org X-Gm-Message-State: AOJu0YwoKTeTlbecWWt3zz2UOoIhs3QcXlavfQOJnzLHNFtXWhV5NWlS n1i5o9jtRr0Nmz0+UMPGv8w1nhuFJfRfbtKiscuAkSpE3K2pTKupNWvV X-Gm-Gg: AeBDievZmqixsvDda73zmC1pWkqqCpS6gD4cw/5JlcqQHlEii867kBQd9fgwzYZX/R5 Fr0he/BwmW6zVfMLiNLRAQPknHJwTNZQ+sH2wJUA0slF1PS3tnUIjgfvXI+MaAk/mEjE0VIpG7p RelPArBIaY8AX81s1XauV9TNVcpGnW1H76gIM2aLVSWyOdoFmrCFKhF+RzWvwVW2ksx8hXLm09k wqIEwfu3c0W+XuzSHymhOEX+6m7GKFV0rgByoJgvUQBMko0I+mbMPLawxtjc4mlwHHOBvd41z+C EpFYyeczKPdtFtSRSidsM3P84dOpTEiY4X0C+A9A2eW61Wy9viRomP63dFoHrDxiyTHBF7bTN16 skXUNHTCJK6lLnl9NCN0Bn+emQSG3l8bOftMgL4P/cjkUuNFMkpPe7VBXlPi5uypz+T/ae94xeo GEolg9+gE2e6zx8zdHkQIwAI1oOdqp0jcXR8TlG8WMsAxzKw65m/D1q36I0NUCN8PsDAm4 X-Received: by 2002:a17:903:2986:b0:2b2:4c58:5ba7 with SMTP id d9443c01a7336-2b5f9f43679mr290188045ad.30.1776948728813; Thu, 23 Apr 2026 05:52:08 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b5fab3c067sm193432875ad.76.2026.04.23.05.52.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 23 Apr 2026 05:52:08 -0700 (PDT) From: phucduc.bui@gmail.com To: Mark Brown Cc: =?UTF-8?q?Martin=20Povi=C5=A1er?= , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , asahi@lists.linux.dev, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, bui duc phuc Subject: [PATCH] ASoC: apple: mca: Use guard() for mutex locks Date: Thu, 23 Apr 2026 19:51:54 +0700 Message-ID: <20260423125154.156257-1-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 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 Clean up the code using guard() for mutex locks. Merely code refactoring, and no behavior change. Signed-off-by: bui duc phuc --- sound/soc/apple/mca.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/sound/soc/apple/mca.c b/sound/soc/apple/mca.c index 39269cc7def6..492165c0e1ea 100644 --- a/sound/soc/apple/mca.c +++ b/sound/soc/apple/mca.c @@ -307,7 +307,7 @@ static bool mca_fe_clocks_in_use(struct mca_cluster *cl) struct mca_cluster *be_cl; int stream, i; - mutex_lock(&mca->port_mutex); + guard(mutex)(&mca->port_mutex); for (i = 0; i < mca->nclusters; i++) { be_cl = &mca->clusters[i]; @@ -316,12 +316,10 @@ static bool mca_fe_clocks_in_use(struct mca_cluster *cl) for_each_pcm_streams(stream) { if (be_cl->clocks_in_use[stream]) { - mutex_unlock(&mca->port_mutex); return true; } } } - mutex_unlock(&mca->port_mutex); return false; } @@ -765,9 +763,8 @@ static int mca_be_startup(struct snd_pcm_substream *substream, cl->base + REG_PORT_CLOCK_SEL); writel_relaxed(PORT_DATA_SEL_TXA(fe_cl->no), cl->base + REG_PORT_DATA_SEL); - mutex_lock(&mca->port_mutex); - cl->port_driver = fe_cl->no; - mutex_unlock(&mca->port_mutex); + scoped_guard(mutex, &mca->port_mutex) + cl->port_driver = fe_cl->no; cl->port_started[substream->stream] = true; return 0; @@ -788,9 +785,8 @@ static void mca_be_shutdown(struct snd_pcm_substream *substream, */ writel_relaxed(0, cl->base + REG_PORT_ENABLES); writel_relaxed(0, cl->base + REG_PORT_DATA_SEL); - mutex_lock(&mca->port_mutex); - cl->port_driver = -1; - mutex_unlock(&mca->port_mutex); + scoped_guard(mutex, &mca->port_mutex) + cl->port_driver = -1; } } -- 2.43.0