From nobody Sun Feb 8 01:51:55 2026 Received: from CY7PR03CU001.outbound.protection.outlook.com (mail-westcentralusazon11010068.outbound.protection.outlook.com [40.93.198.68]) (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 054003FF1; Fri, 2 Jan 2026 08:55:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.93.198.68 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767344116; cv=fail; b=MqC0gfvcokxSva1Tz92Ip9U6WUZdrUcuOYrJe1fLuvg288C++XcARwB5ndXslOSbZrwCDvLcF3UzGGv2dlW35B8GgsiUaFC14KZtFaHlJRRZyv1iwWiRNOVU4RAycO5cM/vrrEnySXMxn9cmKqkKFQr7zBzoFumEs+oUWsxzDzc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767344116; c=relaxed/simple; bh=IQLUJfuGUF4psTpnRJFfyrvZIgeaooMogIF+uiiWexA=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=XlIe3dzdRhrZgCrNRlSX6oyX93ZOQ/be98iuJC1ee2Ra188/qvulEQPBDM2vLFaXUjEWq1ijTO3K7sVlK2QxINFxtQJa7QYtsHr14t3O7JXGV3BabmrkX6FaR8aFdkqNUypl1pDA52MiXeXUTkIOYl5xEFjbblKy41cPDvdAHSE= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=XlKIcNzU; arc=fail smtp.client-ip=40.93.198.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="XlKIcNzU" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=n0vZg1KvBNKb9vsrnD+Iy4Md7HxVKxZQ5SGWLmnGnWibbdXBNUq7GoybeTJzUFcFL3SNdIOUF9crVrhhAG4cy3ixrBFEH64SdqShAg9xbjZRoOOEenq64TinvlGwgQeAQWzisL6kU5Bs731FyA+ynaKpYQEJVxkNFCwKQrhZpf/NPvdhgEv3XTuT9VdASP7Iy1ng9tCSugxjsCowsQRmUK5OKMfBtSbkg7eWcH7sjW29pcp/qpHROjOu65hUpJZ6NjsBwRTdG3qotm+bWOTBnXR+Ncc6Loe4iuoCC53co6iQVR9IYcOoQAJ+rDVm1GrKwb5zSwukv2t+yhbXW9tFKg== 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=5Rmo1G2uqxHyV3vtME30C+kkfROV30KoVzZ3djrPvh8=; b=ZLEj0IMUDriGqEHbeIkU25k7y801ppDK4lsb8r8i67pGX23kZQDucmtyQDgAaTsMEX9NkoCcUklCH/wU1Fa4a5MHo/xvbTCl6o5wRocedxZMgHwCc2OUmARXCpwqxJjYNEta2nVXH5CbCwqo3CMBEwRK3SDLTneIqXIMUFwp3+pWhHGPjzYYmqm9X409X8pCUkGQ+uGjMcEo/V4Xme1xuXfwNa7mTI9F4vd6up6BNzvKhR9baajf+sfrf0Iu2syE1DKzg0v80SXYz/0yu+7Lc+W2ohQiiD6VK9pRd+LG8xtYI4gGPg4YMfFpPwZkzy0GIDDnQt1JR1WdKfl2RRxhfg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=baylibre.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5Rmo1G2uqxHyV3vtME30C+kkfROV30KoVzZ3djrPvh8=; b=XlKIcNzUsjP1ZVkZcf4dcSggNRjYVkBRnsPgCMbbkBrI5PUFYF3G8o8e1RHSQ20Ki4ENaQ1RqkkYSIrPZQ46uvgcJ9pYzl+GWmmgQLEi3amAJvAZkizViT6cHfH+QUpAFM8wtS54qI83TjnFw6SIgyuFw55r5VP/fI3Ap0VWPec= Received: from CH0P220CA0011.NAMP220.PROD.OUTLOOK.COM (2603:10b6:610:ef::13) by DM6PR12MB4332.namprd12.prod.outlook.com (2603:10b6:5:21e::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9478.4; Fri, 2 Jan 2026 08:55:10 +0000 Received: from CH3PEPF00000013.namprd21.prod.outlook.com (2603:10b6:610:ef:cafe::c5) by CH0P220CA0011.outlook.office365.com (2603:10b6:610:ef::13) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9478.4 via Frontend Transport; Fri, 2 Jan 2026 08:55:07 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by CH3PEPF00000013.mail.protection.outlook.com (10.167.244.118) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9499.0 via Frontend Transport; Fri, 2 Jan 2026 08:55:10 +0000 Received: from satlexmb08.amd.com (10.181.42.217) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Fri, 2 Jan 2026 02:55:07 -0600 Received: from xhdsuragupt40.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 2 Jan 2026 02:55:04 -0600 From: Suraj Gupta To: , , , , , , , , CC: , , , , , Subject: [RFC PATCH 1/2] clk: Add devm_clk_bulk_get_optional_enable() helper Date: Fri, 2 Jan 2026 14:24:53 +0530 Message-ID: <20260102085454.3439195-2-suraj.gupta2@amd.com> X-Mailer: git-send-email 2.49.1 In-Reply-To: <20260102085454.3439195-1-suraj.gupta2@amd.com> References: <20260102085454.3439195-1-suraj.gupta2@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH3PEPF00000013:EE_|DM6PR12MB4332:EE_ X-MS-Office365-Filtering-Correlation-Id: 9aabd022-0deb-4ea7-ebd2-08de49dca2c7 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|36860700013|1800799024|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?2dHTyYKb1q/AjPx2JuDg12Ew+ovOC7lH7MeELArV9TAasNG20kKzvCU3gp2e?= =?us-ascii?Q?K66WsWpahDqZ+E8LMDd7iMlwsarKjtU2J0XcCjuLb8fIhS7qq2tJK1Lin53X?= =?us-ascii?Q?BFUPBDuzzEqcsBPNx8bFW9VchSZHN3mt3I7C97pcClDw+6VhPp/sidOX8wu+?= =?us-ascii?Q?0Aaf8nAfvyLOdRnpXET8usWN+XUUxLI6Yl5BO+hSV+j0IxNO70yyz6UNPmxz?= =?us-ascii?Q?ok+w8SEDyFKxGJ1qxi7AvC8PE+GJt5MCnPhclsElRQocq1LVwyDvjgS4+rON?= =?us-ascii?Q?LJCsryz3xVBGne5TsJii5xCtXk7RcTnjH/V+mXgGFOYE2fDJBfKe24thmfv+?= =?us-ascii?Q?u6UxwGCb5u7bcOY8yuT5rqi2Svj7Bw0gETGViNrs4j1iq1yk5V2CdL5MB/sJ?= =?us-ascii?Q?l+VRo67UmXdSy5rfMkVWNe69rKPHM3e3uOj933yVxhvsNVPT0HwBj9Sp5eOx?= =?us-ascii?Q?Q/TCWARCVPCK3q73Sbw37hga9Z447GtZl4Fb/68eb2X1g5EuQ3NN3Re1LE51?= =?us-ascii?Q?mqQffVkXTPIoEDzr+/DIW8X71mlbRBaTQ8SP2zELDTFl15lPrq90eoTB2JmD?= =?us-ascii?Q?6S+2vqzATJhwlf5G8QzpJPV6aXkN0GRbWX+O3CLBEe7P+LlxkT8IlvjbMtXo?= =?us-ascii?Q?4q46PShgfsA8AUIzI+JP2E5bQoZz+nV6njOhqzqJl39dXZ01XqFsgvlUfzg8?= =?us-ascii?Q?RWVT5N/1WMGQOBKdv33ToJqOTxrVGEfwg0DwQUh0JEkpPUDTH9zl90s/Urv6?= =?us-ascii?Q?4wff5FkIfAZg/2IzwflI88TU2cYrnFCSY1x0ENDlqYzxhnVdbUeVy+q2T5bX?= =?us-ascii?Q?tI/By+OynLivd/fAdlezs3wAFhuZBBwWm6T11aeDeT3cu1Z4KtbzNXwIjFK9?= =?us-ascii?Q?lVI/pPjjG0d67DCwKevi7UIb8eXY70rDFQlzT7LaQEse0ZNQ2Y894c4usvmb?= =?us-ascii?Q?3Dj0JkNzLnkZdkrs20K8hD8mh2JQQyq7PuQ5igwEwoqWUDVA1AvEQm8nMnbJ?= =?us-ascii?Q?6raxG84swqTJLVHbSsOKLmouUKq97wiuVfRTS70Md9gSIRX5Aj+ofUcVcSqC?= =?us-ascii?Q?AA0sMHjPMlM9zsQmO4VhQFc1Zl5U7NZQrGwW4UoxhHGofK1R/abBMF6DAj4R?= =?us-ascii?Q?dMqZLfj71KywFlilObUw5YmrQ2sFerAx7JXfaRa7fT4AlVBlr75fdQQUlo0+?= =?us-ascii?Q?tjayh71SgaJ/vOgCu5o3ZvedepavOx9HCrkzkqOx1HLbVWuHluaxzUGHQWUf?= =?us-ascii?Q?jAw4olpawmWo/Ljy+iRHRbAmDytal6KnmBjF0EU/KkcrQ2IQkp1PQz1HQP3x?= =?us-ascii?Q?IDHZkEvmHVxmXwJiwZ5bsT381TIe+0uBhY035tKkOmFjznSjHS3E1kbOSDhm?= =?us-ascii?Q?IWkGACJMa2ZqSgh6PNCy+JNGME24oQ5BgmnyXuN+fr/rHNXxCkJmZkJZb3YF?= =?us-ascii?Q?tU2g4CX8N10477GEAsebbdILfNCkYj6Grh6DJGJP+hNubZI2OGnQXhvDrHHk?= =?us-ascii?Q?+BjXqKBohEf67Gxw7YuXkMkW6H5WYLTSYaq9J3iXKBGlrIRQhZ6QzdmxqWwl?= =?us-ascii?Q?wrVc6Ch7S2B8CyHhNk4=3D?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:satlexmb07.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(376014)(7416014)(36860700013)(1800799024)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jan 2026 08:55:10.5290 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 9aabd022-0deb-4ea7-ebd2-08de49dca2c7 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CH3PEPF00000013.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4332 Content-Type: text/plain; charset="utf-8" Add a new managed clock framework helper function that combines getting optional bulk clocks and enabling them in a single operation. The devm_clk_bulk_get_optional_enable() function simplifies the common pattern where drivers need to get optional bulk clocks, prepare and enable them, and have them automatically disabled/unprepared and freed when the device is unbound. This new API follows the established pattern of devm_clk_bulk_get_all_enabled() and reduces boilerplate code in drivers that manage multiple optional clocks. Suggested-by: Andrew Lunn Signed-off-by: Suraj Gupta Reviewed-by: Brian Masney --- drivers/clk/clk-devres.c | 50 ++++++++++++++++++++++++++++++++++++++++ include/linux/clk.h | 23 ++++++++++++++++++ 2 files changed, 73 insertions(+) diff --git a/drivers/clk/clk-devres.c b/drivers/clk/clk-devres.c index 5368d92d9b39..994d5bc5168b 100644 --- a/drivers/clk/clk-devres.c +++ b/drivers/clk/clk-devres.c @@ -179,6 +179,56 @@ int __must_check devm_clk_bulk_get_optional(struct dev= ice *dev, int num_clks, } EXPORT_SYMBOL_GPL(devm_clk_bulk_get_optional); =20 +static void devm_clk_bulk_release_enable(struct device *dev, void *res) +{ + struct clk_bulk_devres *devres =3D res; + + clk_bulk_disable_unprepare(devres->num_clks, devres->clks); + clk_bulk_put(devres->num_clks, devres->clks); +} + +static int __devm_clk_bulk_get_enable(struct device *dev, int num_clks, + struct clk_bulk_data *clks, bool optional) +{ + struct clk_bulk_devres *devres; + int ret; + + devres =3D devres_alloc(devm_clk_bulk_release_enable, + sizeof(*devres), GFP_KERNEL); + if (!devres) + return -ENOMEM; + + if (optional) + ret =3D clk_bulk_get_optional(dev, num_clks, clks); + else + ret =3D clk_bulk_get(dev, num_clks, clks); + if (ret) + goto err_clk_get; + + ret =3D clk_bulk_prepare_enable(num_clks, clks); + if (ret) + goto err_clk_prepare; + + devres->clks =3D clks; + devres->num_clks =3D num_clks; + devres_add(dev, devres); + + return 0; + +err_clk_prepare: + clk_bulk_put(num_clks, clks); +err_clk_get: + devres_free(devres); + return ret; +} + +int __must_check devm_clk_bulk_get_optional_enable(struct device *dev, int= num_clks, + struct clk_bulk_data *clks) +{ + return __devm_clk_bulk_get_enable(dev, num_clks, clks, true); +} +EXPORT_SYMBOL_GPL(devm_clk_bulk_get_optional_enable); + static void devm_clk_bulk_release_all(struct device *dev, void *res) { struct clk_bulk_devres *devres =3D res; diff --git a/include/linux/clk.h b/include/linux/clk.h index b607482ca77e..ac0affa16c8a 100644 --- a/include/linux/clk.h +++ b/include/linux/clk.h @@ -478,6 +478,22 @@ int __must_check devm_clk_bulk_get(struct device *dev,= int num_clks, */ int __must_check devm_clk_bulk_get_optional(struct device *dev, int num_cl= ks, struct clk_bulk_data *clks); +/** + * devm_clk_bulk_get_optional_enable - Get and enable optional bulk clocks= (managed) + * @dev: device for clock "consumer" + * @num_clks: the number of clk_bulk_data + * @clks: pointer to the clk_bulk_data table of consumer + * + * Behaves the same as devm_clk_bulk_get_optional() but also prepares and = enables + * the clocks in one operation with management. The clks will automaticall= y be + * disabled, unprepared and freed when the device is unbound. + * + * Returns 0 if all clocks specified in clk_bulk_data table are obtained + * and enabled successfully, or for any clk there was no clk provider avai= lable. + * Otherwise returns valid IS_ERR() condition containing errno. + */ +int __must_check devm_clk_bulk_get_optional_enable(struct device *dev, int= num_clks, + struct clk_bulk_data *clks); /** * devm_clk_bulk_get_all - managed get multiple clk consumers * @dev: device for clock "consumer" @@ -1029,6 +1045,13 @@ static inline int __must_check devm_clk_bulk_get_opt= ional(struct device *dev, return 0; } =20 +static inline int __must_check devm_clk_bulk_get_optional_enable(struct de= vice *dev, + int num_clks, + struct clk_bulk_data *clks) +{ + return 0; +} + static inline int __must_check devm_clk_bulk_get_all(struct device *dev, struct clk_bulk_data **clks) { --=20 2.25.1 From nobody Sun Feb 8 01:51:55 2026 Received: from CH4PR04CU002.outbound.protection.outlook.com (mail-northcentralusazon11013046.outbound.protection.outlook.com [40.107.201.46]) (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 87D9F2D6E62; Fri, 2 Jan 2026 08:55:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.201.46 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767344127; cv=fail; b=bZ0MFtVniH4hXKoFOnLgbgfL/hKJ4v2/dZ45IaujQvAS8RsOBXdd4v4srqmyc7il6FWpUI8WHvtG+hNOeH0BSCSRdUf4Q2G2Zbf9Ijejgt/5/V6AIYcSjYNpyd24Ckk7gFdoGsVDPCUvkNbxXE+OWaFx5L75nmOkDsOl6BtUEG8= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767344127; c=relaxed/simple; bh=21GWd8KfbZiY6gdA64kUybuxxFQHtAwlOxnxWD6iD3Q=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=HLMgSmUH+XMtcnzgqDw6/MHge6tjMj5hITZiKZSQhXvlTnJK3kFTgdKNsLgwJfUDT7WeunJ0aEbkvesYTeFt9MuCJyfSn+IF/cRk1aPJt/ljHnmh7c+DwoOQHi8o8L5k/NOOv9CwQHtwb0tUSOMzOiRjYAMwd+gZ9TWmMjg41rA= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=nT2HOoYz; arc=fail smtp.client-ip=40.107.201.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="nT2HOoYz" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=DYfi9D4WShgkPuvTkHdC98gioZ0NwcWfEIeNfGjy5JQrQPIRFZIjB7Gqr3HJv2jx3jJ3p8fbnLwT6D43D3aD7JpO12p70EU9laLhZ7cZHw+zTGbZUX6XaQksE+23XDmTvv3GeKySOlcGMFIyumcGDALP5NcESjzQcXzf6hvGRkDMSaHaursrX/8SRk0L0MKUVv8NfGA05sxpRY1uRr1vQtEpJcYpA+fo8FxrOzNM4vjetfh7Int7P42Jua5Wz9S6RXHouWYwHeeq4kM25SJEN5i/+0RtKyff7MpU0xfE2TILig3jcd11KsfnbBw7yc5io4X9TPxKsdlBF7t0ldC7dA== 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=FgxeFhQEmdQG8SiC1NxXcu8Ej/mmP9Gpsq8A155OxqE=; b=LlaZ53PqPqQAIqxPwZzkFjxYH4fpwB6dFQtgRpO+b9H5rPwt3mMs6U0z3nXrKh2La5enzohgQeO+29z7g4A/VeI/jbejON871FkrKBuoywdFgiBaFQxPXn5r45VGarWUBK96kRoYEqObPvAWK5E+OlehZKdw8I4iPF7ZKsBw5KZoHp9680oaG7V/bSkMO0BlPh8zOVL7MQMcfvdlrR6PBjS5jEsiboTGC0HskyFlv3QDRc34FQje2I8PE+OlPsU3MnoVBqw7MMDwCczNduj/PV62sq6Z5Jm3/MbV4xkvpSCxusj5H/thBJBOZwN4aMcSS+xNulMLqCKoOS22akvMIQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=baylibre.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FgxeFhQEmdQG8SiC1NxXcu8Ej/mmP9Gpsq8A155OxqE=; b=nT2HOoYz3nAjhQqzx4GC4Gld7ep/BoAoaGxHVxh/8xCy+v0Bn68A0hORdhnPgi+jWc2gPrE9XqCx7U+PMyAQrlcYXpbTlXHd4yTcVH1rpjRzTum6tNVNupOb8SBIiNmF/LYFuvtztRAC/8AkjLemXpglUQ3+GCWpHf3mlEd0d54= Received: from CH0P220CA0020.NAMP220.PROD.OUTLOOK.COM (2603:10b6:610:ef::33) by PH7PR12MB7892.namprd12.prod.outlook.com (2603:10b6:510:27e::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9478.4; Fri, 2 Jan 2026 08:55:20 +0000 Received: from CH3PEPF00000013.namprd21.prod.outlook.com (2603:10b6:610:ef:cafe::64) by CH0P220CA0020.outlook.office365.com (2603:10b6:610:ef::33) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9478.4 via Frontend Transport; Fri, 2 Jan 2026 08:55:00 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=satlexmb07.amd.com; pr=C Received: from satlexmb07.amd.com (165.204.84.17) by CH3PEPF00000013.mail.protection.outlook.com (10.167.244.118) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9499.0 via Frontend Transport; Fri, 2 Jan 2026 08:55:20 +0000 Received: from Satlexmb09.amd.com (10.181.42.218) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Fri, 2 Jan 2026 02:55:11 -0600 Received: from satlexmb08.amd.com (10.181.42.217) by satlexmb09.amd.com (10.181.42.218) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.17; Fri, 2 Jan 2026 00:55:11 -0800 Received: from xhdsuragupt40.xilinx.com (10.180.168.240) by satlexmb08.amd.com (10.181.42.217) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 2 Jan 2026 02:55:08 -0600 From: Suraj Gupta To: , , , , , , , , CC: , , , , , Subject: [RFC PATCH 2/2] net: axienet: Fix resource release ordering Date: Fri, 2 Jan 2026 14:24:54 +0530 Message-ID: <20260102085454.3439195-3-suraj.gupta2@amd.com> X-Mailer: git-send-email 2.49.1 In-Reply-To: <20260102085454.3439195-1-suraj.gupta2@amd.com> References: <20260102085454.3439195-1-suraj.gupta2@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH3PEPF00000013:EE_|PH7PR12MB7892:EE_ X-MS-Office365-Filtering-Correlation-Id: 0abf7fac-300f-45fa-4322-08de49dca877 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|36860700013|1800799024|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?0kXEgNxFlcK9bHczVUnEoRLzhKmegCFfFU4t9lfjpwfTmPBVyZLEg+R8p/0y?= =?us-ascii?Q?bn5qxT27237TLX5l5nlM4hZqC7ZEyDItZBWCE+I7O6uGZSW1IFZksZBCa3qd?= =?us-ascii?Q?YXgqf/tc2La+4dbEL7tVZKQCMaLmIBavhtcv5MtfyDKzuH728DhkMnI47nTd?= =?us-ascii?Q?9iZ87GqB64afjtXvBYmTZIQUhXbk5p/ZLlm3IoGfBpcJi4uH6aOGVatCkb34?= =?us-ascii?Q?CNNTuGxgfQU8idlzUzGAK3aQZ8FhFu707mUtpqVwHyD2npfnHzf7U+W+d26D?= =?us-ascii?Q?R/HK3UDlePICn6SZvsWhQ7OdXvYM+pS180OskIuqQpUzOnjCh0MivVi7k06C?= =?us-ascii?Q?VaOYOQ5qdNw0HZYqMFq6D8zgwgN0EzHsejItUrUohm6TDE9y1AhSVzAKTCt5?= =?us-ascii?Q?cDl9QIyKZv2zfKjxiCMTMbhM3IIGElHzscbkGwLfq2gTyC+5cd/YqvFQDNW6?= =?us-ascii?Q?YHc0Xa7DsGq7WFQgnhxii1hzjkiBqA3xxICD3p+nf9hOFensu6mye5z3FehS?= =?us-ascii?Q?KIoq/8VTt6uH/hET65qBRbe76tXiRjx507YW0GtflfyTYdw4ls2SBDUlXS7g?= =?us-ascii?Q?rw7me418Cd9denELZJJApabo648lFYToWnlf9P2ZJI2Ps91fgOOcABozB1bK?= =?us-ascii?Q?ThTM2+fh5lErMQxmMG/YFyPuzg2PEINsqVafCqJQapIefnoX2a3Uo2o9fl5o?= =?us-ascii?Q?YqfTup0o9McIehYWq4qR47+XOjVCBpb/rZ0LRqG/hNe6Znie9OJuC78aukwf?= =?us-ascii?Q?WnJHCjw0gDvOUC1ylxyf6b+t7B/HKYClPRBnomUB+qNED6x2NaV7JX+VvD7V?= =?us-ascii?Q?mMYgLv7jQM6fm1lTuGWe0MoWA4lQr63FGdLY7NbgMinNS03udVRB5+DK2Lpk?= =?us-ascii?Q?pqTYzb7tYXY3EcBojMFO11IRqtN6LjZTz/2ybcxYO9SxTz6mnFuX/JuLPmeb?= =?us-ascii?Q?wnMETWKzB08gdYFbkH4LvnwblJvWygFZCuVFwgL2i2oKe1Lczr+vIJsLkNk8?= =?us-ascii?Q?+cH7wIFl2gnY6LQo7WAoxIYZn3LuGM+PTg91G1OkrynGkYtQwgXKNT+xiZuu?= =?us-ascii?Q?rsIHIoekYpx90Ws29vOTe2SUvUETXKTjyhQwFXLFj+MNL8/CZ3TB729mVusB?= =?us-ascii?Q?LzVkw+f8jpKrmSSAg2xZYmMAi5ylmmUO4V1pvFuNUmYocWeCyEvBUi/0H8oI?= =?us-ascii?Q?z04fMFYnsmjTJbPFY7oQBIdbZn0bwGUG+WowDhVH+cvcVmextn2SPFVi2Wo+?= =?us-ascii?Q?pDCgyLntt0cwLTe0qsIQu1OIWH+P8S2eSZyuFoQeT/SliYYwUB0jwhG2uIEi?= =?us-ascii?Q?mDVuV5m/6RQDKod+obbGQ47nOHJyz93ni8XQF5oasdSq87Xr8QvXfwnYQYP2?= =?us-ascii?Q?HEhYJclcvteSiTf8Clfd47XTPtTimwOkeLUou2PkB3tuKH9vUHAOmRckt8Iv?= =?us-ascii?Q?aOeXFaf+izyw9lYlpnLjQ3m23uTod/RRWkC87WAa+Kuyt+rx1FR9qIeKMAb7?= =?us-ascii?Q?PLmS06I8LcKH+BKfq/kCHmYbd5vzKRWrp0jXR4dtOClJWRTTiF6kab2cMo6y?= =?us-ascii?Q?3p0b+0iNi8tjiBAdHDA=3D?= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:satlexmb07.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230040)(376014)(7416014)(36860700013)(1800799024)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jan 2026 08:55:20.0887 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0abf7fac-300f-45fa-4322-08de49dca877 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[satlexmb07.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CH3PEPF00000013.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB7892 Content-Type: text/plain; charset="utf-8" From: Sean Anderson Device-managed resources are released after manually-managed resources. Therefore, once any manually-managed resource is acquired, all further resources must be manually-managed too. Convert all resources before the MDIO bus is created into device-managed resources. In all cases but one there are already devm variants available. Fixes: 46aa27df8853 ("net: axienet: Use devm_* calls") Signed-off-by: Sean Anderson Co-developed-by: Suraj Gupta Signed-off-by: Suraj Gupta Reviewed-by: Sean Anderson --- .../net/ethernet/xilinx/xilinx_axienet_main.c | 83 ++++++------------- 1 file changed, 27 insertions(+), 56 deletions(-) diff --git a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c b/drivers/ne= t/ethernet/xilinx/xilinx_axienet_main.c index 284031fb2e2c..998bacd508b8 100644 --- a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c +++ b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c @@ -2787,7 +2787,7 @@ static int axienet_probe(struct platform_device *pdev) int addr_width =3D 32; u32 value; =20 - ndev =3D alloc_etherdev(sizeof(*lp)); + ndev =3D devm_alloc_etherdev(&pdev->dev, sizeof(*lp)); if (!ndev) return -ENOMEM; =20 @@ -2815,41 +2815,32 @@ static int axienet_probe(struct platform_device *pd= ev) seqcount_mutex_init(&lp->hw_stats_seqcount, &lp->stats_lock); INIT_DEFERRABLE_WORK(&lp->stats_work, axienet_refresh_stats); =20 - lp->axi_clk =3D devm_clk_get_optional(&pdev->dev, "s_axi_lite_clk"); + lp->axi_clk =3D devm_clk_get_optional_enabled(&pdev->dev, + "s_axi_lite_clk"); if (!lp->axi_clk) { /* For backward compatibility, if named AXI clock is not present, * treat the first clock specified as the AXI clock. */ - lp->axi_clk =3D devm_clk_get_optional(&pdev->dev, NULL); - } - if (IS_ERR(lp->axi_clk)) { - ret =3D PTR_ERR(lp->axi_clk); - goto free_netdev; - } - ret =3D clk_prepare_enable(lp->axi_clk); - if (ret) { - dev_err(&pdev->dev, "Unable to enable AXI clock: %d\n", ret); - goto free_netdev; + lp->axi_clk =3D devm_clk_get_optional_enabled(&pdev->dev, NULL); } + if (IS_ERR(lp->axi_clk)) + return dev_err_probe(&pdev->dev, PTR_ERR(lp->axi_clk), + "could not get AXI clock\n"); =20 lp->misc_clks[0].id =3D "axis_clk"; lp->misc_clks[1].id =3D "ref_clk"; lp->misc_clks[2].id =3D "mgt_clk"; =20 - ret =3D devm_clk_bulk_get_optional(&pdev->dev, XAE_NUM_MISC_CLOCKS, lp->m= isc_clks); - if (ret) - goto cleanup_clk; - - ret =3D clk_bulk_prepare_enable(XAE_NUM_MISC_CLOCKS, lp->misc_clks); + ret =3D devm_clk_bulk_get_optional_enable(&pdev->dev, XAE_NUM_MISC_CLOCKS, + lp->misc_clks); if (ret) - goto cleanup_clk; + return dev_err_probe(&pdev->dev, ret, + "could not get/enable misc. clocks\n"); =20 /* Map device registers */ lp->regs =3D devm_platform_get_and_ioremap_resource(pdev, 0, ðres); - if (IS_ERR(lp->regs)) { - ret =3D PTR_ERR(lp->regs); - goto cleanup_clk; - } + if (IS_ERR(lp->regs)) + return PTR_ERR(lp->regs); lp->regs_start =3D ethres->start; =20 /* Setup checksum offload, but default to off if not specified */ @@ -2918,19 +2909,17 @@ static int axienet_probe(struct platform_device *pd= ev) lp->phy_mode =3D PHY_INTERFACE_MODE_1000BASEX; break; default: - ret =3D -EINVAL; - goto cleanup_clk; + return -EINVAL; } } else { ret =3D of_get_phy_mode(pdev->dev.of_node, &lp->phy_mode); if (ret) - goto cleanup_clk; + return ret; } if (lp->switch_x_sgmii && lp->phy_mode !=3D PHY_INTERFACE_MODE_SGMII && lp->phy_mode !=3D PHY_INTERFACE_MODE_1000BASEX) { dev_err(&pdev->dev, "xlnx,switch-x-sgmii only supported with SGMII or 10= 00BaseX\n"); - ret =3D -EINVAL; - goto cleanup_clk; + return -EINVAL; } =20 if (!of_property_present(pdev->dev.of_node, "dmas")) { @@ -2945,7 +2934,7 @@ static int axienet_probe(struct platform_device *pdev) dev_err(&pdev->dev, "unable to get DMA resource\n"); of_node_put(np); - goto cleanup_clk; + return ret; } lp->dma_regs =3D devm_ioremap_resource(&pdev->dev, &dmares); @@ -2962,19 +2951,17 @@ static int axienet_probe(struct platform_device *pd= ev) } if (IS_ERR(lp->dma_regs)) { dev_err(&pdev->dev, "could not map DMA regs\n"); - ret =3D PTR_ERR(lp->dma_regs); - goto cleanup_clk; + return PTR_ERR(lp->dma_regs); } if (lp->rx_irq <=3D 0 || lp->tx_irq <=3D 0) { dev_err(&pdev->dev, "could not determine irqs\n"); - ret =3D -ENOMEM; - goto cleanup_clk; + return -ENOMEM; } =20 /* Reset core now that clocks are enabled, prior to accessing MDIO */ ret =3D __axienet_device_reset(lp); if (ret) - goto cleanup_clk; + return ret; =20 /* Autodetect the need for 64-bit DMA pointers. * When the IP is configured for a bus width bigger than 32 bits, @@ -3001,14 +2988,13 @@ static int axienet_probe(struct platform_device *pd= ev) } if (!IS_ENABLED(CONFIG_64BIT) && lp->features & XAE_FEATURE_DMA_64BIT) { dev_err(&pdev->dev, "64-bit addressable DMA is not compatible with 32-b= it architecture\n"); - ret =3D -EINVAL; - goto cleanup_clk; + return -EINVAL; } =20 ret =3D dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(addr_width)); if (ret) { dev_err(&pdev->dev, "No suitable DMA available\n"); - goto cleanup_clk; + return ret; } netif_napi_add(ndev, &lp->napi_rx, axienet_rx_poll); netif_napi_add(ndev, &lp->napi_tx, axienet_tx_poll); @@ -3018,15 +3004,12 @@ static int axienet_probe(struct platform_device *pd= ev) =20 lp->eth_irq =3D platform_get_irq_optional(pdev, 0); if (lp->eth_irq < 0 && lp->eth_irq !=3D -ENXIO) { - ret =3D lp->eth_irq; - goto cleanup_clk; + return lp->eth_irq; } tx_chan =3D dma_request_chan(lp->dev, "tx_chan0"); - if (IS_ERR(tx_chan)) { - ret =3D PTR_ERR(tx_chan); - dev_err_probe(lp->dev, ret, "No Ethernet DMA (TX) channel found\n"); - goto cleanup_clk; - } + if (IS_ERR(tx_chan)) + return dev_err_probe(lp->dev, PTR_ERR(tx_chan), + "No Ethernet DMA (TX) channel found\n"); =20 cfg.reset =3D 1; /* As name says VDMA but it has support for DMA channel reset */ @@ -3034,7 +3017,7 @@ static int axienet_probe(struct platform_device *pdev) if (ret < 0) { dev_err(&pdev->dev, "Reset channel failed\n"); dma_release_channel(tx_chan); - goto cleanup_clk; + return ret; } =20 dma_release_channel(tx_chan); @@ -3139,13 +3122,6 @@ static int axienet_probe(struct platform_device *pde= v) put_device(&lp->pcs_phy->dev); if (lp->mii_bus) axienet_mdio_teardown(lp); -cleanup_clk: - clk_bulk_disable_unprepare(XAE_NUM_MISC_CLOCKS, lp->misc_clks); - clk_disable_unprepare(lp->axi_clk); - -free_netdev: - free_netdev(ndev); - return ret; } =20 @@ -3163,11 +3139,6 @@ static void axienet_remove(struct platform_device *p= dev) put_device(&lp->pcs_phy->dev); =20 axienet_mdio_teardown(lp); - - clk_bulk_disable_unprepare(XAE_NUM_MISC_CLOCKS, lp->misc_clks); - clk_disable_unprepare(lp->axi_clk); - - free_netdev(ndev); } =20 static void axienet_shutdown(struct platform_device *pdev) --=20 2.25.1