From nobody Sun Feb 8 14:11:11 2026 Received: from SA9PR02CU001.outbound.protection.outlook.com (mail-southcentralusazon11013016.outbound.protection.outlook.com [40.93.196.16]) (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 C0DB33A0E98; Fri, 16 Jan 2026 19:27:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.93.196.16 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768591669; cv=fail; b=cUr12RX/rBceRH5Krh0091OyXg/ML0uoF7pcWpdpLh1rFwWXpFKsy+nipQt+fzqCBVyDPADdb0fAkyUrG7OqKd9qU5UdHtPahCbqfW79WVeYZ0xOP41wzDJslEitfOJfeVqA+dr5CyhmUfHtJgwkdJiHJS+u0nT/9tbCPSStxBY= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768591669; c=relaxed/simple; bh=t8AIVLWdCaY1rbvsZfZvV1eBZl4zASH50xs1/aO9UI4=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=YCaJca36k1EGYQvERcPOsrh9k2YexFNNGDQKfsKfJ+VpYvuGiRtD7QY7fnhQmF+GlFW104LdM5cM3WAc84xpOl302RBmjnwVtx6DXVF/A9ZhsSi5uXOMzoaRcnhE7fQ2ceoDpYW4Or+3mkAP6xklyA3JIq5IR4IMVEzG5iB/k0Y= 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=sJTI3Rmx; arc=fail smtp.client-ip=40.93.196.16 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="sJTI3Rmx" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=F/bPcW3237nLCs48ogKCC9IGjGg/BeMaCsh3b4nuDf4MQ9wlQy2vna7Ty4sQ2WUrqLef5wbKhqudkFVGxAbu2YbXj0K+I+Nhnp2HkswumwOBR5C1QInGRTI6JNxjNN6bkMTNX0qSrKOjShuKfWXb7ArdyezM1Y/Unq0yuJUKygVdnJ99sYRc5LeE+/jJC7wmL5yJtVcJdI5B+CnN5z0fmpGq7xjBTCAOnzp/nUJE7gin6EdlM8eK83HZUUiizSKdKxIYbItfrzzCGmpVgN6wMZNpQ7kBHvFR8YL1wu+tIv51QRmHXnglzoJSNOlLQc0Kx79Yb4OE3Mr6JFHwNHYfog== 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=jbrVd3eBvdNXPN5GNCFOy5FnXFUpmePFVMTpgaRDJBA=; b=CjciF33Anz22flq5ZtKkhmVCZKPZwdB9Ko9pZLCgadfPrPWTf+8ezeCIyj12/F+xOktOXyGbmuzZj9bfmawF8awdx3Dxt25k39No0MiQbI7bDL7U1mzqCe7TE6wsAwbLvwjkxBQw9+MVSY9OChTdb8fxlXe8UpAIkRVMgh8fwl0Pt5p1Yz+DqmJjhz2sVQKkdVbj5b1FSNOEe26M297xPeXkEHnIlWLz5lLboqlLo6M+91kiNm1HA0cNnRZu/Srtc7Zs53VNouwc1/jgNzIP3YQHtlSGJ5eh4MJ3n5puDyG6r/jKaqdQ5lXpLBjnYkBWWPSx7XjuJiUU4ViaW8NzTA== 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=jbrVd3eBvdNXPN5GNCFOy5FnXFUpmePFVMTpgaRDJBA=; b=sJTI3Rmx6SKLdxdEjM5E8JEkvLnjXar9tsN0ctoKOMWDxgwN257xo50Xex8uphULRU9+FoHy3EaJIKxXzM1m35KnGfBnBARHAxtV6PqB1pNHvNL9nmq0IIESpTcqA4hYSZrPsBLVTPMlAraOPExdC7EL7/5iuOgm6RJL0Rcu8Gc= Received: from BN0PR04CA0085.namprd04.prod.outlook.com (2603:10b6:408:ea::30) by CYXPR12MB9280.namprd12.prod.outlook.com (2603:10b6:930:e4::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9520.5; Fri, 16 Jan 2026 19:27:42 +0000 Received: from BN1PEPF0000468D.namprd05.prod.outlook.com (2603:10b6:408:ea:cafe::f5) by BN0PR04CA0085.outlook.office365.com (2603:10b6:408:ea::30) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9520.7 via Frontend Transport; Fri, 16 Jan 2026 19:27:36 +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 BN1PEPF0000468D.mail.protection.outlook.com (10.167.243.138) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9542.4 via Frontend Transport; Fri, 16 Jan 2026 19:27:39 +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, 16 Jan 2026 13:27:39 -0600 Received: from satlexmb07.amd.com (10.181.42.216) 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, 16 Jan 2026 11:27:39 -0800 Received: from xhdsuragupt40.xilinx.com (10.180.168.240) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 16 Jan 2026 11:27:35 -0800 From: Suraj Gupta To: , , , , , , , , CC: , , , , , , Subject: [PATCH V3 1/2] clk: Add devm_clk_bulk_get_optional_enable() helper Date: Sat, 17 Jan 2026 00:57:23 +0530 Message-ID: <20260116192725.972966-2-suraj.gupta2@amd.com> X-Mailer: git-send-email 2.49.1 In-Reply-To: <20260116192725.972966-1-suraj.gupta2@amd.com> References: <20260116192725.972966-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: BN1PEPF0000468D:EE_|CYXPR12MB9280:EE_ X-MS-Office365-Filtering-Correlation-Id: ae4742c8-8a83-4f59-0574-08de5535501c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700013|1800799024|376014|7416014|82310400026; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?hC2MRO0PSTRutTX93OlfUTO50xfGDGVWpQHonZhDS4X8rYUd4z9sx6BmaioH?= =?us-ascii?Q?ohsaL2FeCC1zE2TV6P5PwjQ4dPKDsur9VftK5q/GRPZLIsk+zrDM2Z31mWOC?= =?us-ascii?Q?JQioEb9tsPFdsORsNB6zIzDZ74MMYbS1oRgVRPkNupUfijuWE6UDATkZp8qs?= =?us-ascii?Q?A5jlG2R7/vOwoaM4ea+JcY4V7fqL+jsgl7n1BV1KzllMi3vgYA3TSBqkqxYV?= =?us-ascii?Q?usIsGSsfuLnZ1CG1npeuCcXk/YLMBUSwO4sg15TWZr/wR7A7gpKPafRMipw8?= =?us-ascii?Q?OwQiCA9bZL2u1eNCp1/g1BAQqq1GXBbpG6/Sd9GYUxIkMlyYgZW8g6Dbou3s?= =?us-ascii?Q?LpUta/8PvdxSLh+n2eoVE++KFBY8y5yP3a1ad0tDFTbI45xpMJhCuFTpc+/w?= =?us-ascii?Q?XG7lNjcBbhORxKJy7bqtHrl66L2ypfIdTShGkgWroYOH6G3lLzdT9pDxmZ9h?= =?us-ascii?Q?6hR5adoSiuf/sEGRZ8ajtRaHeK4xruFsErXN+VKhkmqwvgVDYEr5BFk8Nr7G?= =?us-ascii?Q?r2KB5obWSg96NttroPvovN9wbFh3tAnhsl0uNolPmll+uBmUAERTWBsiMAYN?= =?us-ascii?Q?dp2ihZRpmO1GyIqO9IoGUFZiFQhRtFFrToFjfdVEokuwSnpECtqTdQswmY5B?= =?us-ascii?Q?+dg4ZZUjDS4yWvkCgNU4YhOO2scSsQfdIUmyGM/isMHEFJZoxmyQHNnai/xI?= =?us-ascii?Q?OYXPFuyyJfL/xUb1+evclOu+oRlDobB8gRgSDWTtWpY4ya85tAQOx08oOpV1?= =?us-ascii?Q?5h3nQsyMHtr2AbxaeMKEQZU1gk5rAbTJyoJEkPmuzGiUjbuSHTDwRf8vE9K4?= =?us-ascii?Q?8MfGPH5daeIqF7LgaVGjxoTiXkn40f0wCXDf/zJ1yr4ejXme75SXHYariNHK?= =?us-ascii?Q?SnGc/81J8RUNBDgMnLTzNsxy++l/Tv+gZOAl2TzPBBAxLbLRGnfpnQgiaEae?= =?us-ascii?Q?RKuazqsXZRGDQf7lGGymOmB3rzpVqIeWCg//WuXMQudrF8VR4OrZm+tgqxmO?= =?us-ascii?Q?zXxWAGgr7gGndDtbUtyI4S/kO/GfJdWAvF/z5LH43V9/FPCRY+oKj24dSwJp?= =?us-ascii?Q?bh4tSKVRUpoHKLCLAabdsnheAqXQrxgRSE9rpm6p6A5Mq5GdxoVPQdYZqFhX?= =?us-ascii?Q?Cl/l0+zXlDSE5lIw65maOwTgdyYlnTdMsJug0mgPuwqx11AjsDSskexWGn8P?= =?us-ascii?Q?ZR55MSdlJsHUeewSOatt+iA6dVV8KD5nl29DmFyNjJNozcS/6t5EzGI6Xplg?= =?us-ascii?Q?2DEauA7qZW4Gd4kIVULnOX3C9aHpIBhSwWQnYu68+wYJJLK1iZmSE3KwezYf?= =?us-ascii?Q?Luzb3+e+tfgYCHbniRmU7X7LYN2sZwjgzRjTcUreT0FY2sIp0mT6WLVir9d/?= =?us-ascii?Q?B4mAEqM5Su7toLabJeqCRnFTryrfQXkoPSX2Vd1S7sEjyCmFIXl+MWANl+KZ?= =?us-ascii?Q?tak4nQk8eHQLfZf2P5V1BUzfT0AHIsJiTxDVCGpGg7wM+qVJuxWU+NUTkszY?= =?us-ascii?Q?4J7wXnnufIer3PsolHILoZy6dYt6aQHESGl+/crAXBnMeKQ1vKalfQjNzxEH?= =?us-ascii?Q?myy0K/o9J0IUm2DXuSpgLfuYMdqSD10E61D5JIBGiLVZ7tAcmN23u51UUIIe?= =?us-ascii?Q?xZUPxSWGiw7/fzf1bRiPgV9PsIqYz4a0qILpS4Zpzbp0e7RfViXz3Fh2lsKm?= =?us-ascii?Q?Fne1Ag=3D=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)(36860700013)(1800799024)(376014)(7416014)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jan 2026 19:27:39.8394 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: ae4742c8-8a83-4f59-0574-08de5535501c 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: BN1PEPF0000468D.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CYXPR12MB9280 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 Acked-by: Stephen Boyd --- 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. + * + * Return: 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 14:11:11 2026 Received: from CH4PR04CU002.outbound.protection.outlook.com (mail-northcentralusazon11013059.outbound.protection.outlook.com [40.107.201.59]) (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 2B10F3A0E8B; Fri, 16 Jan 2026 19:27:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.201.59 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768591677; cv=fail; b=m1VhtN6rwHyB1kMiJLpU8cKOwbj3D61CQGgeLgGY2CzBF+5EJF+8nJwGgVL0SDRKAOgHzIi1tiUq4/23U2dEoomoI+GHygr08/oOZr5dK3kg2iwaPevpZhbgLb99X9LlGxTKU4KpDjkEVKq7GX7O8663Fm6okiB6hAGapWp/El8= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768591677; c=relaxed/simple; bh=D71eyvQE6h67pES+eJnRBEEEk65C5WCtug0DI4CXz6A=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=iN8d2wCZPhSQL/1g2Ok82YZeNOBTXCaxT9pw4ythaOcP1YXkHzlIDuXzyHnFhGoUMZJXEG0IKU1XF3A1yx1yqpiyTrCrJebCDetzAYAJQJilUmWygcfIbagLtkxCyLZvzaVXmwJ7TsabDbjBPVn1CB7vrRz2gHrrRLSHMPKmfjM= 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=a7iYQ7bL; arc=fail smtp.client-ip=40.107.201.59 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="a7iYQ7bL" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=oEbmx4pP0MrY8jAbjkEXvMySzrSKvhhKYiZo59t8GzB1W2tN3TwOBCV2fwuDlN1AWVZ4lvLj1rx147vEINDRgUAgRBoGA4yPAV2X7CCScXdrJtqSebR93uLjtvwLZGFjXW6+v4QPLjtXUdR4ZRW1uJhcBOtO58bgxtXJGebhqB4BqnFw01JBLIBBauqzBbBajWiPKS5BWnWUSD90XmosW5kycJScOHK45Z6mW5ypvEr8WIPORa0OiuoTVhwrAzToSPt09uheDKn9sngI/ycFEdUfJpM6TMeGyLhg0RUUYtvHsQniZ0SrLJMcnA44E+o+c51TbE5NRxWc0cBJ701Tog== 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=sRDXY4J/uFGB1yQXyfnnJwWjtcJGqG541JFygF+15PI=; b=jNNkg0GM5XqgfhwUTc9m46JvnHsdwCUgN9Y03TBuTBlc3Tf4jMrj4imKnBuAuaPjaW+Lk/Q6b3s3wpbZMlEyKfCf2CQ7J8xtR3qqZC3vZ/mdT+oINp5RNKhOzloNsHOHX+IupQ+KXB7yxgI38+/mhqAH7bRDil3Q1Y1FVbLiJUDAvA9r5E5cpK2A8Kj8aEJupsU6NJ1upidwzAS1edP6X5oXkzwNEfUM87VUv9Ro682RaMWcyxjF3dndZIB++49cels7hq5se6acCf6ewvhZJ5xCh/BmLl96gXp8zq6JyPTsVspje0GOXl/JlGR/Plj4RWVwsMu+RlxTWM5+WQWy3g== 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=sRDXY4J/uFGB1yQXyfnnJwWjtcJGqG541JFygF+15PI=; b=a7iYQ7bLCmPkg1uC3xD8B6PFXLmexYvpfzn6JaFaM5bdC8w2U88d9tcfGRo36RGd7C7XFMANibBA2QlY+8ZKp2YR/3J0lso6Nxv4XFm1gtYYpYPUqj2Wy7yAnRoe7quqgFoVJOML7Qypdg+7Ofs2WdkbK3Ofj5vXFSZel8XPVnc= Received: from BN9PR03CA0455.namprd03.prod.outlook.com (2603:10b6:408:139::10) by CH1PR12MB9600.namprd12.prod.outlook.com (2603:10b6:610:2ae::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9520.7; Fri, 16 Jan 2026 19:27:48 +0000 Received: from BN1PEPF0000468A.namprd05.prod.outlook.com (2603:10b6:408:139:cafe::99) by BN9PR03CA0455.outlook.office365.com (2603:10b6:408:139::10) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.9520.8 via Frontend Transport; Fri, 16 Jan 2026 19:27:37 +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 BN1PEPF0000468A.mail.protection.outlook.com (10.167.243.135) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9542.4 via Frontend Transport; Fri, 16 Jan 2026 19:27:44 +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, 16 Jan 2026 13:27:43 -0600 Received: from satlexmb07.amd.com (10.181.42.216) 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, 16 Jan 2026 11:27:43 -0800 Received: from xhdsuragupt40.xilinx.com (10.180.168.240) by satlexmb07.amd.com (10.181.42.216) with Microsoft SMTP Server id 15.2.2562.17 via Frontend Transport; Fri, 16 Jan 2026 11:27:39 -0800 From: Suraj Gupta To: , , , , , , , , CC: , , , , , , Subject: [PATCH V3 2/2] net: xilinx: axienet: Use devres for resource management in probe path Date: Sat, 17 Jan 2026 00:57:24 +0530 Message-ID: <20260116192725.972966-3-suraj.gupta2@amd.com> X-Mailer: git-send-email 2.49.1 In-Reply-To: <20260116192725.972966-1-suraj.gupta2@amd.com> References: <20260116192725.972966-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: BN1PEPF0000468A:EE_|CH1PR12MB9600:EE_ X-MS-Office365-Filtering-Correlation-Id: 012f6e78-ded2-48ad-af32-08de553552bb X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|36860700013|376014|7416014|82310400026|1800799024; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?/GYMisNUQG/hNSjmjcTnN1v05ZRydEI+UffXILuceajjLGVCbpdanKfKZJlb?= =?us-ascii?Q?Zmi5dIVTYtnwyRwbfoZX2rpunuSRwoU7o5Df0PbBLgUz1KzT2Hjwnd1m1Bz4?= =?us-ascii?Q?tF/BurExSsbxYlCREgPbVO1sTpD6RNFfst2v0f7ffvTcyGTsMJ0Jkpz0HyiK?= =?us-ascii?Q?GEKeTc8StIyJCBHwjz2SBZKQmheTx4IqQU5XZjSzFUshAsN8qkbxtSGqUO0g?= =?us-ascii?Q?aGAyksbmS0AOj7X8gQz41jNDklNhAqBBKDWxmvEoJfDSgQVqG8Mcm668eNUL?= =?us-ascii?Q?in64c3R1CbPoGRzeiWSpJWteqbyWUObZV1vtdrmVMUqvlpRhUfGu0Y3LVYE/?= =?us-ascii?Q?loKHXNULKTWhNtcSa2PsjK2Wjk0etKETDVvDYSQHLKhA1qhbI470hrGbsXIC?= =?us-ascii?Q?Uah5P+doI/PIgy4zpCrhVVe7zu2rwaTyikswdTnh/IS4Z0NAsvgOr21Vb84u?= =?us-ascii?Q?1iJRkqxrd9O+bJZ1G0A+uqRHrwtrIpg6uv7MndhsJNeSdz2GnUXI9Nptmx7y?= =?us-ascii?Q?rbdmILoNSlFPsm8yDsdCB19vX17qoVkrWAjbmB3n0gzbualVTcEfGCDZ4V3Z?= =?us-ascii?Q?1/Z2VMHIb85rWnIc05UQ1fqEgbDB5peQUySDWiZfv8rSJ2aEzhwW6EJsq6ww?= =?us-ascii?Q?yCcB5g569+sDRK3WSGh1Ly2U1eItai8Q7cBFZ29QP4NitPxHcNBGYlFPQAm3?= =?us-ascii?Q?WdKul4iC4v8YV8ObjzJCKbSC3nU8r//12tFbdnJY6+WaJxK4jQSZtUGq0sEJ?= =?us-ascii?Q?qfDl2YKJGBePZf0bjiLEZ+IRHBJh2D3rcQu068ptKBPVjIt+2ALcNS94XL/b?= =?us-ascii?Q?UtCAMe10dRT0UmyR1x3CEynWbpPpbm2/nH+2JI7/kuvuLBsiXnDhFzwGWpp5?= =?us-ascii?Q?xx2bGX7YZo09SwTxLtEx/LRkWgkZuLbIYk2M4eo1BeVGl+dz1QJOIlMM78OX?= =?us-ascii?Q?KKStPUnVGf/fWsoGQzRnnu5i+WTT2hCKM6p+/lnjjXLZf3UaLpS0c+XLe6zS?= =?us-ascii?Q?kezu9q/TouPm1WEkiB2+K4kSiCoFfaQ+sxbfk5bSObMT00zDnuap8MEe8Gtu?= =?us-ascii?Q?6KEfVrWXSJ5/rfTW3tq2Kq0vzsekL12wdV0WwdcyA7pv96cgW3XRtzUf76Sf?= =?us-ascii?Q?MWGIG5mOHL91NffH8zf3uuVVhhJUq9+NkEefGrPyFfFRhGsdsR9sklezkhA2?= =?us-ascii?Q?O5jDjvYpwQzb18lv+QpPWAaDY8xQXdEBJpp5CELWYeifwsuJH3L6Zpx5QCZL?= =?us-ascii?Q?i1okuOIVfuYW+/Cii3/gAaoCmYt74ifw73vBsmCACh7YRHeo2NdcxSEv9FsP?= =?us-ascii?Q?d5HuHANr4lRyOnYy5U9kxZpNNEYDslyfgmI/UfaUnFLENXb9yoihpJfSHHt2?= =?us-ascii?Q?hiY30BvAfOCehHWsU9K8M9UjXy2bV4Lzyc9fLDiA5tplGz0EBwFBLe+3PcKN?= =?us-ascii?Q?F9ipwbtVz/l+VodSMB7iQ5redit+WPiEIS4/oV3hAIHjtevjHQm8c6qFmwqD?= =?us-ascii?Q?UI1bJXtHon61P4Dh0xdaGqyynT3v71H+fC4B7vsWScUqA9dKcthP/c6/lKKT?= =?us-ascii?Q?e7L+5SU04DTyR9grSHBWFeQH2f1j1N3lG1xaWk/lMCzGwWEMmLt+h1wc0GoH?= =?us-ascii?Q?guepPdOb1iA3DJ4IbRSbYLMQkF+My/EtYEj0nFJ7+EojI2b8T8eBhIKr8ywK?= =?us-ascii?Q?eq2JaQ=3D=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)(36860700013)(376014)(7416014)(82310400026)(1800799024);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jan 2026 19:27:44.2428 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 012f6e78-ded2-48ad-af32-08de553552bb 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: BN1PEPF0000468A.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH1PR12MB9600 Content-Type: text/plain; charset="utf-8" From: Sean Anderson Transition axienet_probe() to managed resource allocation using devm_* APIs for network device and clock handling, while improving error paths with dev_err_probe(). This eliminates the need for manual resource cleanup during probe failures and streamlines the remove() function. Signed-off-by: Sean Anderson Co-developed-by: Suraj Gupta Signed-off-by: Suraj Gupta --- .../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