Commit e0221553 authored by Ethan Nelson-Moore's avatar Ethan Nelson-Moore Committed by Jakub Kicinski
Browse files

net: remove unnecessary module_init/exit functions 



Many network drivers have unnecessary empty module_init and module_exit
functions. Remove them (including some that just print a message). Note
that if a module_init function exists, a module_exit function must also
exist; otherwise, the module cannot be unloaded.

Signed-off-by: default avatarEthan Nelson-Moore <enelsonmoore@gmail.com>
Acked-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
Acked-by: default avatarMichael Grzeschik <m.grzeschik@pengutronix.de>
Reviewed-by: default avatarIdo Schimmel <idosch@nvidia.com>
Acked-by: default avatarPing-Ke Shih <pkshih@realtek.com>
Acked-by: default avatarToke Høiland-Jørgensen <toke@toke.dk>
Link: https://patch.msgid.link/20260131004327.18112-1-enelsonmoore@gmail.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 14e6cf84

drivers/net/arcnet/com20020.c

+0 −16
Original line number Diff line number Diff line
@@ -401,19 +401,3 @@ EXPORT_SYMBOL(com20020_netdev_ops); 


MODULE_DESCRIPTION("ARCnet COM20020 chipset core driver");

MODULE_LICENSE("GPL");
 


#ifdef MODULE



static int __init com20020_module_init(void)

{

	if (BUGLVL(D_NORMAL))

		pr_info("%s\n", "COM20020 chipset support (by David Woodhouse et al.)");

	return 0;

}



static void __exit com20020_module_exit(void)

{

}

module_init(com20020_module_init);

module_exit(com20020_module_exit);

#endif				/* MODULE */

drivers/net/can/sja1000/sja1000.c

+0 −16
Original line number Diff line number Diff line
@@ -705,19 +705,3 @@ void unregister_sja1000dev(struct net_device *dev) 
	unregister_candev(dev);

}

EXPORT_SYMBOL_GPL(unregister_sja1000dev);
 


static __init int sja1000_init(void)

{

	printk(KERN_INFO "%s CAN netdevice driver\n", DRV_NAME);



	return 0;

}



module_init(sja1000_init);



static __exit void sja1000_exit(void)

{

	printk(KERN_INFO "%s: driver removed\n", DRV_NAME);

}



module_exit(sja1000_exit);

drivers/net/ethernet/8390/8390.c

+0 −14
Original line number Diff line number Diff line
@@ -86,19 +86,5 @@ void NS8390_init(struct net_device *dev, int startp) 
}

EXPORT_SYMBOL(NS8390_init);



#if defined(MODULE)



static int __init ns8390_module_init(void)

{

	return 0;

}



static void __exit ns8390_module_exit(void)

{

}



module_init(ns8390_module_init);

module_exit(ns8390_module_exit);

#endif /* MODULE */

MODULE_DESCRIPTION("National Semiconductor 8390 core driver");

MODULE_LICENSE("GPL");

drivers/net/ethernet/8390/8390p.c

+0 −11
Original line number Diff line number Diff line
@@ -91,16 +91,5 @@ void NS8390p_init(struct net_device *dev, int startp) 
}

EXPORT_SYMBOL(NS8390p_init);



static int __init NS8390p_init_module(void)

{

	return 0;

}



static void __exit NS8390p_cleanup_module(void)

{

}



module_init(NS8390p_init_module);

module_exit(NS8390p_cleanup_module);

MODULE_DESCRIPTION("National Semiconductor 8390 core for ISA driver");

MODULE_LICENSE("GPL");

drivers/net/ethernet/mellanox/mlxsw/pci.c

+0 −12
Original line number Diff line number Diff line
@@ -2542,18 +2542,6 @@ void mlxsw_pci_driver_unregister(struct pci_driver *pci_driver) 
}

EXPORT_SYMBOL(mlxsw_pci_driver_unregister);



static int __init mlxsw_pci_module_init(void)

{

	return 0;

}



static void __exit mlxsw_pci_module_exit(void)

{

}



module_init(mlxsw_pci_module_init);

module_exit(mlxsw_pci_module_exit);



MODULE_LICENSE("Dual BSD/GPL");

MODULE_AUTHOR("Jiri Pirko <jiri@mellanox.com>");

MODULE_DESCRIPTION("Mellanox switch PCI interface driver");