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 5EC453E49F5 for ; Fri, 11 Sep 2026 02:35:58 +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=1789094159; cv=none; b=bbyvXtiLgCmwtCzKmz3vthR4PFLkx07SKqWaxMejLFt8ZbxNnsBzBMJoM4a9PzCydoQI6TmOSxucWAh3bXF7oTI9j2K8BLM3/88otjJ5S6xR8kg24M1ZL3unSPPOoXBN8osHGDKO3AOym35gBQadxgyJo5ZsWhkZ6lhcQaW/aQo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789094159; c=relaxed/simple; bh=aGG9GySrHo3aT729lj4kupW+yah0H41IU1upRHxXT6I=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=WSf38/y9WVV/taVNg51VEMTBgjj4Fn6EC60dTb19cpql4YgRG0votXKneotsiNVOxw2pKPnEaM+FujxKUPu3lTJozY6+mUsJuITvQHiQszK6TJw00PlgJxCipM/JvLIfJN5fyOj7nAPY+iBx28ETYAbElL6BA6Ph6s30nbDRpJ0= 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=BcaKzGCJ; 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="BcaKzGCJ" Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-2dd1dcdcf95so3797545ad.1 for ; Thu, 10 Sep 2026 19:35:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789094158; x=1789698958; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=MBqtsP/CWLSVd4X3g+IUHWQOXUwsfrAeb1YA314853Q=; b=BcaKzGCJJxBVJi2N6/ZlOEwsq8mWT/aE2uun3ZPGK9qgJLCXH7kCzD6Ud0Zs/SDH/K p8E1s4NOb2r7KbPc1lbTlKQ7p4SvMAU8NCbehgb7FYRQPaDIr0V/dg6PDKu8atr3Sn8l ZfG0uuMXkHjGVI6zKQe9KJzQGJu8z0MHVSl1uN3JWqBTZfJmuk3MRSd2ipDDG+xOOSE1 p2TDEB/W38zhN3VYPGVP8w5I99d1B4iWIHHMk8vEki/3KKABOoFxNiz/7JiiMuLlPV+t POcsPLQtbukQPg/KV+QGxCwDzUOP/dGp6h4Y6GIG+920prhCLDkxydRd02h2wUTN5BxJ tWww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789094158; x=1789698958; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=MBqtsP/CWLSVd4X3g+IUHWQOXUwsfrAeb1YA314853Q=; b=Q7eqBQ1KyB8wJlycFWWmuHVvEwKRtK12lMdw3Diw8i8Ls9uZMJj26FL1nCf6cSB9xT iynx5ETFEfolg/NZpwR/lAJcts/Awsn6s6DoT36iLPKFOF6D9glv3LxY+k/DJy0bePwC uGTNiTwllrHGbfiI9fXYShfLN+snOywVaRSM34Lq32FRyu0pF1CxTiCkvU/3CwMb/7w2 v5M0uIgy5XCUewVtAcw3PbdOpkEW7P+KoabjgR8BaoZw0uqKmwKyAZUx3xXSmGtUDd+b ofl28cry3AprzZDehU+h6Gv3gxui3YTixMkV1SZqP8Km2hShhejEgpYhcuPfka8ZCAwS VgsQ== X-Forwarded-Encrypted: i=1; AKwUvBzd3+u1/02woetAH0ID1TA3qw0KWdgGGyFwiI2WlWiEj11lsOT95r8UCqRDkrss4Zwoce8e3ndeLr6h@vger.kernel.org X-Gm-Message-State: AFuF++ld118uYGMunfQ20FYmMkZpgYwEnjBrD1DioDTNOTxD65D1Icg7 Ro+QdjzfflfYWOfy4A/JWJc0F0Qcu7XQqELFYyrX5CMgnTI2HunujcjU X-Gm-Gg: AYBFou3xlvK68yBlKwHVywomQttcFKDGHM8JKi2U7QmiPz4CGJxgAWP5CIt0aWSdWZ4 hhV7nh4IZqD4P1VcHOrS8bLIcV48+KBDS15QYhv/kOt9wYrHkyY6DORNfe6N3bi9wJaJXh3V8yy jDjeRPDaOsT45gmvTM+NzX96q/PpolHlUIJJeSMEgBl6giLVK6zvCNe3aHQwsbdY0Wk88836cNv sGz+K/Ztq7x9VUkzFWwwfdH6FBsh4c54f1A+NSX+JZnZMcvnEAekNu174YkPSXbxicM7WDmYEXU EINudg2ULFr/Vw5kEclYnI0+smV2VqOLXFwkAu6NhObI6hSH2bEiqMcKsDwW5Ml8e7qo9D5uJu8 cdlEgWXWq+bTPPtfiQYBdvhLLbGP6ns/n59CS5d6Vsn47e8HSXPx+Km41ro02nQZeoQn1fFJi/k nZwxGIDT9NtDtTkZbiUDElFFddlzsV/O17tlA0j7lO725J4l9MQLO4cUE7/HJb622aZA9LqP/RQ kPeei/0wIVCcdaaRKGbbxXIIr5VpWly5GA= X-Received: by 2002:a17:903:388c:b0:2d9:14e8:bcb7 with SMTP id d9443c01a7336-2dd0e8a213dmr85013585ad.3.1789094157678; Thu, 10 Sep 2026 19:35:57 -0700 (PDT) Received: from [172.20.10.2] (114-137-148-9.emome-ip.hinet.net. [114.137.148.9]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-cc4c6550bbasm397520a12.17.2026.09.10.19.35.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 10 Sep 2026 19:35:56 -0700 (PDT) Message-ID: Date: Fri, 11 Sep 2026 10:33:39 +0800 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v10 2/2] ASoC: codecs: nau8360: Add support for NAU83G60 amplifier To: Dan Carpenter , Neo Chang Cc: broonie@kernel.org, lgirdwood@gmail.com, perex@perex.cz, robh@kernel.org, krzk+dt@kernel.org, linux-sound@vger.kernel.org, devicetree@vger.kernel.org, alsa-devel@alsa-project.org, kchsu0@nuvoton.com, sjlin0@nuvoton.com References: <20260825021644.1203076-1-YLCHANG2@nuvoton.com> <20260825021644.1203076-3-YLCHANG2@nuvoton.com> Content-Language: en-US From: YLCHANG2 In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 9/10/26 22:33, Dan Carpenter wrote: > On Tue, Aug 25, 2026 at 10:16:44AM +0800, Neo Chang wrote: >> +static int nau8360_dac_mux_put_enum(struct snd_kcontrol *kcontrol, >> + struct snd_ctl_elem_value *ucontrol) >> +{ >> + struct snd_soc_dapm_context *dapm = snd_soc_dapm_kcontrol_to_dapm(kcontrol); >> + struct snd_soc_component *component = snd_soc_dapm_to_component(dapm); >> + struct nau8360 *nau8360 = snd_soc_component_get_drvdata(component); >> + struct soc_enum *e = (struct soc_enum *)kcontrol->private_value; >> + unsigned int *item = ucontrol->value.enumerated.item; >> + int ret = 0; >> + >> + if (snd_soc_dapm_get_bias_level(dapm) > SND_SOC_BIAS_STANDBY) { >> + dev_warn(nau8360->dev, "changing path is not allowed during playback"); >> + return ret; > This is another case where I would prefer return 0 or return -EINVAL. > >> + } >> + >> + mutex_lock(&nau8360->lock); >> + >> + ret = snd_soc_dapm_put_enum_double(kcontrol, ucontrol); > Not related to your patch but the comments to snd_soc_dapm_put_enum_double() > say "Returns 0 for success." when really it returns true/false on success > where true means it changed something and false means nothing changed. > > regards, > dan carpenter Hi Dan, Got it. I will change return ret; to return -EBUSY; for the playback check condition in the next patch. Also, thanks for the heads-up regarding snd_soc_dapm_put_enum_double(). I'll keep that in mind. > >> + if (ret <= 0) >> + goto unlock; >> + >> + nau8360_dsp_switch(component, snd_soc_enum_item_to_val(e, item[0])); >> + >> +unlock: >> + mutex_unlock(&nau8360->lock); >> + >> + return ret; >> +}