From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) (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 22B21477E38 for ; Wed, 3 Jun 2026 11:50:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780487445; cv=none; b=FOxer1ZBmP50STqu2DK3AwAQf5ELSfLxnpUXaxq2kcca+dzVj05fWYV+atmOZbqaQKdcgHtFZHWq0ts4rHeyYFtWDMhzb3Dv+VsFSc+gzdj81hgPNi3AEPpN8MxWwZE09mbwG9fbaoMWcb/Ng9gGYOyz1a1yQ53d2QwiF6hwaqI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780487445; c=relaxed/simple; bh=mW4eQXqnfxaB0c/JC7r7EXwbppQQR/dLCcxN+G/JGWI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=riLFte4OaUdQY6OiOYKPpkfFn8lwhhpIqk8MM9m427yyUIFnWtZo9OjK4/A8mrESZgOyp3xvuI+W34e8xAnLE7p2iVGT2bpnLH1bE43MqUW+3aGU1Z2GxVns7RM8Uc9PC9qgIcfKy0/M0DBhuR76NrDfoFUrhQZ/D5Pq0Aqt7Yw= 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=dOeD0Csh; arc=none smtp.client-ip=209.85.214.181 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="dOeD0Csh" Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-2c0a5354da1so4613335ad.0 for ; Wed, 03 Jun 2026 04:50:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780487442; x=1781092242; 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=v/cb7/Oh3wrVW6lTEgQSTGPGYUEEQIjXPznLNOjAVMo=; b=dOeD0Csh2J9hCpo1AaEEl1z+VNNAE/NsFnLdeIyzE78YecNEmFb8cAdZFGAhzJOLIe pwy93amfXkQO1ilD+mUH4EZfev+CYpfcU9ghQ1RFgFH2iDO6ilBUWt7ypt6qtFH8CG56 +DL1sFCze48VV8uv1hyGALHt4A5hXzGRKg79Ho2Ysv5/FWB0cBiiob4baZOAlJZhMb4K I7KeofRucMuV/HIGtak77/KJ7OG5fKn1nmjMHY0/G7ofwiSKU3OY3NtCSjd+srx/jTLo dH4Cc8LccLrAxSphyy8bALrk7vgoruAmJVppb8m0jtBle1UsCGoXFVN6O1+zzw752QdA fZnQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780487442; x=1781092242; 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=v/cb7/Oh3wrVW6lTEgQSTGPGYUEEQIjXPznLNOjAVMo=; b=WEbFlgX8JOlwDSW8yW1RkPebYgzoYS5inK1jYcsH05u0aKuPJ5QIQtEsiydhwHgmZF a2YTJnmj2lHQ4smSlM9WT4YV2fYJ05l+h5gCYleVmVP3wlXUxJSfWhv9Q/qd2SeypAj4 ercl9G5Hu6YEeU5Mp30lQFz2rboauopZgDpiMVIooE2uPs2AjfdjYAibcD1Dt0d0pJdh l9gsiv7VpelJSsqFIvab5e+up9oLSxNXkqTxrfEU33ZIcqev92uSra/snznM8AWVbYHl ziak7Jsr0wEo3ykCeSDCF8QblNf2pprX6JQFVzCXbRlpB3cyDbqqGUBWEKB3k+saSdBD 53gA== X-Forwarded-Encrypted: i=1; AFNElJ/sYtdiWRZ9C7O80hTz8CODClVfTaAZ2My3DloFb1+FamLwke75M7msmA32TE5Al0PsUrfvi7hTRnpkdOs=@vger.kernel.org X-Gm-Message-State: AOJu0YzbIoU8vr+ImKoXygnjhrzsPhdgTSeC8ZTjL8IEbAc3X9Xn/ShS 2n0bpiBlPWN6CyUL+O7f/t0+7JDh7CCmA5+VNBWxVPoIv1y/VVpQCHwj X-Gm-Gg: Acq92OHbj0Ru3D7W+L+tN2OfeKURXFakggjfvIwuXdM308AJdr3deZ3gn7v3zfFhBsm t/j/rcppWc2xtBLLu51IoqPSbhNVYki0JYE5l7A5/Ld5N/3x9+AoHsWd8jZ1OgN84BE67CgbBpu cWhAiTg7/XDWjxbwldWV/nngx7xULNQ1qiKg2O3cC2dNjaBNV7S/vKgXzng30qpvokAc21U0qra qxXxwbGtXzouqHeoxQK946dqeg7BHoSXPkgfGKLzR/d1t0d2fFW4/jDOLXIV4BoKvw6lO+5KeSG d8BViY08Q6yKxkp+UJYZVE1Mx3pS9Hwpmzb3MXByUj7iDHq+CuNJ88W++tR8DdkfRzcR+2nYVk4 jInGfDyCq7R+cRDOZRMNRTZN18YiKCxSB/AlK7/JWmO0XU+BKlCMnBia7CGfxo/hPl4T/LxjEna YPYVzQwdv4O8dhp/ywStY3GA3eyRlbfDM1Enk6w3vgThzrYbmDRnJWqeYQduc0Eq7UquA6 X-Received: by 2002:a17:902:bcc3:b0:2c0:bd40:17e5 with SMTP id d9443c01a7336-2c164b2c190mr16899325ad.19.1780487442382; Wed, 03 Jun 2026 04:50:42 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2c16609df63sm22731455ad.40.2026.06.03.04.50.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Jun 2026 04:50:41 -0700 (PDT) From: phucduc.bui@gmail.com To: Srinivas Kandagatla , Mark Brown Cc: Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Krzysztof Kozlowski , linux-sound@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, bui duc phuc Subject: [PATCH 8/9] ASoC: qdsp6: q6usb: Use guard() for mutex locks Date: Wed, 3 Jun 2026 18:49:48 +0700 Message-ID: <20260603114949.149595-9-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260603114949.149595-1-phucduc.bui@gmail.com> References: <20260603114949.149595-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 Clean up the code using guard() for mutex locks. Merely code refactoring, and no behavior change. Signed-off-by: bui duc phuc --- NOTE: This patch is compile-tested only. sound/soc/qcom/qdsp6/q6usb.c | 28 ++++++++++------------------ 1 file changed, 10 insertions(+), 18 deletions(-) diff --git a/sound/soc/qcom/qdsp6/q6usb.c b/sound/soc/qcom/qdsp6/q6usb.c index 6381b289c55c..ee6de9635342 100644 --- a/sound/soc/qcom/qdsp6/q6usb.c +++ b/sound/soc/qcom/qdsp6/q6usb.c @@ -61,31 +61,28 @@ static int q6usb_hw_params(struct snd_pcm_substream *substream, struct snd_soc_usb_device *sdev; int ret = -EINVAL; - mutex_lock(&data->mutex); + guard(mutex)(&data->mutex); /* No active chip index */ if (list_empty(&data->devices)) - goto out; + return ret; sdev = list_last_entry(&data->devices, struct snd_soc_usb_device, list); ret = snd_soc_usb_find_supported_format(sdev->chip_idx, params, direction); if (ret < 0) - goto out; + return ret; q6usb_afe = q6afe_port_get_from_id(cpu_dai->dev, USB_RX); if (IS_ERR(q6usb_afe)) { ret = PTR_ERR(q6usb_afe); - goto out; + return ret; } /* Notify audio DSP about the devices being offloaded */ ret = afe_port_send_usb_dev_param(q6usb_afe, sdev->card_idx, sdev->ppcm_idx[sdev->num_playback - 1]); -out: - mutex_unlock(&data->mutex); - return ret; } @@ -203,15 +200,14 @@ static int q6usb_update_offload_route(struct snd_soc_component *component, int c { struct q6usb_port_data *data = dev_get_drvdata(component->dev); struct snd_soc_usb_device *sdev; - int ret = 0; int idx = -1; - mutex_lock(&data->mutex); + guard(mutex)(&data->mutex); if (list_empty(&data->devices) || direction == SNDRV_PCM_STREAM_CAPTURE) { - ret = -ENODEV; - goto out; + route[0] = idx; + return -ENODEV; } sdev = list_last_entry(&data->devices, struct snd_soc_usb_device, list); @@ -227,11 +223,9 @@ static int q6usb_update_offload_route(struct snd_soc_component *component, int c q6usb_get_pcm_id(component); } -out: route[0] = idx; - mutex_unlock(&data->mutex); - return ret; + return 0; } static int q6usb_alsa_connection_cb(struct snd_soc_usb *usb, @@ -244,7 +238,7 @@ static int q6usb_alsa_connection_cb(struct snd_soc_usb *usb, data = dev_get_drvdata(usb->component->dev); - mutex_lock(&data->mutex); + guard(mutex)(&data->mutex); if (connected) { if (data->hs_jack) snd_jack_report(data->hs_jack->jack, SND_JACK_USB); @@ -257,7 +251,6 @@ static int q6usb_alsa_connection_cb(struct snd_soc_usb *usb, if (data->hs_jack) snd_jack_report(data->hs_jack->jack, 0); } - mutex_unlock(&data->mutex); return 0; } @@ -284,12 +277,11 @@ static int q6usb_component_set_jack(struct snd_soc_component *component, { struct q6usb_port_data *data = dev_get_drvdata(component->dev); - mutex_lock(&data->mutex); + guard(mutex)(&data->mutex); if (jack) q6usb_component_enable_jack(data, jack); else q6usb_component_disable_jack(data); - mutex_unlock(&data->mutex); return 0; } -- 2.43.0