From patchwork Fri Aug 23 07:21:14 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yangtao Li X-Patchwork-Id: 13774694 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 61C79C531DC for ; Fri, 23 Aug 2024 07:09:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-Type: Content-Transfer-Encoding:References:In-Reply-To:Message-Id:Date:Subject:Cc: To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=mI0K8rSXqPnuSQqybgOp1bYWqZhcAzOMfVU1VL/kzuc=; b=PAq9isktKpw79E+EVex6MYVRHo JPDJMt7qTtXQWYIGKmf315m+Al2xXL1UX4BH1xCw4Oo0mjO/8cdoH146sYSB7F2rkvZLtvJL5B8eu mF2c5g3ZwkN1MFOkohiP+YA/vKGOCZHqHAbAgxEjFxWG+I/GUjS54kZWeeDdQJxwogwZR4i6tuKnQ cxwAOqplI+x9fzixVeQqnpZPyBrtKN/Geri3XNOMivOTq1MSugiMwkQw4ChFzSgvIenewyB/mUhVM HwDbNqSMOVkCVHnaQb8dm14B+Q3f60t805a5ERRYk1diQ4jFdG9j7O0szNt5BKuZnZKb0jRNSugsZ GLuyL0sw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1shOPt-0000000FaQV-19BF; Fri, 23 Aug 2024 07:09:01 +0000 Received: from mail-tyzapc01on20606.outbound.protection.outlook.com ([2a01:111:f403:2011::606] helo=APC01-TYZ-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1shONi-0000000FZeG-0bTV for linux-arm-kernel@lists.infradead.org; Fri, 23 Aug 2024 07:06:47 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=V2XqJyBuiBaBqYz6lWfcaWt00Yypr2OqNfg4jG8lJnn3eNXck0P/eeP+lVZPpNTBOdN6zAET9pgxOl1JrBPnwoNpKP4xzWr4pT+YToUTimX3VsrQUpQWW3fGHnhxxtnqVGntuqecNk2YVWsrIMK5pTUQGDBEJ/ufN3kkQMiIm99XzWamgycSe6qSvu0GzfPekY8uXxCsu1Cv7BDqOerg5G+jt8DKaQ9SCPvQYjNGMf++ZbcT1aNQyyHtJNn5tvi9foCSKAXvUEU4ppxwOQOiqSdd7Hu2ucskzxkDBZxLQx9aR9ranYXklTjb9NpUVN/vT2Rn4mDkuKbsQ2Z2OBbFkQ== 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=mI0K8rSXqPnuSQqybgOp1bYWqZhcAzOMfVU1VL/kzuc=; b=EnRqHXtUl7BeofEex/GAK+XTLzGBTHGk50Zb1H/yYHdK8bOYIPQZaJHp2GWzWzjgt7HcZo4+wrIGp3dm1v2ZDUAo/Pm8Tqere2k/NJMX64VTMvmIJhXquKWc94r5rTYZXxQCL980j1gf/nWl6nVaETZov+u9a5BDuZ5hKISqgkth+oRQtYVTIGTgYTJI6nuMZehOKHUf0tBGHsTKm9ojvgrV7AIfyfEcVopZIlQp9R7tFKMLS6vOy8vEmentvu41hidj0Lm6R2ibyN7WviK5Zq1vIg9Oi3utr8EyWtI4eqcel+0Qs5D7LmTUfI8WecdtutB8/os6WZ4lomCviby3tw== 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=mI0K8rSXqPnuSQqybgOp1bYWqZhcAzOMfVU1VL/kzuc=; b=Ijze0GV/JRwxE84JbpPTQdtc0dTu3EW5jnZpsMExVPUhff1Vz8l4j827uM6I9DU2VCMuYStacq82osSEncTy1PtToXj4Y7rv2zT8m2TsXXcK+nuDrEmsqQRLaDYpY/Keyu/HAoKZanF0LklTsPd67asKgjclQB6lAO+3aAkzyGEMBQX2HMym4Z7w/Ry+6KNJYVSN4wuRPtHPUE0K4ScxwV985eSx0n68M8ZGkfF4C7oIdQdqjEZJknclajFlCR0WjlPLr1Aw41qwDY0pfFVSIRmMVWm7DNAH3yL6cenejXktWCueIGISChOEP4YsSIpD6/XXNEGw0ZjXewxLaukdMQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; Received: from SEZPR06MB5269.apcprd06.prod.outlook.com (2603:1096:101:78::6) by SEYPR06MB7069.apcprd06.prod.outlook.com (2603:1096:101:1d6::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7875.21; Fri, 23 Aug 2024 07:06:35 +0000 Received: from SEZPR06MB5269.apcprd06.prod.outlook.com ([fe80::8c74:6703:81f7:9535]) by SEZPR06MB5269.apcprd06.prod.outlook.com ([fe80::8c74:6703:81f7:9535%7]) with mapi id 15.20.7875.023; Fri, 23 Aug 2024 07:06:35 +0000 From: Yangtao Li To: clement.leger@bootlin.com, andrew@lunn.ch, f.fainelli@gmail.com, olteanv@gmail.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, ulli.kroll@googlemail.com, linus.walleij@linaro.org, marcin.s.wojtas@gmail.com, linux@armlinux.org.uk, alexandre.torgue@foss.st.com, joabreu@synopsys.com, mcoquelin.stm32@gmail.com, hkallweit1@gmail.com, kees@kernel.org, justinstitt@google.com, u.kleine-koenig@pengutronix.de, horms@kernel.org, sd@queasysnail.net Cc: linux-renesas-soc@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, Yangtao Li , Maxime Chevallier Subject: [net-next v2 2/9] net: stmmac: platform: Convert to devm_clk_get_enabled() and devm_clk_get_optional_enabled() Date: Fri, 23 Aug 2024 01:21:14 -0600 Message-Id: <20240823072122.2053401-3-frank.li@vivo.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240823072122.2053401-1-frank.li@vivo.com> References: <20240823072122.2053401-1-frank.li@vivo.com> X-ClientProxiedBy: SI2PR04CA0001.apcprd04.prod.outlook.com (2603:1096:4:197::12) To SEZPR06MB5269.apcprd06.prod.outlook.com (2603:1096:101:78::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SEZPR06MB5269:EE_|SEYPR06MB7069:EE_ X-MS-Office365-Filtering-Correlation-Id: 27bf6480-d3b4-49c4-f4fd-08dcc342202c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|52116014|7416014|366016|1800799024|38350700014|921020; X-Microsoft-Antispam-Message-Info: jkMcSo5NHHu2p9IMrktwdoUcchvuIQ3g013ybIaMu5JkilOnYEoQKyHM4QxyivnclIvuECwKilOyS7iX+0BxJSZlh7BffqKNfPg9EDh35frQpl7FUYzus9kBtdVknVG9HQy0UtoF4ftJwkBwXkJKAUdnjJsnZqigU9M+YfvQ/t3I69XZljZh6VWjA1EIhWo9eKdYwRlC9L5k6aAzBxBhXeUprO+Prz/PAf/0RkIGLdRO9pr7ZLI+l95hRgk2IuzZ1IW1noC8jgl3G3avN/vl8dujzd42uoNk8EFcerqg+dgY7JIk3dyiZijm3DNk2Kx9nG1YBdPKr0IW2ragEw6ROazOngA5MzpBpz3/uQMHYHkpk9MVkbBEiziayur69DcFrEWbhRBI4phDYRi1Gdu5QQIHY2WLFqVLDVt1TXZK/e9PmzNsCOkbcQJO+tTJeVaykCqHn1pJFcowl4MTQW7h3+mnzcz2jQ/SXyqw0xRDyFoPeEqJfbFRC8jpe+s+LPvIdBW7xWO7Dnw3GlFQehboR9ESLCCIttQigv/bQqG8s8kLuA0dPZ9GzNdXQz4xwk+vsprrkmRW1u1TFj5XJ/XsUZytegHeb8aSHl1X7t2QvRW21mrKOrgHXUhlNqM19T11J93iie6K16m8f1UTcGYx0TlN35S4lroexzE4h/tFyzto8KdIJKkOID49qrF5rl7TGyZjtyCxs4TLjRopNeJCAt/M1OaX9aF8STrwB/kNFeNr5rHByI1x1tJAJqvWYioxvxiYStEHDWiujvBCEQLrODybmjk77iqR0ABun5WFcD8zhqn/9k8HRujssLm7qPzP+m2P6WPRgrFEyL/W9UL17cEuhTN/wmslFqgzM5yYlOV7cuOw0ywYUaUDekMVp8WrjfZMkAmkf+ONGIfEoRmHsg0zdSTr6aRyJFxw7eehXtZuYi2sN7jYDIXp4Dsg9Pp7lBRJ05i3U6ebcRhr1IYYyAQj3z7SA/6H1cHyRJ22H6m9VmPJAHpUH6SQanSxs9eCclYpcTMOeY3rl1SlqQUVdS6+xZZSbgZJ30Of4veG7ivIAEl2VzRmU1cFM/kU0dgz/dX9+wh1VEXJOojBYJ3NxzW3ly6tqyWwk87GgHpMl1m033RrNlkZ7NRyLz411xxT0tiekRkIrmL7BH9ZJSwYguD0EKO5bXfe4OAnPB3Vkm6CfOLhhMXR6Hcq5h3iZss4P6+v5xdu11mTIZcfVEbra0GfFDITM2RuRwVeiu7u9DcTaZTsXRC91EZiflULzjyiyuFt7xKtAOlAjMT6/kgGqwIbMPL5Vh8O1UuvTb6Rj051tQoswWfHT8NCnPu/24WyYL+aaQA+rGNM4WzLE4vgLRSDqr+mj1TWMMnsYZO8fxpT5c4mGVAYQwBOJEk4xKOxgj95gEk6QR2vJzZQSKMRxnFMjSyCINe3wNo3WfYOqgA= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SEZPR06MB5269.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(52116014)(7416014)(366016)(1800799024)(38350700014)(921020);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: f7t1JkJLLWA9qGccmbYE8XifZOReHu8/CjhcFIIEzSyWsuurlsiYt5oPsBbuioFMwfesL2XB6brUXEG2J3S+nnoiyB4nfYiUjjF+yPp1CGmC5K4U7Ey687/2DhoDiJhoCbMdVLKnVn37SvTnSIzewWHHVlMaXqTMa9FuPwxnAQlFyoqpL6u0SlwwVgjg3F2YdvgcdVgSwXWCk2vb/HL/8stH+mRQyHHtvqm/TFUcWJegVFh+wg7bMe43itKSpweWXvYNt1wfX9sH0ieJs1tfLyBZTqC/UkzlRUDcPtNGhLvu97pcXP2eWlxIXxpwlL67MwUv3RJwvDaj1aNoWqoVvlVZnocLajtLPwV4lcpBJPCKL8cQGdEmf9h7UjaKQrRWkx7ynz62UlsVddlzcRwEPMQSXtgunk8W+Rc/7bvIgsP5zyu1qBYzeSmIqiZUHba6ayl/USe4yjw4xe6Kvda2kHIaXAMZvJmrGkyqdFoFUHWGsbVIkkR4itLeWf6YchcojVzC+6LR7q1CcC003xNSfa1NhBHVDMH++7GQwQUMogz5izzzcQL91gKzjw0+jL80JSjb7dmArE252st2qNBVj1YkgZWLaSlYVxdr8yRFmjoOM3KNuF/f2iL3I6Uzmgfp6KatMBbsGl1PIUoJxVx//0fKitwkNpYwg7SpzcZRVRv+1dyRh08YPdWQVqL+DFSmfdv/NQ3ig//K70v1M1zaC0rax9tLzam2/VxfxNcvNbTlnyu33J0xFefyP0p3O42Z37/OJypxfyj4a4rkFGUH5kEqw/GQLguFUWFdWfg3ikI1JweXDcym6B2Dq/Cts0bg0Rxuiuynv3m/tDRKq5Yk9CE5gL8wAcTlVyRrW41nAZnBonzEr+78mTJl+Imq/z8gz0VKh/Rt1aj0oHUFUhbUgTGcJop+DoQvYoa365wC4hIBjRamrk83GeX8cP2GotktvRPtWMiQhB6UHyxRfiPUbepxAE+803E7Em4krgIMvnFXI/ICLb2mM2i2rCX8/AMLFHlIY1rUAF8sM16kYgXjaa5QDZHgycoitM6Xe5JO30AYzY/HWbrakfM3WZlI4AJM2oiwjcrBmJvKOX1+3Gq2jRsfoBwrmVhvCsqpQPkE7I6fBpnnyzJk6kADYnnkx6hDS6gZSCNys/wSgxtaZCOSGTnwNQNGeLvBKmqfLhW7mgtJbfAl12L+910TbOHH9q72cQ+nRkUXnaiyRArgb/OIJWoBubQN+JR8cVkpTbGujzOtM6QcSIfxdKz9vA/Za4+KeOpsChkX+PKXIHJTa4OJBpzKJ9DJVlEDM6yQ7iEUETksrNC9oT+lJZixkuWCKaw8zQFYQMsN3hkvcEU70mpNTiA7BkZisVFpfl0Uiw3F0WRGeFNwX+gAKecdBvnWI/jsM4NiBFYsP7MDOEScfSrYsXfsCDkS0LLGxXNIOrthXXW8CIjDzNrCAolzNmpZdDmMIwvLCeZV6lxRa4ZGLbsCA/Zekf+XaxbCLMd/nMQOf2NTcEVvMdTn3BO3U6JrM1dGxGBKKm/VzQ6HWkMaYVQeOwFp5UzeTeYdct9taxYe9OSKIfRERzOigNhgIZoANvwU X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 27bf6480-d3b4-49c4-f4fd-08dcc342202c X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB5269.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Aug 2024 07:06:35.6439 (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: W6c4Zj9/d630eUn5u1SuLXVODqGG6GB84nZRtR6aH1esL09uZoy7VmgU0fSM3JOAifTLcc4Tdwx+WxGBOtfCeA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEYPR06MB7069 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240823_000646_218762_25115F67 X-CRM114-Status: GOOD ( 10.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Use devm_clk_get_enabled() and devm_clk_get_optional_enabled() to simplify code. Signed-off-by: Yangtao Li Reviewed-by: Maxime Chevallier --- v2: -remove unused 'ret' -fix incompatible-pointer-types .../ethernet/stmicro/stmmac/stmmac_platform.c | 35 +++++-------------- 1 file changed, 8 insertions(+), 27 deletions(-) diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c index ad868e8d195d..1a66baaa4081 100644 --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c @@ -415,8 +415,6 @@ static int stmmac_of_get_mac_mode(struct device_node *np) static void stmmac_remove_config_dt(struct platform_device *pdev, struct plat_stmmacenet_data *plat) { - clk_disable_unprepare(plat->stmmac_clk); - clk_disable_unprepare(plat->pclk); of_node_put(plat->phy_node); of_node_put(plat->mdio_node); } @@ -436,7 +434,6 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac) struct plat_stmmacenet_data *plat; struct stmmac_dma_cfg *dma_cfg; int phy_mode; - void *ret; int rc; plat = devm_kzalloc(&pdev->dev, sizeof(*plat), GFP_KERNEL); @@ -615,21 +612,16 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac) /* clock setup */ if (!of_device_is_compatible(np, "snps,dwc-qos-ethernet-4.10")) { - plat->stmmac_clk = devm_clk_get(&pdev->dev, - STMMAC_RESOURCE_NAME); + plat->stmmac_clk = devm_clk_get_enabled(&pdev->dev, STMMAC_RESOURCE_NAME); if (IS_ERR(plat->stmmac_clk)) { dev_warn(&pdev->dev, "Cannot get CSR clock\n"); plat->stmmac_clk = NULL; } - clk_prepare_enable(plat->stmmac_clk); } - plat->pclk = devm_clk_get_optional(&pdev->dev, "pclk"); - if (IS_ERR(plat->pclk)) { - ret = plat->pclk; - goto error_pclk_get; - } - clk_prepare_enable(plat->pclk); + plat->pclk = devm_clk_get_optional_enabled(&pdev->dev, "pclk"); + if (IS_ERR(plat->pclk)) + return (void *)plat->pclk; /* Fall-back to main clock in case of no PTP ref is passed */ plat->clk_ptp_ref = devm_clk_get(&pdev->dev, "ptp_ref"); @@ -644,26 +636,15 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac) plat->stmmac_rst = devm_reset_control_get_optional(&pdev->dev, STMMAC_RESOURCE_NAME); - if (IS_ERR(plat->stmmac_rst)) { - ret = plat->stmmac_rst; - goto error_hw_init; - } + if (IS_ERR(plat->stmmac_rst)) + return (void *)plat->stmmac_rst; plat->stmmac_ahb_rst = devm_reset_control_get_optional_shared( &pdev->dev, "ahb"); - if (IS_ERR(plat->stmmac_ahb_rst)) { - ret = plat->stmmac_ahb_rst; - goto error_hw_init; - } + if (IS_ERR(plat->stmmac_ahb_rst)) + return (void *)plat->stmmac_ahb_rst; return plat; - -error_hw_init: - clk_disable_unprepare(plat->pclk); -error_pclk_get: - clk_disable_unprepare(plat->stmmac_clk); - - return ret; } static void devm_stmmac_remove_config_dt(void *data)