From patchwork Mon Sep 18 20:41:32 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= X-Patchwork-Id: 726989 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id AB3F7C46CA1 for ; Mon, 18 Sep 2023 20:44:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229436AbjIRUoe (ORCPT ); Mon, 18 Sep 2023 16:44:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40940 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229449AbjIRUod (ORCPT ); Mon, 18 Sep 2023 16:44:33 -0400 Received: from metis.whiteo.stw.pengutronix.de (metis.whiteo.stw.pengutronix.de [IPv6:2a0a:edc0:2:b01:1d::104]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1279F10A for ; Mon, 18 Sep 2023 13:44:27 -0700 (PDT) Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qiL4x-0006iV-Nk; Mon, 18 Sep 2023 22:42:47 +0200 Received: from [2a0a:edc0:0:900:1d::77] (helo=ptz.office.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1qiL4k-007IhV-K5; Mon, 18 Sep 2023 22:42:34 +0200 Received: from ukl by ptz.office.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from ) id 1qiL4k-002mnC-50; Mon, 18 Sep 2023 22:42:34 +0200 From: =?utf-8?q?Uwe_Kleine-K=C3=B6nig?= To: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: Grygorii Strashko , linux-omap@vger.kernel.org, Bagas Sanjaya , Yang Yingliang , Greg Ungerer , Arnd Bergmann , netdev@vger.kernel.org, kernel@pengutronix.de, =?utf-8?q?Andreas_F=C3=A4rbe?= =?utf-8?q?r?= , Manivannan Sadhasivam , linux-arm-kernel@lists.infradead.org, linux-actions@lists.infradead.org, Andreas Larsson , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , linux-sunxi@lists.linux.dev, Joyce Ooi , Shyam Sundar S K , Bhupesh Sharma , Alex Elder , Wei Fang , Rob Herring , Iyappan Subramanian , Keyur Chudgar , Quan Nguyen , Atul Raut , Heiko Stuebner , Simon Horman , linux-rockchip@lists.infradead.org, Chris Snook , Justin Chen , Florian Fainelli , =?utf-8?b?UmFmYcWCIE1p?= =?utf-8?b?xYJlY2tp?= , Broadcom internal kernel review list , Doug Berger , Li Zetao , Steen Hegelund , Lukas Bulwahn , Nicolas Ferre , Claudiu Beznea , Russell King , Marc Kleine-Budde , Hartley Sweeten , Ruan Jinjie , Hans Ulli Kroll , Linus Walleij , Kalle Valo , Nathan Huckleberry , Gerhard Engleder , Maciej Fijalkowski , Alexander Duyck , Paul Fertser , Sergei Antonov , Vladimir Oltean , Yisen Zhuang , Salil Mehta , Thomas Bogendoerfer , Michal Kubiak , Douglas Miller , Nathan Chancellor , Yuanjun Gong , Zhang Changzhong , Hauke Mehrtens , Karol Gugala , Mateusz Holenko , Gabriel Somlo , Joel Stanley , Sebastian Hesselbarth , Thomas Petazzoni , Marcin Wojtas , Michael Walle , Andrew Lunn , "Russell King (Oracle)" , Krzysztof Kozlowski , Felix Fietkau , John Crispin , Sean Wang , Mark Lee , Lorenzo Bianconi , Matthias Brugger , AngeloGioacchino Del Regno , linux-mediatek@lists.infradead.org, Asmaa Mnebhi , David Thompson , Horatiu Vultur , UNGLinuxDriver@microchip.com, Lars Povlsen , Daniel Machon , Shang XiaoJing , Qiheng Lin , Claudiu Manoil , Alexandre Belloni , Yuri Karpov , Vladimir Zapolskiy , Timur Tabi , Sergey Shtylyov , Geert Uytterhoeven , Yoshihiro Shimoda , Wolfram Sang , Biju Das , linux-renesas-soc@vger.kernel.org, Byungho An , Ralf Baechle , linux-mips@vger.kernel.org, Nicolas Pitre , Steve Glendinning , Andy Shevchenko , Jassi Brar , Ilias Apalodimas , Kunihiko Hayashi , Harshit Mogalapalli , Wells Lu , Florian Fainelli , Thomas Gleixner , Sebastian Andrzej Siewior , Zhengchao Shao , Kevin Brace , Francois Romieu , Michal Simek , Radhey Shyam Pandey , Harini Katakam , Haoyue Xu , huangjunxian , Dan Carpenter , Krzysztof Halasa Subject: [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void Date: Mon, 18 Sep 2023 22:41:32 +0200 Message-Id: <20230918204227.1316886-1-u.kleine-koenig@pengutronix.de> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=12339; i=u.kleine-koenig@pengutronix.de; h=from:subject; bh=uRL0xD4tSDbk8HBljakKJR0DpZM1z56i+WaCKgjL7Kk=; b=owEBbQGS/pANAwAKAY+A+1h9Ev5OAcsmYgBlCLXkgHb1AYcXBsm+1AoNI9Zz8jTCH/W620zP1 6IbyUIVcSCJATMEAAEKAB0WIQQ/gaxpOnoeWYmt/tOPgPtYfRL+TgUCZQi15AAKCRCPgPtYfRL+ TjL4CACpgZdTUBHbAfkKPQoauoamflwuznvGP/dcoFsENWxzxBbtLfH0gbmcJ04pfaUb1DK5dBW YO4QKjUDWoYoUSneGv+deG/vcPg560G+Y5CDgpLT0YwLukZUD5uYyNZTxQlRTf675DhQByb4PCF m+MnMNJLSYH1uHImncdNbyWsFXDIi63fVKBqvkqz9HM8HBZ/SdE9hpsbQ/B1D49tMn+culsC6fz LH7duV3+tPcEf1ygOlVi6zvDLLIWfAWzyRiID9X93wWWMjGlJL5aAQJud/RLUfIs5k0CrlmhseN 9zRLNiIurYbtHwHEai7pwclDIy1u4elXf3JkJUD5j1yvAkhD X-Developer-Key: i=u.kleine-koenig@pengutronix.de; a=openpgp; fpr=0D2511F322BFAB1C1580266BE2DCDD9132669BD6 X-SA-Exim-Connect-IP: 2a0a:edc0:0:c01:1d::a2 X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-omap@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org Hello, this series convert nearly all platform drivers below drivers/net/ethernet to use remove_new. The motivation is to get rid of an integer return code that is (mostly) ignored by the platform driver core and error prone on the driver side. There are 3 drivers I didn't convert (yet): drivers/net/ethernet/ti/cpsw.c drivers/net/ethernet/ti/cpsw_new.c drivers/net/ethernet/ti/am65-cpsw-nuss.c These are a bit more complicated because they don't always return 0 in .remove(). Unless someone is quicker than me, I'll address them in separate patches at a later time. See commit 5c5a7680e67b ("platform: Provide a remove callback that returns no value") for an extended explanation and the eventual goal. There are no interdependencies between the patches. As there are still quite a few drivers to convert, I'm happy about every patch that makes it in. So even if there is a merge conflict with one patch until you apply, please apply the remainder of this series anyhow. Best regards Uwe Uwe Kleine-König (54): net: ethernet: 8390: Convert to platform remove callback returning void net: ethernet: actions: Convert to platform remove callback returning void net: ethernet: aeroflex: Convert to platform remove callback returning void net: ethernet: allwinner: Convert to platform remove callback returning void net: ethernet: altera: Convert to platform remove callback returning void net: ethernet: amd: Convert to platform remove callback returning void net: ethernet: apm: Convert to platform remove callback returning void net: ethernet: apple: Convert to platform remove callback returning void net: ethernet: arc: Convert to platform remove callback returning void net: ethernet: atheros: Convert to platform remove callback returning void net: ethernet: broadcom: Convert to platform remove callback returning void net: ethernet: cadence: Convert to platform remove callback returning void net: ethernet: calxeda: Convert to platform remove callback returning void net: ethernet: cavium: Convert to platform remove callback returning void net: ethernet: cirrus: Convert to platform remove callback returning void net: ethernet: cortina: Convert to platform remove callback returning void net: ethernet: davicom: Convert to platform remove callback returning void net: ethernet: dnet: Convert to platform remove callback returning void net: ethernet: engleder: Convert to platform remove callback returning void net: ethernet: ethoc: Convert to platform remove callback returning void net: ethernet: faraday: Convert to platform remove callback returning void net: ethernet: hisilicon: Convert to platform remove callback returning void net: ethernet: i825xx: Convert to platform remove callback returning void net: ethernet: ibm: Convert to platform remove callback returning void net: ethernet: korina: Convert to platform remove callback returning void net: ethernet: lantiq_etop: Convert to platform remove callback returning void net: ethernet: lantiq_xrx200: Convert to platform remove callback returning void net: ethernet: litex: Convert to platform remove callback returning void net: ethernet: marvell: Convert to platform remove callback returning void net: ethernet: mediatek: Convert to platform remove callback returning void net: ethernet: mellanox: Convert to platform remove callback returning void net: ethernet: micrel: Convert to platform remove callback returning void net: ethernet: microchip: Convert to platform remove callback returning void net: ethernet: moxa:: Convert to platform remove callback returning void net: ethernet: mscc: Convert to platform remove callback returning void net: ethernet: natsemi: Convert to platform remove callback returning void net: ethernet: natsemi: Convert to platform remove callback returning void net: ethernet: ni: Convert to platform remove callback returning void net: ethernet: nxp: Convert to platform remove callback returning void net: ethernet: qualcomm: Convert to platform remove callback returning void net: ethernet: renesas: Convert to platform remove callback returning void net: ethernet: samsung: Convert to platform remove callback returning void net: ethernet: seeq: Convert to platform remove callback returning void net: ethernet: sgi: Convert to platform remove callback returning void net: ethernet: smsc: Convert to platform remove callback returning void net: ethernet: socionext: Convert to platform remove callback returning void net: ethernet: sun: Convert to platform remove callback returning void net: ethernet: sunplus: Convert to platform remove callback returning void net: ethernet: ti: Convert to platform remove callback returning void net: ethernet: tundra: Convert to platform remove callback returning void net: ethernet: via: Convert to platform remove callback returning void net: ethernet: wiznet: Convert to platform remove callback returning void net: ethernet: xilinx: Convert to platform remove callback returning void net: ethernet: xscale: Convert to platform remove callback returning void drivers/net/ethernet/8390/ax88796.c | 6 ++---- drivers/net/ethernet/8390/mcf8390.c | 5 ++--- drivers/net/ethernet/8390/ne.c | 5 ++--- drivers/net/ethernet/actions/owl-emac.c | 6 ++---- drivers/net/ethernet/aeroflex/greth.c | 6 ++---- drivers/net/ethernet/allwinner/sun4i-emac.c | 5 ++--- drivers/net/ethernet/altera/altera_tse_main.c | 6 ++---- drivers/net/ethernet/amd/au1000_eth.c | 6 ++---- drivers/net/ethernet/amd/sunlance.c | 6 ++---- drivers/net/ethernet/amd/xgbe/xgbe-platform.c | 6 ++---- drivers/net/ethernet/apm/xgene-v2/main.c | 6 ++---- drivers/net/ethernet/apm/xgene/xgene_enet_main.c | 6 ++---- drivers/net/ethernet/apple/macmace.c | 6 ++---- drivers/net/ethernet/arc/emac_arc.c | 6 ++---- drivers/net/ethernet/arc/emac_rockchip.c | 5 ++--- drivers/net/ethernet/atheros/ag71xx.c | 8 +++----- drivers/net/ethernet/broadcom/asp2/bcmasp.c | 8 +++----- drivers/net/ethernet/broadcom/bcm4908_enet.c | 6 ++---- drivers/net/ethernet/broadcom/bcm63xx_enet.c | 10 ++++------ drivers/net/ethernet/broadcom/bcmsysport.c | 6 ++---- drivers/net/ethernet/broadcom/bgmac-platform.c | 6 ++---- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 6 ++---- drivers/net/ethernet/broadcom/sb1250-mac.c | 6 ++---- drivers/net/ethernet/cadence/macb_main.c | 6 ++---- drivers/net/ethernet/calxeda/xgmac.c | 6 ++---- drivers/net/ethernet/cavium/octeon/octeon_mgmt.c | 5 ++--- drivers/net/ethernet/cirrus/cs89x0.c | 5 ++--- drivers/net/ethernet/cirrus/ep93xx_eth.c | 8 +++----- drivers/net/ethernet/cirrus/mac89x0.c | 5 ++--- drivers/net/ethernet/cortina/gemini.c | 12 ++++-------- drivers/net/ethernet/davicom/dm9000.c | 6 ++---- drivers/net/ethernet/dnet.c | 6 ++---- drivers/net/ethernet/engleder/tsnep_main.c | 6 ++---- drivers/net/ethernet/ethoc.c | 6 ++---- drivers/net/ethernet/faraday/ftgmac100.c | 5 ++--- drivers/net/ethernet/faraday/ftmac100.c | 5 ++--- drivers/net/ethernet/hisilicon/hip04_eth.c | 6 ++---- drivers/net/ethernet/hisilicon/hisi_femac.c | 6 ++---- drivers/net/ethernet/hisilicon/hix5hd2_gmac.c | 6 ++---- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 6 ++---- drivers/net/ethernet/hisilicon/hns/hns_enet.c | 5 ++--- drivers/net/ethernet/hisilicon/hns_mdio.c | 5 ++--- drivers/net/ethernet/i825xx/sni_82596.c | 5 ++--- drivers/net/ethernet/ibm/ehea/ehea_main.c | 8 +++----- drivers/net/ethernet/ibm/emac/core.c | 6 ++---- drivers/net/ethernet/ibm/emac/mal.c | 6 ++---- drivers/net/ethernet/ibm/emac/rgmii.c | 6 ++---- drivers/net/ethernet/ibm/emac/tah.c | 6 ++---- drivers/net/ethernet/ibm/emac/zmii.c | 6 ++---- drivers/net/ethernet/korina.c | 6 ++---- drivers/net/ethernet/lantiq_etop.c | 6 ++---- drivers/net/ethernet/lantiq_xrx200.c | 6 ++---- drivers/net/ethernet/litex/litex_liteeth.c | 6 ++---- drivers/net/ethernet/marvell/mv643xx_eth.c | 11 ++++------- drivers/net/ethernet/marvell/mvmdio.c | 6 ++---- drivers/net/ethernet/marvell/mvneta.c | 6 ++---- drivers/net/ethernet/marvell/mvneta_bm.c | 6 ++---- drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c | 8 +++----- drivers/net/ethernet/marvell/pxa168_eth.c | 5 ++--- drivers/net/ethernet/mediatek/mtk_eth_soc.c | 6 ++---- .../ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c | 6 ++---- drivers/net/ethernet/micrel/ks8842.c | 5 ++--- drivers/net/ethernet/micrel/ks8851_par.c | 6 ++---- .../net/ethernet/microchip/lan966x/lan966x_main.c | 6 ++---- drivers/net/ethernet/microchip/sparx5/sparx5_main.c | 6 ++---- drivers/net/ethernet/moxa/moxart_ether.c | 6 ++---- drivers/net/ethernet/mscc/ocelot_vsc7514.c | 6 ++---- drivers/net/ethernet/natsemi/jazzsonic.c | 6 ++---- drivers/net/ethernet/natsemi/macsonic.c | 6 ++---- drivers/net/ethernet/natsemi/xtsonic.c | 6 ++---- drivers/net/ethernet/ni/nixge.c | 6 ++---- drivers/net/ethernet/nxp/lpc_eth.c | 6 ++---- drivers/net/ethernet/qualcomm/emac/emac.c | 6 ++---- drivers/net/ethernet/renesas/ravb_main.c | 6 ++---- drivers/net/ethernet/renesas/rswitch.c | 6 ++---- drivers/net/ethernet/renesas/sh_eth.c | 6 ++---- drivers/net/ethernet/samsung/sxgbe/sxgbe_platform.c | 6 ++---- drivers/net/ethernet/seeq/sgiseeq.c | 6 ++---- drivers/net/ethernet/sgi/ioc3-eth.c | 6 ++---- drivers/net/ethernet/sgi/meth.c | 6 ++---- drivers/net/ethernet/smsc/smc91x.c | 6 ++---- drivers/net/ethernet/smsc/smsc911x.c | 6 ++---- drivers/net/ethernet/socionext/netsec.c | 6 ++---- drivers/net/ethernet/socionext/sni_ave.c | 6 ++---- drivers/net/ethernet/sun/niu.c | 5 ++--- drivers/net/ethernet/sun/sunbmac.c | 6 ++---- drivers/net/ethernet/sun/sunqe.c | 6 ++---- drivers/net/ethernet/sunplus/spl2sw_driver.c | 6 ++---- drivers/net/ethernet/ti/cpmac.c | 6 ++---- drivers/net/ethernet/ti/davinci_emac.c | 6 ++---- drivers/net/ethernet/ti/davinci_mdio.c | 6 ++---- drivers/net/ethernet/ti/netcp_core.c | 5 ++--- drivers/net/ethernet/tundra/tsi108_eth.c | 6 ++---- drivers/net/ethernet/via/via-rhine.c | 6 ++---- drivers/net/ethernet/via/via-velocity.c | 6 ++---- drivers/net/ethernet/wiznet/w5100.c | 6 ++---- drivers/net/ethernet/wiznet/w5300.c | 5 ++--- drivers/net/ethernet/xilinx/ll_temac_main.c | 5 ++--- drivers/net/ethernet/xilinx/xilinx_axienet_main.c | 6 ++---- drivers/net/ethernet/xilinx/xilinx_emaclite.c | 6 ++---- drivers/net/ethernet/xscale/ixp4xx_eth.c | 5 ++--- 101 files changed, 213 insertions(+), 399 deletions(-) base-commit: 7fc7222d9680366edeecc219c21ca96310bdbc10