Re: [PATCH net-next 0/9] net: freescale: Convert to platform remove callback returning void
On Mon, Mar 13, 2023 at 11:36:44AM +0100, Uwe Kleine-König wrote: > Hello, > > this patch set converts the platform drivers below > drivers/net/ethernet/freescale to the .remove_new() callback. Compared to the > traditional .remove() this one returns void. This is a good thing because the > driver core (mostly) ignores the return value and still removes the device > binding. This is part of a bigger effort to convert all 2000+ platform > drivers to this new callback to eventually change .remove() itself to > return void. > > The first two patches here are preparation, the following patches > actually convert the drivers. > > Best regards > Uwe > For entire series: Reviewed-by: Michal Kubiak > Uwe Kleine-König (9): > net: dpaa: Improve error reporting > net: fec: Don't return early on error in .remove() > net: dpaa: Convert to platform remove callback returning void > net: fec: Convert to platform remove callback returning void > net: fman: Convert to platform remove callback returning void > net: fs_enet: Convert to platform remove callback returning void > net: fsl_pq_mdio: Convert to platform remove callback returning void > net: gianfar: Convert to platform remove callback returning void > net: ucc_geth: Convert to platform remove callback returning void > > drivers/net/ethernet/freescale/dpaa/dpaa_eth.c| 8 > drivers/net/ethernet/freescale/fec_main.c | 11 --- > drivers/net/ethernet/freescale/fec_mpc52xx.c | 6 ++ > drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 6 ++ > drivers/net/ethernet/freescale/fman/mac.c | 5 ++--- > drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c | 5 ++--- > drivers/net/ethernet/freescale/fs_enet/mii-bitbang.c | 6 ++ > drivers/net/ethernet/freescale/fs_enet/mii-fec.c | 6 ++ > drivers/net/ethernet/freescale/fsl_pq_mdio.c | 6 ++ > drivers/net/ethernet/freescale/gianfar.c | 6 ++ > drivers/net/ethernet/freescale/ucc_geth.c | 6 ++ > 11 files changed, 26 insertions(+), 45 deletions(-) > > base-commit: fe15c26ee26efa11741a7b632e9f23b01aca4cc6 > -- > 2.39.1 >
RE: [PATCH net-next 0/9] net: freescale: Convert to platform remove callback returning void
> -Original Message- > From: Uwe Kleine-König > Sent: 13 March 2023 12:37 > To: Madalin Bucur ; David S. Miller > ; Eric Dumazet ; Jakub Kicinski > ; Paolo Abeni ; Russell King > ; Wei Fang ; Wolfram Sang > ; Chris Packham ; Andy > Shevchenko ; Damien Le Moal > ; Christophe Leroy > ; Michael Ellerman ; > Mark Brown ; Marc Kleine-Budde ; > Pantelis Antoniou ; Claudiu Manoil > ; Leo Li > Cc: net...@vger.kernel.org; ker...@pengutronix.de; Shenwei Wang > ; Clark Wang ; dl-linux-imx > ; linuxppc-dev@lists.ozlabs.org > Subject: [PATCH net-next 0/9] net: freescale: Convert to platform remove > callback returning void > > Hello, > > this patch set converts the platform drivers below > drivers/net/ethernet/freescale to the .remove_new() callback. Compared to > the > traditional .remove() this one returns void. This is a good thing because > the > driver core (mostly) ignores the return value and still removes the > device > binding. This is part of a bigger effort to convert all 2000+ platform > drivers to this new callback to eventually change .remove() itself to > return void. > > The first two patches here are preparation, the following patches > actually convert the drivers. > > Best regards > Uwe > > Uwe Kleine-König (9): > net: dpaa: Improve error reporting > net: fec: Don't return early on error in .remove() > net: dpaa: Convert to platform remove callback returning void > net: fec: Convert to platform remove callback returning void > net: fman: Convert to platform remove callback returning void > net: fs_enet: Convert to platform remove callback returning void > net: fsl_pq_mdio: Convert to platform remove callback returning void > net: gianfar: Convert to platform remove callback returning void > net: ucc_geth: Convert to platform remove callback returning void > > drivers/net/ethernet/freescale/dpaa/dpaa_eth.c| 8 > drivers/net/ethernet/freescale/fec_main.c | 11 --- > drivers/net/ethernet/freescale/fec_mpc52xx.c | 6 ++ > drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 6 ++ > drivers/net/ethernet/freescale/fman/mac.c | 5 ++--- > drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c | 5 ++--- > drivers/net/ethernet/freescale/fs_enet/mii-bitbang.c | 6 ++ > drivers/net/ethernet/freescale/fs_enet/mii-fec.c | 6 ++ > drivers/net/ethernet/freescale/fsl_pq_mdio.c | 6 ++ > drivers/net/ethernet/freescale/gianfar.c | 6 ++ > drivers/net/ethernet/freescale/ucc_geth.c | 6 ++ > 11 files changed, 26 insertions(+), 45 deletions(-) > > base-commit: fe15c26ee26efa11741a7b632e9f23b01aca4cc6 > -- > 2.39.1 For the FMan and DPAA drivers, Acked-by: Madalin Bucur
[PATCH net-next 0/9] net: freescale: Convert to platform remove callback returning void
Hello, this patch set converts the platform drivers below drivers/net/ethernet/freescale to the .remove_new() callback. Compared to the traditional .remove() this one returns void. This is a good thing because the driver core (mostly) ignores the return value and still removes the device binding. This is part of a bigger effort to convert all 2000+ platform drivers to this new callback to eventually change .remove() itself to return void. The first two patches here are preparation, the following patches actually convert the drivers. Best regards Uwe Uwe Kleine-König (9): net: dpaa: Improve error reporting net: fec: Don't return early on error in .remove() net: dpaa: Convert to platform remove callback returning void net: fec: Convert to platform remove callback returning void net: fman: Convert to platform remove callback returning void net: fs_enet: Convert to platform remove callback returning void net: fsl_pq_mdio: Convert to platform remove callback returning void net: gianfar: Convert to platform remove callback returning void net: ucc_geth: Convert to platform remove callback returning void drivers/net/ethernet/freescale/dpaa/dpaa_eth.c| 8 drivers/net/ethernet/freescale/fec_main.c | 11 --- drivers/net/ethernet/freescale/fec_mpc52xx.c | 6 ++ drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 6 ++ drivers/net/ethernet/freescale/fman/mac.c | 5 ++--- drivers/net/ethernet/freescale/fs_enet/fs_enet-main.c | 5 ++--- drivers/net/ethernet/freescale/fs_enet/mii-bitbang.c | 6 ++ drivers/net/ethernet/freescale/fs_enet/mii-fec.c | 6 ++ drivers/net/ethernet/freescale/fsl_pq_mdio.c | 6 ++ drivers/net/ethernet/freescale/gianfar.c | 6 ++ drivers/net/ethernet/freescale/ucc_geth.c | 6 ++ 11 files changed, 26 insertions(+), 45 deletions(-) base-commit: fe15c26ee26efa11741a7b632e9f23b01aca4cc6 -- 2.39.1