mbox series

[net-next,00/54] net: ethernet: Convert to platform remove callback returning void

Message ID 20230918204227.1316886-1-u.kleine-koenig@pengutronix.de (mailing list archive)
Headers show
Series net: ethernet: Convert to platform remove callback returning void | expand

Message

Uwe Kleine-König Sept. 18, 2023, 8:41 p.m. UTC
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

Comments

patchwork-bot+netdevbpf@kernel.org Sept. 20, 2023, 8:10 a.m. UTC | #1
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!