From nobody Fri Oct 3 16:45:44 2025 Received: from TYPPR03CU001.outbound.protection.outlook.com (mail-japaneastazon11012062.outbound.protection.outlook.com [52.101.126.62]) (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 589DC277C94; Thu, 28 Aug 2025 07:54:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.126.62 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756367671; cv=fail; b=cBa3lAyoagd/fdRDw1MOlAIlgI061s3CY+6gFhDJ/fF7YI5AwIbBfKDyBbJf1eI2pMlKjYVLjm37LOASNiNLK5aaiTENKKZ3yK7+AP2td2hoo2ZEhWfn0gKTvvfMOGJfNP7p7WV48ycBR7hORW0cMEkgATidLjaPGyNXEfUeOFI= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756367671; c=relaxed/simple; bh=lO91waDNWtZDwT0V5LwsrE3t3/bMmgClZ6tFToSp0IA=; h=From:To:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=R3adCeFKnKztEHm/w7+Z3sV62DzYWk/FIufzSdbVT+3HW8u6dXqFFomVwnDlAexLhnghm9WbU5ckSO1pBX4fpeqnXe1Y9IZkqH/FJXCBAU0ndEIlncDdjX146U/xgEjMrFmPTW8pAYZt9tFaMrPbsS8hvaIs1L01tLy3TZ3kf+Q= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com; spf=pass smtp.mailfrom=vivo.com; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b=SxMKloxU; arc=fail smtp.client-ip=52.101.126.62 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=vivo.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b="SxMKloxU" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=AWxH5B25MK49WPIPjwzuSbsNk2gdWFfgiU2IhziAyIaVg1/5yqDCHj4JC2pD8YUR5xuPMdAiG+DnW10w3XHhzVPwQNw5fs4adbpXWYREJP/xTuVyb9cikNBSTeLjDva3jKxCP3TluNEKXADuL8DDvdS4vgC3IZQk1wgEdRFr3i2gm28bv0cRq1YjIhtClIwec1LGCsSsCHqlCIayVuCRhn05YJ4jnf3JP29TFNsCr2gv6jLm6CFmn+flsA+frnbgiXAvqKeqxtnHjIK7IToC5q9IB617PF0ssCvtZxadAFSjJW+o8hayQ0rAYLnpXIUkDb0IKfJeaDWSB4pATRvSGA== 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=2JzJN8TyWIHMhYc0+grtQD1CJ/mPU4UlaZgyQr2oi9E=; b=ChSdO1HN5M1TzFSq6cf/ZxdXGYmJ2nKtmUhrupVfM3FMWaUVqMqOIW4laU/MCKhpembUQ9tdBMe29Mqh3IekCPps2QXjV/L7tOW3o+m8fmS6vRHAVlJD7CMj+gc7rZsBXkwfauMGQ203ouOEgsSl4eVp/JZb4LzqwfdOrWtP+EOAvYsb478GB+kkyGMQCAEthLW7YJbqsVGf9hBvvyqKK+7bJBXc5/ABfWywPH+pKe6EL+62Tey9UHwuXXAabhlFWAC1n379qEX2GFHi3MiW4bvcK9L5RbFiuKFxidZ+IaXL2DNIjhCs2AupUzbWw6BtQQLv0zapj5aD9/WtXZ1a/g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=vivo.com; dmarc=pass action=none header.from=vivo.com; dkim=pass header.d=vivo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vivo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2JzJN8TyWIHMhYc0+grtQD1CJ/mPU4UlaZgyQr2oi9E=; b=SxMKloxUpiv2SYgbub2vHTp7BLjiNffYlI1GyBzXBzjNOijjT6mqd3x721u32oJaNfHpo1JZ1RQN37BTNQ27V45KlyNWGgTC1FLHEPLEs2Bp2MZzEO/UuNiKoDgFZ2nDiP8EOA9NUj9JKE+ZnxBe6ElqJN/mtmLz0BiQAWAwCmWYAidDUqFA5pikNCa45ObnCGViLF+jlrFTqhAtwB6mXD8cnQT4e8KCXzZt56q63a6vitGqrQ7V0mWtGJxWzVXX71gG8yKCinzICYDJMjsqymcnhVIPB6LUsnZdU1tYH3hHBH0qna/WeBtzsRZULWjQgjUPMDJr3erzZ+IAJ0KYow== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; Received: from SI2PR06MB5140.apcprd06.prod.outlook.com (2603:1096:4:1af::9) by KL1PR06MB7035.apcprd06.prod.outlook.com (2603:1096:820:11b::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.16; Thu, 28 Aug 2025 07:54:26 +0000 Received: from SI2PR06MB5140.apcprd06.prod.outlook.com ([fe80::468a:88be:bec:666]) by SI2PR06MB5140.apcprd06.prod.outlook.com ([fe80::468a:88be:bec:666%5]) with mapi id 15.20.9073.010; Thu, 28 Aug 2025 07:54:26 +0000 From: Qianfeng Rong To: Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Venkata Prasad Potturu , Vijendar Mukunda , Charles Keepax , Mario Limonciello , Peter Zijlstra , Qianfeng Rong , linux-sound@vger.kernel.org (open list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM...), linux-kernel@vger.kernel.org (open list) Subject: [PATCH 1/3] ASoC: amd: use int type to store negative error codes Date: Thu, 28 Aug 2025 15:53:57 +0800 Message-Id: <20250828075406.386208-2-rongqianfeng@vivo.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250828075406.386208-1-rongqianfeng@vivo.com> References: <20250828075406.386208-1-rongqianfeng@vivo.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: TY4P301CA0019.JPNP301.PROD.OUTLOOK.COM (2603:1096:405:2b1::13) To SI2PR06MB5140.apcprd06.prod.outlook.com (2603:1096:4:1af::9) 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: SI2PR06MB5140:EE_|KL1PR06MB7035:EE_ X-MS-Office365-Filtering-Correlation-Id: 573a232a-b4c2-4c08-b00f-08dde6081c2a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|52116014|7416014|376014|38350700014|921020; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?Ci9jvH74zncq47/vWqGQEZdA+Sl7i4tZBScvMGfuo2nYzGjXTGIdjFkD7i7Y?= =?us-ascii?Q?AUFVAC6z7/Y96yEH+0h5j6zPk+MnX0lVaKoH2I8SHSmPehcNhqjTRFnnyfWW?= =?us-ascii?Q?eAEHapmtL3y92ObaPh6wW2qW9+xibr5G9RPR/hmBazM8FYypDXUlD4jEUMJh?= =?us-ascii?Q?/ZcsiH4dUyMScyns3Sg1gYcx9wn6ZFzmAXLPjzThJTyKdQ3JMU9YGlrk/l76?= =?us-ascii?Q?WEj1Oh6ZOFH21ayKsODwSo6zJZF/9yJGlJ7dp3Y8ucF7VxXdXKccYvqi0EZU?= =?us-ascii?Q?xbd4x6v+0mBMkz1kyxJ7ipMys+6EvykkUWtbM9MYuKFhp9bcq4DmTHqajNZD?= =?us-ascii?Q?eWnOxuoz8bMoQmKiHO9thBw6JH3mWwv4SqKj/zppuH2OTSXFUqmbDrewabtA?= =?us-ascii?Q?MP4OWQzlIDGeCn3tuCPnp4CPSp4qwBtq8Qj91jSIobOXnYRap9RF5VghD/nw?= =?us-ascii?Q?sEyBbObB4r9ycTqEcEz8GKgrQF3jSkzjoy9+Me4pOjoTPtwn2NZU4RZKWqV7?= =?us-ascii?Q?8jJxxlPkurCDu5LbBnfQjQvNRVoKRdwJpa7rDjmXElzRe6+FGWrLkQmL9Y6U?= =?us-ascii?Q?+8tlMaY7ob77UHefaASD4TggZWitD8kawwM/YOX2qPtBYAQvmM8KeQ8sbLYW?= =?us-ascii?Q?nWj9SyrhK19M2GLaCr/34ia8nEPone1yd+6GunxyC0kG6phkTEqI2yp+Be+A?= =?us-ascii?Q?YsK5gMMuodh62fWiljDgEUreGjsgOuXdW6zpCSaIv5Fxl4uoiTBvRat+aeSK?= =?us-ascii?Q?oyrpcED8zq4G4rgGFwmCgKRoDt4Afk3zYETnioG5kfMr8LO4MzNjDw2SAXS6?= =?us-ascii?Q?jFwjtHcSB6NFvKwlnCS3TV+ALd3jtJfSi7UR9EL9ntn1eZuVB7V9/RKqfnOD?= =?us-ascii?Q?WLVNfKtfKq8WQCBxU9q8B1bnPNalm7vc37zDAIdp/9MIhSsn3F5Kb4scqhVv?= =?us-ascii?Q?VohfKStJLCsXr9BwQJNBJ27x3bc1d1CoNS8R40tzl+zAIHf+EHLGuRxsGWiP?= =?us-ascii?Q?JYUu5uGFyAj8iG12WvNSzrgPBdadPScPKxkCecVJBZBa2WFANsv6w1chSD/v?= =?us-ascii?Q?wtgEOg+lo8p96ny6Fm2Ro9NM0ZKCt1meX/D5n+AF0O3nw+ZimenxDPsulVw8?= =?us-ascii?Q?GoIeRitxQeUbk/RY+Qhk2aQW/z/daIiJu9DxN3nqKjX8u0XmO3RP2MFQMWKo?= =?us-ascii?Q?rf3NQPBvBtCLZ0WS+peFrys5IEN0D8cWloju+zA1ovG+fqWsUlrkKth5vg7j?= =?us-ascii?Q?32xwVKI1OA5i0pFKEVMhvWnhct39J7DYeKQhgErNh1E23+n7K0DY9iUH0DVN?= =?us-ascii?Q?eX3PePEd8Mj4jIRF/k2nVCHgmmdk+GkWr8OH5oY1mM8LwKGt9GwUA3B2QLPS?= =?us-ascii?Q?mjhAiw1vPoN1ZisZiaBikYxz22+jFIPrkeRfW3bJMaSzZEIBSEcYdoavfcab?= =?us-ascii?Q?75XttNRzDmtb5UJryvDMa0tnnpvaQBW/ofuI0o/brQCpfPfZSqKA8McEqalE?= =?us-ascii?Q?aqeTiE6B1sQSNAI=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SI2PR06MB5140.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(52116014)(7416014)(376014)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ViQJp06GtOCu+YCGComEA7YoJ2nw4b1ycGfGnO3kXpXirjQBcz0vgxaz1EeP?= =?us-ascii?Q?n81eWiOaEmm5HzagKyT3Ssw0nXzBZd6spqXf8zZrO9LgtzEReovohuDzbkjM?= =?us-ascii?Q?JuUPNJbywpxBfSqKHXBpHidWFwR6eu9rSFRRgEG/7bARCkYTGUgNisnlNEgW?= =?us-ascii?Q?UJlwr+2tUJyC0JINBwQ2R4qTBwA5weLJsSFSXrpgIif4iOT9U4KJX5QbO2C0?= =?us-ascii?Q?d9Vhp4zieZb9WwYad8weE61irhAeDQ9s+UhTXEzkacOMU3s8TjUsSru6gGAG?= =?us-ascii?Q?f9wlLdutdKkB1J/iGtpfJg10kgK0P7yBuYTItLfwmZIkG8QqN362JW3nfi/p?= =?us-ascii?Q?9oCh+POklXt0kM/PWAgzmaO3UoWyTytw2b6cwf19PIfwjQHIeEncz2ytGgij?= =?us-ascii?Q?sDUKxbqoPDumB1cNGdK0LuX+FF5CIeg9fb+sxMarlp/QiGUq2KTgQ7FKTahX?= =?us-ascii?Q?Xiswdoj3i0PM1VNPjYRVFhx99Hx1pehgYMjUNb1Nmzlg2jeJv99TT1wmV2QD?= =?us-ascii?Q?5Z9iSexRUid0ozbQpDzdS54YXFC2cETBufrwZHTODyQ+c/gzzOrqWB39tmm0?= =?us-ascii?Q?XNfeI9mulOsDRjPR183e7K+dAnwwP74rQiSfkVkjTsTefRp7fUNaZ1s3l1v3?= =?us-ascii?Q?Qwa0Jjfq3UnOwsgQeSoIr/xthtD18z83VD8q69hBLF1Hi+tnOTSIYo0N3fX7?= =?us-ascii?Q?qloNLQlEgpLC7Xub8PA/vXzaIwjiGkE6jF2WO0GJKmtqsq03NMKnYFsygv/A?= =?us-ascii?Q?E/U7Ixitj/GpI15v2ysZvoRiIKn/XtxUZE9yz8YwSHhehOTB8VSAjx/B7rP3?= =?us-ascii?Q?KL29WqMMoG8qeiqreAgxReP+5wIicOy1eRcC+jtV7zb5zbduuFPvhGlL4kbv?= =?us-ascii?Q?/7LQjc76YsoudMjrlwTHLT8V1TO1PC0iginOk4XvvurasioQJC3UlrpkkGmK?= =?us-ascii?Q?5U3YZe0oVblY8QRT6OLPNhuBSM7P6Ifd4N20jTGi+T5NIQh5Y6tbhrJJkjuW?= =?us-ascii?Q?TquKYqDkcnsKYM+4FhA6KobdhrDsgYiDixQqhY5FE7hc64qpqBK5W97PhnrX?= =?us-ascii?Q?EP7iSmyalNbC6s288gkCjViiWFh7l8Nm3uI8e0sA8d+xx2ahpLBIrArv3F88?= =?us-ascii?Q?aUwKQJFLAVwhNGPJRkoUh+HS+yZ+/B7ecHryPiQyAoxMou34okxLeKvjhUkV?= =?us-ascii?Q?P8Sl51EJyIZ5vCDKsPCgp/0KjLdl+QtciF2NQKtXEClvxn7WVMzXvAtb71iG?= =?us-ascii?Q?Yezy8Jq/myS/rEDDP8Ac4kQSrI9TSzsssrGd1pCoKfCUbmzfQG4RcUf6aZak?= =?us-ascii?Q?SstOdZqXdhWj3hEc/J+O4cL8wY6ZW3daciO56PfW1g1kLYW0Prh+bn/lpwm+?= =?us-ascii?Q?VJCuDKB4/++blJwsqgdoD8ptQbTkEA7FG1DsNukeiJL/Fv36TicLBCGRTMF3?= =?us-ascii?Q?hWjw+wPt3eOicjFJs5WZ4hwG3GXX1RSdvtshlmWJwT9MiZgnj2n7EoKp5BJs?= =?us-ascii?Q?+ls6Xj3QYFWAxRqndUkHzo8oRTBVz0TOkkkdgWKyAJyDDPbdPgu0lNmrb/6A?= =?us-ascii?Q?3mvcEGo7NpEF1z1POchd+Y6Uq+IHuq9ZA8Qmu8jW?= X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 573a232a-b4c2-4c08-b00f-08dde6081c2a X-MS-Exchange-CrossTenant-AuthSource: SI2PR06MB5140.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Aug 2025 07:54:26.6297 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: oMKs0nbVmZLjLDEv+HkMm6qWCQWQalxS6zKifpsybJmwCGykXl+GJJmYXN4wUdRMhi8IqK7gsnDRq6w0EZLRMQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1PR06MB7035 Content-Type: text/plain; charset="utf-8" Change the 'ret' variable from u32 to int to store negative error codes directly or returned by other functions. Storing the negative error codes in unsigned type, doesn't cause an issue at runtime but it's ugly as pants. Additionally, assigning negative error codes to unsigned type may trigger a GCC warning when the -Wsign-conversion flag is enabled. No effect on runtime. Signed-off-by: Qianfeng Rong Reviewed-by: Mario Limonciello --- sound/soc/amd/acp/acp-rembrandt.c | 2 +- sound/soc/amd/raven/acp3x-i2s.c | 3 ++- sound/soc/amd/vangogh/acp5x-i2s.c | 3 ++- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/sound/soc/amd/acp/acp-rembrandt.c b/sound/soc/amd/acp/acp-remb= randt.c index aeffd24710e7..7e9c07488dcc 100644 --- a/sound/soc/amd/acp/acp-rembrandt.c +++ b/sound/soc/amd/acp/acp-rembrandt.c @@ -147,7 +147,7 @@ static int rembrandt_audio_probe(struct platform_device= *pdev) { struct device *dev =3D &pdev->dev; struct acp_chip_info *chip; - u32 ret; + int ret; =20 chip =3D dev_get_platdata(&pdev->dev); if (!chip || !chip->base) { diff --git a/sound/soc/amd/raven/acp3x-i2s.c b/sound/soc/amd/raven/acp3x-i2= s.c index e7f2a05e802c..352485dd98b1 100644 --- a/sound/soc/amd/raven/acp3x-i2s.c +++ b/sound/soc/amd/raven/acp3x-i2s.c @@ -149,8 +149,9 @@ static int acp3x_i2s_trigger(struct snd_pcm_substream *= substream, int cmd, struct snd_soc_dai *dai) { struct i2s_stream_instance *rtd; - u32 ret, val, period_bytes, reg_val, ier_val, water_val; + u32 val, period_bytes, reg_val, ier_val, water_val; u32 buf_size, buf_reg; + int ret; =20 rtd =3D substream->runtime->private_data; period_bytes =3D frames_to_bytes(substream->runtime, diff --git a/sound/soc/amd/vangogh/acp5x-i2s.c b/sound/soc/amd/vangogh/acp5= x-i2s.c index 7dbe33f4b867..bf719f628617 100644 --- a/sound/soc/amd/vangogh/acp5x-i2s.c +++ b/sound/soc/amd/vangogh/acp5x-i2s.c @@ -234,8 +234,9 @@ static int acp5x_i2s_trigger(struct snd_pcm_substream *= substream, { struct i2s_stream_instance *rtd; struct i2s_dev_data *adata; - u32 ret, val, period_bytes, reg_val, ier_val, water_val; + u32 val, period_bytes, reg_val, ier_val, water_val; u32 buf_size, buf_reg; + int ret; =20 adata =3D snd_soc_dai_get_drvdata(dai); rtd =3D substream->runtime->private_data; --=20 2.34.1 From nobody Fri Oct 3 16:45:44 2025 Received: from TYPPR03CU001.outbound.protection.outlook.com (mail-japaneastazon11012048.outbound.protection.outlook.com [52.101.126.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 3E9472D73A7; Thu, 28 Aug 2025 07:54:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.126.48 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756367675; cv=fail; b=CKxAYsAj2a8noIeTGu6wxuCJHecmqZKtW2HdmRPP6PBuT0LU1VBl23xlf/mxWhKcAO7RMm8kCQ+faggynQirQ8z6voMVgifjfT9k0UQrO5UF2tmM19CNMQJ+nmkVq8IlHegxyrO2Ty5CZYtf5jr94YtdWSWouO0hSVZqGOXGxaY= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756367675; c=relaxed/simple; bh=ewC5Es/7muztZJ7dtHgugSUzuPkJkir92wv48mvyn54=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=OewNIENg6VOV2gH18f243kCEABz+A9Fvw7ZVmVeZaWlpCM0IN9zJBNhDYEVvVztyMkRQJOV3fXDNOpyj9xue+1KxX1IZ+yLqgnsbovzoCWTAREY8Zmi5KZ1uvGJrqiXyWukpqbHaJlo0KlK/e6UIheoMdBPfpq9OzYyjq7dZ0FE= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com; spf=pass smtp.mailfrom=vivo.com; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b=qj+usrFO; arc=fail smtp.client-ip=52.101.126.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=vivo.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b="qj+usrFO" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=jh28s7cWHjuBZUyva/fZCEHRDpqOJO0atpgZkJzboZr7R9jaBQKE+Ry5JQPjFmYXlD53H74FIYQJhUBgX9Mzsl7M4EMJveHTlrqWywXRESYkw3gjiGm7fGOZ26OU1eOtivrxH0Fshj4TkVhStjOUHNSaABuvdzK/ITToEQTGzHuMpTIgUzHBhCggm622PxyH2xwxLhG20nmZRyZ3TVZlBoH8bsrY2hkqMAg6ynw2aQOYVSiUuPh1LGVlR3lNBaQFVaUhSMpI6iJeyIntfHEf1rCyslevkU7JFIvvXgUvLN3I61BznwGgek0wG6nXa59jjcwJLPOIVg6QrQ8yiJXxfQ== 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=KBjY9hzMQs0asCXjfKcALrhLv/t/UyJLsTBDWAFjUS0=; b=ADTdJy+xSJXy2ph3DhNVhFyx61EnYGE6y/NYFdtvZx9I3jelo2Cjme4CL2l6uskAl89FESBc9evFAHPtojh6d/NrWCyPNrq8VvNOrFCn/lntSjW38PWoq8ZGRCqcIAzL/y9wm35cb2nazspZqZo5ufphL+RSReM1eb+ijkf5TxAo+0NrhMomNV4K1a0AX6QtMhjBOWbOr8xSsGjdN6PLKyt6b/X+faIOYAv2nwhtw3LqToeKMypUy1i42Vd846362R3n+mSdNNgZ9NNe6OOGdMZZiZwueLyn0GQiHRIW21yCRHXvLDqt3Zeyh/Frv9rbDG5QUviOm5/IPkbx/cPHTA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=vivo.com; dmarc=pass action=none header.from=vivo.com; dkim=pass header.d=vivo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vivo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=KBjY9hzMQs0asCXjfKcALrhLv/t/UyJLsTBDWAFjUS0=; b=qj+usrFOs7fM7Hx/QKA2W9vbmQ+CKB+FTDJ22UENVL5lqJMaxEuB3GBXXdROwI62lvRagCgYbeFJovks7jrop4JaV5z1d1HOz5kD49CXNmGOoqxGavcR105XAWwSnUluJrQ9XIcceGl9YSQ+BAW94bDW3sXJYifHxbhdPyPy0CuPsgcD6ibXK2xcfF+sAKt+43aENgeZY8k92PGHu2bofNwX2w6MT38qDVbIOP8SuNEoYczLZuIEetMPt2eVjKzXOyd2hXAKItUrfZCBfkpNDwwS9Po7UScusa069Lx05WUF0mQKZI6bhRbA6NjpN9i0f5bkghxUz2VKdDNnRAFtZA== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; Received: from SI2PR06MB5140.apcprd06.prod.outlook.com (2603:1096:4:1af::9) by KL1PR06MB7035.apcprd06.prod.outlook.com (2603:1096:820:11b::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.16; Thu, 28 Aug 2025 07:54:31 +0000 Received: from SI2PR06MB5140.apcprd06.prod.outlook.com ([fe80::468a:88be:bec:666]) by SI2PR06MB5140.apcprd06.prod.outlook.com ([fe80::468a:88be:bec:666%5]) with mapi id 15.20.9073.010; Thu, 28 Aug 2025 07:54:31 +0000 From: Qianfeng Rong To: Lars-Peter Clausen , =?UTF-8?q?Nuno=20S=C3=A1?= , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , linux-sound@vger.kernel.org (open list:ANALOG DEVICES INC ASOC CODEC DRIVERS), linux-kernel@vger.kernel.org (open list) Cc: Qianfeng Rong Subject: [PATCH 2/3] ASoC: adau1977: use int type to store negative error codes Date: Thu, 28 Aug 2025 15:53:58 +0800 Message-Id: <20250828075406.386208-3-rongqianfeng@vivo.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250828075406.386208-1-rongqianfeng@vivo.com> References: <20250828075406.386208-1-rongqianfeng@vivo.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: TY4P301CA0019.JPNP301.PROD.OUTLOOK.COM (2603:1096:405:2b1::13) To SI2PR06MB5140.apcprd06.prod.outlook.com (2603:1096:4:1af::9) 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: SI2PR06MB5140:EE_|KL1PR06MB7035:EE_ X-MS-Office365-Filtering-Correlation-Id: 5252cbca-c0cf-4e49-8dd4-08dde6081ee1 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|52116014|376014|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?+YRFWJJZyO/2Q4Iw+QzfzSlbHMA4m2rvXb7eIkUr9whRwW4GYfe8FhdpOXR3?= =?us-ascii?Q?2x06+huTKlCdGRSfkj0BVUSptrcmV32+o/w8nEIPMRHQdI2gpWApSdERzlRE?= =?us-ascii?Q?9wwzAHsGNqFQUfGtszlR9Ib231aIlGI+S0+je1nQjdeSIva1hJ2PUrLR3AjO?= =?us-ascii?Q?CNg8jDcjsNGnDT+WitM3k0FEu6kXUbWgzVDmvIbstRZxkS1zQHwgyzW+RCgR?= =?us-ascii?Q?a7lbCkhBfBDNN0m3srXz60Un+K+oZnXd8z/r0UmuHkxhn2O0Wto3UFy4Jo1X?= =?us-ascii?Q?GVqPnXIGXcb7tKXfM+PbsrXTSLDAZ6B20lrpT0tL2sxo7aOCmCNjQ/s/AaYC?= =?us-ascii?Q?2bVuFUlTa4ZgV4X3tPRsbTUtvl8q3Dhm+a/os6auwLDfIVO7T3KV8+wAISca?= =?us-ascii?Q?7rFqSy8kGsCc62JA+YvmKwcGGawKE9iaTeociCcYjU4vKL2C0w+V/0eLOT/S?= =?us-ascii?Q?UIblTkysna314Yo1pjoMNQbgx0m5bR8fOVaOU2/yMLYwA1xQ5OXxho6jrxQu?= =?us-ascii?Q?nZnlgt1SS1Sfcvgs48EC2Q7HMCU2P66/LZ1JFeodjHlh1a45MYvTKkbIG69t?= =?us-ascii?Q?o2F9sRlNhWKX7yek7krlnLDlDbr2aThNgO5EZ7PnW3iI+QjlN0DNPUw+mC/e?= =?us-ascii?Q?weUMEYvS5UoCe3QpCyuOBxZ+6m2byk/XEKniKZ1/hwmx7IQwMtFPp/wlBfDd?= =?us-ascii?Q?jvuAZmgtsyQo5lc18Kf3cYDsCB16m1hIcxehfVaDlI/EDEtcuAE8JFbUG1v3?= =?us-ascii?Q?P9piT7jzEozR2Yp0z25VYI9WZJDjpjE3YE5SMeh9djDRALB6ysJYZvvmMZ1f?= =?us-ascii?Q?1ECWo9NyvEJCUnl9fcO9UZACxha4mIdZRXcPY+rFtPRu0izH3gUrtY4eIkOC?= =?us-ascii?Q?Tf4bXGv8t1qIczb72aSaSiYgA3sS2gdDiUysK8lgHtM0Uk9RGOx3fRQ9xNIx?= =?us-ascii?Q?VnSAYM+LAbF/4yiMqFizIr1MkqgIYTVrX21Nmuv1D05QQaKb2z5fzNHdyflg?= =?us-ascii?Q?McUA75KZVV4Mccl/rUJWQJ7lIgaPTjvK5UlJ/NAeK3a1COGvedDTfkiG6p+2?= =?us-ascii?Q?qP8jAEAiroFrNkTHHJtKs46bufmXOBx3RJTMXacb8ErzUUeMNgv38vPBLo3d?= =?us-ascii?Q?DA1ST3HY+oy/v7KT2OsVSJIHZWppVvzwyhHdZeb/yR7idPyMdOLYCCEJ0MZh?= =?us-ascii?Q?8VWIVPubZeIIfnvYP8QjVp2ACvB9ULwJ/ts5Vk7D6qMqg5RHIUIzs1msCjjG?= =?us-ascii?Q?So2vM8E3tCmg6OAb19+Qn1euAsuHQxfKM8Es9t4lwofNNq3nscnC6f5EzCDa?= =?us-ascii?Q?moTv+H6JA5YrLRUqdn/Gd0CXPX7uBluMo3z916HF6PMxGoaOeY9mu97xPihe?= =?us-ascii?Q?8VbdOqpuuT46HtbAC87QZwQrwaPdJdO6yyge6KBJvSdcyPdJ0w05car4A2Z8?= =?us-ascii?Q?WwbNaHIg2QhdCeKqyLX11mD7ggnBdYoUnbF/4qHj0F7bxcrWXI1ukw=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SI2PR06MB5140.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(52116014)(376014)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ZpLiuW00f0kNm7wF3j80B/QUdWZ9PGqyQBaWR8K9t5oluIuJvVf/bIGg0EXJ?= =?us-ascii?Q?iPrhb77u0RcjFuY6ya3qnYuaf43g3dtuhEtrDDcw9+2qq9KQt/YNJK21+Xkl?= =?us-ascii?Q?Ag42qni/7oqZ2paE+hT/dMQqqlg+3lfb9nmODfMucjhk/8qD3WJmcGe+8Zo8?= =?us-ascii?Q?WExlt4hfWKevbWVwzD1FeX3ATvDzHK3lYYiuyCCVn1aL1nDuO8XNVjLXm26e?= =?us-ascii?Q?F7pjwwTHi0s9a0GtJeFz2rdDru9NrdbhzCy+3hvbULmWckSBpOLNulleYgWA?= =?us-ascii?Q?UaWctG9p/thjZyXd42AI6qDom2EiIqg68WKywKV+05BvGRfaiSVRJXJmb32d?= =?us-ascii?Q?wb9mttfhDzLYL45KLnLbMLwf8N0UiHtBnmeDPZckfSHwB1biIMnnBZRHnys4?= =?us-ascii?Q?8UF0a/Yx5Y3o93/UEqeyB3qcDpHEn4cu7zozm6x2aErf0jE9p5kDgZuMneoB?= =?us-ascii?Q?rsothz5L6vfF6ADbYwmPVd1ZqYr4odQqv0KE6bFTbhPgzP6+I112y46p0AB3?= =?us-ascii?Q?LFJUV0sPpsm4p2Eh4QczZLE2RDdBT6nP5ugc/KUpNB8VYIRf7xB9WtBqPP8T?= =?us-ascii?Q?l2YEm2TjqBg6EF4PSBPCtTX1UwEKOZnJt3Dt02wdHj0qe+xTz9bJhbaZgfJY?= =?us-ascii?Q?BBh4JsoXrdcLzQ24Ia8aCHi85++l4j9Rz47kAknteijfKswkhcdK7PqsExsB?= =?us-ascii?Q?kZRWh4jOZjoirwRyqx4ToxnwEmO9olZIuRLZ0mXpIABW88qPQnQruImyYvOn?= =?us-ascii?Q?AtZWqf/EW0kokQLsoOLaUFgO3gUoxcHaiABcBeb4AnI0fbJbLbbvwZXYdv3D?= =?us-ascii?Q?hLA3xsY79+r26TBNViPi/+zsgyQnjD+E9hM+iEGMjZPRjSw+xC+fk40aUUzB?= =?us-ascii?Q?3aKU8SLVzmqzcHybSdhM2+ozASvZqawyC/C/c8oRjMfc1dVYBTCoXkFdxFpQ?= =?us-ascii?Q?7orBy/o6hf7riH7mr2FNmDk/JhZRWPrg6qXI9YFRfYQChUiLQgk5I37+1JhL?= =?us-ascii?Q?7yuKPp3ARRd3zxbiBMEGFQPJom09CoOqkAWsMhjM2BsJrDPH7uHHSUcHMS69?= =?us-ascii?Q?ZsXKF8T3OSimeHmoD1EBJWGjIe2+NerhdqyTQYVRzdJyYmAtz6Quz2rc2A+K?= =?us-ascii?Q?GUqGroBdI6s2M2ZKF9VpKqr2jDtVJulkp5/2O4XawBimh2HWib5ZA99IN8SZ?= =?us-ascii?Q?2jWFRPdEYkHsve2pe074sTQV5nhE0OITX1pb2+DnndSAioR79/BopScylq6K?= =?us-ascii?Q?zsapkUvEuOrFHKlbZxIyzhkKc1dKPWkVbX1RM3AGlfmaKtd+kD7j1tluvEdO?= =?us-ascii?Q?W720PtM3zLM+XQrBBytOzG8LweoruMNRpXMz58ee+zhe7SRZuVqcmdgRErM+?= =?us-ascii?Q?Ao98DgqX8Dx2xQK8KAYnsz8emu1o32d4B5e7j1xc/ntzC7+oCGjelKabdVAP?= =?us-ascii?Q?568y/rdpxqq7x/7JY/ijFmwxRirN9+K3PW/sv9n+G1Wn9lOh6WwyqRcBJ8gR?= =?us-ascii?Q?X/kBiSibjrLeiRa1fndxrrSXbgnYlgouA0ckp0JOxZUsfTNUJ8QbbonG+UQi?= =?us-ascii?Q?Rfoy3kpDINlYUBYDG1JJzMgmlreTEEHBmwX+TtXJ?= X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5252cbca-c0cf-4e49-8dd4-08dde6081ee1 X-MS-Exchange-CrossTenant-AuthSource: SI2PR06MB5140.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Aug 2025 07:54:31.1227 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: CCxVyoRk2pRX+bJaPod0Dm1ThNGbVnVUkQXFjAt8y6mMoE8WxLJqbavzvc/RhRM+ofe3bL26trl34iKuHcjIyw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1PR06MB7035 Content-Type: text/plain; charset="utf-8" Change the 'ret' variable from unsigned int to int to store negative error codes or zero returned by regmap_update_bits(). Storing the negative error codes in unsigned type, doesn't cause an issue at runtime but it's ugly as pants. Additionally, assigning negative error codes to unsigned type may trigger a GCC warning when the -Wsign-conversion flag is enabled. No effect on runtime. Signed-off-by: Qianfeng Rong Reviewed-by: Nuno S=C3=A1 --- sound/soc/codecs/adau1977.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/adau1977.c b/sound/soc/codecs/adau1977.c index ae59efb38f26..c193a9f22f59 100644 --- a/sound/soc/codecs/adau1977.c +++ b/sound/soc/codecs/adau1977.c @@ -795,7 +795,7 @@ static int adau1977_set_sysclk(struct snd_soc_component= *component, struct adau1977 *adau1977 =3D snd_soc_component_get_drvdata(component); unsigned int mask =3D 0; unsigned int clk_src; - unsigned int ret; + int ret; =20 if (dir !=3D SND_SOC_CLOCK_IN) return -EINVAL; --=20 2.34.1 From nobody Fri Oct 3 16:45:44 2025 Received: from TYPPR03CU001.outbound.protection.outlook.com (mail-japaneastazon11012048.outbound.protection.outlook.com [52.101.126.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 1BC332D7D2A; Thu, 28 Aug 2025 07:54:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=52.101.126.48 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756367677; cv=fail; b=Z/75Pq+R4I0I5ksb0Tl/oT76dhtOOB2QB96jJOGB52jN5S1scFoawTt8xVY1DBctWPLzRGcyS9gnTXV9j6DC241knXifYzUyV6XvpiGUJAF6XI0ufnRM8kWpTogoMDqOk8PwvU0N77QoSUM0dCTK+hpgfgJbIyIjjYTdd4vUooY= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756367677; c=relaxed/simple; bh=6Z9gTnMQFVKSiUjys6UZxcO3oh61l8t9zs/LHDoSofQ=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: Content-Type:MIME-Version; b=Pn9qtljL6J6kbrqPbiyqY8TuYYfxKCuCZEb51gUywcOnz/kQ+uWhasdD1Fv13hY4NptcyEwiQa4XxZs44iFBYO2NoYsu6rwfzu6xCgKf2rirL4FfOwkS15pDR/YLrEV9gFxD/EFzdjw2ll4Wn6+iW15YF+qbxPZvACs5ka3rZlU= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com; spf=pass smtp.mailfrom=vivo.com; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b=M9iY5eWB; arc=fail smtp.client-ip=52.101.126.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=vivo.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b="M9iY5eWB" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=FLPkexem840dfSYZs7ZrRYsWsOF5+ZA8ke7XD+P0Hx9kLh0A5rV1vvUv9TPnVYV2kwi7gtLlF7d+eorjVsyvoVKhXibwmkW/IGREQcP3X4BlI/SPg+3H3GZz2uNyEUpnSx7vnGGrayfXJT9zjcRcftg1ynQ/MO/w2DN2icZhwQukuDgPDzbBs9RNk7nVmeztBHUK2ebAgyjYc8yavn62AmHtM9y0L3GFolPO0OV8lmO4YCCSs70G2O2eiBcwyzv2o4VZo8390w73Ms79GHcnYPoT6E2HGKKpVIEgF1fuigvo8qG4ihR1Xxb4yjlRiJWViyBIxCNWA6rPdghAJfibwA== 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=+exmfFy50U66Vpx636GetyUys/+/8Ndzm5Raf+BPJGs=; b=CFNo6cCnTmBqaWdODc3j0PC8CZ44iAsNHB+aEWGVzkbNM5BjZDgyvmGWE7GIfFJYJMC4aQIZr738dc7E3LVkyPJumYm+BldhFavU9UjVY6S6q8KCQIA+zMrbn6cCl+VpETbn57bRzHsHnYGG3qND3KWJ3YcUnEVUoyw2oczlW1dpArxVVVzun60kLDvgwUvlJMcBsfLjZ9wp5jqazLEljRPztAbr6rjgrCgmBazoHUZna/YM+/nw6+G+ZKH75o/r1+joTspuGE/a6ym4vgXvsmDUhP4A83R07nAqN6lB0uUK1HDAyYaGsTXM51hnivlnud7khpH2+XtoviykZooR5A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=vivo.com; dmarc=pass action=none header.from=vivo.com; dkim=pass header.d=vivo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vivo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+exmfFy50U66Vpx636GetyUys/+/8Ndzm5Raf+BPJGs=; b=M9iY5eWB6Q958JIOYDfzMcVXp8qto88Joo/wLvLzEIGUhAn5gdGAjgpNdZBHEEP71+ySdotpYIUi7EZBo4Rl2Sdhu16oftRpwN0af/wSMFj663IQ/KWyUl6tFFlhbrC3hrl5sPI6wyd3nHn+eZyzgjkCqgAW7/DW/GWHO8mKoGNXDKeHpagtTmE98MoQiBZix0a/bbDhSLjQQbzudfzbZ9HlBiHPvxf1bjQL4K5uHYLWBcTbdWhr+8X9AuUd2YypliVA9Ar/7PurQoRQoDc3a1X6uOVMoEwS8i3eRcBowwnVoJsjs8ga9eHqeZKXrdQh8egn0m1zNc1S+exidOvV/w== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; Received: from SI2PR06MB5140.apcprd06.prod.outlook.com (2603:1096:4:1af::9) by KL1PR06MB7035.apcprd06.prod.outlook.com (2603:1096:820:11b::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9073.16; Thu, 28 Aug 2025 07:54:34 +0000 Received: from SI2PR06MB5140.apcprd06.prod.outlook.com ([fe80::468a:88be:bec:666]) by SI2PR06MB5140.apcprd06.prod.outlook.com ([fe80::468a:88be:bec:666%5]) with mapi id 15.20.9073.010; Thu, 28 Aug 2025 07:54:34 +0000 From: Qianfeng Rong To: Srinivas Kandagatla , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , linux-sound@vger.kernel.org (open list:QCOM AUDIO (ASoC) DRIVERS), linux-arm-msm@vger.kernel.org (open list:QCOM AUDIO (ASoC) DRIVERS), linux-kernel@vger.kernel.org (open list) Cc: Qianfeng Rong Subject: [PATCH 3/3] ASoC: qcom: use int type to store negative error codes Date: Thu, 28 Aug 2025 15:53:59 +0800 Message-Id: <20250828075406.386208-4-rongqianfeng@vivo.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250828075406.386208-1-rongqianfeng@vivo.com> References: <20250828075406.386208-1-rongqianfeng@vivo.com> Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: TY4P301CA0019.JPNP301.PROD.OUTLOOK.COM (2603:1096:405:2b1::13) To SI2PR06MB5140.apcprd06.prod.outlook.com (2603:1096:4:1af::9) 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: SI2PR06MB5140:EE_|KL1PR06MB7035:EE_ X-MS-Office365-Filtering-Correlation-Id: ce1d5d7f-4ae9-4dd0-cde0-08dde60820ec X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|366016|1800799024|52116014|376014|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?wrjNAP3hIRz/GeA5P7aR22qnlGTXzQgNbz1/ZjYnH0w2PUx7SRdW0VtV2Hy+?= =?us-ascii?Q?tfjQnhftjGbZzwubOQfUl5hxcWF6SLjxE/M1CcihvZmzzFugMyJnBjlYb2Ko?= =?us-ascii?Q?KHbKzFOuyOREsb4kmmzajdRgG1NJKhnD+A0pdjnCi2mN+c44/TXNVtyC26hX?= =?us-ascii?Q?6Tg4MG/J+GdYXO8GCunwFlzzU5385qmYlRLFm8hy4aFYhxHrkJN7CmsjHp44?= =?us-ascii?Q?U/h1lkuJ/Bdx8Lh1wjbCm2z4qndkZIX2D68Ac5HlDTn55K6hzjj4uQUD/6LJ?= =?us-ascii?Q?aBjKx7i7XZEpRECyjxOaDqg9WIc2EM8GwoTQroq9bx3+3gla3zt0Y1BlDk75?= =?us-ascii?Q?ZTdgMYDFMAa7fjVexlopZqcemovOayPuKTFExZmsuFOjLSDH8vp/7LvW6Eks?= =?us-ascii?Q?qEqmvdJYhEaVbSoKbFRPbiVB8tUT1bjOHVr9eDTcSMrUCTNokl9RLl63kWS0?= =?us-ascii?Q?U3soSYKe5jm0oWrwqkCLwdiv4+u1RotOpkTJxwxfNcLmE0idTaFDb1MrQ78d?= =?us-ascii?Q?7uoVEPMop34ym9/OgQp9hSDWZJpvWPWn93EGnvK1ShGrqLONif2VJGovbsPL?= =?us-ascii?Q?UyQx23/VR5Cwyne9Rh/D2MCpfd2jdBm2Vft+LtXmwcrfnjDBFkq+w253y8A0?= =?us-ascii?Q?fnlveF8ky9RPwlv6/a7MiclEbVJbc+oZNda/RA6BDmokb7fyVgaz4Iny+D6t?= =?us-ascii?Q?+83u187+BXPALCbb8dyWozKjXrsr0uAy8obhYb4rZ6WhWeboHmtpuU8vLX4k?= =?us-ascii?Q?Okzxcj6VmMmBWaGh44mDbV2yywKidZxPlg0fbR9M0MOvNeneTOWQKM8XRqEV?= =?us-ascii?Q?FqY+mvzv/Yps/pTZU/0E14FxkfLGHVgSPuejZWZDHtJXsNJDCKhe/+i5AqIi?= =?us-ascii?Q?6zMczgO9jbXrbImJNKGcCCx7cmENZL+mBt3UazOtUfqiJv+wA4JTechxgqk6?= =?us-ascii?Q?pTc8WQCXSmjLVH9lIA7I3jlyE81LrDKv4mWzrTX0/7pWHsvC2kS7SSPfzAja?= =?us-ascii?Q?3pXDqLv7+DPQ4z0bDLawcM3H8TIxDy+3128ifDj/7j8ylcZzByEhFx3QJr3I?= =?us-ascii?Q?Ee8Lhz1ddjxVGZ45m7ZojC1j6W0FZ8ctyQBXonSoSvptO6Pd5NwwqkWa98lk?= =?us-ascii?Q?/J/D8Y+BiOEGuXN7DatTqvnhUaQA3RZhsWNH3tpoaJISDjMeuCiVS98v0WnF?= =?us-ascii?Q?HEppU0bW6BeXIEaEf/0kMTHs9U/FQRFRMeQ0Z1QAqbezgrQw8H49TTRRAs0J?= =?us-ascii?Q?wXqGrh1P0quoph85jI/wxpNcVjHYPCaoDEq1D1aYtYyR3R5o/rTZQaOE/yOk?= =?us-ascii?Q?CPdrChw7Ppa+OvpUaXTkfzMrxNa9MRwGErPFvb2lZEOw+Fh75uAR0hiuS7H8?= =?us-ascii?Q?qTX23G9RQ5MuHsEwijPSAGZJFGURVxqaY454CynvLHOpc2MgXZctwB6houmj?= =?us-ascii?Q?tFf9XeNWmz8CnqXhHj+uWHmvYOswse1lprXXeDTktaB0+Nxi6JgYaw=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SI2PR06MB5140.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(1800799024)(52116014)(376014)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?g+7if/bpymBk9EYVcZEIlXcI3yLZuCx9aXPOWRcrx13lI+7nyvIiijXokjlm?= =?us-ascii?Q?OdX8Wot8lHbIpcifRwPx7UNHRGBRJNspCXjPmcTf9mjIvMOFgShThjKK0c/h?= =?us-ascii?Q?vuRI2YIxoM2yngGtX39R2AwZi+fW3I9/OljfKuDx/6K0FPsNGbFGSbvdM8ji?= =?us-ascii?Q?EvS/qvqnd3Pw/aBfQyY1xtBLOZMFWkzNK15B97SvLfzoOG5Jja5I2ePAeb3j?= =?us-ascii?Q?S+V/wpIiS0JnayE1JmuwnMA/vIEM5pLtQromZhvPQ2YdGEcnJZFC22PHAxeP?= =?us-ascii?Q?KE4Tzj5NbmyosNYPeBHlatYo6MND3Q+oFBqfdpBX+uRDQRIo3myunp+3OfLC?= =?us-ascii?Q?XWf05eoExt7t33wZKqxnQQ8QWwkxerqiixULWaM4EQkpbk/AnOFPwVoF3xni?= =?us-ascii?Q?Oh7mjAwRDAI13eKb+CpDvFQ+Fk6BK51X8Ef5jumicQdIWorm16uFZtxpcfv/?= =?us-ascii?Q?vvVnB18Gy0f/hgkYGLUifh49rI45o7yzWoRnCLFvlNt34YhsvDRJhelfEsND?= =?us-ascii?Q?FEjyidI4jbmnr7NH2S0T4Pxce22mIcl4OP0GHRyLH8lWtPtQc99MBbtAtN0I?= =?us-ascii?Q?GE/9X04W1Mvm6dlFLKYAvsKVmQLR43rPpLD3IYq2jAbyci9izakbBPSky6Zz?= =?us-ascii?Q?uWt+SqcSmG5hKnObQ4iYpS6J+UjAq1PlxYAzqByWCDgplhSkVuM/IQWAGI4v?= =?us-ascii?Q?j17Tg4WhInuTERv8W+ysCXYLYvBokubCabIRpHaO+iUd2EAntBLkl/MEX2me?= =?us-ascii?Q?f0dg0pQDqw+npAyJZ+hFpG5UQmYlyroo+ngoT31VJTGn5rf9xTMPbYQYHE3t?= =?us-ascii?Q?xobuei9LrJIeMHLucOAd2kzVSJO6Eh0Xxwaqcd5O9Klf/15mf+uD5oOddnKp?= =?us-ascii?Q?wfTJpbfz1lCGGAmA684mZfC/GALQUaVs1DceoiBYS15D/WLNMEqmOJiRIjZk?= =?us-ascii?Q?cvuEfJw1nBZTs4gnFcgdOOSQMpx1Tc+2OQ0lgToigzvYKZCh3q0dG5kK2r2h?= =?us-ascii?Q?jI82DTsmIrDNZm+dGpNSokeUi3FFRrEnToVF9q4xbm//Iz1MOsFx34rhY+zm?= =?us-ascii?Q?+j9ltC86LsjJgmmoX0UKd2XnvFZNk8L+myGkNDV/s0B/7mf233g3aa0iWXYB?= =?us-ascii?Q?FvekTo4o5YHjXictHtLwgcsTTfRTtUUqPWtMJdbjUtknbCXJwcHtMgWVm6Mb?= =?us-ascii?Q?gh0+fWzIuXmutcqf+rt/kdYIXBJuy4INzSPTwuPQDBF9N2Gn5IeZg73UgnG+?= =?us-ascii?Q?5ziJjB5FZORfQyKaFAlMFZIRggbcjORRG9cwpS4AWwXMo0+fFJfRIWUkMkRj?= =?us-ascii?Q?d9m2Mb8QdAAy/ARABVY2u+kDMvaygFIBwR7kZuBzZyEEfjX8bKUB9/IO5eUd?= =?us-ascii?Q?vK6rSbZLkSgTmQszML20m3MjkNGI0AUPFy9R9saUFbyqmWq5TqIOm8q1u/Hj?= =?us-ascii?Q?w2FVPPJ2IVdhrSp7dFVzgwV5d3VPSjxlJFylRgRMxABqjXv8QJ0kzfNULEQJ?= =?us-ascii?Q?6X/Dpxx3rYwpUZUmd5ix+Ad/WStYjlOQuFw/d5xU+9TsFwUQ7RWWMy5TONF1?= =?us-ascii?Q?OdF83wXHtvVRwNwNTm7aQTMpQH+/rcGN/MLgVBJS?= X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-Network-Message-Id: ce1d5d7f-4ae9-4dd0-cde0-08dde60820ec X-MS-Exchange-CrossTenant-AuthSource: SI2PR06MB5140.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Aug 2025 07:54:34.5124 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: M5b0j3LsV5np7gMQqfpbNTCYvIqSkxESqCX+Pko6HJH5T0iGgp4lMZZDwsWUwIakem/a97VGQFrb9TjIF91sRA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1PR06MB7035 Content-Type: text/plain; charset="utf-8" Change the 'ret' variable from unsigned int to int to store negative error codes or zero returned by regmap_field_write(). Storing the negative error codes in unsigned type, doesn't cause an issue at runtime but it's ugly as pants. Additionally, assigning negative error codes to unsigned type may trigger a GCC warning when the -Wsign-conversion flag is enabled. No effect on runtime. Signed-off-by: Qianfeng Rong Reviewed-by: Dmitry Baryshkov --- sound/soc/qcom/lpass-cdc-dma.c | 3 ++- sound/soc/qcom/lpass-hdmi.c | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/sound/soc/qcom/lpass-cdc-dma.c b/sound/soc/qcom/lpass-cdc-dma.c index 8106c586f68a..2dc8c75c4bf0 100644 --- a/sound/soc/qcom/lpass-cdc-dma.c +++ b/sound/soc/qcom/lpass-cdc-dma.c @@ -217,8 +217,9 @@ static int lpass_cdc_dma_daiops_hw_params(struct snd_pc= m_substream *substream, { struct snd_soc_pcm_runtime *soc_runtime =3D snd_soc_substream_to_rtd(subs= tream); struct lpaif_dmactl *dmactl =3D NULL; - unsigned int ret, regval; + unsigned int regval; unsigned int channels =3D params_channels(params); + int ret; int id; =20 switch (channels) { diff --git a/sound/soc/qcom/lpass-hdmi.c b/sound/soc/qcom/lpass-hdmi.c index ce753ebc0894..6d9795306cfa 100644 --- a/sound/soc/qcom/lpass-hdmi.c +++ b/sound/soc/qcom/lpass-hdmi.c @@ -23,7 +23,6 @@ static int lpass_hdmi_daiops_hw_params(struct snd_pcm_sub= stream *substream, snd_pcm_format_t format =3D params_format(params); unsigned int rate =3D params_rate(params); unsigned int channels =3D params_channels(params); - unsigned int ret; int bitwidth; unsigned int word_length; unsigned int ch_sts_buf0; @@ -33,6 +32,7 @@ static int lpass_hdmi_daiops_hw_params(struct snd_pcm_sub= stream *substream, unsigned int ch =3D 0; struct lpass_dp_metadata_ctl *meta_ctl =3D drvdata->meta_ctl; struct lpass_sstream_ctl *sstream_ctl =3D drvdata->sstream_ctl; + int ret; =20 bitwidth =3D snd_pcm_format_width(format); if (bitwidth < 0) { --=20 2.34.1