From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010027.outbound.protection.outlook.com [52.101.69.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 69D0A3CE498; Wed, 1 Apr 2026 09:41:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.27 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036471; cv=fail; b=ddAHyUNt2yCyPAgQEnUa9et26C0vawmNX9+hqmeLyz/iP3DM/Xv5SeM0TTq5GUuqsKLIZQAT+tDKxlO0qXgARejMiiUYhYZaCp+8si1YUfbtd9XiAnP2UI7k+QR/3YdQtftQsxE0fbCi6oF+K1/dxc7O48SI5oS5QxbP7Q5mB30= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036471; c=relaxed/simple; bh=3O/ORVmeE4LTreSezBLRTWEyBELACMx8WGbdYHcnhhk=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=CLw8PzVNJN+7ZSHMLqpmaqeJO1uiIPWHjppZLviQv65gbQq3hIoxwje8gL0o2l6Xpk2t8iqPmNJd4CPc1S25WjaRSbZW1/La0jobk/VK8To3wO0uK3H8QZD7LXFokKsbdisyUpS1l+/XDGCi3IL2MVYf6dzhldqYk5xUcUKIuLI= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=CvrmQAMM; arc=fail smtp.client-ip=52.101.69.27 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="CvrmQAMM" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=TG9ZDwnIS9zwqCrLyEK2Jy2roHpN4DbA1JXi/Tp+CHrLOYZExwzBQ19jWEI3Yxfome/5uf+1cdpfHuIOFU+7N3B9mCHEqiMv0VJTtIaKwwUezZmwkTkM1rFx5GwR/nBXzzZWPduiv34ydChLtoaoMhkXk45KEMmPzxoycY0R+Rz5ElrMX9RzH/08cHLK2TJSY5XvlJ2QUDfNlpGb0ECUcgoIzuxwvHnxrF8q6uySoMacCbfZnJBiloBcio77mN89DOLe6FAHNPH1HsNKg/qLVWxlA+YAY7gzgs9KHad9KZ5AWu7r7h4Am4qWOmY/WTXbKer9ulLC6WS7TwxAG+7zzg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=iYtWKN468AYcmwQsYSWAgH/Fo9OGSrF0j8fEL5WYu3g=; b=v3n92wxNftqOfhXOnOLxxNf8eYwv5wUY15MpbGYjpUg3NHxRbWh9TSQUU+xhLiyY9bE7mFT7OtdzUqtaa3Op93JaJUTzhCUQ3a7oaval2uBusRKVZaq0QMfJKfl7WrDMMz9O3ELSb5FctVzHdtK1QvilBuvWlfDVPamKqXUD/US0Em/xyc0aMgAjgIDgB0zRFmYUMEqRic/qmH6MJz7b1nkO1f3B1t3mNb5sUK5PwjQGuHmr/n+GEP9yQr3YrVY0u+NA1FJj0nbprfunHxWfS9tb5yW9adJGHjz0ilxNkE5mKjysHEKBHo6AOCF3io47bE5rBdF31+U1/v4cI1/lfg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=iYtWKN468AYcmwQsYSWAgH/Fo9OGSrF0j8fEL5WYu3g=; b=CvrmQAMM7AORGTI95c+nfUgHVYlLZA+WihsHKIaiWtbrpOqwDxaDYSYDfxhmotyz9ErzDVz/ySb2PzzC9ps06xcEqofYfGA0PaCFBc9erxPqtEs1r+x/KhCvTHEZrDIcE33sBDN0mttkXn9m3+sFmk2WIyc0I+2tiftuEo1E75vd32YOiIOV2awje1Q7O2JGbngkFYYQWfHgdNRl5vCS39OmzP0NZF6IidlgFTC15wyuD4SEln7FzHEpx+vOSHFhzOVhjoniPxHpK4GrUxmGMPi8sh0RlUXujEq/BvaorXie1yCTjb/gqg9IK0TMc7kGxLsHureaTash0ByLooDbZQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:04 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:04 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 01/11] ASoC: fsl_micfil: Add access property for "VAD Detected" Date: Wed, 1 Apr 2026 17:42:16 +0800 Message-Id: <20260401094226.2900532-2-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 52d43c43-fd56-4c79-d324-08de8fd2caba X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: 8mZOGgJ/kZPzBlG1XrawGiZTjN7IG5R75qBj1PurbNypIX8eYVAFB7k+fbBGSWTr5HRsRsUEt3Hf1lhWoPd0Qj1X5G6ZyUs2QFsfMJJbea1xA6oZxPZQWM1m23S7HP1WYOQpN4vilWjqegbgWr3njZRee9tutC71sNaxrsWYl/oJW9+Oe+emDwVxku0PLMAifnhefnwlibob9XPJMtZTCQNPt9UrczMMcTM8f4arGiu5zmPKgpJCVVCBYfAKl4U4vWZebLIV/eYldHkyb3aPP6/WMa63mfkXqq3C8NBDZwpUKUtAbf3rAq+jSw4c2TJNsy12ho8jNQ8M+UymGDYhX5V1ZVUy1lSG/BWMMSt43OcxC126m93dsSD/eucrjd5eBRGYcXJr6dSqRO/v225ZRchfBm899yeY4tGZ5Gm7tfwcVkKbG6CFFyFSDdL0uVcwz+9uBc+xM8YyvdcSYAlRR7CuupcCzDlEwS+mcIXTAP4voohbVSqF2nTbtuRrhE0KDqKAY4+G+F3Jz2xHHWwFuxP1HPpzDct9Sg3vCQYWSRzeMzmqyzoedH0VRzekBHSVmRR1FPns1Xpc0gIrkVMWGhVMKuYHqqTOI6l61d65mvzAg9e8CiUZNocUGOp7AeL6tjM69i6HNVOyJguFAVw5ycbDBoeeE2V1sNqjAGxRSBwxHEFrh91Xa/MKKyhQjqtnIL36rkzSmqOT3UB9QURwutgNpjgukd5sEjmMRixwUuoIme2TLI16lBQmInO6skkkBPGV2E+oPG+aq0cy0+QokHT2YCajgGgyT5Rh/cIFCS6BgPTEbppcL/f9+ggq3hOz X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?1ThNPPm73R96D665dZpSO+MzGVOQH+XzlOaL/UB60oc7ZVLxjVBs+iw2tAyP?= =?us-ascii?Q?VLI6uf9JU9vgy57RW7P9uPQMjw9tYx8REydkEJ0lxzs5FhbnX9SvEI/HqQ0o?= =?us-ascii?Q?TcCmHX9vxvBtiV3G0KI3S2Pz6YjsW+JIohEbr46Rt2TzB8BuuptaB12dzvy+?= =?us-ascii?Q?0Jt+cjnInXlP9lWK4453X0tTV3Por/OCIiVBEuGL067nmPc3gRfNtHbZ92hs?= =?us-ascii?Q?quPjzPwKs1kmMCTi9NSHhNVFBUmiMXEWgAdU47ucFbJH3bYk1GYdk/8t96GV?= =?us-ascii?Q?UpUdFm6FM+5UdVHQWPIpLYZR+y/qGjxWVdFWiX57TO6AXX0JL1xlXmnbuPcg?= =?us-ascii?Q?tntoGmWWmwUBYQv4DFAPHL6FBGWKRbVESNsm9uC1P47c9gndlxYjJ1LCk6T7?= =?us-ascii?Q?5h+TTLsEo28lU5EHP98RmaevkEQTwmHfzpAi04Gbafze2xUsqJdizqxeGaU8?= =?us-ascii?Q?45f6oy+4WgsyK7wHNeV9I3i8YqwmlEG1v6LIk3uEYlQRCJPOAtWE05V/Eo/K?= =?us-ascii?Q?efH1rfIbUe3+/LY2ha5ZCxtiQO73zExUy6ZuVocJo7xlxBsdvDuJeefwPpZd?= =?us-ascii?Q?RwkCg2+jbmjd2A2iQ12/WH45ZSnBujVy3DaDvT4aaFoPiUDusf2pcE7SfFXl?= =?us-ascii?Q?mxEUYAfXSNM11+WU/2XBPhYr/DCt5IJl6Mo+xaKJEJhP8qUEfrDQKAQ8UaG2?= =?us-ascii?Q?YL7RIm++4cEPm0yiXBEHqdFM7jHYBY+VDxl/R8qQoQpO13Vtexmrn2PTtNwz?= =?us-ascii?Q?3y56t6gYOlNtkCdfGMctJsiXMSdF7Ek/OFuruyd54qrS0z285bs7Ve5kTD2f?= =?us-ascii?Q?FJ33X9QsIZtsgwY0tSEUG5zigD7GSoYzATIOH6M600nYOLLO2E/3mcAfD0Nc?= =?us-ascii?Q?iX8n8SnAUdWX6qVfNLOxBnGP23CDdPUsTBZpltpZ0c5HpZQ3i3ck2rfaDvF+?= =?us-ascii?Q?6HP9YLTwLMENGdOa/y6Q5TIfEn4k2OzbBWVow4exihQ0i2zAl+IKUwsiiqyA?= =?us-ascii?Q?OVjSZmM1ZEPSo5nGbqYvapYfcLcYusHrM2U+Lg9Y+54EV9PjiHYx8IYHc7+4?= =?us-ascii?Q?rftW2DysSiRia+kIA6EjAv8f8bAqxyrh9iT7M12Q3Cgh1Q46a3zHQPAi2zaV?= =?us-ascii?Q?ZgDCb/4nvILN+7tMsg0dOXEom1zE5XVnldzlxSZ8kJq3HW8yWSnhCr2Xylp6?= =?us-ascii?Q?oJ37WcKKDgkmKRTWX4Bh/Rg/Vf94GQaE2b3ksnjSjXfj2f/ncuugjKtodU9G?= =?us-ascii?Q?5CnucjTBCQC0PAv6iqMcPNR+vTBLa0616eDAywtmKVWBtPNDocAh2yMa8rFP?= =?us-ascii?Q?Cu0uB8IcXNKG2B1K69G+MpPLSN7cKeKO15bwPzMXDjSDfki/Z8+Zuoa6f1xJ?= =?us-ascii?Q?ikKLSnz2oSjP3jnlfVwnEBkv43c7ZOfcFIBe3JkU2a4DWIdVxWxE/8pNJz8v?= =?us-ascii?Q?pWtA77VO5EynGY7bynqj6zNnhPPiaTWCFhwMX8jDy9ptM2qet2WvJbhxmG9h?= =?us-ascii?Q?8aD+VgJLnRv0Tt3DFT9lJXGN6LilbzZ8BFcXk/iyBHognqtbt6JNKdrIvMgu?= =?us-ascii?Q?GVulY/9rAx66QV2H/mkQ0JeaWVMYnWuasNUrRJ0wfYr5fP2Hv4p8Y/Tmbq44?= =?us-ascii?Q?PqfjmaiVMWJU2VK3dk6n8VEkgjcd03Om5DuOEBhB0fd3ndDmKZGY0dAr0Oxb?= =?us-ascii?Q?QC6aZ3/YHOYPO/C3HSfxTpIeT0e1C9Wdsy/RxNj76MfpZ0pVXVZrSeIjZJbg?= =?us-ascii?Q?tE0HM8rQNw=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 52d43c43-fd56-4c79-d324-08de8fd2caba X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:04.3810 (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: oOHVKfXfAX0zOye10XXUem0+tBeSloSZf+iJu875njeNt3ZtwdQwSybMfJtbvpFo9N9VF97rvKvOmIHiKtWcoA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" Add access property SNDRV_CTL_ELEM_ACCESS_READ for control "VAD Detected", which doesn't support put operation, otherwise there will be issue with mixer-test. Fixes: 29dbfeecab85 ("ASoC: fsl_micfil: Add Hardware Voice Activity Detecto= r support") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_micfil.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c index d6cde2757c6d..79850211742c 100644 --- a/sound/soc/fsl/fsl_micfil.c +++ b/sound/soc/fsl/fsl_micfil.c @@ -503,7 +503,13 @@ static const struct snd_kcontrol_new fsl_micfil_snd_co= ntrols[] =3D { SOC_SINGLE("HWVAD ZCD Adjustment", REG_MICFIL_VAD0_ZCD, 8, 15, 0), SOC_SINGLE("HWVAD ZCD And Behavior Switch", REG_MICFIL_VAD0_ZCD, 4, 1, 0), - SOC_SINGLE_BOOL_EXT("VAD Detected", 0, hwvad_detected, NULL), + { + .iface =3D SNDRV_CTL_ELEM_IFACE_MIXER, + .access =3D SNDRV_CTL_ELEM_ACCESS_READ | SNDRV_CTL_ELEM_ACCESS_VOLATILE, + .name =3D "VAD Detected", + .info =3D snd_soc_info_bool_ext, + .get =3D hwvad_detected, + }, }; =20 static int fsl_micfil_use_verid(struct device *dev) --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010027.outbound.protection.outlook.com [52.101.69.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 484283D7D8B; Wed, 1 Apr 2026 09:41:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.27 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036475; cv=fail; b=NOY4CkXpgecWs86O0AcD5JuAByNqSbkEOHk5rlOBjMR/OFdx/wlwJOgt8xSIXBp/uWAQbPHWxBzAJofUT3DxnpXGvPw7KyDW97cEFJZDC3YlzoXw0JXLX3pryrzCAusra1xeNzV31GSh115vVUOj18YJcGEve0NPeStp9u4HR88= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036475; c=relaxed/simple; bh=0WOC96N4xvo6manO/Shd2Ycy3favvBt32Q3LwDDJ7qo=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=toL3AJYoIbrniat9a+mZauhRT1VZHkv2sQ7t0ZPKbm6+C+1KNl/kpdMWYHi8tO3xbqrJeKrmoV0E7gplLPAuM48tObM7m4cljQbc606TdhB9i6gPamjpg1p5T7fYOfPwZYJWPoTwyhkReER4O5kZQxbgRMUMRLzc8JGTnqOd7Fk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=eV6t/IdF; arc=fail smtp.client-ip=52.101.69.27 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="eV6t/IdF" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=WIawdC2n8T04AHtrc0FkWh51ZL8Of8fgLHPf0m/BJcT9qMUgV2jAjrkp4cig7+9ebKwVhEMRpCZCnnwyHsw15YZ+rW+we0QNw33mFfkKi9qryzXPh7YGsL3K+eG+ZrqAS1TrFUh0CYSblkbLfHErH40hh6YKeTpoYgFr78ggu4pmJJ6BO53yzkEVXziDWJU8kS/cArx6HatMOWRTBZsI5Tdmc9A+MYZaW97FNocjtcmUQQ/fEsXiR42HYTXvLHrHOu/TojOfhlL/XsTQEuj74P6ZufTRlUKe4qFifR+54h+OuPJtqgtNXZOpQio8ssDK5ab86V5HxQRZ65oRCePFyw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=VlIrpXKqus6x43WSMgpfF/1iWwelY/PhqPKK3iQxXAY=; b=M6J1sMy8sPzwxXcstGfIpyExiGTFUgRhlAI0bBtDkDXC7MLmhizfgYKI521aOPSIgXLpMJCjH4LcUcFdFvYqlVLfu7cDELVYMeKcWifHCuBzVa6toTMed5N4oXRKUCm7zBI/gXfdQE7HZ69izbJl8oV6JhV5bBlsKkGHT5lP3NavipzaOALiWIXMv6/jWFe42KOTGkCM3U8aOGeVGhvzx79K4ahfmeVTG0SH6UOK2bDk5pQf6ANkBKaQ2WrGDz1deyYqIKMizEJSqJSeH2lc3v4HrJAnMZF0B1M+TwKy7+vMGCVTtQGd0JFmNyeMQvbszH7k6hoTQ4AzYX49IZSJ0g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VlIrpXKqus6x43WSMgpfF/1iWwelY/PhqPKK3iQxXAY=; b=eV6t/IdFT7t1bmDGgsD4qA/vHZzAPExVlW11ZGPAt9WkUXRYVLZ9wtAvWjSeX5rD6s69nrB3Za1pdJBqbrzxpu/CWGE/nqUhdeLMJkLHRl7+uCKul3oF/Qq3WgBkaFPBEjrfLvo7y0wFW1WvqVJ/8Trpvs1krR/ZU/CFEPdsn66baHzPCUg2Q5+fZyjgqFji29SO6GDONOmFSUokQ16gMP2/3cTOpLa5vkfOPrEbj5slfA7FxP0QTsJ3ikzqfMddZ3d9dnLDS+QXP2x7AdEdZEXQq/h8kYVY192yzfHGYo0K2YJ3Ly0bny4WO6i+37m0l1xYx92GpgjQE/NOkDp74A== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:08 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:08 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 02/11] ASoC: fsl_micfil: Fix event generation in hwvad_put_enable() Date: Wed, 1 Apr 2026 17:42:17 +0800 Message-Id: <20260401094226.2900532-3-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 73a65833-bd03-43c5-5e46-08de8fd2cd04 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: Mg7EPh+TzEYtzL5dFgfn2zAM0hX7wLffmzZ6MkPbP1OaH1znYRcfmLJmCRpSr7CrpX98E8oNCdsqqagXY3xtRDVcA0J8bTiwIeEZZNp0TrBaHE8LkOS0oF8urRju64YAvDT4GhIl913ImgAsZLlVhxqwGKYkcyHz1Cg6P+hXkKcLH8qjPrWyxwID7vIydvOVLUPoiKLWgRC2qB7KAL+f9bo5HllWX1mhovyfR32/4QMuUbxwPaINEZq5ig2FgHuNfZ0BgMvxcBjN3EBXKA6GI5I6/LDKWSLzaj/s9PKzmZgMrwLV/DQaDohduNhfoE/08nlzSo2FHNmXAHwl9rJJvWuPs4jkqwB/V5m9v9ZUlK58fMxkkG4atIZ0OLsKsV10nRcd3TNwvfGfv0enjkPL7mjRgVdmJgTsY/pTPtcOIrTDL+pPRFMQ6qn7GCEFP6/p8ElCUN0T7h+mA6GkI41A+G0xy8+7ppzxqs5BUi/x3S5s0hoL3MQ7MirfbIuszioEhSpxNZn5VUSvW8cIau8AYr5jH0D+czTofdMGSnlmYVODIm5GO9wviym2j/6eEqNi+Q8A5vlp0jPGZt2qDfce1QwUE5C6Tx223vjIBDU5ZakAMzvIQRzBMvx1rzrPTjK4TCLyKvXjNyE9bVYrFBvbaIcZaKRzlhiyIj0WVoJAFYRzHdRdiVwXbYuMgQyKxtzCzZHpXKkb8Lv8LECnC2ezSMre0foYkTnuyKkigCBg7Wn+0mUgm+57FayopSkZRJna6sSzZWbSZ+2I3Ys6RNjAaRBXJvxXeOWAWSDorCYrvhJ2P3jQkJL73iyVEvoiV4LW X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?rqgCw80I8e+t72UIU+vjRG+c8qpCBVE/fx/YD0LsfItlJZAotG3PVdvtHDJ8?= =?us-ascii?Q?0QA8qOD9YurZ4FkZbqmdCfFMw5pRXoQx9yeT0UZu3v48ualWoTlsLvo6KkiS?= =?us-ascii?Q?8MISycKTtjWcYUmijN3u6rDHNaB7xu0YgP0DPqz100DTkkfYUZfaLdmb5ehh?= =?us-ascii?Q?Xvnhwq1jMj1/ui0R0N5LOA2tYuhcmGUPIXiZQtRy0HIbUuJrMJ3pYkf83M4i?= =?us-ascii?Q?3IOb8pBZBFGa8h6hDDXjnYKuiKf8LWDjLWdzKjyY6AK+pzPflhO0ubvjNM9m?= =?us-ascii?Q?4uFbl10u2GQth6kLHKJtVFz75aHERvzlGgJpQFZqN3JjWHD91E2Fd/awcrwC?= =?us-ascii?Q?LiZoH89eRjFDg/uA4HcDgT2lvuNpMymRTTOw22nmVjhh26RJy2a8zy/xF2LD?= =?us-ascii?Q?CebsgRfmOiFwEeFj8pMUSJ9fvZoz9CVKyNDJk8ydlrcVjCVgqtcTht0PLPRo?= =?us-ascii?Q?USPGjrsyWaSmHnePZhKHPIiM3+7qafToBM8dzUDAmjK5asumSP1woEl7qqKB?= =?us-ascii?Q?h6ZsLwC2qU0913YWtybe4ayzkqdrvY95YOo60GXoSLjAjR4GZ6tdmvYMbKrd?= =?us-ascii?Q?6nFqNu701TUq9p+thcqh8RhlI9xHvnvdVGTA801RnPivYB9RqbuDu6HwPADY?= =?us-ascii?Q?BRMejLzPHR7c0g/mY2ab1V6kNHr6Fw7fGKE+X8/V4RaTuGYeTO1R0LrkDWBE?= =?us-ascii?Q?VORFk9lS7e40lcSeTLuKf15kzZohJzr6H1OQ1Vaxm0Y4qPgVJxc32rWHsbYK?= =?us-ascii?Q?OLiDxRXcgdLUPXRgoFpYNKrjMjPjJ6PS+wmmPYiyQ3EqVUC7ZhjjM9LW8cBU?= =?us-ascii?Q?j1NxkSdROsxNC6fDgoLkV3Pvs8XD1PxtGG1PzLnm/+qsLqIliYrvaCmkIU3x?= =?us-ascii?Q?6xBxQobxRDnXADfhwxid18GswtamKrVPILLq84xzuk5QIXVfeDMs49+L0ySC?= =?us-ascii?Q?PVlazvbOrnePyv6zPzMpqJxlAORFU9IFG1k2nY/yVBHgkjuIEmGWFb0Hg4JL?= =?us-ascii?Q?7bw/LjhwpLvdthg0zZ+KM3Dx3GIgNvTLauV89flYLT+GRu3nkY3FS7RaYf+W?= =?us-ascii?Q?1JyROgyytXjqK1kycrZehPUFXHO7zkqS6tC+v+QzfoREdYgLUs2ZtKHSoQ0B?= =?us-ascii?Q?i+3Bz9e16vDc+gkzyGhoEEvXTyCyU7z1RohXmedIuke8qFplud3+8SkaYwDy?= =?us-ascii?Q?ElnVXbMG67c+ur0P4Hs61Mi6E2GRIGL0oAPHTHlB8np1kqLRu3ZI7/AWjfgz?= =?us-ascii?Q?4fB8PnUt4oqObtKHZsZ/U6s8GJn4dQ77lvJwIzHenir2bJZpIJU+NvvtRGHo?= =?us-ascii?Q?IrjNpjDEfAxD+5rrv1yNozp8qw7j2tjE7jKk68TRYWYF1M1LkLokffYy7LgX?= =?us-ascii?Q?Okp+2TH3j1wbWnMMyVJuuPU+GWs3C2EafcN4vKIkc4x8hChmH/z0V9hQ1lXn?= =?us-ascii?Q?k7wnuzg/hZomCvNfJ3pH8lP2MqzZVpsL3I+ak9+iQ2EenBRr58XRA7+3Bv6T?= =?us-ascii?Q?eJwe2YyYDSgqZRuAhfpnpJPjgBdZiF9QISem4J46VOzuUpdJfYAsfOT8nTmY?= =?us-ascii?Q?1qZJXIvFeEgwzUEPiseh39KrxjFq3xJmgmHJq6y3N1W70fRr9Fw5i4pfUvZ+?= =?us-ascii?Q?yl+FTPcTIuyRl/2H0X4aqiSMA9MhRE+iPkh6tpvLmNTgttkyW9UBClECVwi3?= =?us-ascii?Q?hgcE1zMaSFmkSM1sCZNNXdsflqVgAzXqW+WStnVL9aamuQzx/sJ4KawG0cHD?= =?us-ascii?Q?OUfZjDPbzQ=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 73a65833-bd03-43c5-5e46-08de8fd2cd04 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:08.2152 (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: rB+RFVVVpAVsT/LK215ShEz0wtTym6sMp72zWMC0vblj3ZGa76f82HIxKWe/v5hj3nMyqdzI6ux7woWOdk3NSw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation hwvad_put_enable() only returns 0 or a negative error code, causing ALSA to not generate any change events. Add a suitable check in the function before updating the vad_enabled variable. Fixes: 29dbfeecab85 ("ASoC: fsl_micfil: Add Hardware Voice Activity Detecto= r support") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_micfil.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c index 79850211742c..97f24c9bdd68 100644 --- a/sound/soc/fsl/fsl_micfil.c +++ b/sound/soc/fsl/fsl_micfil.c @@ -377,10 +377,15 @@ static int hwvad_put_enable(struct snd_kcontrol *kcon= trol, unsigned int *item =3D ucontrol->value.enumerated.item; struct fsl_micfil *micfil =3D snd_soc_component_get_drvdata(comp); int val =3D snd_soc_enum_item_to_val(e, item[0]); + bool change =3D false; =20 + if (val < 0 || val > 1) + return -EINVAL; + + change =3D (micfil->vad_enabled !=3D val); micfil->vad_enabled =3D val; =20 - return 0; + return change; } =20 static int hwvad_get_enable(struct snd_kcontrol *kcontrol, --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from DU2PR03CU002.outbound.protection.outlook.com (mail-northeuropeazon11011048.outbound.protection.outlook.com [52.101.65.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D194D3CCA1A; Wed, 1 Apr 2026 09:41:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.65.48 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036482; cv=fail; b=hvNsCum6BTZ1AX8Wix9ZrSQ30+a8N+NI2HYUN/B8y2AuUJWvahznuiUcYKZl2qScuWI52W9mhlJB0rA3q+fb7pEKQh2CfPJ74/WZBGlLrlRjNOxxO5AokXx6Vub37zdoTqMD0ifHrfN1OWnQ00ciD1EtSIaTbfc6jwcCX1i2Vnk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036482; c=relaxed/simple; bh=bc0YvWcAF8bR32JzvTYcwUyh5BvvQ3EAJW56frLTC5M=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=YTvul9myAQCKZCfhSI5dTNpe2wp3+Tlj0L/XQBs1PMas5FgGlzYIK87I/d4QbNcey9sL3w6kD7B5Tm+vRM4AaRFtU0jMAutpU7x7CuFIedYLSBL6gjnso2FwQuNv0LRDIY8+oeJJxm55U0Ay9HiZ5rNElD1xen5lPNqblaCc7k0= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=Zwl2Z8mo; arc=fail smtp.client-ip=52.101.65.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="Zwl2Z8mo" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=RUlg3QclsBdugZMFdBtd5qitFCOJn88+WW6BfuB3OYSfHHb8rzaitTRIC3fMndL5RugdfzqE8ljsA2NfIq5cfYnCi0tgY2rJT65nHv/w9eFPxcyI1W0HEDWgAUUtthTRawfJeQmDwIHnYvBNjzog6EPm3xG1PSnHRwz5CdBQJqb8CRsTDb9NB6AgLbOEJzWOnN5TTM0dcR8RZ1vxZ/afypT7govkhrVaqcGmk1ZK5aH793mZXZNTwao+Y/9BuRnXk/wk1HkXlfihGHJpIlb6RYySNUGbovS/e59bXRwLCfqA0L81gET5coSylJEg1hdj0CAi2vUnLad8rmaQpDpstQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=y04GCk7Dn5EiZCT7lizHtIqx7R1Rt2FBz+pto40UiW0=; b=N7bgmk+PvVwJ1mruCiBErTBAcPKJD3b4DYruCOsWeGMNXMi/4MhH8cqUy5cm/am0L0e6rQZN75uULzk2iyJJfwL4n4l85hc3hiKyCZI017ved7orvEG1kNM47MRTYdslKGheXXVshy/eUZfPnLE5HsTZHb6ittYLksB03qK+56b+WVLLYIsQY8/CNex7lcXH9yLTaqWOW4D+3cW9/2IBKC8R9qW+HReV5mREpSlY61wHZeBgIYD/G4k1SMn0MK8unVmePcUyo5Rd/VoBO7WG+jqoGtPLFlMNHnJEqdF37KUeDpjWW/y9xS395tq6Me1qfYWSCINT6KIkd6Y4okbvxw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=y04GCk7Dn5EiZCT7lizHtIqx7R1Rt2FBz+pto40UiW0=; b=Zwl2Z8mohebS6C/eh0u5i19o5TXJ6bR2RiNWepJQELsqMKFvPFBY1u3oEd8iRrjEscqa0Li6kmadj6eDEpMYIgA+b6WdYJ/UZ5wNhF0WNkbP5arGqz9ytwwobp3NzmB1ocp/Fj3Sfu88owWUP4RMVlaloi9bafp3VZ822gVy3KYHXnHmH1e7JviCXABBRh0bdJ7/d6SWu2rgn7P4Lw+xZvpPBOXlYJDa/2mlY2mbre7CGfyWKq03oPCuMfWlZ60svGdp4Hmy2YECEiWSXFvH+DRpgOd5mjfs87+Dgj5tBK9AdYgHscRyyMBQ8fcFfaxTwQQ4fhVjPHTUC4WRkiNf4Q== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:17 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:14 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 03/11] ASoC: fsl_micfil: Fix event generation in hwvad_put_init_mode() Date: Wed, 1 Apr 2026 17:42:18 +0800 Message-Id: <20260401094226.2900532-4-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 3ac9e8d2-3b77-4f4d-9cef-08de8fd2cf5d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: JRh+KL0rI1n+Zm6PJ0ZsapTGyQOVJL45AMb6hyy9GpMi6NLLgmwt/DIE3J2pynou9IEFrjksSGMNMvnx3fFJU8IFxAEHGTvMk9SKm3AK6coBZM+OQ+4essgm+QrBZWU7n4oUbzZ3KPGsNrAkuxjyjdDrLFMitTGAWMi1D1T28ltlrKIeJB7ubiyqQLDCBfHOlI5LeshAupj/PH9yp96HdiDxXBCGwyjFuBZ8xlvfkjvumbTonUiogd2TM9JgWFSBiZvL5LfRiSCv0M7wjn4+Sw+pMoFyjmMDnYGX6lPXdRhzy/umD5oI2lDtElJDTlE9y4rkPQdp/GYDOey8dy2FquJqeaGbGtyXx/ggAZGYHKOQEdWZZNE8dXK3epwY2j9vcGzit3+memzAHQJhEZrgYeuSFxMJ8wCBPAuqoA6Ltj2laDTUUH9BCsERXrJboIjUpetc2L8NYueb769BPCSGqDY8fjdyXRk5ryx2dmCEyIZjGcA8I8LnrfvhJmbYqTCsv6/TTyyOLWvst+Bwog1ugNsvfc5CiKvYLp2RA0Lk9YcCjY4j7gxx9tPI0eXbYRtSDhSpmWbNw6nJYF/lKb9NEfp28ZJ3SlWjkebfATKxPkglj+X6qswFoSrZNqXdtlHO3pRtkSjTDqFR7Ydoicx5DYrH20Tkvw4sUbFajq3/5O4Plz0g3POD8fcWOnSbjBQDofR0wTb8Dvtr7IC7SZyzrMaMkiStZoHNPcCMtas3aZAttMQ3YM+gcW07XehtzPxYlIrd3u2h7gmrjl3te3wZ/m30WPn4ycbBPAyL3ZdCfMQkmj7GBHOmfZFRcR+idBiF X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gcYLlZX+D1savFWdP3hBh9AZrWa1j0cRZvFP4AupW4kd+ZgES4BOgv0Wq17V?= =?us-ascii?Q?suYpJYNpwYrc8jNyUUCbaW2FmlaoneGMfIdDGMyATuBvjyu8Mjto1DC8Xwgn?= =?us-ascii?Q?4pLxJsSz7dU+uYVmaO2ltHZmsWPVdyhpF8xHNF8PLwZlObqMkPfDWd2Dok3Y?= =?us-ascii?Q?FTihUtHTNPeZJ1pPOW0rXLTkRwIatGf/Etc1JqMjDSTF5zLSD1cxQPWsJ6ud?= =?us-ascii?Q?VMtc7NOJNpEIURLFKcd0f7rGmS+wVItpWLnoXV3B/4bNgeYZ7TABGGd5H5TV?= =?us-ascii?Q?qyUFBWzsAdPMjXJ0/00Tcf6LKgAO5UtqFfebekDWCPwoc8yFU9ZzCUv+fuLv?= =?us-ascii?Q?H/zyBOiW5mMHEZQxLkIUH3MahDKn0eJIGX+BykiF9SzgalQHJalwnzI7Jzcs?= =?us-ascii?Q?Ol3C4iJWLooZpiiIKdhGaqJ3/vrzb8Et/cJn6nnHfVZK3+iAl74rk22hxWP4?= =?us-ascii?Q?0s7/8ZIHW4ZmG0xtUuG3BwdA6F8fncIryQXll7HXxeGhbefDRKQB+PDztZvL?= =?us-ascii?Q?gz7qjB8vdL66UiowD5bcBW8U+ipkBbkLQ+jV7J9105wzn6xo39WK3u90ONJQ?= =?us-ascii?Q?uuO2mSueZem56s0PBKsnkiYslnDR/BEJDgkJRZEPFP3MlZVwcd1pkqudUBAS?= =?us-ascii?Q?q4uTxxPZtFFqKkwO4KPpGV8k4H23vlfHL/j63fiCqd1kHXTaSbnnHT6DNPO7?= =?us-ascii?Q?iAZxPp9x3Q7vxngaD+S/uLDfPJvasOuUjFWniaUzQ1gn8zkVs/jtri8gPAeG?= =?us-ascii?Q?1TNSV4UePwS8bVvIM9mwMQg1KMgOWuCID3ZlxypV7/0wps4FIZTSE3kFvUyq?= =?us-ascii?Q?6dIs1U+lys2msJohRBIO/znqhEr4VXYqN3Sc93IYCYQOOtVJgoEL3dgyJK/5?= =?us-ascii?Q?2UGUKyC/+FmENVqGvSljuAZ3puyCGmfrG+rkuni6E0rePhx5WhhrlPa7yCWv?= =?us-ascii?Q?r/d1azH09uMtIkdU2efNMfbzsWB3c9Ka6zCXixOnMroTw2fqkrFaYrjw/BE7?= =?us-ascii?Q?9WwlpC5srhuNtQINa5dctf3K2KZsVFQaBv/yYw/FGXXmsl63Q9jH3ZnxHBug?= =?us-ascii?Q?h0VnrAvebqguI4N++pAeQZfLlusffllfctGm5fMkXSX/t2OJ9k5AKrcJDRhD?= =?us-ascii?Q?w3v2Ly24kgqLLkmFrJM4rH+mq90k4PEa9i0m6WVTRpgGwtqw2v38ntqfmtU8?= =?us-ascii?Q?vBKF+Nd8U4PMrTAIiMVPMXbB3uo8VSMbWO8GjUHVUnejf1iKWscLK3jUXPpg?= =?us-ascii?Q?CvpzlsNxqE8U7AjQ9/otbyP48NrXZWwfh2Pc8sjunpNj+jM8GPNRBQI7XU9V?= =?us-ascii?Q?Gat4oZrvrKwZ5rTVs4mnPW9+05Hyr1XJPiM60OFSpsJU6nUhEt0Sf1qvhmhQ?= =?us-ascii?Q?U71o4C0/FGvQWWTU9kaezbyMgUs1fr5d1Sn7RCTs4JIRtqDC0TQKoEiEj8uc?= =?us-ascii?Q?esJepdyefTct7mOUn6bw5i6yN+/Ou5PRifYWvMN8QNBP+TgYPVfDHO3xsndL?= =?us-ascii?Q?lzMCE3ZF+AKKQf3PcG3ab/s9z63gg5oQ577fAnZDCO7f1K2ZxPSisM1pSiXc?= =?us-ascii?Q?JNFVkJB3UuNWr7aLKtRXWpXVt6a/rvIDbD/U0acHppa1o7f72F+BxoznHVeA?= =?us-ascii?Q?ZYVj9rtsrzqWX3/VwGJJQT5WMLKUH02tgWX7BY8+2KAKMVODR2Uy1PoDgDO7?= =?us-ascii?Q?feiT7IW5S5mcxTiOUaUTv6snpBCyTiPvETV2p6mbQ490xDU/oGBiVwo5Ech3?= =?us-ascii?Q?HGNOZPd8wg=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3ac9e8d2-3b77-4f4d-9cef-08de8fd2cf5d X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:14.5880 (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: h9iDMTws4r4jZh5GNsu3tHg1vl/LVOt3EK/88LTx5/qu5usBz3G2wb7ueBestzF1tBun7zx/KOqX/mqMM/m2ug== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation hwvad_put_init_mode() only returns 0 or a negative error code, causing ALSA to not generate any change events. Add a suitable check in the function before updating the vad_init_mode variable. Fixes: 29dbfeecab85 ("ASoC: fsl_micfil: Add Hardware Voice Activity Detecto= r support") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_micfil.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c index 97f24c9bdd68..1c826e0cb1d5 100644 --- a/sound/soc/fsl/fsl_micfil.c +++ b/sound/soc/fsl/fsl_micfil.c @@ -407,13 +407,18 @@ static int hwvad_put_init_mode(struct snd_kcontrol *k= control, unsigned int *item =3D ucontrol->value.enumerated.item; struct fsl_micfil *micfil =3D snd_soc_component_get_drvdata(comp); int val =3D snd_soc_enum_item_to_val(e, item[0]); + bool change =3D false; + + if (val < MICFIL_HWVAD_ENVELOPE_MODE || val > MICFIL_HWVAD_ENERGY_MODE) + return -EINVAL; =20 /* 0 - Envelope-based Mode * 1 - Energy-based Mode */ + change =3D (micfil->vad_init_mode !=3D val); micfil->vad_init_mode =3D val; =20 - return 0; + return change; } =20 static int hwvad_get_init_mode(struct snd_kcontrol *kcontrol, --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from DU2PR03CU002.outbound.protection.outlook.com (mail-northeuropeazon11011048.outbound.protection.outlook.com [52.101.65.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 47E423D9034; Wed, 1 Apr 2026 09:41:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.65.48 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036484; cv=fail; b=c8bfafWdCPBqumugwBt4yTa/lbJfiPZdWj0kGnKp4NPFqzL3po3geKan3dcqnk2J+Tuf85Ry/FarJWVhld132DbpTYV64WE0YgRyiAnziOSwDblmR1quRznE3gRGxOwj5kN3bjvaYw+VwY/Qa9a1A61vUOX/wznwJUyQwAeWwcw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036484; c=relaxed/simple; bh=eYMFVQaCAmuTgSX97pLRtA+GQy7mr0h63BXRTT9qpqM=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=SA1EgSBK84N6no0S1v41Fzji4eIdWBTXP+y0gklFaK1vKQMya548d0ISP0URGEoUceciolFe/gIrTTCq9Cta2pKmbp/biDvyIRrTsFLq3LbGDyoGeJyPLfdqYvnUyP1bTBez3ml0GTLhY50OLBw0q8G76UC8pBAzV5m0lIH9NU0= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=lF3c3hC2; arc=fail smtp.client-ip=52.101.65.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="lF3c3hC2" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=UX/s4rddaoJ54+QbvPT7PCe7eoALg1u6BG0OSNthbis4wrkkQCbtYgahjutZFmmENUPfoV8gKahPaWaANvt8rypByDwC/IHqDDMAuJ6vMU4Xi2de6zSXJdWQqN6BZLf6MQJrwTCL0nPRmJQNX7128fBUihITUdFhD0oUCnAUYoc7by0xliWzFgjlLz8t19U452quLESS94/ss9kBDey8Co3+QspJ5+F2dBz2aa+VOgX4MNrSAx4LCfcIvmoEjEz/GfcJzIqsjiultNewGDL2idOU0UbIz7vaxFNFCJmZ7R/ZkzzF2CZh9g0y4N9dyZDQJgVVDpjEYIglirTl0fDovA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FmrTItx47rJ0HdJvNf6wHuSfObpE19m86nMqe1jrQsE=; b=Gd4RoVWvRddfc5qpbEnmpIpQTau6u51+NEMhrHaEGaEf/xWj2kmZ2xw2jn5FJKBWUqgo1TZ30zkmvApie8jc/oREwMq5lh7rCdC/BwAdjtc5R8/TAj2kTt4D8DuYRGchtDOnoh9OOLrSjgxpcvEwY/whNk/6jMzGD0U4GHJGbdX7S8qu70UzZ0/h6JBLN1Qlk4A0Q3CMaQB3hqZNbTnG7nXfHcK1EfRUXQSGhlTmPf82fxoZnUA02jej/9pBfg3VfHIdwr1Qtdv8jaeCfga5M2PB5HCGtaYFOP/C7jfht0IVi0CnRkSpe6y6f/uwQZAkcKXsgxIbbP1pO9cz58kSDQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FmrTItx47rJ0HdJvNf6wHuSfObpE19m86nMqe1jrQsE=; b=lF3c3hC2/S1L8g6OtulJrxqVB50mCfmcCFTtssQLAvC06RFbYi1OwHRZGHkevuMN9l/TLKgRj6EUjGyshZLC54XKxcvsZ4QvTA9mdAv/L4Xi34Ro4k17uhA5Czpl46GSEmHfYjnGbB+AsTOHPe4G7kNlaKo3q7wDg0JhqswDMoTzIvBuEYRm292e2Lw7PPYvsST6pll3xQB7vB2D+Zf+k/3+U9CHlBRqqwonylI3RohjLUOssE42ghbgDARUWK49DQ4/7RygTL5+amNicbuGF5uGlhODehn+6PRHriYy14RtAs4gGRXGIrK+XmD7TfTI2lq3b868uzyMu0U79bW7kg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:18 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:18 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 04/11] ASoC: fsl_micfil: Fix event generation in micfil_range_set() Date: Wed, 1 Apr 2026 17:42:19 +0800 Message-Id: <20260401094226.2900532-5-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: af69e423-6348-420a-c7f0-08de8fd2d32a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: F5f7GE1GOix02N8IW3pBJtcnIpS52Y3WP5A275el7WVRU9gm84bIT00ZLzMk2AOPiGLlHSXZfcsTfz/iCsdSFuCbCssu6For6VY6oTK1fAeKW1eBkhNzdAz+eHo9pwwjEXapR7Hh2XrZFEO/sOHpVG4O5CVDBFcovVhmfQCmarae5uP+PKDKkU+x05cxebmxihFlt7PYS+BDLO6VvZDpEr+fmZ0whX8SDEL73CcYAp1tGILA88ESPAz7MJ6lejXKXYrpS127v2dzeuRVmbK8b4oF2rcEf+qhDPPQ5SQaPKKmGTqPKCiIdhW6qTSziUEIpkxlRId0Gnwpal8Vzkho4bVHq+GTioFape7P1zQVoxhaTphlr9vDI7DyfgdOH4hWkw4USrv2eQomv79dKzFZUwEv3ZomOfaXQIETAVsPYWoY5d3xzFTozdC7uPHm5PIFNVH/KeGICnZX0I/aevUJVD/wcC9s1lAOrabJJgWp7LWCP5b5TBGa4JUOOBiNNr3id1Kfcm4zHF589xZpu127MNtGWmRpeW0abMWrAe+LzLc/j54U5x5hIgZHMcrxd0zc5tGG8uZHUFHGdjoYJOHyLG9xLRTe92CjbFhLLpFf0wrgXxyYmSK4vcM/AfDkytLUCoEG5OUX8tqNHHPJ1VldqBcBUs88796f2IVGvhcgGwBLjcqiJa4Fdn37XoIW55MsGkDpOo/tSavl63Cx5NnomMy/nl1AW3gcv5XP1RyekizuOg1qckGbaX0s0WJ9iRKTZe4wNhX1V6mGHLO5AGfUBhUgEpNMX7f6YIKD50yHvpZHA1/XqplepGJk8KVCgvgp X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gJ9BK06/4fOHrWGSUn5/HUOfUeZsF+oRSQkTag6M+UvLN3bV4kGBd5lPhC5I?= =?us-ascii?Q?dyy8nXnm/ipWEncwoR3vf7CGSp6SPrXbXCrSX9gtYnbd2G1SftEhtSjsadbz?= =?us-ascii?Q?X3IxV6JWzksYRhE67vzJZ8B9q3QTuKQ3SVjpGQbUJou8mFSem7cWhypgPP+C?= =?us-ascii?Q?3mwC5OJWR55qAbdoET2rhBv7+I7jg+WCDgM5TCytoYCUNC0HQR1BIBn5Q2ll?= =?us-ascii?Q?u5mE1ITnj1anQFhvoKmftgvXtrxCHmqdl2CT61aVhcaIJnpwV2/m03y+Pmbb?= =?us-ascii?Q?Wd6U+ulv+VF9d9zGbB+L5wCLMMTeBx/8GIEBXNwrZWbP1GZ0A6IYOIWFY3pb?= =?us-ascii?Q?i5g/wm/1rKf3LI+lrFiHll4WkwG5SXCujupSwgms6HlupS2dhDNDgeJc8rTs?= =?us-ascii?Q?QH//jyCqCjfCArd4wH68tbaGNZ7ra28Yw8cX3Pm7vygkQvofWoucpw4xjJyb?= =?us-ascii?Q?iE/AhX8i6AKjshQZZEZbRyVe90Bpo60LDruZTOl6pxG9UqhmppyoD1NTX6wu?= =?us-ascii?Q?tIWI8cH7z3OsKVHgcJB0AY0KP3r5lzICFtbID1dmtddEXwmFwJZuHo3fSTHa?= =?us-ascii?Q?Wijz5bm6zSpo1HpSbAZLIOD6mlnheP4Hv804pKjg9hG3i3kZk5Ec7KdtUzqe?= =?us-ascii?Q?1I1BY2nnBmuMM2sDFTI5m7HCILFLA295LQAQP64eGa86rufeepVMvqpcdLKa?= =?us-ascii?Q?h+DP+Tt4PFCl7/dxL+uCf6n5rydiurOPtvqLoiJRGU//OtxzZeSN87//E5Xd?= =?us-ascii?Q?8irqs3x43Nyhpgqkywfb7yGAbigi95WxaZBL6zYbbN4XSAkNvq+j8cVovSCk?= =?us-ascii?Q?2a4lha5+DM3BPMvuyDs+Owm2CsVF8gaBTsNMNac8ZndXFH79S7YEL9UCXBN7?= =?us-ascii?Q?Ppn0wtgjZqKWuGWt24Mje3/lqq5vgW/g3BNFpOuZFHJODRSIt6p4Euuvo49a?= =?us-ascii?Q?gt1NP5de1hhvuoJ0TKCLFQ42ZzMorAfdyI4IHeCnz/J1AKro/4rUIxINhdKn?= =?us-ascii?Q?Ud5nSZi3WPH0d965tH5gyWZ7p6ZkRvvfVBv3buZyW2NjeomCq9NUfbD6QkRn?= =?us-ascii?Q?D+KAOsMiCAIhmx/wKVxUv+QPQf8lKISeH+aA2kMTJPmb7ytmDOnOqhkyxr+Y?= =?us-ascii?Q?8d4dtV9bHNY7ar73xjLTv9WleofWolGwT+KMYpBiJmZzDcOeJQExUMFdMczM?= =?us-ascii?Q?iGRewCAgYe1ge8FwsWSUk/lhKU2X/eX31jnvkQYQmbqLcknEqM8DP3ed9BZV?= =?us-ascii?Q?wiVmyKOLl9Bbq+4TsbTmxvI72/VvvvtI21oi7dStNGH4Hfp3SO0/DKoTZEXe?= =?us-ascii?Q?6g0WSPPhMFq30rom2NBHaxPR7XkfuSah0rJWoRGA6on2TotlnLQiLH88lgSM?= =?us-ascii?Q?09q9zsa27XbE97nBSV9HRtQdbpKo3Pls/UwaUCXggxj3ILOm8i+mTMtotwCT?= =?us-ascii?Q?QRnILIwRtwy5GyMZzcYvzVn7rQq+WWIHjkfw67mX50kOIFNjDaifrAtpUB/V?= =?us-ascii?Q?5BEk9LNqMa3ekPoLQ51Ga2Fmr+MaWhuur0xAt7K+t055FtkcOlRf97tLU48A?= =?us-ascii?Q?nxzMQt9uVxxIWRs3lviPgsY1N/2ZgZ1AEGM7KM4akXhBUFgBiZy/o3GlW3VM?= =?us-ascii?Q?wX7E72U+s9gMxKg7TbBHN5tX0SOTp8xwSh3xAejjgU48enobkPoUUBJVp1vp?= =?us-ascii?Q?fWykB/NaffgU1zes4rwll3vZYO2AB795/FC6PbWrYGMLFBe6snE9lpZc3Gqk?= =?us-ascii?Q?vSjpPUmVoQ=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: af69e423-6348-420a-c7f0-08de8fd2d32a X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:18.3295 (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: 8HiAw0VRp8iK+4kp1ckSwSKG8/d6pb0bfTZGjxec8zqo0xx13mdwy7SQ2iynNCRPh4DNP6JvF7tpEcdhurXLNg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation micfil_range_set() only returns 0 or a negative error code, causing ALSA to not generate any change events. Use snd_soc_component_update_bits() function to replace the regmap_update_bits(), for snd_soc_component_update_bits() has the capability of return check status. Also enable pm runtime before calling the function snd_soc_component_update_bits() to make the regmap cache data align with the value in hardware. Fixes: ef1a7e02fdb7 ("ASoC: fsl_micfil: Set channel range control") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_micfil.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c index 1c826e0cb1d5..0cfdd6343291 100644 --- a/sound/soc/fsl/fsl_micfil.c +++ b/sound/soc/fsl/fsl_micfil.c @@ -210,15 +210,23 @@ static int micfil_range_set(struct snd_kcontrol *kcon= trol, (struct soc_mixer_control *)kcontrol->private_value; unsigned int shift =3D mc->shift; int max_range, new_range; + int ret; =20 new_range =3D ucontrol->value.integer.value[0]; max_range =3D micfil_get_max_range(micfil); if (new_range > max_range) dev_warn(&micfil->pdev->dev, "range makes channel %d data unreliable\n",= shift / 4); =20 - regmap_update_bits(micfil->regmap, REG_MICFIL_OUT_CTRL, 0xF << shift, new= _range << shift); + ret =3D pm_runtime_resume_and_get(cmpnt->dev); + if (ret) + return ret; =20 - return 0; + ret =3D snd_soc_component_update_bits(cmpnt, REG_MICFIL_OUT_CTRL, 0xF << = shift, + new_range << shift); + + pm_runtime_put_autosuspend(cmpnt->dev); + + return ret; } =20 static int micfil_set_quality(struct fsl_micfil *micfil) --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010009.outbound.protection.outlook.com [52.101.69.9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 17F843DA5A5; Wed, 1 Apr 2026 09:41:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.9 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036489; cv=fail; b=k7GuP8DySeTqfC9tc46hDyH0Gqt6x3/Te0KSpnDP+DXkHRPvJyCuINNZLMKBe6E2imDxoM6Ybg3t3tjLQv+qnxQiuYcZNPNBWw9MrTxhmXZxNlzg+75pT9h0Y1j426IqUgTiE7E+ugg8RxS5Z1SBmz7/xr8qs++cmJrsU5eFpOg= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036489; c=relaxed/simple; bh=vBQw3ZG7rpuuLWUReMNccXW5ZTCfv/VY/nlCZREKNXc=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=HJzYAZ93JbIewC2s+LyjqWsPE8XQiqW5nTHWNRwwYOHq8dHb9yHiAh5yrAAezk693XBisk8ZEH83MqoKiOARM2B836iSH42GHAvkfN4Qqkr+mYiFUpz6rsRvuBvFk41/GlUTkhoAWOLpfRJVyBIZEF6MdGxI3ICjYAL+HS9qKZk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=ZjzipCzz; arc=fail smtp.client-ip=52.101.69.9 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="ZjzipCzz" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ZEtvrCRvpZ5ohUFFCBwBkEf6qZRCsTEE72F5qMAib1CSdu2xiXin2RUlZZ8yrNXG6Z86d6ZHpODkfJ6HrZOhI9p1QHSSmPd4mVZUsngKVisE4c923K5jK/pyeABdsuH7YqICZkcJajLb920Qy0WtFuQs0eQL/6GWS5OC72GYBaHtRpwOVKXbIs+kG9MlipdtNHqFvRvvF48Vt3V5QhnCZq9cb20J3jWWCl8XWmtohbA0/eq8wQMwrj3/GmCF+QDFKeQzfBMBob2FByH0VSGxPZoaLnExOGDe/kcdqtcjbCaU0BfBXbfxHZh4IwvUw6GkHMT9yKGNwAwdliw9taSVWw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=haS41qD/fAlt8cHSvubSuj5P3Zr9kdx2zsfBPqlcmMw=; b=dkoUMfO5l92tji3arbeO1IwpkglopBqrqP04qBLH6Leqrow+K+SmabbYzQFzd2f4cBZjG9rZF/zK62qfgq18I04aqPW0pHZcuE1JHMllO/mOUq4dEORmgONtPd5aWOlzUvILgp4uhdAppn6DO6lVidzfoFE7B+2dyhSvjvUsb2U6BjvdQEACAvWUvwGAkJ3uy77iu2p0yP7UpZSa2N5CdOb51rQiTfIpNCqbIUo7LYK56+PxrXJw/KRQQOrpBApTgXiwT3ruKQlCDKs1u0IhvSHohwB+Sf/uKTgUYf4gZ2kapPdsh+NxtqIVKcgybOuMLHSSteIjCN3CunUF4ZCBIw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=haS41qD/fAlt8cHSvubSuj5P3Zr9kdx2zsfBPqlcmMw=; b=ZjzipCzzryGzdavJHXPlGUUk8lv3KT1Ba+gB5NAuYEGbOTmBVWwP5sOh96FqwrZe9IvjVRnAOYA+5W3fcjV3vf7Q7YuqLBThmbqyUoGIRdaV1r+WHBPPTZ0L3NsdO1wXIfSJXFUQc9QI55+/WQkEOYJwvyW0HEWAQbPo3QuQJkn0jmi8ngAUUoQ9Pyo2L5i/kP5eUMnFEXqVnoDBIHeE6FuH8YWbcj8ObPXxb7XsOjwkNMJVay1OHTd/QFzoEzwb8DzSq4m5OUL5nh2nAT/zElREILZUY04vfOzEwgLS9bdgcT9322d1N++Bw/65vB9BxihclWruWz2/iBzij2x6hQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AMDPR04MB11607.eurprd04.prod.outlook.com (2603:10a6:20b:71e::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Wed, 1 Apr 2026 09:41:24 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:23 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 05/11] ASoC: fsl_micfil: Fix event generation in micfil_put_dc_remover_state() Date: Wed, 1 Apr 2026 17:42:20 +0800 Message-Id: <20260401094226.2900532-6-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AMDPR04MB11607:EE_ X-MS-Office365-Filtering-Correlation-Id: 5f7b284d-2085-444b-e2af-08de8fd2d598 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|52116014|19092799006|7416014|376014|38350700014|921020|56012099003|22082099003|18002099003; X-Microsoft-Antispam-Message-Info: g9uxAggcjl3rgP5MvfRJBHWScv+6KUpsGmQoxIeAcb39wk4Xy/8d4SmRb0GudcF2UeN5yxeR+ADNFBvbCDQF5FbB9gdigyRju4fk9Z1slB4UGSAULK9pUU8Wu1x879UmcUxdsZ4C4MBqVIUBdDaDV+yxAg+n1+hUyyicAshUdUWO38doyxftjP7TKiM/ABB1tsymBu7317Ux+UG2QdkBUbqBaUGZ/9x21dV2yCCtJ5LN8rUParBo/5YXItAY5F5kEIkZ00JeR+r5cbnLcTp9zOl0hODOdF7zh17KWeTftc2Pm9vX5f2VuSpz2LLDk7yMmgCDJArubJKra9VB40XnD2B5ONrDKPRyZCQ2B3IXKEKMcqNFxFAVjijH8hNI+zXQta/cBFj2KZZD0I/Nu/hAwjcr94jLOG0iGU3OinGJduxtjSmZIDMYUGsqwHhWt8csYypHztBK7uFDyLD42RMjMBlEQfZc+DnfHeh7SFHXam8I5QntyLu32yZ7KtNGIiWS/0Ez7ONsXYoR8nxCureIJf6hDSdYtS2LVIUs5bID+XULvfT392/odPzJzLdGrT4NivlzJJ87jj5J/HuLVKldqrOn0mmaTyujUKLMjjmY1YZhghvFr5SVIM9WxwEB61hfJMbEnIxZip0SKz3CHorOQRTPzaXc5hMBivyBF95QgnDkgnCZ+1FJ477ehEIoLNqZsYceIQJoEynwwd7jRdgmqOFawhWEz6bL0ecTjBWaKQArdnkRCkP72OIo4SJzQP7c4myJBCY1db6/K47Geuu3XAmnsZjAEuplKejR3SzagOm0aXiJia34rHOnoLixcmpN X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(52116014)(19092799006)(7416014)(376014)(38350700014)(921020)(56012099003)(22082099003)(18002099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Jyzm+EQQhTnygq0+HxWuoqeJsmD9SBaMMmCWa/EGATebdgJ5+TO+NrqFPkQu?= =?us-ascii?Q?GApftQcqgtUeExrGu1S2vNf1ZSaWr3M9fjZBfUz6zQPQyiwvjMYbFyIzBSYV?= =?us-ascii?Q?0g3Gx83WyYhCEK4NLs548QdkO6kgEu0+UdhLkh9DEyaj9ClUAJ4HZchUyBA9?= =?us-ascii?Q?eUiuFrqpPln26lyhi/NeKm8hfvRtaUbFUaM5SvxOkQ7Me71LFXv9GNPtnja+?= =?us-ascii?Q?ajd76jGyZqDjF2GT7hdGjToQSgmxEnV4LRXgEVrdvODw/7z5EdPKFOWazyf+?= =?us-ascii?Q?IKqtJM2rI0ebJcrFErI8LRiL3PtMds5ALyIdzh4L0OpddD+EOhYk4Z2hursf?= =?us-ascii?Q?0pZ8vZb505o9qX3K8lVYTR0SDnZbPnkpDomIFa/rqd76tEdxZG7OzD9UyhNW?= =?us-ascii?Q?bVBTKn8Z9HP6xqf4l5wjovta+iNlKQ4cmjaaGJATWCDqrw9gmWmC4RxKdfqR?= =?us-ascii?Q?xi8r8S1rMsELgQyQBx9FrIVya+dR5xN8ACtzz+w3fv0W2IyQ0OpP+4ybttVO?= =?us-ascii?Q?L0PF7LShrI6oLE2TF3MKlPtFWiX0fgLYWKzyeSXAv+rBxwYbAcoIKnBj3cfX?= =?us-ascii?Q?ysb9vm49Z9T0x9egk/8LNvtwJTpewT549EEC3W18Y/QKMsfXc8apLh1GE4Qf?= =?us-ascii?Q?Du3J4nMuVVCO9Vjn3eIKNWmJ6NN2hZf0VEqbX8ORROPevG3jAhmOwffOLjW9?= =?us-ascii?Q?91FtYGIO5ckBUh2GUr7j8jspScBIXdjU/FxTb4hLh2WFEkB3wq7dzSGg6xuM?= =?us-ascii?Q?32xpbsCWdRAhYU7rnRyL8Q/r+PcSVA5VazGf6IUNIFwQEcHIFFTRe10pzhF2?= =?us-ascii?Q?Nz/1llplAkSyeMBxW/95D+aKex/0FcYUiswNKmvwAeB9SS9AFrdecyDnv3fZ?= =?us-ascii?Q?cMTacK9LO2FtLrEm391hZiBhsrNUt7xtX0A1Ki0RCtuPfErZTYj7xhg7PSd2?= =?us-ascii?Q?pC2Zqvs9i63fuwQ72WHfDdOcnRr5osyq5L9YtXMvcu99jb1acBDnehSuyXRn?= =?us-ascii?Q?PjuCxbfPf2lWHSOX+G0lx0eYE0do3P4FnmgwsdTAUPbVek8MszP+6yK9LQNI?= =?us-ascii?Q?qBIjS7a4vp/yQQODKSKMCvPNxiprbChrCmBDSubQs3562n8OA7paMG2JZ4Bv?= =?us-ascii?Q?DP03f1MoRNgYmPk6cjpisnqFMqpO5l9VoELOmVTKiisom6ZoSfoaaGvxWAQQ?= =?us-ascii?Q?qwSEaRWlgSgbHwzm++kkCfGQYGmyDabXZT9NmgP0K2lxg5KaZ4NP3dqCxHsQ?= =?us-ascii?Q?LaFOPRsl4GSYqNu4IxOe0gbf8D5Nyt5nMzqkPZfOVSEhbMZU4thDjWOPvZW0?= =?us-ascii?Q?Rm0dS8qV8dBEqDwCA2KoReOp40eU5tsnCL5Jb55CDjX3cqh3A7Ef47fCa9No?= =?us-ascii?Q?NXFVXfGhyo/zlESjPZwu0XVeSFvK295AYoU8ACgtLJnuJ9psFXROuoH8UgYi?= =?us-ascii?Q?729m+dSLdLeXsk091MEdqbwDco3oTlWdfc2WdOJ+/S5+b49wL7sKehJZ8xKH?= =?us-ascii?Q?AD8ezARIHPtLmwWcbgkenukbhy6Y3Il4adoroMLk7lJQYScYh3xu3JPJ1+q6?= =?us-ascii?Q?1xJLvAOs0afxSq74TGz2b4vyXFZwlIJQMwi/DfuDOQBYjPVRBukbB8HDXwf/?= =?us-ascii?Q?SsjTGs/yyfeYjQLDOtZO/s7Bz1rWm7zWMwevtM4Hidz1cNHMdY82+B9lm+xC?= =?us-ascii?Q?0kWUSfh+QDPyL9nGWjUSQBqc63rYbzj3MJTn5BjT/1pwk0umcKZtFLKjLlbG?= =?us-ascii?Q?E0HGcOIuFg=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5f7b284d-2085-444b-e2af-08de8fd2d598 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:23.9148 (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: zcoCWndkBGf5X5COTT//O1YnAkj6zeJU28lrblTa3gvSLGTWqofQmGQNnEz4WH4/OlLwwapuPA1xqC8gL++Tug== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AMDPR04MB11607 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation micfil_put_dc_remover_state() only returns 0 or a negative error code, causing ALSA to not generate any change events. return the value of snd_soc_component_update_bits() directly, as it has the capability of return check status of changed or not. Also enable pm runtime before calling the function snd_soc_component_update_bits() to make the regmap cache data align with the value in hardware. Fixes: 29dbfeecab85 ("ASoC: fsl_micfil: Add Hardware Voice Activity Detecto= r support") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_micfil.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c index 0cfdd6343291..983805bbaae2 100644 --- a/sound/soc/fsl/fsl_micfil.c +++ b/sound/soc/fsl/fsl_micfil.c @@ -351,6 +351,10 @@ static int micfil_put_dc_remover_state(struct snd_kcon= trol *kcontrol, if (val < 0 || val > 3) return -EINVAL; =20 + ret =3D pm_runtime_resume_and_get(comp->dev); + if (ret) + return ret; + micfil->dc_remover =3D val; =20 /* Calculate total value for all channels */ @@ -360,10 +364,10 @@ static int micfil_put_dc_remover_state(struct snd_kco= ntrol *kcontrol, /* Update DC Remover mode for all channels */ ret =3D snd_soc_component_update_bits(comp, REG_MICFIL_DC_CTRL, MICFIL_DC_CTRL_CONFIG, reg_val); - if (ret < 0) - return ret; =20 - return 0; + pm_runtime_put_autosuspend(comp->dev); + + return ret; } =20 static int micfil_get_dc_remover_state(struct snd_kcontrol *kcontrol, --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010066.outbound.protection.outlook.com [52.101.69.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 311E23D812E; Wed, 1 Apr 2026 09:41:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.66 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036491; cv=fail; b=BmJYz/UzkZJGS/gCqiymmOuIvsRZwSIZdidjzuogrDKiJVIwYtPl3nmP+n50v3GJymMOLWyoC9eAWOEPwW8DAHKEP/ZeOoazhBYch6xVW1Je3UnKWJVnDIj5Id482W4rBCCGUBR16x0+UjuR7XkInIyvcCxn5LMgic7tBU1onm4= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036491; c=relaxed/simple; bh=aBA8GgaliWchNnTrDHeATWGIxXo6wFBqLqskkaY0O1k=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=k62vhbUV2t4TkL8rc6+B4XbPYz1ehcSrDUHUb3e/LX6v/uFLiFgx1mbBzpEaYcFnF1yDFaGbf0JvX1Cgc4gTs/TQ545RXSXfqwcYv1PdOdVsGfxdVDOD/J+AJ0wL8U0ouU31fL70GbV4oMAy2yAHVJYGSAOFLJzU72m9KMt67tA= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=E8w/SDHg; arc=fail smtp.client-ip=52.101.69.66 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="E8w/SDHg" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=EOhUx1qD018s5ALMjfGup/KPG1CBoFqXAOxTcBNpwwY/Of0he0rbjiSf3Y3DeulC3TgKxT/HUZryaQ91ItsD71DgC7XVH9Xs60OHoSli0UcP35JyFK5e9DRq73Kz+c/3Ggrk242YT4KiU1M5g11vPEwgrtUuZw0hjK4r9LTMmEGK/dc/8poKW2h5kJUC/t3qUsHS3Eeep0g+EUVL138CRixAeg/DgEB8P2nLKHoLKWAnzz6I+9e9dQWZCsbpZBzcgD6Kr2W4WGbbMMguS4eeK8+b8OWC9ZmbL8chB9NfIVAesRA0hV8AclzWce0Y9IIi7vcugYHjEDNsp0LAxenPEA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=9Oi/2KTsaG7nxKIQclwlTR7Vxue18R5nbgO2D18qPW4=; b=ZQbEIs1RWm+BoQPpx8vvHY6E/JGYZpXqvh7J4eTeUDexQUt9/33P2Co78eurx/vavG+YeK4GWI5Dk+wmdKexXFpxX7dqXCi2YNDa3NFAxBHhUT7KmBV0td9+uqQfTH7yrXAgUU4j5NSI2udmpLzwFWIxT0GIQzaIt7JU10WE5wkEvi7UH0RJzylgFuissQef81iyUG0tLLcSVgOmqq9c6nQ4o8BGWzFmW1PtrgKsVoLKAwkLays8Q2FnkxOSKvfOdw2B4g8l3lVphPg7Onu91xZIy+0O3Ocs73vn0T6G2S1uVuQioQKxzAvD/qaXk1d1Q+UgSKRODn6TjUZlGYzzCA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9Oi/2KTsaG7nxKIQclwlTR7Vxue18R5nbgO2D18qPW4=; b=E8w/SDHgHotgAfWZlXqLOOCdSBeZxkbsy/c0R/01bF7nNSWWmAuML1+/l+KPJfJcsdq5h/M6+5bQtC620uv9qifddq27/WdmHIwlYtXRj6NERQFwXt4D0fVulI8fL1mLOlgYvuCK2ega+/dm5PqH0yEVqZNZH/c9wZk/qWzdMSCZQxs1MbTbHwuaQeIFHNHwI33sa3NnJycIZmcUlZfMDQnNfmPi6VTzCnEkK2Id7WBdruxoPenSEnOaZWwEPrxf6LErNveEvHE0dAFstOOq+7U0/7B4qimp5jTvUegTLwfivIXxUOYmVPgFf+5XLHJww68f/2vFJBT9npq5z7y+tw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:27 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:27 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 06/11] ASoC: fsl_micfil: Fix event generation in micfil_quality_set() Date: Wed, 1 Apr 2026 17:42:21 +0800 Message-Id: <20260401094226.2900532-7-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 49822465-9f1c-4d01-b101-08de8fd2d8ab X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: 8GR1takhoBIam+kT+HaooapJ5FOlhQE89vaeKmQUhPTnnL+Hvoj2U4Kxn2n60PG/mJf+xYInksXi/73SeOGuvs3aTf1fbohcuA9+LNyvf7ltF+SdDAPHpHFogb1VpYvHNU42DwtHuVkz7wvAqSIutzGIvjMcwboljdBnbepcnToA+8z/lpJ9M9AtQLMzyCUluGPjDZfV0AVTt56sI2H6UEKIB+qrHe114lS2PmRK48dsiB+UjbDngf1teRZBuICgj+bzwiskMISLAfLAB8nlHnGZ8nNyfNe+Scn/+Ac9WBRJtAc6LrJkj6SbV0+I5gegtUYTouh+XQPF4DPpCy4SD0xX2f4JkVh3R8X29uR5Gu+9ZGACPinyae6Nh681o2+iVYGXHScsh5xNM8OhiW3Fpm3NGodP4s+hWSj0eQ/AbEnKJLKrrWFUyi2RQhGXnTwmPZ5Xt7RAiKHaPUMwBbLZg3xbODeQFxKjsezbKiDoZznRViejs+CM63blVWoB1gEI5Q7cRKIVDntZ2BfpYPc5omr1A979FvdVhYo9Is+Y6RnQNGIXCQdcuMICgBvl7s8Wp3/1vum+kQvezsFPMOPYdwdS0/bgLl6t2W5jeO3K38JRCaC43DUApuTJkfMKmBFLeh4VK/5P5jC3ovjeRlutT7KROY4fOhMmJWMvd4rX+YcD93NmyXIGBn2BB29ZwJzrgP1k83PvWyrXRUCbQd1sio27raWWiIbB63V40ugWHIn5LQ+TMX9uuYMLx0EFppMjmDo8piW0bHbj9oUhClEKfG0TvwubF4h2EZXFiZG/JdBQa4qxMS2zHHNHYLKDl7lr X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?h9Jsjik8QTKGgLNrYO4EQoQU3ServwyJoUFToZiK6qYkDfNVU1XcKnf5ks/6?= =?us-ascii?Q?E/gLIMtpWdvC0AqIfnFVxNrqZkThAGX7cyD6cUyopQQdBXxEjx2llbhXqCab?= =?us-ascii?Q?2wd1SocoNBJMyppiqUymjRY3Bl5NGIg8m7muaZ8l9TnQlErVCjzXIdIxbtwR?= =?us-ascii?Q?hNGvoFD+axsP8n9oxs8aXWh6PjFbAB8TdlnXa9fZO4LFt6f8meEBdliwPR/U?= =?us-ascii?Q?HH+dO8uLXshE4sfkZNrcn0VuYNT2LbXn/o5ecPAenuoIP7ulrtN26G847OrX?= =?us-ascii?Q?2df0F8QzWyBRYr5QG0fethC7D/k3MMSlcBlAYGcHg0jXY39ESDK3q4QSfFik?= =?us-ascii?Q?qTjabzSImwDUzdhzv2rxKB1bHARcOHjGhfs/tOe/qgnvBFqyjuHxj1oLrMwT?= =?us-ascii?Q?x+ZqPN+BTe47IO23+c23mIIuYHStxHNHLmnuYWdmmwte8zU/SNrRdV5gGjT3?= =?us-ascii?Q?Iap675in64FwhbeQOsuP7423w+RBKSS5/8rKTAXP3BWnZgQfxgtudtlYgBaN?= =?us-ascii?Q?vxXoqtln/GCGnucCF5sNs90b+Lb6DL/EqbaPHdftrZ2qaJgr4fbemnyOYffL?= =?us-ascii?Q?X49KmmwUZViIA/bir5YSe8SvNHH+TrmmNa/xlSNhs6VPUZiFm6DbFZQ4fyVd?= =?us-ascii?Q?bj8nRiENWCZPJPD0w5x7cWqgqL38BirEAU2twvfoKGZokmNxxQ8XQO3AaeSk?= =?us-ascii?Q?MV+apkGkGCNE+5FdgG7TLIyIfEt7NVUTmaLKcH06SAZwXCXeuPPaEQ3GeutU?= =?us-ascii?Q?ERwT+vrWjWmtozJ9iJgCCLK1FYJ/V67pUSv6VTR11+G2ox/r1/JZM/+6TWNM?= =?us-ascii?Q?wrlIequNlt6hQgsFB3Mxo8k9nZjCX/ytWYmycTr7ItUGbXI5KjRnmydd84EZ?= =?us-ascii?Q?nKyVGeyDbZk/yMFqQdReEdZLXP14PS/JkByIx1dxrCZdkcuXsqaRaSLR3/SS?= =?us-ascii?Q?fXj+iNuNoGzgwjm9f/viATZ1mWJDL+7HDWE3xxg3IaBUfqYsybTmhaGXZkMb?= =?us-ascii?Q?GrSwMaVje/p6OxSQ/zSfeTP6Byc5dMyEqSZ9vCr3TqzvXqe+SJ0Ho8v7Ifno?= =?us-ascii?Q?rmf+jxnxbOehqHQfuelX2tJvV0RmKK/jyThr2RdFVJo3v96I2PUAQ+JDqEcm?= =?us-ascii?Q?1hdOeAeQ+VvYypjpdzZwWrPWOHx9nuyZNWxfOAG9Z6+co+UmW3ODzrcdRzyG?= =?us-ascii?Q?RgPhnLwBJGJPmwe1jzukrSUIX+4zOOzGRyt+V3RLk3ayU/UHA0p3S6mnMTMc?= =?us-ascii?Q?nV8LuJCW+JS1SDp44vACxeihhjydWgJsNfY16aFoJZyfqCtyD1Vf6I+FpFm2?= =?us-ascii?Q?BCb5TIQt9O54R+aYqtSxwSTgkpvq3B4kQeXWe9av68y0dtUqdQHSQKhVWmaT?= =?us-ascii?Q?iLhR36GLZ87DXWMV3TaFINiXAGMzvk9KQ7eJ6fZtpp9zwqqpF9mLsmg6T3m4?= =?us-ascii?Q?NnraQyqHYcaqlvP6CE2n4DBrHZmG45ESpT46iezSwlhdXc3vNi37gaFun9Bu?= =?us-ascii?Q?9+Vv0FAAgHdzoNhIXPtF+uEOS/iXLDVGa3eULUxCjfWKIIPi1QSH9zgM73Hz?= =?us-ascii?Q?C0RPgQ1FuVNsbw9wYfy9OOQ8qRe7614yYqKpy0eLNqkc7vq3nBVUiYmiF1AH?= =?us-ascii?Q?Rs8GUY89KtGYqtkFA0nBwoO+fohoVM2uHtkrycVDoFPUxOORVm1IHN7LQeyc?= =?us-ascii?Q?GBNRQSU0FB6XT/s4pzndyA+O/uqK4+jUJXfoQ2vWMh3+FKa5XQBNXtu48ajS?= =?us-ascii?Q?ALd4Av9wag=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 49822465-9f1c-4d01-b101-08de8fd2d8ab X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:27.5836 (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: 9xCz7IhW5CijWIDRfboZ7LwXdAYduHN5ORsXmEaQmjQbe0uMh6VZWsKWkZqzqqsOilKraUa+DwK652d4cJj7Eg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation micfil_quality_set() only returns 0 or a negative error code, causing ALSA to not generate any change events. Add a suitable check in the function before updating the quality variable. Also enable pm runtime before calling the function micfil_set_quality() to make the regmap cache data align with the value in hardware. Fixes: bea1d61d5892 ("ASoC: fsl_micfil: rework quality setting") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_micfil.c | 28 ++++++++++++++++++++++++++-- 1 file changed, 26 insertions(+), 2 deletions(-) diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c index 983805bbaae2..2e887f1f1f36 100644 --- a/sound/soc/fsl/fsl_micfil.c +++ b/sound/soc/fsl/fsl_micfil.c @@ -289,10 +289,34 @@ static int micfil_quality_set(struct snd_kcontrol *kc= ontrol, { struct snd_soc_component *cmpnt =3D snd_kcontrol_chip(kcontrol); struct fsl_micfil *micfil =3D snd_soc_component_get_drvdata(cmpnt); + int val =3D ucontrol->value.integer.value[0]; + bool change =3D false; + int old_val; + int ret; + + if (val < QUALITY_HIGH || val > QUALITY_VLOW2) + return -EINVAL; + + if (micfil->quality !=3D val) { + ret =3D pm_runtime_resume_and_get(cmpnt->dev); + if (ret) + return ret; + + old_val =3D micfil->quality; + micfil->quality =3D val; + ret =3D micfil_set_quality(micfil); =20 - micfil->quality =3D ucontrol->value.integer.value[0]; + pm_runtime_put_autosuspend(cmpnt->dev); =20 - return micfil_set_quality(micfil); + if (ret) { + micfil->quality =3D old_val; + return ret; + } + + change =3D true; + } + + return change; } =20 static const char * const micfil_hwvad_enable[] =3D { --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010015.outbound.protection.outlook.com [52.101.69.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1CA653D8126; Wed, 1 Apr 2026 09:41:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.15 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036495; cv=fail; b=msY46B5dK/YxvQsRlU9Xg5Qr4NoXCZqgHkoOgvdkJVxaqOLPA6rsbxR+d1qEbCgXgSYx/YfiDNr9vbtd7KQ0oTPpvuFpaU5E2ModkqZN7VypKqhWSuLqziNZhbOpT/LSCqPC/UGHyyV3c/Gq1lIfq6R42GMswyew+cKHmDxEHFc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036495; c=relaxed/simple; bh=1glDNbAfvfcl/h+iTuEYjPIYg3fpyPZ4SERugOz/q7s=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=Ag0gjn11TRCwDrfSOx1J33/r4XRiVI3UfulPqjuYx1IoIC/vFqFnHrYcTkvx5H8pMAdn2MBERJfE/bwSxwax35J8IBVKumtO5R85v34VzDrEexshQepDFdRH7h/X+aabyzqdF/8Fc7ghfpodsQvoq27nOXIVfl92tIZGxOZ5hqQ= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=jfMzOegl; arc=fail smtp.client-ip=52.101.69.15 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="jfMzOegl" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=rzSuGOqY8ckTTwzOiUksj8nsSVFp/XMFS5e5DBK1ySnGSoBvyfFmZDysqWpKjqPM5RhKVloyg2KvOCAz9sUUIFnBkN8w86IpkrumgmV5X/5dve8W3iIheFUT3RKnH2hYGcwfxPS5NbfrThmVm6c9hrAfmwyAuVYKjJw0Rn372gR13FrXXxDI9V/sf/oz/ktp1egRq7mEHIO2U8Hcemdt92lykV3GxzQVZ1liCMn2u8rhmJPiIltzUeW7j/A8vNmCgbmCafLeJU5BrkBCKgJKGRp1eqno4SMmr64+07Phy9Tba0dvqJa7UUvC681W5z39Wp63G3wHmANQkzaBYe+cEw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=vjz7/cKn+uswz6wnJFHTG2Teqay8TwPjwvJMoHSg6vA=; b=BqCFV84TXghXX7pxxhs0pGiBdI+ey6gltB7yQ25TypYExseKPNFVUSLuiQagMwaLtygPa/m6V92zjUj9YOlQoTShIJiQsXnxgGjnhe4tzi5d2qzLbpQlsOU9X6NfH8zHBVqxGtT8/w8SQx/Rs1e6OA0Q+VtWz7uY16d2mjF6ap7ykRizz7jk5AEFYYVM3MmCcGu5jbCS40C8ieGkbBT0xjaIiyphyuMkg1jJ+qoJuHdYvawPx2dKGpAiN2jJI8lgGuikB9JXBABO4t+2fdXzkiunIHmfuI26LueYusxO0QnH5N9blcEDRZV3yWjpwtDHxlTkI+oH1v2eQr2Xl3c0Qg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vjz7/cKn+uswz6wnJFHTG2Teqay8TwPjwvJMoHSg6vA=; b=jfMzOeglAV2gjCOBUWBthY97KEon+r9bPpfhs0CKSNhK4kjVGq2c6dlue338wImKHPZvHmGfr5A5E9fxDjFGBZTIxdgOVo3tb3TZHK9Yn4Z8UP/S9UwZaCQUv2mrCWRQpDZeefhXneM3fzfv5EDvla75EiEyr1JsYPMfPdqOXzwD+EKqD6D3JDv2CcJogL0e57cuim0gipzCejoyjWeLBIBIfhZ4zc1mMPpVaxUft9LNj77gZgBXqOKC5RBZKLzoIItBzjj/TFajRc+sNNKky5TqA/WzKG4S7RjT6M28YHbaNQr6cdJDsLZFImVraClbZ/NDxpsk2cxAPLr5hCcI/g== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:31 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:31 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 07/11] ASoC: fsl_xcvr: Fix event generation in fsl_xcvr_arc_mode_put() Date: Wed, 1 Apr 2026 17:42:22 +0800 Message-Id: <20260401094226.2900532-8-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 118ee732-f917-4e01-53ac-08de8fd2dad3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: WnQYHGZrpTwlinK5k2N/68v4xCugVUJTSJHA8giBDVOUTiDat0RL71Z6AeERk1Y7N2yv54kBZD4w6LocVXVB/bC4qZT6EEuiLdPCAiTIE7dVWpru6LWlp2rKdq+WEb3ztalKYa1i2llFIqzpQKBxBIZxpHMEoCKBkoPTZfJsnzvnumxPn8364u/Qi5S1joItW6ZoJv2ihCAwzBmVTvkECnNr7O7Ncc+szHmqa6CQjcqZg4fgetet0R8D/6UbGMj5AlaEBBvQqQzmTqua+eWTftpe29F4zUVr8hB8OWq0xMl7vHhUx6zRxB/pEMtjX250t/JgbJwUGB7e2UhJipddGLdkAHZSYnWzEdpsTqGri23vLUuS+9z/WNWNxHPWKenChtCLiejmlmehtIULnc1jMdK8TgtmFWcaUI+AuBradtnG8fSTBf4uaDSqP4y0jqYlPqYbS1d3zHApBAiDCNT9VWqOnuB8PW/+08WVoWqHCN4m/OqzEBbOOlV1ORwCZ+Vd+F9GvyCWonWiFArbLPdvlJlZC6Frn+ajxCcNkE/UTVVKWOHgu2c5g6otQwDFn7+EkoZLVDaGgt4ggIi1ug4dkDMdIAS47LSsdD/hj892cKji8VpMjux86Xx7sfJHzeEGRVKIwzpVauHitW9jxAlEztUWI2tWJIWGVMN1JpQeVa2qZXREpbXuLGR8HTRRxzIc/cE/DQvQw3kkozyoFx0egpzMGPHHQlrmlgs7a0ItFZOqDlswOWL1ZuEUrX4nj1VobqEfaRu3YIOog/AuE5R3GzmxZC8ZesdMilPa5OUcVMyaffPymBWYxVXfstBRND6M X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?tSXQpJmp4De27XQAXVNufVIKbIDHShEXzE8JLwdP6HYWnukbBLrHXTgD+zxQ?= =?us-ascii?Q?gpGFdyv2huyGJhHU45c2oxhxOzckEUZhjHJ9JqsS10S0erg6pohM7AMmGC7I?= =?us-ascii?Q?nNfsYZ3aknsSysXFAzE0ZkYKkGhZHZewKE8WdfyMSuN+wgMNLpuUnLHYW5t4?= =?us-ascii?Q?l/pUYxai7etr/Px/6GYpwBKIl+yBKSejv9ZxFhNOjDM0zL+br4GJh2EFrZoy?= =?us-ascii?Q?IcImUF+sLoEvhXocIfQyThAgV6nejmAY6f/UV5hq1TMlMNy87W4ZJEAwAXwv?= =?us-ascii?Q?cdpmz7Cn7X/Aujv0qUgo2hIYRBL1eAY2cm9mCfi+SvFvy7IQpHkOP2UPVM94?= =?us-ascii?Q?BkEOiY5Orq0kCz2QTH0aogdaVT4R6sJSMfFlCDWo31to5GD+CCgwZkEwsMi1?= =?us-ascii?Q?5JxPSwyvHtwhHDpWoPSn8SECNMc/LmDvVQM+wJgsXOKWwttyc8DLxy7/4+Yv?= =?us-ascii?Q?CAFolGtBiRJKF9D9htP6yjy4bGxQxQqQy343FOHxrAeFKncVs/hiL+B1gWvs?= =?us-ascii?Q?wjVI0J/xJMi1/hNqRSIOjQRd2ZN3YDAZEWsQ5OmXC0bXD4siEqEF67TmwtoN?= =?us-ascii?Q?afLNAvG7HcBbxSLqpZyjaX4lJE4swcNM9vDBV2l3iF0ofuhK5CIsrsGgrmoj?= =?us-ascii?Q?u745hP3njecoq86CFP11fs1ZSllOtfrecoyw3GErx/Aa0z0yT7ep1xCJfAAB?= =?us-ascii?Q?UaXAGrKIVOXuyHMFW3yGGM7T5o0+ff2+o5ARx+p0+CjTvOEj8VE6+UEFDJbX?= =?us-ascii?Q?sO/HkSTIM1QeG2GZ8hovJln6iyeGPJmfHL6EpfWdyUpafX1jrmzqs9GNylM6?= =?us-ascii?Q?QFAgYAqOiAIGHha1BsCQxDaubuPSv2XQiTFQgo7J4ssPLJvyzi0hy7MjUdlS?= =?us-ascii?Q?OmISvzPUG8Kc3DRHP0zrf2DvFsfOqqn8xwg0SMfHyFZvP7EiQYjMR6X6ZTVs?= =?us-ascii?Q?nW5jFT+TnD86UZ8N5E1xU1tuxGtEpDxlHwscgedW+LSPL4VCKSQclrw19bh1?= =?us-ascii?Q?i6mP7HqQnHl2zottEzvA+Yc/ufaT48jrfHYpjRyQpSk0pmdWqx7O9xycUW9O?= =?us-ascii?Q?0ISrZ1WmFmCZMK+26iuvW4DkSGpQfcu/iCNGoIAnXfLxPDocNw3SkydFjL7f?= =?us-ascii?Q?q8cwsU58zZrkev+3SuWHZa+WL7kLsqNptP/Gw7JQivvEkQr3Z61xgTEdUkhX?= =?us-ascii?Q?ffalmfSne7ZYkBjNhKqXzEuiB4DTfJ5+YRe8ZgM6/Q6whdAgzCMFCNo9luSD?= =?us-ascii?Q?Rl2hPWkZASnuIsALbc2iY20DdcwIWeeoFy1euY3ZakaLe3OmHdHaPa//om8W?= =?us-ascii?Q?otZpyWDfVET+7uh5iq3Zb/yAWPQk3BWJ+1QBiIkuOk2V1rqtKNAIKu/ahQnB?= =?us-ascii?Q?eaehvPoYaz0ksoExJGGXqp2+f5E87kfp/xUPnJVf3oLTkW3VM1wMk5Im/H6u?= =?us-ascii?Q?9420fUsXAEx8CzKOx6U28RGxfzfYATb3GjVbRepXjjnOF74Lh12ppgD0E1lV?= =?us-ascii?Q?Q+dK50aKvJCnoCZS0XpqHwR0/E+t/K6aAC4XzxiNxsuuQ053oPaxAJ75mkq+?= =?us-ascii?Q?EdXxL8kQydoo65EaMM4zXYkNWLqcFi422890dw53+InxtOoyvxjTbRc/iMpQ?= =?us-ascii?Q?jSiChzGFCBzBA3tuAE7Ivz7gcyMl+5NXrPH3XkU3efgqvByEWPPWZ7YyUt8d?= =?us-ascii?Q?1vXilI3PfOOqWuh+M94njP2NE23aunNDdhMj1g8G3QtmeGu7OVcGavNLtiqB?= =?us-ascii?Q?kyzl1/wpoQ=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 118ee732-f917-4e01-53ac-08de8fd2dad3 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:31.3281 (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: n6QFV1Z/21+52P0b2AGEPK9PR38hKQT+k9b6HPS86sj5mP7+HOjOuJsICa3zkD0H74JW/DQh6uicsGB+lUVRZA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation fsl_xcvr_arc_mode_put() only returns 0 or a negative error code, causing ALSA to not generate any change events. Add a suitable check in the function before updating the arc_mode variable. Fixes: 28564486866f ("ASoC: fsl_xcvr: Add XCVR ASoC CPU DAI driver") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_xcvr.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/sound/soc/fsl/fsl_xcvr.c b/sound/soc/fsl/fsl_xcvr.c index de25d9e667ee..109e3e03f455 100644 --- a/sound/soc/fsl/fsl_xcvr.c +++ b/sound/soc/fsl/fsl_xcvr.c @@ -167,10 +167,17 @@ static int fsl_xcvr_arc_mode_put(struct snd_kcontrol = *kcontrol, struct fsl_xcvr *xcvr =3D snd_soc_dai_get_drvdata(dai); struct soc_enum *e =3D (struct soc_enum *)kcontrol->private_value; unsigned int *item =3D ucontrol->value.enumerated.item; + int val =3D snd_soc_enum_item_to_val(e, item[0]); + int ret; =20 - xcvr->arc_mode =3D snd_soc_enum_item_to_val(e, item[0]); + if (val < 0 || val > 1) + return -EINVAL; =20 - return 0; + ret =3D (xcvr->arc_mode !=3D val); + + xcvr->arc_mode =3D val; + + return ret; } =20 static int fsl_xcvr_arc_mode_get(struct snd_kcontrol *kcontrol, --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010000.outbound.protection.outlook.com [52.101.69.0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B2AC03DBD71; Wed, 1 Apr 2026 09:41:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.0 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036499; cv=fail; b=cPgX380VPDGJ3k31SJu+E4r2saKCcVlMnKakursM7TStWJsyUI3Ds6/ZrS+y9C6Nq8Yaj6Fla/7ZQVOjW0ospVXEZRv8+GeR5ZQhA6SAmqSNFv5P6xxrbDgAvqTLIQhjUzhUfvdLUszypAgGjgyk2q3iArphCfYqYE2KS2hOlOk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036499; c=relaxed/simple; bh=TM5AhqMBYBdLNkojU/lS+ZkRQqL/VeBqMaW1q0K9xLs=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=cB/IOsH63w3b6uINf2xu8pTjCpsf+tXSQsRapFwCr70zidVvZrb6L30kEKd9mz2YyesJ/ZB3H09Vgcph94UrMfDLq6PGrujxyMRrVLEksNKA3onw1Av5xpimQaQahFj+XSiV3bkGJBT+ziS3+I4Keh+WtAq+ORsqN/99au5B3JA= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=atIPUcjL; arc=fail smtp.client-ip=52.101.69.0 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="atIPUcjL" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=UW1FTmftt6y/ctIW/vHkPLN3KplezD3PAK9lhF44KpTjArjFftMpssH7lrQIPStfWa0ijvOdN3uBzYhGBuHb76qUgZRa6CDNGw3PJ1/xP90ssNtXKtk0LxaU/YE78V2sOPdDtQqtrpUUMjWNeaQw2VLVTjhX8IdiXDpk2zkjWwoRaJGfmZ0Qie9WkEa7+Kj4iZ+tOhssXVqosmwBc4sudcsk7KlKqgcQ8yk8dFTWNy3L1QdHE34A1TZUAajcvkr7smBEsIIPdGs4vXfQfjJVfAmyrbr4W/AnSRvmShTb+EDdNU99anBvTUS5Kyfx6tRtFXXJ2X2HjHoXAOcEXE7rBw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=z1Ua7Y/Yo6TxbvyaRDoolgjhe+iyuzuyd94d3RcAmdw=; b=js9/mv3zWp1iZhpkjJm7O4Kb0ma00bZSerMjR6WBh99h76wxaD9bBIoml6qMlSCLwFpcj5wrs17KchKkB9/kIb3uOgStN8FhMs0EpZ1r06JdPUEfJDs4YfrwpQbYVYztTHV/TsyYVrXsiayQMfn/nYwCWMNoU0C4vdAW3QnIk4UIee0u9zg+VC2oDhY16R8ujYQLnafhIafjxzTM/daiXSkeil2nglqxPMjwEhdDEqT4ObXReXurXiVJL6GFihyaoB9apjzMBuaxz/zX53ajxK/l5G04IRR/OXrsj2V7HOkcxuqRuLD3mDl2ZQLB40qC1P4BYKHTw3R5GblVb9CK1w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=z1Ua7Y/Yo6TxbvyaRDoolgjhe+iyuzuyd94d3RcAmdw=; b=atIPUcjLRqESicYggX4KLxNBJZwigI6So/tI1+VSGJxKRz7/O78QxymMVoxDvTM4P3eBgX5Vr9Jdta3WeUcc/QoQDkDAzo1dao9pfqXqlosYmfGq4uQqa8knZ4VT3V35um+IFhJdjhBTzcGZA4aJVqct3JXiBprfCfC3fo4h4Yih0SHJbMcGomK6Ll2l3s2N36IeSHYDTPqP9Cbd1mGkbiE2oJfDaS1z4MIG7gxUsk5XcBgLsFoDb/Axb+Gp5bSai6C78Q5dRL1dKiLnefn3T6mo88d9Y6wcUZKTZQLF+Fb9ubpMx52ei0JWPklmQlbSK7qQ0lDlpDODH4kCTcRwSw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:35 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:35 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 08/11] ASoC: fsl_xcvr: Fix event generation in fsl_xcvr_mode_put() Date: Wed, 1 Apr 2026 17:42:23 +0800 Message-Id: <20260401094226.2900532-9-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 50526bc8-363f-4759-658b-08de8fd2dd19 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: QFjjVopwy5XmuMClGhkD9gfyvYr+SKHFR9dSBGULLzwJu762Eth0ZbUTlhpHoDQM3kQD1C6KZfAxwvPpWfiUpo51adcBKBV+tN3B9IYGwkZVe3AXdb0PvS/t0K5rChCrj7K5aY8DwQpFAcO5e6tDjyGo2k6o7au6atZdV5zDnwRt+lFEm9RcGHQ414zC5uMy5aJL2SAj3Ycn4jaWlIOCnWeI5EnqJPLucdjST6X4GQ3pt2lSwOBrN+siBYOMvm4Wk2FsV8tO+9TAvx8CgGw8incu2aWTgJudJEl3j/+BlXT4kc08HcMJxiC+0newBLttnazQnW31BbkXKgjd/vH6dQ8gUulremviO6umVF3nOmK0CfriuzaBbLjGMgXLdoZ5CNkltwZgoIURXLxvvlY1/R/Cs1+G3TKWAxJCs/BMY40NN9u9ghtJVNdKoIFlj9/P2dVkFNXl/WqevGXGH05JU9Akzk+iuQJX8WD5Hg2Yv4Z1re+Hac5dW6V8WMxKg7Z/kVrBXt2WwXdJztCEgl0LVflli9e27JI+H/h7qqvrG8r/4t72cgRYyd9D+0cOV1oZ8m0Grtut3PKTC5fkQywEHvdIAKO/V/TBIlFcnXEO9Umg+32mv5YIV07Fob5Xyzx6AhP1SauIv4B+8jhyUm9ax4yHNhnDsGMc592AQrUESMbfIf7//HjfN7/Eado3ZY256Ni0SHbkMvjfRQpO7O+MfxppQ4OGbTuM4pde7j4L+fscvdavwoN6sZktwiVthjLKGkJabgbHceUcUWreuQlV5Ldsi5MykLYRIGucNStqPH3mAGCWHKJeKYCBd+hoDPvf X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?5rWd+zPma3CX9fU0ncjLupWzjE9QIgmbMFIoYe1UlAGn6ZoPagqTseMk4GXI?= =?us-ascii?Q?3fSRhJHN2yl3bt9Nuskt8qQEfsnIKuQouApdsQqHNgd7d3IDw3QtYxBk+Ire?= =?us-ascii?Q?K1m2BFMPY+rfq3h67tfmgXlBv9Pk8TwqTcZPkH/DXFKnIB1gGGF9es77xRYY?= =?us-ascii?Q?skiW65en+rg9alGOv1N83Mx06KDGIIlgPEkOIDzi0QyKzAGsEUyNZDG6lStg?= =?us-ascii?Q?L4CWURYQOqIyZO/CcB3+92qLJ/QFQMAAljvefFhdmuQaX9W/OBpD3JMNyB2A?= =?us-ascii?Q?fvfJFLpK/e1a8J12ftWGYM+dZVdb7pMwfpamwX7RYkDZ1DYvuwGFB1U9bh0w?= =?us-ascii?Q?wCzPutpdAS3+rWdE7UE8HXZZlTJtjw/R4XdORK2aT5cGC3KqV06WuqSgJwrx?= =?us-ascii?Q?axQJI6zy3nCToxo9IqmefFkTkTxyqHS/987U79kowTrGRuYDTEpOJVKXH0Tl?= =?us-ascii?Q?pD8sWnMac0hlsG/lpwDuv9oui2tMWfD/AGBVqUCelnyX+BuotkE+NZ+wp7sn?= =?us-ascii?Q?dDNceZ8xgwHkg/WS0ygnA5OI89AXnkwXeC1erzfiKlk7brzYZNS4MO1khOU2?= =?us-ascii?Q?72Hn/oh1Gyx8FRHtysYZ6CO9fkOGUyt+KQ3Z+/aRc2pjn/HSjlq84VqRF2U8?= =?us-ascii?Q?f5KL88qhZv/jx34nXXkQ1axuJYlPkeP38Uv/q/t86S7m+3aSAJLbMPzZClbl?= =?us-ascii?Q?1XS3fk+PUMqI2ErPQD39LMXIozmUklGxh5GMX1tzDG8j/Ic9Q+jtkcmGPy1S?= =?us-ascii?Q?+HejcxGqS/mtlAtFLIGWc7NbTd3JbGYbfkhcAhLzDtfciPQv8WbO4168p13I?= =?us-ascii?Q?+JTIBmd7Djl2zSk6EQyvIaP1ZJ9v3QyqQuI53o+Ar0VL7yBxxRglh3GIAu7B?= =?us-ascii?Q?0/t72yx9FiYh70Lnu1EYXSwykY77S/ABiSQRHqz0S0kgtsCVdAdv7WRRlW0n?= =?us-ascii?Q?WF0HmFDNgqbpinOWam0YBixrUmJCn+9b87+7a2js821iT6o5kXND5NWSrF1Y?= =?us-ascii?Q?E3gM9E+l7ML7FfQ6yjbzvR1ZsUX5Wck3PDeVwUHF8CQzuHCkATk5lWS+rZ99?= =?us-ascii?Q?Wiedgq5XGMhT3jI5hsmRDG9xt281yB3QLUwDXfTOIcUQs3JXb74L8333gtlD?= =?us-ascii?Q?mlE/tqLr+Edh9JJJu7LJbUMBxC13HHUx/CsRtPdx6/4h5r+8rgHYxWIUQ/I0?= =?us-ascii?Q?tUI42d2vBrbkVBLhJYl99aYH5PqvfqOBUNdpL0SicMFdLGYnG2l8IuFFjXNh?= =?us-ascii?Q?t/izutZzsB+aOSvsHCChPKqEFEGjzDfi9zRMh/rxSOOcm9Pn0XiXQR2HXpW8?= =?us-ascii?Q?hDwDIXQps4EHFavtFPrN+DA/t1KnunsQxY5LFKfQ0pbFQdvdDIUp1LfAyJuR?= =?us-ascii?Q?8COJ+hoWHFarepze/ZHRZ8lgkOLIx6q5wyj6OdOseg1THheSL5XI5sV/BFJ9?= =?us-ascii?Q?gUfhOROr/4Fv1nADV3yUc8Ko65bXsOtc91qLb2u4PECkghN8brZEFdgvikd8?= =?us-ascii?Q?1/kcxc4rrzKpmudMn2tJJScU7HKWrOXLMf7TW5k01ZbUocg/k8z4S0K2Z5Ts?= =?us-ascii?Q?Dx81nSouqaWA5HjY72MPSXewLG36DPSN5mV3r3K8qxHFM16UyOBcuDKRPJgA?= =?us-ascii?Q?ANyFYPlDm37zpNG6W48OZ3tP3rFPHRQtwIgOyS+wstHKG83aHSsSDQ0/Ic0z?= =?us-ascii?Q?hXXpQTMUB4tV2OjxDlMxm5apazTVojIJi22HSiZGTsMU7O3SZk+ajcMdR/S7?= =?us-ascii?Q?cyVsjZywvw=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 50526bc8-363f-4759-658b-08de8fd2dd19 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:34.9738 (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: 7PzpRsEzo1hlM/DifBaQvb9VrEydLyOnQZWwTjoNzndnBTatgxF0pd/WjLyJi1nJDqjocVpfmADzLb/MHCsSrQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" ALSA controls should return 1 if the value in the control changed but the control put operation fsl_xcvr_mode_put() only returns 0 or a negative error code, causing ALSA to not generate any change events. Add a suitable check in the function before updating the mode variable. Fixes: 28564486866f ("ASoC: fsl_xcvr: Add XCVR ASoC CPU DAI driver") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_xcvr.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/sound/soc/fsl/fsl_xcvr.c b/sound/soc/fsl/fsl_xcvr.c index 109e3e03f455..ee16cf681488 100644 --- a/sound/soc/fsl/fsl_xcvr.c +++ b/sound/soc/fsl/fsl_xcvr.c @@ -277,10 +277,17 @@ static int fsl_xcvr_mode_put(struct snd_kcontrol *kco= ntrol, struct fsl_xcvr *xcvr =3D snd_soc_dai_get_drvdata(dai); struct soc_enum *e =3D (struct soc_enum *)kcontrol->private_value; unsigned int *item =3D ucontrol->value.enumerated.item; + int val =3D snd_soc_enum_item_to_val(e, item[0]); struct snd_soc_card *card =3D dai->component->card; struct snd_soc_pcm_runtime *rtd; + int ret; + + if (val < FSL_XCVR_MODE_SPDIF || val > FSL_XCVR_MODE_EARC) + return -EINVAL; =20 - xcvr->mode =3D snd_soc_enum_item_to_val(e, item[0]); + ret =3D (xcvr->mode !=3D val); + + xcvr->mode =3D val; =20 fsl_xcvr_activate_ctl(dai, fsl_xcvr_arc_mode_kctl.name, (xcvr->mode =3D=3D FSL_XCVR_MODE_ARC)); @@ -290,7 +297,7 @@ static int fsl_xcvr_mode_put(struct snd_kcontrol *kcont= rol, rtd =3D snd_soc_get_pcm_runtime(card, card->dai_link); rtd->pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream_count =3D (xcvr->mode =3D=3D FSL_XCVR_MODE_SPDIF ? 1 : 0); - return 0; + return ret; } =20 static int fsl_xcvr_mode_get(struct snd_kcontrol *kcontrol, --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11010012.outbound.protection.outlook.com [52.101.69.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6C9983D891E; Wed, 1 Apr 2026 09:41:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.69.12 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036502; cv=fail; b=ipqDBkIf8po3+8ATq6Ry5Es1VQTH+QFgSQ8//GkdikXcH0Fp8UQV3JwU0vbIQiBMK5W9MgGNAX+zMJXvOjLMt18Q0J9v6LAKGhYIYXSzM53jmAWcdIL0p/PSjgT4ro/kHXRIQgNHm17W3/+kcNgR/r+d1WnYmvfsVofxkkH4du8= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036502; c=relaxed/simple; bh=CzMqE8pxtKS4TRaRNNLH7ScJ9KZpknbuBwcanfVA+Gw=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=PjxbBFg44TJa4kluwd2PF0iML352tJw6p2WZ5bi2uuwF3Or/dLlv6VWy3oMRDBTHI0JLYdPLLsdjy1MFTUJF6+tOPLtsQsOremF3VG5HgEnzSzrGmgYZTP8zavKL5Ak9lY5EQrNpd7V04SBnOtjt2u38MLbEr/6v0hwCZ/JQwAM= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=ITTs4sSe; arc=fail smtp.client-ip=52.101.69.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="ITTs4sSe" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=kYD5WVKUUYFdE0ntXwHy/caeg0zQsMrjhUFZ+CTv2xPjsrK1Hu1leu6tfezodDBL3m18p7gVoWCIcAf59bh2R3WXZ6LGOPoawMcYYG3hp4NTH49dO9ERifefKuTbFPU4ZQXE5E2YsKVcFXCSfTxiWGXyJJ2YdVZKMYc4x4YxuRck4uqNyzrCiB4xvCkR1BXipSJgxdQ1r0Rdc9DvN7LNMyGTArE1ktK4YQqb/ShQbXir3nK+Mr2BdfZOJK6+s/IYIhiYMVWMsWKKiQbKbl1uwcwEc2Ma8koMQKMb2jaEcvBjteQP0kIwX/Ov4ImPZsKQtisrSgDVpTTMp/XGo/kMwA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=9Uzoe5gXhM8SPvOT7oaHjCJYRsmb2cWNtkzMLoIVxUU=; b=s0MBqrUsFwEwOA7KzuvDqlyt8TwBSyDpLOVVth4L+QL+DmnJgEBqM/p2p/q6B4A0GLbPj8o7oKXhW+CZFKjuUVni5ewghkUNPNk5KuvhRy50YKHl9os0A18DD0WfXyFRIyEY/P97elxSIkFIvtpeOnurrSdNt2/ikx9tGc4Pvy2X6M0sPpWRKdPcS6sTtc/T+oD4/M1ibZ2c0EVzOpSOmoezKoQhLyVNqZZJTeqkc+jc9YCgQnVH0OOkHszQUXEto+njQvQSpYWae/cnCjkdFJqj/n50SdscETcHfLkib4QPEu/Btu8OPWZI4NqsQ6hfF+FYJFj+bbJpDUXGBrz+KQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9Uzoe5gXhM8SPvOT7oaHjCJYRsmb2cWNtkzMLoIVxUU=; b=ITTs4sSe6nLR9Fu9FB4y+LJ9UB5Tx997Yn1+7YCeEHcicwc6KpWM8z2++O3GneUt4rDSCr4OB02Zo5Y9VlMpfTMCyr7riinzKvA2spibKikLZ0ldq0YTx5DcZ/tUp2LgyZtawU3JovqlipVC1uEZVQ9FHpqH1tfpcFjeaaUrfD+F3nI8voe2zy+uKI2UqdXhhdyyR4jTCxJZQPM/wrudmdO/jTR2/eejQLo/0CC90EbdggxUR4CC+4FqUMKPetytzXjb/HL2ZMWXVlgHijk0jT2XsALah9HpyJHTsJX/b3xa9f+v+ruIBgJeua+yKEOZ/gVYmpwQFp/He6gBeKnT3w== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AM9PR04MB8241.eurprd04.prod.outlook.com (2603:10a6:20b:3e4::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.15; Wed, 1 Apr 2026 09:41:38 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:38 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 09/11] ASoC: fsl_easrc: Check the variable range in fsl_easrc_iec958_put_bits() Date: Wed, 1 Apr 2026 17:42:24 +0800 Message-Id: <20260401094226.2900532-10-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AM9PR04MB8241:EE_ X-MS-Office365-Filtering-Correlation-Id: 36a858e6-222a-43b4-a384-08de8fd2df40 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|7416014|376014|19092799006|366016|1800799024|18002099003|56012099003|22082099003|38350700014|921020; X-Microsoft-Antispam-Message-Info: 6PnY1QkKX/z20Cikmg+7/Yf30GtAHv7g+awYP1EvwCPOSPt3ydJbOgd6/CIs0aU5Q2Fbk+3itg2MyEmBUFHJ0+8ygj42HfxfBNkonVZIL+BSPTTgxTaMjke49Tgdfg+VUkF+xonDa+rowwaMy6jAvBwm3EImKTRNRMjGnWN8YMolix5kAfHXz5Ogh9E7kNVOmZDB7PaJqMA5D/2cGew1qnOqdmS2s2PZrtOoMRZKSgRaBP4u7jtKAKEhhb6SVJS3WklcyGV/aRFvVoQ0dyR33WXeMGsbSMyhI88//mLb/NL1UmuKFivKLtGYaofraOTLNsSrAxoiu2IvIK1Wl+EDWKXaJVo7pYQdVSQf7dDFDLP+eXk+vo5B1m79vb3VSp0PFMFTK6nXvcR3xbk2GAzVtj1b29evKq88ZVi6UwyhREscSbmNJci8MK+nIBce3Q0yCS3BGpUj6DxBn40tJ/Hsur32B0Cy75OCFtLHVhta3z236n3k2n08t0iLIuBX3Im8BAIyNrYjvX6wc4cjRwgs9XrrPyuS6X8IEzdnF8R1YIKiMsUFB5oPHkSPwdncOxdOfnZAu2OCsVTvQ3TEQM7aK0TcOVvEgttuaY3vsMzkhCxbZPJqa+Zu7V4X5N9BvJ1UV2DQ34AtFygVG/yO8katN1IlpkyFp6nHfsqb1MoNtqRBAuHre7NE/t6jkoCNwXJ/t+YbrhkFDRiZ//5oMEaFJSuShBoRPcSbKgqgFxrdqAZOibF3Epb862gx+/gIFNHNjuVsuq5swEGP9mPRPHtUPasadE6d9PS483Em/58grpLrkzRLPbH6K+sr+rf5a6IF X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(7416014)(376014)(19092799006)(366016)(1800799024)(18002099003)(56012099003)(22082099003)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?0GEIh8yr14h3AtBQiUOStOgmSrwwZN7td/z+mLFw+TMPSimrHnQPF/I9ydmm?= =?us-ascii?Q?ftKQMvUNvGvdK95wnMq+L3ZMiXccn+O9Tqigz9UOtPGSBwuUmra27MIL9Wwn?= =?us-ascii?Q?O/1GnTsUz5fluwCzn1dENI1G2aFanOGz6aJsEz74lSxLw9FvujxTYYWhVhEk?= =?us-ascii?Q?EshLafg0eJ4QCniWo1P/dY5Jekc2az8OcuSnADGDLrG8PhTMvpw0hydyaj32?= =?us-ascii?Q?yzWn9t81RWGhmFmrq/BRnybcbQhoiFee2lszkLp/cv3f9F2t8uGCUWJ3hz/d?= =?us-ascii?Q?86dFx1ZzP9e9HPlV1MzBIVBiScOEp4RTWLPIvk6kb2/jlfu4EE78eou4nLux?= =?us-ascii?Q?y2Wag0Lkp8WRjplZahYuFez7L4wUpMsjsg2zEQukTammzVBY9o6HQpKeAhuJ?= =?us-ascii?Q?APbbrHohanoWTBLrXXuxRcBJ9mMiik/tHWkxXFxNdDwPMo0NfguQpFCLOKov?= =?us-ascii?Q?AVyjQZMOIf3VWyzG0DmuxwEGU849hAFO2awBxgF7eK8R7hhSaA2/c7SXDWDV?= =?us-ascii?Q?epGSEMo6spRDEjsFO9tu3+S60u1L0+dqt6zECsiGafYDWJF1qSB+Zlf46fuZ?= =?us-ascii?Q?4MSNioXuKtHuB2nZKNKBTAg4xYAEhjvVRTL61vap0fcU6OmluBhCqSL6RcUy?= =?us-ascii?Q?t+hXCVz1/PiahpJKsgm533a3cNwdxURl9aEsmv04Wz42EARs/HdQnI/41JFS?= =?us-ascii?Q?/ZXv20xDKxcKZYEEYx015whWC12lzGglZEv4CRG0pBleWw4S3dUEiyqdlshL?= =?us-ascii?Q?mCLYeaCZfnuhE7VCqIy5wcW37QNctWofQZHFS/iZmf/70gTCcB16nK8r9zBv?= =?us-ascii?Q?OPvjjN+fYV9AAwccKC6MGiBRAdOfuotutbPAGUFalcwmvvMI6BZITrfnvEbw?= =?us-ascii?Q?95lcXlJFvCSt7fdEsbn/COOnmPaaB5OYnzId63x7MqTVyxn3Hnpk7H1swbnj?= =?us-ascii?Q?n+gIjJVMQ+hmyv8uBks6RsYhRVkhCerMo1PvvcyNLC03R/qncnRnRIzXPkuJ?= =?us-ascii?Q?WMCUQtaPm/Z7Kxtpl7o2y+R0svcqwqeYsPSlrYUOlF8Ey6Qy8HlF8qMzMI9C?= =?us-ascii?Q?VvUgOAF7V3UFmpjFK8UPYtYXC//MokOgGx2MB5gWt/K5ezok1fWv+Ts5QkKP?= =?us-ascii?Q?wjwIfOwRNA7rA3TcHcR4mbExqV1mD7iQPZi+aNYNuyczCeSrqigg44FNUBv2?= =?us-ascii?Q?jGMYxuyTDlPPunwmQKosXcbM57HSsCWo6ROGGyKDBsImZ46DI+NARWvR+uvr?= =?us-ascii?Q?YHD6Tz8fOvrFneziKQvj/773t0g0aUgyLJH4Nyjc2hkn9pVO1bZAMiQJhndf?= =?us-ascii?Q?Ci+Dw/ratyoZMQawetOvVvdaKAoSpRmBHyk6CyNl1SXG7pJMv9sAUv261m4H?= =?us-ascii?Q?USSzmEtwa2vX7cERWsWm9L73nFQRUZJOu8CkbIuEl8VTDROimbQkuTWotjXe?= =?us-ascii?Q?Z3qESl/sfdwrUIDNMJ1SQrUkE9PVVdYcoQA0h5MTZhdAtDlM9SeAma3dAshl?= =?us-ascii?Q?6HPbqnf74dpfSzrbPsrCr+KOi9hZ8ZOJet2zWepxmoUbr3lCIxWJo1BrWvdu?= =?us-ascii?Q?4d5HXV5HGeIKlPboHGax2CWqjwsb2ndkWZeuq+YKXztEl786RZB8E/XF4HnT?= =?us-ascii?Q?oPHNaGy/n0jRlCH5zR0dgmLfwpU/KkRhIDtufdD399GQdRw4Tbag0NVzoH/q?= =?us-ascii?Q?IAJYtpmILVO6PbnTKUmW+gneUN+sxp4Yz2ArdQ3gNvf2c+ZSwAW3+V6pBItE?= =?us-ascii?Q?4stYfGBoUg=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 36a858e6-222a-43b4-a384-08de8fd2df40 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:38.8308 (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: l5qvg2bQ7YnnY46PtreRpmZxe2rkLLdMlvuL8oHVzHgU3ACbGX3uw/ANrnwU15B8ZpX5Q1hOEiDdYvIjQuoDCg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8241 Content-Type: text/plain; charset="utf-8" Add check of input value's range in fsl_easrc_iec958_put_bits(), otherwise the wrong value may be written from user space. Fixes: 955ac624058f ("ASoC: fsl_easrc: Add EASRC ASoC CPU DAI drivers") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_easrc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/soc/fsl/fsl_easrc.c b/sound/soc/fsl/fsl_easrc.c index 599e439b359a..f48e43a0edcf 100644 --- a/sound/soc/fsl/fsl_easrc.c +++ b/sound/soc/fsl/fsl_easrc.c @@ -54,6 +54,9 @@ static int fsl_easrc_iec958_put_bits(struct snd_kcontrol = *kcontrol, unsigned int regval =3D ucontrol->value.integer.value[0]; int ret; =20 + if (regval < EASRC_WIDTH_16_BIT || regval > EASRC_WIDTH_24_BIT) + return -EINVAL; + ret =3D (easrc_priv->bps_iec958[mc->regbase] !=3D regval); =20 easrc_priv->bps_iec958[mc->regbase] =3D regval; --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from DUZPR83CU001.outbound.protection.outlook.com (mail-northeuropeazon11012022.outbound.protection.outlook.com [52.101.66.22]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E43943D9DA0; Wed, 1 Apr 2026 09:41:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.66.22 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036506; cv=fail; b=oQmy2kojPglfGiz5p7Uj0+Wa5/nkpsf406QFUznbKIhcpwuk17wRXigMl/UCCzunB04NuFWZYdzznrB10WXumC9l03n/SrWjG8ZTfmGqf1GNT7YqjlXMGi6tjC0tNtfSe34+Uv2v1RuBsUWeyNmdAsU+UklMRte9uWbIWlP1x8U= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036506; c=relaxed/simple; bh=72BKGs8TzH7Tv6r+rEjW4XSfQ3tWbwVnDZlIzhmbrRA=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=WBN5KxPfKUsMmjnFcumEdwkzCAQk8ULMbd9z0hYkVK8OuF6cZqrC4ixd3T3t4awLaw5rHhf0U6QSRUs97ZMEvr6homaUsJ5g453YEhDaE1OpUgencyX9wJORzncL/6vVATsK4U4pLRFiY9Yer9mlOT2714W51wGc7/1YKUyiQxk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=Nf/aUkCJ; arc=fail smtp.client-ip=52.101.66.22 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="Nf/aUkCJ" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=nH4aaLX4ZoPPWOfMzy8rl507RHf8gt0zMHaSaMgmgu2H7M8cuu3CHnKkK0ZzKYMfYT7Y54pByxl7rgUEThDYGfw1ngTTP9sW8kz+u8Ook/FDNjcwoDFdzqCamd1dtFpAMsSMteTG+Y02gD129WMg3HbToPeEQSt2lzo2DbIviUqRGqaSZdPf4j9jEJnT/GSbrzoJiNGWu38xsFOqz7ML8VLJuS37RlCRd47aFZylfAKae8thuAIXV3lMriDqZ6gxGwDmTx5CBmY/cIyhO9OhuYftdR7L7LItYNOreQyVf5OTKxV32MQg+uYXujIrlyqmvSyjNL5BK1/Umw8Ox9h6HQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=1W8Z6I5k4qC1og9LoTkBGolQOg+dtplXEcqMJbQxjaw=; b=NDq9Ul9/yBkh+LyQCn8J1ib09WGDXNzuawSWRoduxrJI9FMDZTm/9Smk0tnBuat3fg/2UwRA8dD1CS54oMYTsjS3kcRk2S/slwx5SIlu9ji+rhJc13E7c/d/XHdHCzjLaScUaSaLKH3+RnkL++l2Yh9uBbTdv3nH8YEFQ4WY+qT1S70stjqCHFJQPzSW4jlutR3mJaowMPlb3Db61Vnwg1X5ymlShXYzH/FDN330yxLWDosB6b2wuispgYg+vONvQzbnCCLYii6RTamXzLhJVS/dwiL1ZbwicrVCygJrzAhSJ5DA5tX8Jkf8MkeC60SywfJB4IHuzQHznZ2nKfCUzA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=1W8Z6I5k4qC1og9LoTkBGolQOg+dtplXEcqMJbQxjaw=; b=Nf/aUkCJK2Bw/n4Wst2RfRapMh050n5gYUw+DLwz5Gm8VITUuSEd1eIgX5D6rqXeHBToSWBrqK5q7eN/ykGSXZcy5vx54VwbBr4604OOlmyVl7YiNCHC87T7vKyhAg/0pilOqp3iheoGtuGBJiKlqdCsCOo7h6jZnNilrCRE0YVj180kXgqCCj6bu7nisqBfWmGaRYPYQp1KTAZPw3Fqvxy+wOx2C95thAKcfHwAlK3I5SlxbKPJKcM0fh6N26ZBuSUHfCm0qHKQleSIxJMcfU1j5RhbV5LFOVgI33YIj0oK1t+vQ89kmX0L9T1Ji0ElgZoabpoy3VoE9PAsn7/l5A== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AMDPR04MB11607.eurprd04.prod.outlook.com (2603:10a6:20b:71e::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Wed, 1 Apr 2026 09:41:42 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:42 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 10/11] ASoC: fsl_easrc: Fix value type in fsl_easrc_iec958_get_bits() Date: Wed, 1 Apr 2026 17:42:25 +0800 Message-Id: <20260401094226.2900532-11-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AMDPR04MB11607:EE_ X-MS-Office365-Filtering-Correlation-Id: 6961ea43-751e-47e5-f015-08de8fd2e189 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|52116014|19092799006|7416014|376014|38350700014|921020|56012099003|22082099003|18002099003; X-Microsoft-Antispam-Message-Info: 11LHnqmNLnwO53z9Q+pj0Gk73gXJa/Nk9HnHsAr2mjr6p+pd6197eIGg7oUIEvqX0jG7Gpd9Lm/K5GoqmPrwBBL7t3wyioXYH8nCrPocHUtQ3hDUg0aoJt2fJoCiDo7fEUVcsat+xn1a6w8rUwlIu1AC/AD0yCadYOCkwwJ8c81gTnARToiJeuh95i6OQaDoEOYwTQKF11kyIjCpxgYhCHhtGxdsgmvB7CTB180FIEvDYaa4lKw8xH8t/ZFtH2kWfru3D1IVhz6TbG/tyhlWJbqqiZoDS5VjYLcbdVgDKvIeoLE4xEfHa4d0axsLFL4xOjD7pK7FAysxUtRXQmvssdI1y6tfAjR4VD/1fj2cS5CFW4Zo06KE25v3e2XiVPTXOJUpN6N81shVTZfnLnFraUsYdB+vBUcNnlDalKs3CurldeqUl5eDPM+ZtEuFgkkMnF9ufQhomeXwUbuq6cuWlL5dfhzcvvDbN95wB/keU8kyrDGJGO4jozkehABlahSVferZkXGAbKZY6QPJ47FSQtmd1kwLxs1jlYcZc3vRJzcrHv4sNmpz1YAwvxo7dFGkv/wAR5o3mQpo69IwI437Y7yd/oD7zQQnXUCECCKgKCirgI9JB1Lfcsx4fIOTfYlPQBhnGE7uOKxBreqNdm9mJ7AY8BziZrf08NWFtn0+zttPxxyTi1x6F3kxDFG+vvPG6zfXbPimclHFODHslIHygZaLLxDEt0iDxBt+n//PenNih+E0yLvIjnssza6+xTmO85PFNCvr2kEOij+ra/L+qDSGOV0jIcgovnTz43H9WEvj7rhD2njAVtA/Qg/vsXmI X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(52116014)(19092799006)(7416014)(376014)(38350700014)(921020)(56012099003)(22082099003)(18002099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?WnLoOt7HMnAOMkd3D1c2N929LcNvhgBIvs7mTLmayqYjOTPvxdGvbwR8tFtS?= =?us-ascii?Q?Rm/SjVBi4ciYxTgH6WS7L9PDIlAaUXRUlK8ZlgIeo+4uVryc4YbBsp/eyfqa?= =?us-ascii?Q?rnr/ewai9tBEunI8PwaKv/NyiGkpsCzRoDlYUDp/zPO4H5I1ECLTnuk04GuS?= =?us-ascii?Q?CBWy1r2TfWPIj0wxWHeXLzC8TmXFre6mBPknwUNZtPAIbGmvOm+83/VtqoPp?= =?us-ascii?Q?feNdueDkhH4svg+m5jnQ/4y6/QZSl8R+3ukSD3LcTqSEYrrWhT6SxxQRIala?= =?us-ascii?Q?vLS4IgGDDu3Z2Pzg/XBs+V+q+G67+NCLbF87mpwUh6wlhPbiQGfFpG7nSlJn?= =?us-ascii?Q?/YAUqpM5xqbihJ/c+XZt4olZI7ggf6/7irMeqwg9ZkiDt39Pllq7I0J+gkTN?= =?us-ascii?Q?1ZFTM8Cd3ZrzYQxnZEL3Om1s/3+/mB6oMzQHuPVcJlzKLR/RFM0eOw0c74FM?= =?us-ascii?Q?aCdFqCQ+gRbMNCglL7e32wHBtRqaaQdB0p5NBiJP1ZGgw7dhjB4/XzQD2j8/?= =?us-ascii?Q?qaZnGHTgipyQJCBW21Wre66Gs+BJOswRnBRW7ujJFj4b2ORp3kuteaIV+W5K?= =?us-ascii?Q?8xiQD9kHbYbD1ziIiLjmvb4Rt3xr03w+E799viUur7LDKwj3hlr0OJkVxkDQ?= =?us-ascii?Q?lXSRhsWwlNGT7UrC3R5T8lsEhAbZV0wKNHdPWDRN1baLn3q9smN3UYwOj+Dl?= =?us-ascii?Q?KPz1xZGCaoQOse9b2pzALGxl30rE0VXmDH2giwvCq7jahR8+KgtpNUTp+aLa?= =?us-ascii?Q?EUds8skvf8U+YJfWqL6rTGFtxUfYWjD2jWaJaW8lCvh80Q4YqLmZXAIajSmI?= =?us-ascii?Q?9sMALPxOFSnFte9OLtXRct26lhFHvadpToMqBV06E0OxULYeGQDvpaiBR2UE?= =?us-ascii?Q?wADWrh2BwVevasKuinv8IzScCqERyze0RXcS8b+RZNeF10241cx+Bj11EAaP?= =?us-ascii?Q?S3LxOxf7lWE/0/Q52IV57a0WYSin7mZ4Qcw8XPbmcbFbD6fXvwmNRBEsCZYG?= =?us-ascii?Q?ZE6PVuGGMR0tEyMp7b0xZ/YBslja944kBReZtz1398jm0HB8J9j1z75CB1Yx?= =?us-ascii?Q?PpbeNiknBgOmmhzT5VzbFD/ebokkM0atdeq3HKXkzzpDFI0m5E4ny/NmYbek?= =?us-ascii?Q?4f12psQYbi06SeVWOAsk1hSL3jOK+evNE5EZqjMkQetrd6NNvN8Zd67yHELy?= =?us-ascii?Q?ZygVarvJ0RkEi69IHmUR6zmSq26QQHmajN7/sy3T+Hhoy1Vd7CWmRadZq6Qk?= =?us-ascii?Q?9AGqvrkHvvt9+oG748ZyJPgCyGp7zESBQ9Emnc4e1f1eFnFRKi6qJUt6H+BR?= =?us-ascii?Q?WTw2F1YxTpnbWIBo8W64UU2IaHGvGYoyZbEXCIXOAMP5Vg1tQ40FbjozkYU9?= =?us-ascii?Q?sXolUX24Txvyuv9GoWmUauA1lqeMoBSQTyTSx/hAOd2S4qNK2o32daKSwYiW?= =?us-ascii?Q?3qZOcvdrXjGVnRGci2ZuCz7GnA9ep44FQ+DGtJYaRxIQNZ7oA6Pr2D4tYbfc?= =?us-ascii?Q?JI8yeR+BhlFfOaDE/hXE1t6PO2aFfZesdlp3VG4+F+neB/WBf/pXHHoxdi94?= =?us-ascii?Q?C7LbsJChkRMlRTVWjwSd7eoS7dtFyV2EKpx1QIeZVEpKl/lJajFqzDsng5Da?= =?us-ascii?Q?3pPY91UvxrsrHcqVb+I0tQMngUzGeSlOLxw7JbI3zHwGGoUaR+tVOK7Mq7kW?= =?us-ascii?Q?EMLGBW4V2ZloAOBvseCSco26OwGiBluFYjIGx+SZ1Fwf5q6HJOuv/ws/BsI/?= =?us-ascii?Q?nkV42SBFMw=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6961ea43-751e-47e5-f015-08de8fd2e189 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:42.5950 (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: HMw6kmyCrKl9A6Ed4MqlLKn8zzzZGD6wz6jHsiBvf7fwyvEqZVGr0dZI7YAh+0GiGXHjjabfxYT2nlsi15ejVw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AMDPR04MB11607 Content-Type: text/plain; charset="utf-8" The value type of controls "Context 0 IEC958 Bits Per Sample" should be integer, not enumerated, the issue is found by the mixer-test. Fixes: 955ac624058f ("ASoC: fsl_easrc: Add EASRC ASoC CPU DAI drivers") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_easrc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/fsl/fsl_easrc.c b/sound/soc/fsl/fsl_easrc.c index f48e43a0edcf..3086cb758beb 100644 --- a/sound/soc/fsl/fsl_easrc.c +++ b/sound/soc/fsl/fsl_easrc.c @@ -73,7 +73,7 @@ static int fsl_easrc_iec958_get_bits(struct snd_kcontrol = *kcontrol, struct soc_mreg_control *mc =3D (struct soc_mreg_control *)kcontrol->private_value; =20 - ucontrol->value.enumerated.item[0] =3D easrc_priv->bps_iec958[mc->regbase= ]; + ucontrol->value.integer.value[0] =3D easrc_priv->bps_iec958[mc->regbase]; =20 return 0; } --=20 2.34.1 From nobody Wed Apr 1 22:35:53 2026 Received: from DB3PR0202CU003.outbound.protection.outlook.com (mail-northeuropeazon11010056.outbound.protection.outlook.com [52.101.84.56]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8E6203DF000; Wed, 1 Apr 2026 09:41:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.84.56 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036510; cv=fail; b=hJtQTXchNL7RT1lZE0eyuWQFe+3z5UWwolUxzhKS+q9AGVZl7wwTL5Q2jC/hRQHd97PhK7fBG2ib93V8gQEpfdPhw4hRJunOAg6q7zlS3s2z2WWdz6/d3NhyRODZa2XwF+ulzqkKduituYsk2ErX4xsgAvTP+mDgiFCYaYUL6as= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775036510; c=relaxed/simple; bh=3wA/uymqTPE35r80IsaKAW3If46nVepyaYjWQBlwUyQ=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=dXu/5GLtS5M38IiXp6N23ePp0F9DLBlV6iYzgs4REZJrE4puRQjavtnJOH77Uq02SD7W9PBviJuKlM+e/VakwvNWbeShVZrokOmdV1mo0KwHVNpdA2XLnBEj2Sju/ZhcZXIBQygLa34S6GNFR9Kl5rIN0uPnmAt6OjL8AduVZNE= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b=XVipNUdY; arc=fail smtp.client-ip=52.101.84.56 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nxp.com header.i=@nxp.com header.b="XVipNUdY" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=SDsR9vkt0e/45AR3+ZQJt+LxAek1zPX2qeB0mjhVFbB1uECCZx4eZoIGWaeaJgG9GDOrTK1RuydkeBRP3/8NpiuGEMAAyTuwyeiSXyyX7CB+OHAt27qIKOShXE345NIto5xiXHvv1YkUmEp+xR7nj4vhsXWiwF7PkGI70MnOXqS6tcvTI0RZavxg42hJHuVHgUsqyaTXvyKfdvdeKuma8/nnOCb6Xps05dfCBiQ8gjFvPueYvEnV2cwktHYrSekQglxpGfszX7n4aI5teS+pgUcxALhri9Yyyfvmk5chDxR2wgFUAbLCRrpfKgDWceLTGYC3txNJOVPhAXtl1F1nFg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=lcy5UmZqh/o0ySld2VRlYssBwZ+fCg5A6+bAmvQGrUI=; b=A1xsVv7aawVfW59jTXhyX9n1pUf/+JUCmabz+U2zJZSr6WARvKnxdVj9GfYzgz1NoTrkKvOcQMOvPqRASglpH0GLzeX/dqk47ey714YoSh4QnQPI4Jae6eaVOQtwYMgHeWiNVxgtInXzAngIOJYlMCjdxK9sVloKGBpaYqVSkpKtax1XyeGqR2okyxzW9pfXoIUXrrIgcvdbNGoZDthSINLbTkplAg32gN3gmsib6azyCscX4X/MglI6pJwUgZUa0ghrgHfmFJwURJwIX6PUQDpOCbEvQUB+G+OZGORHETT4mE+VJ2rvT3wj3TfeX4vL5p5qj3tLN9jv20KFuf6T3g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=lcy5UmZqh/o0ySld2VRlYssBwZ+fCg5A6+bAmvQGrUI=; b=XVipNUdYnmSn1UvJFOQ2m94PMNgxFlsaO1ROslYW+PqfcKgVsR8v1e9/Rip7xOpsrBriL3ivD1X8EihB7AhPoIo3ja+vmfumyiq+OyPlgpl7ztTccb/PuK1eoeU+U6oS8sWQGmM2tpccA/iHlK3a8ecjd6u9QhCUU4+NkzUrqHu8hZ5tqLiawsdKBaxWVKiZZ1JHUkq6RLeqUgJ5Nrxgst2lQbObiyxnjxs1T/uh2tALcuMopwhPDwr2q+y0Ikvm68qVQzePOOmHzYYGUnAryYn6Q1olelaHcQ4yhg0e7YWtGjlicR6QkPmusc5zzKoQovLJ8btu8TWe+5XUsGzEIQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) by AMDPR04MB11607.eurprd04.prod.outlook.com (2603:10a6:20b:71e::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9769.17; Wed, 1 Apr 2026 09:41:46 +0000 Received: from AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b]) by AM0PR04MB7044.eurprd04.prod.outlook.com ([fe80::bab2:d15c:fcf8:ef2b%4]) with mapi id 15.20.9769.016; Wed, 1 Apr 2026 09:41:46 +0000 From: Shengjiu Wang To: shengjiu.wang@gmail.com, Xiubo.Lee@gmail.com, festevam@gmail.com, nicoleotsuka@gmail.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 11/11] ASoC: fsl_easrc: Change the type for iec958 channel status controls Date: Wed, 1 Apr 2026 17:42:26 +0800 Message-Id: <20260401094226.2900532-12-shengjiu.wang@nxp.com> X-Mailer: git-send-email 2.37.1 In-Reply-To: <20260401094226.2900532-1-shengjiu.wang@nxp.com> References: <20260401094226.2900532-1-shengjiu.wang@nxp.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0029.apcprd02.prod.outlook.com (2603:1096:4:195::6) To AM0PR04MB7044.eurprd04.prod.outlook.com (2603:10a6:208:191::20) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB7044:EE_|AMDPR04MB11607:EE_ X-MS-Office365-Filtering-Correlation-Id: c4201a8d-6710-4c06-7743-08de8fd2e3d6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|52116014|19092799006|7416014|376014|38350700014|921020|56012099003|22082099003|18002099003; X-Microsoft-Antispam-Message-Info: pi68TQ4Hi17gfnqYV9ElSAE+VliIa0EN39RJeGd57icFzBG4iCqr1jsN8aYaqmrVjhJVK6Zq6ceZS96vEJgjOb2JW/qFeoWpBueS4Fwhyq1GbTp0STotKVMECO2onbiN9uuiOfV0+7tigv/lubcrUm16e8mIDQJo9nW3ATWyXz6BDSKotrP1fOE3dwggxAolUtav+qaeqV/GnqcbTP9oyXiYnLgtpBHNg34C0hCO/e7gIYNOEzDZ8qur6aJ+XR7N6eL2rt4SL2LZ8+QgGOcewTtz2q1tyfvYqE6bITjDQ+3wd30DLcJF2S/ed9BThp8v4hbungO8gFqwrGDrWYLCuPAf6rvSKZS6LV4rFdzJsGtPVu+DlBz/gSY9slxQEP6MEKXS9HXTmewXOfRCA5I7x0lEBecd6hNu1D7e+hC2Xw0T49hP8HZXJeldemR0MPsVtd2D5irX02KViDCDuw7FCXqNK45j2fcAqQUIQS9SvT+SZgm8XtWHL8g3QiMRijo2LuDbUVXrM/sg6ugrwnwlSQ/BMgE1dYegadydKabHzdTAMGa78oWs47I2ls/NxGhOPZSW4buw/qQeZpUvgY4kvoymuV9/Ij0g8y9INIkyzldhUnGtkbtnXD3qCjOBEtxb9fpuJUgo6kXhFwLKgHpawrQPuXC7PVjIAoEHqD5MrMVnITdNK1c/xgtRPzoI8tGB+u+qHb5zPk7IcZrrqDAdks5rUHuSHzmb0n9HKw+3N4XPqWmBxwl6QxXgkIpTVM7LPDy/qBRQomV0x9PcV0a7yGkMxTSMJL2LujGR2yXRicRpbvWD9+TqSEWBI9gxE46l X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB7044.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(52116014)(19092799006)(7416014)(376014)(38350700014)(921020)(56012099003)(22082099003)(18002099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?vXJGfKBDxkbB1INgDshPvVx2hTL3nfebkUR6vfxl+1cVYkT+k9ov/moWDuZ/?= =?us-ascii?Q?xfip2Gecpdq/5qlT4hPWZDQYL1uENZdEM4ooRXbaVoQKn3vrz6+suJzkhsCE?= =?us-ascii?Q?OTFEsd5iif8hAWB6yoVvIDff8u9l6oF6wrz2iW9IbTsxllGTnQ+CjKq6U//K?= =?us-ascii?Q?Gpf8tWoZ6WiwHL1qmfd5ti7QIhClO5mL6x4NYaE1OQH/c8dR777R+f/gaiv8?= =?us-ascii?Q?c2oerUFzPACxV8byBq84gfC/YNDqpAOXNkQtwVZIg2BWhpoTgG/IbvLAm+af?= =?us-ascii?Q?33K8WSqcbfuCIlnCab2RXK3iO5PRvETW4JFvMwO4UBDFdpPEgmwNHAaI7IfG?= =?us-ascii?Q?HhtnXVdERsGiREMx3u8lC7T/rYtvObxUZ+BMeYKcZBhvLEsWdQakbDjgvQIP?= =?us-ascii?Q?sPWZtgCdHDp0OvjQyNL8Bz6DMHGm7SHZvdrT8Ut7azz5TeZ3lBdUaFKS0qgo?= =?us-ascii?Q?M5jcmmydTW1CG29w17SrJJsT4+P5QpDrXlihqStTPKJ9br73dO6mQnUkuzLO?= =?us-ascii?Q?v0J+ygQFVQjpVAjq9S2mQs/oBvB88FsO53JO9+VquEsj4cx4Vwcya5QBT6Id?= =?us-ascii?Q?nny/6wMF2u8eli+Snur5GS2iKqiiz+V2r9ELc26qFBE1mjmOZzeOgL5Df8aM?= =?us-ascii?Q?I3YFmOR4lsGreLxnsoEPIFs1iMSgRhmQ2cLxUtDPII5xDR0x7krmluQ5B9t5?= =?us-ascii?Q?d9GZUsniPnnzqtUHBlTnfEA6C1jVz/0+2HB5x+om2Hzctiq4cSxiANgCV9Al?= =?us-ascii?Q?U6TgwMb2eu/mw/uZR6GB7tmndacB8KedgJ2fTFe8wONkYptWeiLGpddf904f?= =?us-ascii?Q?CLOcwMqGDvCMugEo0dNi4dIt3HcMtj4DMhh7vthe+/0T9uGr98Rb4RJynop/?= =?us-ascii?Q?06/N7Uihx36DwAGRw+b6/y3v1+IoSgid0Cbl17ExMeQC2m6XKUkW0mg/vs74?= =?us-ascii?Q?kV1cSaJvl7g6NS6+Ze64HQqXreh2nJHvDObOBdXkdr0cePdXBxPlS3ml5sEF?= =?us-ascii?Q?Gi61hRFdmn+bXpq9msMhqSJuFyX+xoqtV17PH74Mcz+rnphzDdO8IiYmHY7D?= =?us-ascii?Q?IFzXzSlIEWT7xpFlvOHSHVhDw9zRLNf1gHWRz9YPfRGoZbOiUMuEnkkeK0by?= =?us-ascii?Q?SSZa3DLChcNncxMbNDccY0WN2r9Oh1DS4bETsh37oqelAL3T4YVV6BzNET1Y?= =?us-ascii?Q?EcbApDYi5hYRKyupuL7wmW0hF9v/g3u68moz4MGOA3WCs75/7QAOoulVRpF8?= =?us-ascii?Q?eUlfr5iOLwpuCkPT2wghV8Iu/CKvHdB5ow7l8ineNGCb+0E8hrCrFeryhzqT?= =?us-ascii?Q?BSeNjov8HKSEwBE9gaBoNZEn5SbpqWWdLgM1Ds8nZBZClA/ygJ3HgzeqxnII?= =?us-ascii?Q?aY7P875uiXNBwOCJAv11xXlcyOgrGwOxhoXRIR2cuLaI8ditmdiJtigc6Q54?= =?us-ascii?Q?orGrC35rK9W4hwQv5VNacMwet4NAHurWkP+tptVqiHpREQ3gDgbfhDGeqx5C?= =?us-ascii?Q?oZGghMGDuAsyJBFKvEq58pDERyRSbzdkYceGjtE4iYxmamvZN18F+/jFpbgv?= =?us-ascii?Q?1vtoajbcYhUR+/UEHjd0aL1lBAsNUs4hedUjjrzHqry4m0yb6Rxdp5LXcO55?= =?us-ascii?Q?nK5rS5pz9FiSHNEeVKHKNL+S0a1jndeuvu0n6Advq35MYMC5s5qDUQVYZbVP?= =?us-ascii?Q?xSNGNdbA941aS48dag3LwAfPcMIx7vsNVBMsVD102AVFVGXKOSZixtBIJYa2?= =?us-ascii?Q?bewgudfBZw=3D=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: c4201a8d-6710-4c06-7743-08de8fd2e3d6 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB7044.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Apr 2026 09:41:46.2431 (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: pHYSYdkeGDmZvjFlmZcyX+XGI+eKl0Psw+7vm6+lVwboDpt2uL1hX82wQ921jAcX8Pin1/yMZsDarpy9xR5UsQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AMDPR04MB11607 Content-Type: text/plain; charset="utf-8" Use the type SNDRV_CTL_ELEM_TYPE_IEC958 for iec958 channel status controls, the original type will cause mixer-test to iterate all 32bit values, which costs a lot of time. And using IEC958 type can reduce the control numbers. Also enable pm runtime before updating registers to make the regmap cache data align with the value in hardware. Fixes: 955ac624058f ("ASoC: fsl_easrc: Add EASRC ASoC CPU DAI drivers") Signed-off-by: Shengjiu Wang --- sound/soc/fsl/fsl_easrc.c | 118 +++++++++++++++++++++++++++----------- 1 file changed, 84 insertions(+), 34 deletions(-) diff --git a/sound/soc/fsl/fsl_easrc.c b/sound/soc/fsl/fsl_easrc.c index 3086cb758beb..114a6c0b6b73 100644 --- a/sound/soc/fsl/fsl_easrc.c +++ b/sound/soc/fsl/fsl_easrc.c @@ -78,17 +78,47 @@ static int fsl_easrc_iec958_get_bits(struct snd_kcontro= l *kcontrol, return 0; } =20 +static int fsl_easrc_iec958_info(struct snd_kcontrol *kcontrol, + struct snd_ctl_elem_info *uinfo) +{ + uinfo->type =3D SNDRV_CTL_ELEM_TYPE_IEC958; + uinfo->count =3D 1; + return 0; +} + static int fsl_easrc_get_reg(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol) { struct snd_soc_component *component =3D snd_kcontrol_chip(kcontrol); struct soc_mreg_control *mc =3D (struct soc_mreg_control *)kcontrol->private_value; - unsigned int regval; + struct fsl_asrc *easrc =3D snd_soc_component_get_drvdata(component); + unsigned int *regval =3D (unsigned int *)ucontrol->value.iec958.status; + int ret; + + ret =3D regmap_read(easrc->regmap, REG_EASRC_CS0(mc->regbase), ®val[0]= ); + if (ret) + return ret; + + ret =3D regmap_read(easrc->regmap, REG_EASRC_CS1(mc->regbase), ®val[1]= ); + if (ret) + return ret; + + ret =3D regmap_read(easrc->regmap, REG_EASRC_CS2(mc->regbase), ®val[2]= ); + if (ret) + return ret; =20 - regval =3D snd_soc_component_read(component, mc->regbase); + ret =3D regmap_read(easrc->regmap, REG_EASRC_CS3(mc->regbase), ®val[3]= ); + if (ret) + return ret; + + ret =3D regmap_read(easrc->regmap, REG_EASRC_CS4(mc->regbase), ®val[4]= ); + if (ret) + return ret; =20 - ucontrol->value.integer.value[0] =3D regval; + ret =3D regmap_read(easrc->regmap, REG_EASRC_CS5(mc->regbase), ®val[5]= ); + if (ret) + return ret; =20 return 0; } @@ -100,22 +130,62 @@ static int fsl_easrc_set_reg(struct snd_kcontrol *kco= ntrol, struct soc_mreg_control *mc =3D (struct soc_mreg_control *)kcontrol->private_value; struct fsl_asrc *easrc =3D snd_soc_component_get_drvdata(component); - unsigned int regval =3D ucontrol->value.integer.value[0]; - bool changed; + unsigned int *regval =3D (unsigned int *)ucontrol->value.iec958.status; + bool changed, changed_all =3D false; int ret; =20 - ret =3D regmap_update_bits_check(easrc->regmap, mc->regbase, - GENMASK(31, 0), regval, &changed); - if (ret !=3D 0) + ret =3D pm_runtime_resume_and_get(component->dev); + if (ret) return ret; =20 - return changed; + ret =3D regmap_update_bits_check(easrc->regmap, REG_EASRC_CS0(mc->regbase= ), + GENMASK(31, 0), regval[0], &changed); + if (ret !=3D 0) + goto err; + changed_all |=3D changed; + + ret =3D regmap_update_bits_check(easrc->regmap, REG_EASRC_CS1(mc->regbase= ), + GENMASK(31, 0), regval[1], &changed); + if (ret !=3D 0) + goto err; + changed_all |=3D changed; + + ret =3D regmap_update_bits_check(easrc->regmap, REG_EASRC_CS2(mc->regbase= ), + GENMASK(31, 0), regval[2], &changed); + if (ret !=3D 0) + goto err; + changed_all |=3D changed; + + ret =3D regmap_update_bits_check(easrc->regmap, REG_EASRC_CS3(mc->regbase= ), + GENMASK(31, 0), regval[3], &changed); + if (ret !=3D 0) + goto err; + changed_all |=3D changed; + + ret =3D regmap_update_bits_check(easrc->regmap, REG_EASRC_CS4(mc->regbase= ), + GENMASK(31, 0), regval[4], &changed); + if (ret !=3D 0) + goto err; + changed_all |=3D changed; + + ret =3D regmap_update_bits_check(easrc->regmap, REG_EASRC_CS5(mc->regbase= ), + GENMASK(31, 0), regval[5], &changed); + if (ret !=3D 0) + goto err; + changed_all |=3D changed; +err: + pm_runtime_put_autosuspend(component->dev); + + if (ret !=3D 0) + return ret; + else + return changed_all; } =20 #define SOC_SINGLE_REG_RW(xname, xreg) \ { .iface =3D SNDRV_CTL_ELEM_IFACE_PCM, .name =3D (xname), \ .access =3D SNDRV_CTL_ELEM_ACCESS_READWRITE, \ - .info =3D snd_soc_info_xr_sx, .get =3D fsl_easrc_get_reg, \ + .info =3D fsl_easrc_iec958_info, .get =3D fsl_easrc_get_reg, \ .put =3D fsl_easrc_set_reg, \ .private_value =3D (unsigned long)&(struct soc_mreg_control) \ { .regbase =3D xreg, .regcount =3D 1, .nbits =3D 32, \ @@ -146,30 +216,10 @@ static const struct snd_kcontrol_new fsl_easrc_snd_co= ntrols[] =3D { SOC_SINGLE_VAL_RW("Context 2 IEC958 Bits Per Sample", 2), SOC_SINGLE_VAL_RW("Context 3 IEC958 Bits Per Sample", 3), =20 - SOC_SINGLE_REG_RW("Context 0 IEC958 CS0", REG_EASRC_CS0(0)), - SOC_SINGLE_REG_RW("Context 1 IEC958 CS0", REG_EASRC_CS0(1)), - SOC_SINGLE_REG_RW("Context 2 IEC958 CS0", REG_EASRC_CS0(2)), - SOC_SINGLE_REG_RW("Context 3 IEC958 CS0", REG_EASRC_CS0(3)), - SOC_SINGLE_REG_RW("Context 0 IEC958 CS1", REG_EASRC_CS1(0)), - SOC_SINGLE_REG_RW("Context 1 IEC958 CS1", REG_EASRC_CS1(1)), - SOC_SINGLE_REG_RW("Context 2 IEC958 CS1", REG_EASRC_CS1(2)), - SOC_SINGLE_REG_RW("Context 3 IEC958 CS1", REG_EASRC_CS1(3)), - SOC_SINGLE_REG_RW("Context 0 IEC958 CS2", REG_EASRC_CS2(0)), - SOC_SINGLE_REG_RW("Context 1 IEC958 CS2", REG_EASRC_CS2(1)), - SOC_SINGLE_REG_RW("Context 2 IEC958 CS2", REG_EASRC_CS2(2)), - SOC_SINGLE_REG_RW("Context 3 IEC958 CS2", REG_EASRC_CS2(3)), - SOC_SINGLE_REG_RW("Context 0 IEC958 CS3", REG_EASRC_CS3(0)), - SOC_SINGLE_REG_RW("Context 1 IEC958 CS3", REG_EASRC_CS3(1)), - SOC_SINGLE_REG_RW("Context 2 IEC958 CS3", REG_EASRC_CS3(2)), - SOC_SINGLE_REG_RW("Context 3 IEC958 CS3", REG_EASRC_CS3(3)), - SOC_SINGLE_REG_RW("Context 0 IEC958 CS4", REG_EASRC_CS4(0)), - SOC_SINGLE_REG_RW("Context 1 IEC958 CS4", REG_EASRC_CS4(1)), - SOC_SINGLE_REG_RW("Context 2 IEC958 CS4", REG_EASRC_CS4(2)), - SOC_SINGLE_REG_RW("Context 3 IEC958 CS4", REG_EASRC_CS4(3)), - SOC_SINGLE_REG_RW("Context 0 IEC958 CS5", REG_EASRC_CS5(0)), - SOC_SINGLE_REG_RW("Context 1 IEC958 CS5", REG_EASRC_CS5(1)), - SOC_SINGLE_REG_RW("Context 2 IEC958 CS5", REG_EASRC_CS5(2)), - SOC_SINGLE_REG_RW("Context 3 IEC958 CS5", REG_EASRC_CS5(3)), + SOC_SINGLE_REG_RW("Context 0 IEC958 CS", 0), + SOC_SINGLE_REG_RW("Context 1 IEC958 CS", 1), + SOC_SINGLE_REG_RW("Context 2 IEC958 CS", 2), + SOC_SINGLE_REG_RW("Context 3 IEC958 CS", 3), }; =20 /* --=20 2.34.1