ARM Sunxi Platform Development
 help / color / mirror / Atom feed
* [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void
@ 2023-09-18 20:41 Uwe Kleine-König
  2023-09-18 20:41 ` [PATCH net-next 04/54] net: ethernet: allwinner: " Uwe Kleine-König
  2023-09-20  8:10 ` [PATCH net-next 00/54] net: ethernet: " patchwork-bot+netdevbpf
  0 siblings, 2 replies; 4+ messages in thread
From: Uwe Kleine-König @ 2023-09-18 20:41 UTC (permalink / raw)
  To: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni
  Cc: Grygorii Strashko, linux-omap, Bagas Sanjaya, Yang Yingliang,
	Greg Ungerer, Arnd Bergmann, netdev, kernel, Andreas Färber,
	Manivannan Sadhasivam, linux-arm-kernel, linux-actions,
	Andreas Larsson, Chen-Yu Tsai, Jernej Skrabec, Samuel Holland,
	linux-sunxi, 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, Chris Snook, Justin Chen,
	Florian Fainelli, Rafał Miłecki,
	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, Asmaa Mnebhi,
	David Thompson, Horatiu Vultur, UNGLinuxDriver, 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, Byungho An,
	Ralf Baechle, linux-mips, 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

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
-- 
2.40.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH net-next 04/54] net: ethernet: allwinner: Convert to platform remove callback returning void
  2023-09-18 20:41 [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void Uwe Kleine-König
@ 2023-09-18 20:41 ` Uwe Kleine-König
  2023-09-19 17:22   ` Jernej Škrabec
  2023-09-20  8:10 ` [PATCH net-next 00/54] net: ethernet: " patchwork-bot+netdevbpf
  1 sibling, 1 reply; 4+ messages in thread
From: Uwe Kleine-König @ 2023-09-18 20:41 UTC (permalink / raw)
  To: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni
  Cc: Chen-Yu Tsai, Jernej Skrabec, Samuel Holland, netdev,
	linux-arm-kernel, linux-sunxi, kernel

The .remove() callback for a platform driver returns an int which makes
many driver authors wrongly assume it's possible to do error handling by
returning an error code. However the value returned is ignored (apart
from emitting a warning) and this typically results in resource leaks.
To improve here there is a quest to make the remove callback return
void. In the first step of this quest all drivers are converted to
.remove_new() which already returns void. Eventually after all drivers
are converted, .remove_new() is renamed to .remove().

Trivially convert this driver from always returning zero in the remove
callback to the void returning variant.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 drivers/net/ethernet/allwinner/sun4i-emac.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/allwinner/sun4i-emac.c b/drivers/net/ethernet/allwinner/sun4i-emac.c
index a94c62956eed..d761c08fe5c1 100644
--- a/drivers/net/ethernet/allwinner/sun4i-emac.c
+++ b/drivers/net/ethernet/allwinner/sun4i-emac.c
@@ -1083,7 +1083,7 @@ static int emac_probe(struct platform_device *pdev)
 	return ret;
 }
 
-static int emac_remove(struct platform_device *pdev)
+static void emac_remove(struct platform_device *pdev)
 {
 	struct net_device *ndev = platform_get_drvdata(pdev);
 	struct emac_board_info *db = netdev_priv(ndev);
@@ -1101,7 +1101,6 @@ static int emac_remove(struct platform_device *pdev)
 	free_netdev(ndev);
 
 	dev_dbg(&pdev->dev, "released and freed device\n");
-	return 0;
 }
 
 static int emac_suspend(struct platform_device *dev, pm_message_t state)
@@ -1143,7 +1142,7 @@ static struct platform_driver emac_driver = {
 		.of_match_table = emac_of_match,
 	},
 	.probe = emac_probe,
-	.remove = emac_remove,
+	.remove_new = emac_remove,
 	.suspend = emac_suspend,
 	.resume = emac_resume,
 };
-- 
2.40.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next 04/54] net: ethernet: allwinner: Convert to platform remove callback returning void
  2023-09-18 20:41 ` [PATCH net-next 04/54] net: ethernet: allwinner: " Uwe Kleine-König
@ 2023-09-19 17:22   ` Jernej Škrabec
  0 siblings, 0 replies; 4+ messages in thread
From: Jernej Škrabec @ 2023-09-19 17:22 UTC (permalink / raw)
  To: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
	Uwe Kleine-König
  Cc: Chen-Yu Tsai, Samuel Holland, netdev, linux-arm-kernel,
	linux-sunxi, kernel

Dne ponedeljek, 18. september 2023 ob 22:41:36 CEST je Uwe Kleine-König 
napisal(a):
> The .remove() callback for a platform driver returns an int which makes
> many driver authors wrongly assume it's possible to do error handling by
> returning an error code. However the value returned is ignored (apart
> from emitting a warning) and this typically results in resource leaks.
> To improve here there is a quest to make the remove callback return
> void. In the first step of this quest all drivers are converted to
> .remove_new() which already returns void. Eventually after all drivers
> are converted, .remove_new() is renamed to .remove().
> 
> Trivially convert this driver from always returning zero in the remove
> callback to the void returning variant.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej

> ---
>  drivers/net/ethernet/allwinner/sun4i-emac.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/ethernet/allwinner/sun4i-emac.c
> b/drivers/net/ethernet/allwinner/sun4i-emac.c index
> a94c62956eed..d761c08fe5c1 100644
> --- a/drivers/net/ethernet/allwinner/sun4i-emac.c
> +++ b/drivers/net/ethernet/allwinner/sun4i-emac.c
> @@ -1083,7 +1083,7 @@ static int emac_probe(struct platform_device *pdev)
>  	return ret;
>  }
> 
> -static int emac_remove(struct platform_device *pdev)
> +static void emac_remove(struct platform_device *pdev)
>  {
>  	struct net_device *ndev = platform_get_drvdata(pdev);
>  	struct emac_board_info *db = netdev_priv(ndev);
> @@ -1101,7 +1101,6 @@ static int emac_remove(struct platform_device *pdev)
>  	free_netdev(ndev);
> 
>  	dev_dbg(&pdev->dev, "released and freed device\n");
> -	return 0;
>  }
> 
>  static int emac_suspend(struct platform_device *dev, pm_message_t state)
> @@ -1143,7 +1142,7 @@ static struct platform_driver emac_driver = {
>  		.of_match_table = emac_of_match,
>  	},
>  	.probe = emac_probe,
> -	.remove = emac_remove,
> +	.remove_new = emac_remove,
>  	.suspend = emac_suspend,
>  	.resume = emac_resume,
>  };





^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void
  2023-09-18 20:41 [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void Uwe Kleine-König
  2023-09-18 20:41 ` [PATCH net-next 04/54] net: ethernet: allwinner: " Uwe Kleine-König
@ 2023-09-20  8:10 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2023-09-20  8:10 UTC (permalink / raw)
  To: =?utf-8?q?Uwe_Kleine-K=C3=B6nig_=3Cu=2Ekleine-koenig=40pengutronix=2Ede=3E?=
  Cc: davem, edumazet, kuba, pabeni, grygorii.strashko, linux-omap,
	bagasdotme, yangyingliang, gerg, arnd, netdev, kernel, afaerber,
	mani, linux-arm-kernel, linux-actions, andreas, wens,
	jernej.skrabec, samuel, linux-sunxi, joyce.ooi, Shyam-sundar.S-k,
	bhupesh.sharma, elder, wei.fang, robh, iyappan, keyur, quan,
	rauji.raut, heiko, horms, linux-rockchip, chris.snook,
	justin.chen, florian.fainelli, rafal, bcm-kernel-feedback-list,
	opendmb, lizetao1, Steen.Hegelund, lukas.bulwahn, nicolas.ferre,
	claudiu.beznea, linux, mkl, hsweeten, ruanjinjie, ulli.kroll,
	linus.walleij, kvalo, nhuck, gerhard, maciej.fijalkowski,
	alexanderduyck, fercerpav, saproj, vladimir.oltean, yisen.zhuang,
	salil.mehta, tsbogend, michal.kubiak, dougmill, nathan,
	ruc_gongyuanjun, zhangchangzhong, hauke, kgugala, mholenko,
	gsomlo, joel, sebastian.hesselbarth, thomas.petazzoni, mw,
	michael, andrew, rmk+kernel, krzysztof.kozlowski, nbd, john,
	sean.wang, Mark-MC.Lee, lorenzo, matthias.bgg,
	angelogioacchino.delregno, linux-mediatek, asmaa, davthompson,
	horatiu.vultur, UNGLinuxDriver, lars.povlsen, daniel.machon,
	shangxiaojing, linqiheng, claudiu.manoil, alexandre.belloni,
	YKarpov, vz, timur, s.shtylyov, geert+renesas,
	yoshihiro.shimoda.uh, wsa+renesas, biju.das.jz, linux-renesas-soc,
	bh74.an, ralf, linux-mips, nico, steve.glendinning,
	andriy.shevchenko, jaswinder.singh, ilias.apalodimas,
	hayashi.kunihiko, harshit.m.mogalapalli, wellslutw, f.fainelli,
	tglx, bigeasy, shaozhengchao, kevinbrace, romieu, michal.simek,
	radhey.shyam.pandey, harini.katakam, xuhaoyue1, huangjunxian6,
	dan.carpenter, khalasa

Hello:

This series was applied to netdev/net-next.git (main)
by David S. Miller <davem@davemloft.net>:

On Mon, 18 Sep 2023 22:41:32 +0200 you wrote:
> 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.
> 
> [...]

Here is the summary with links:
  - [net-next,01/54] net: ethernet: 8390: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/3c000faaaa65
  - [net-next,02/54] net: ethernet: actions: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/d8c6b3fc918d
  - [net-next,03/54] net: ethernet: aeroflex: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/a0fbde8f647b
  - [net-next,04/54] net: ethernet: allwinner: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/9f07797c332e
  - [net-next,05/54] net: ethernet: altera: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/d0b0245b84c9
  - [net-next,06/54] net: ethernet: amd: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/9ecae52f7db4
  - [net-next,07/54] net: ethernet: apm: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/61471264c018
  - [net-next,08/54] net: ethernet: apple: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/e5d0a380b2b4
  - [net-next,09/54] net: ethernet: arc: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/de0afa130ed6
  - [net-next,10/54] net: ethernet: atheros: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/c6e817838902
  - [net-next,11/54] net: ethernet: broadcom: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/d4295df3e043
  - [net-next,12/54] net: ethernet: cadence: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/6b5940402f9c
  - [net-next,13/54] net: ethernet: calxeda: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/ca76b17922e6
  - [net-next,14/54] net: ethernet: cavium: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/de413f46a56e
  - [net-next,15/54] net: ethernet: cirrus: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/8ca0ae6a2ec7
  - [net-next,16/54] net: ethernet: cortina: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/1f69574b5cba
  - [net-next,17/54] net: ethernet: davicom: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/ffb928290ba5
  - [net-next,18/54] net: ethernet: dnet: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/5db6c1fed1f9
  - [net-next,19/54] net: ethernet: engleder: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/d2e6c2ea996e
  - [net-next,20/54] net: ethernet: ethoc: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/4cc08098d197
  - [net-next,21/54] net: ethernet: faraday: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/dc715af33b42
  - [net-next,22/54] net: ethernet: hisilicon: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/e5835a0a5119
  - [net-next,23/54] net: ethernet: i825xx: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/e5a6d397ca2e
  - [net-next,24/54] net: ethernet: ibm: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/ac35a3c4c8da
  - [net-next,25/54] net: ethernet: korina: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/551598a120d1
  - [net-next,26/54] net: ethernet: lantiq_etop: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/e791aa06a2dd
  - [net-next,27/54] net: ethernet: lantiq_xrx200: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/adbb270655a1
  - [net-next,28/54] net: ethernet: litex: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/36b80b8353af
  - [net-next,29/54] net: ethernet: marvell: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/fee02f498c9a
  - [net-next,30/54] net: ethernet: mediatek: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/5b6ce1734d9d
  - [net-next,31/54] net: ethernet: mellanox: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/e184700959ec
  - [net-next,32/54] net: ethernet: micrel: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/bae04ae2be07
  - [net-next,33/54] net: ethernet: microchip: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/5253ae3a6223
  - [net-next,34/54] net: ethernet: moxa:: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/253f9c47273d
  - [net-next,35/54] net: ethernet: mscc: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/0eaca7a9e9d6
  - [net-next,36/54] net: ethernet: natsemi: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/9f0243d9bcc4
  - [net-next,37/54] net: ethernet: natsemi: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/b83fb509c758
  - [net-next,38/54] net: ethernet: ni: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/0f09f4e7d401
  - [net-next,39/54] net: ethernet: nxp: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/241882d79f99
  - [net-next,40/54] net: ethernet: qualcomm: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/5180236592b9
  - [net-next,41/54] net: ethernet: renesas: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/0b4f04e2f60b
  - [net-next,42/54] net: ethernet: samsung: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/6e8b0ef7b389
  - [net-next,43/54] net: ethernet: seeq: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/02c74030beff
  - [net-next,44/54] net: ethernet: sgi: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/506a327ceaaa
  - [net-next,45/54] net: ethernet: smsc: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/e8e2c863f941
  - [net-next,46/54] net: ethernet: socionext: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/d3cd32451c7e
  - [net-next,47/54] net: ethernet: sun: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/8f4e45a0135e
  - [net-next,48/54] net: ethernet: sunplus: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/ede778ed3ea9
  - [net-next,49/54] net: ethernet: ti: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/231ea972ccaf
  - [net-next,50/54] net: ethernet: tundra: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/2f961952e19e
  - [net-next,51/54] net: ethernet: via: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/70680f178280
  - [net-next,52/54] net: ethernet: wiznet: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/46b5668494d0
  - [net-next,53/54] net: ethernet: xilinx: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/2e0ec0afa902
  - [net-next,54/54] net: ethernet: xscale: Convert to platform remove callback returning void
    https://git.kernel.org/netdev/net-next/c/d74a5c154b6f

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-09-20  8:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-18 20:41 [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void Uwe Kleine-König
2023-09-18 20:41 ` [PATCH net-next 04/54] net: ethernet: allwinner: " Uwe Kleine-König
2023-09-19 17:22   ` Jernej Škrabec
2023-09-20  8:10 ` [PATCH net-next 00/54] net: ethernet: " patchwork-bot+netdevbpf

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox