lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20230918204227.1316886-1-u.kleine-koenig@pengutronix.de>
Date: Mon, 18 Sep 2023 22:41:32 +0200
From: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
To: "David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>,
	Paolo Abeni <pabeni@...hat.com>
Cc: Grygorii Strashko <grygorii.strashko@...com>,
	linux-omap@...r.kernel.org,
	Bagas Sanjaya <bagasdotme@...il.com>,
	Yang Yingliang <yangyingliang@...wei.com>,
	Greg Ungerer <gerg@...ux-m68k.org>,
	Arnd Bergmann <arnd@...db.de>,
	netdev@...r.kernel.org,
	kernel@...gutronix.de,
	Andreas Färber <afaerber@...e.de>,
	Manivannan Sadhasivam <mani@...nel.org>,
	linux-arm-kernel@...ts.infradead.org,
	linux-actions@...ts.infradead.org,
	Andreas Larsson <andreas@...sler.com>,
	Chen-Yu Tsai <wens@...e.org>,
	Jernej Skrabec <jernej.skrabec@...il.com>,
	Samuel Holland <samuel@...lland.org>,
	linux-sunxi@...ts.linux.dev,
	Joyce Ooi <joyce.ooi@...el.com>,
	Shyam Sundar S K <Shyam-sundar.S-k@....com>,
	Bhupesh Sharma <bhupesh.sharma@...aro.org>,
	Alex Elder <elder@...aro.org>,
	Wei Fang <wei.fang@....com>,
	Rob Herring <robh@...nel.org>,
	Iyappan Subramanian <iyappan@...amperecomputing.com>,
	Keyur Chudgar <keyur@...amperecomputing.com>,
	Quan Nguyen <quan@...amperecomputing.com>,
	Atul Raut <rauji.raut@...il.com>,
	Heiko Stuebner <heiko@...ech.de>,
	Simon Horman <horms@...nel.org>,
	linux-rockchip@...ts.infradead.org,
	Chris Snook <chris.snook@...il.com>,
	Justin Chen <justin.chen@...adcom.com>,
	Florian Fainelli <florian.fainelli@...adcom.com>,
	Rafał Miłecki <rafal@...ecki.pl>,
	Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>,
	Doug Berger <opendmb@...il.com>,
	Li Zetao <lizetao1@...wei.com>,
	Steen Hegelund <Steen.Hegelund@...rochip.com>,
	Lukas Bulwahn <lukas.bulwahn@...il.com>,
	Nicolas Ferre <nicolas.ferre@...rochip.com>,
	Claudiu Beznea <claudiu.beznea@...on.dev>,
	Russell King <linux@...linux.org.uk>,
	Marc Kleine-Budde <mkl@...gutronix.de>,
	Hartley Sweeten <hsweeten@...ionengravers.com>,
	Ruan Jinjie <ruanjinjie@...wei.com>,
	Hans Ulli Kroll <ulli.kroll@...glemail.com>,
	Linus Walleij <linus.walleij@...aro.org>,
	Kalle Valo <kvalo@...nel.org>,
	Nathan Huckleberry <nhuck@...gle.com>,
	Gerhard Engleder <gerhard@...leder-embedded.com>,
	Maciej Fijalkowski <maciej.fijalkowski@...el.com>,
	Alexander Duyck <alexanderduyck@...com>,
	Paul Fertser <fercerpav@...il.com>,
	Sergei Antonov <saproj@...il.com>,
	Vladimir Oltean <vladimir.oltean@....com>,
	Yisen Zhuang <yisen.zhuang@...wei.com>,
	Salil Mehta <salil.mehta@...wei.com>,
	Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
	Michal Kubiak <michal.kubiak@...el.com>,
	Douglas Miller <dougmill@...ux.ibm.com>,
	Nathan Chancellor <nathan@...nel.org>,
	Yuanjun Gong <ruc_gongyuanjun@....com>,
	Zhang Changzhong <zhangchangzhong@...wei.com>,
	Hauke Mehrtens <hauke@...ke-m.de>,
	Karol Gugala <kgugala@...micro.com>,
	Mateusz Holenko <mholenko@...micro.com>,
	Gabriel Somlo <gsomlo@...il.com>,
	Joel Stanley <joel@....id.au>,
	Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
	Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
	Marcin Wojtas <mw@...ihalf.com>,
	Michael Walle <michael@...le.cc>,
	Andrew Lunn <andrew@...n.ch>,
	"Russell King (Oracle)" <rmk+kernel@...linux.org.uk>,
	Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
	Felix Fietkau <nbd@....name>,
	John Crispin <john@...ozen.org>,
	Sean Wang <sean.wang@...iatek.com>,
	Mark Lee <Mark-MC.Lee@...iatek.com>,
	Lorenzo Bianconi <lorenzo@...nel.org>,
	Matthias Brugger <matthias.bgg@...il.com>,
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
	linux-mediatek@...ts.infradead.org,
	Asmaa Mnebhi <asmaa@...dia.com>,
	David Thompson <davthompson@...dia.com>,
	Horatiu Vultur <horatiu.vultur@...rochip.com>,
	UNGLinuxDriver@...rochip.com,
	Lars Povlsen <lars.povlsen@...rochip.com>,
	Daniel Machon <daniel.machon@...rochip.com>,
	Shang XiaoJing <shangxiaojing@...wei.com>,
	Qiheng Lin <linqiheng@...wei.com>,
	Claudiu Manoil <claudiu.manoil@....com>,
	Alexandre Belloni <alexandre.belloni@...tlin.com>,
	Yuri Karpov <YKarpov@...ras.ru>,
	Vladimir Zapolskiy <vz@...ia.com>,
	Timur Tabi <timur@...nel.org>,
	Sergey Shtylyov <s.shtylyov@....ru>,
	Geert Uytterhoeven <geert+renesas@...der.be>,
	Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>,
	Wolfram Sang <wsa+renesas@...g-engineering.com>,
	Biju Das <biju.das.jz@...renesas.com>,
	linux-renesas-soc@...r.kernel.org,
	Byungho An <bh74.an@...sung.com>,
	Ralf Baechle <ralf@...ux-mips.org>,
	linux-mips@...r.kernel.org,
	Nicolas Pitre <nico@...xnic.net>,
	Steve Glendinning <steve.glendinning@...well.net>,
	Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
	Jassi Brar <jaswinder.singh@...aro.org>,
	Ilias Apalodimas <ilias.apalodimas@...aro.org>,
	Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>,
	Harshit Mogalapalli <harshit.m.mogalapalli@...cle.com>,
	Wells Lu <wellslutw@...il.com>,
	Florian Fainelli <f.fainelli@...il.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
	Zhengchao Shao <shaozhengchao@...wei.com>,
	Kevin Brace <kevinbrace@...cecomputerlab.com>,
	Francois Romieu <romieu@...zoreil.com>,
	Michal Simek <michal.simek@....com>,
	Radhey Shyam Pandey <radhey.shyam.pandey@....com>,
	Harini Katakam <harini.katakam@....com>,
	Haoyue Xu <xuhaoyue1@...ilicon.com>,
	huangjunxian <huangjunxian6@...ilicon.com>,
	Dan Carpenter <dan.carpenter@...aro.org>,
	Krzysztof Halasa <khalasa@...p.pl>
Subject: [PATCH net-next 00/54] net: ethernet: Convert to platform remove callback returning void

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


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ