RE: [PATCH] e1000e: rename init/exit module functions to avoid confusion with e1000

From: Loktionov, Aleksandr

Date: Wed Sep 16 2026 - 02:01:28 EST




> -----Original Message-----
> From: Filip Balluch <fballuch@xxxxxxxxxx>
> Sent: Tuesday, September 15, 2026 2:27 PM
> To: Nguyen, Anthony L <anthony.l.nguyen@xxxxxxxxx>; Kitszel,
> Przemyslaw <przemyslaw.kitszel@xxxxxxxxx>
> Cc: andrew+netdev@xxxxxxx; davem@xxxxxxxxxxxxx; edumazet@xxxxxxxxxx;
> kuba@xxxxxxxxxx; pabeni@xxxxxxxxxx; intel-wired-lan@xxxxxxxxxxxxxxxx;
> netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Filip Balluch
> <fballuch@xxxxxxxxxx>
> Subject: [PATCH] e1000e: rename init/exit module functions to avoid
> confusion with e1000
>
> The e1000e driver uses e1000_init_module and e1000_exit_module as its
> module init/exit function names, which are identical to those used by
> the older e1000 driver. Rename them to e1000e_init_module and
> e1000e_exit_module to clearly distinguish e1000e from the legacy e1000
> driver.
>
> Signed-off-by: Filip Balluch <fballuch@xxxxxxxxxx>
> ---
> drivers/net/ethernet/intel/e1000e/netdev.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c
> b/drivers/net/ethernet/intel/e1000e/netdev.c
> index 844f31ab37ad..2a4cf75bce34 100644
> --- a/drivers/net/ethernet/intel/e1000e/netdev.c
> +++ b/drivers/net/ethernet/intel/e1000e/netdev.c
> @@ -8203,31 +8203,31 @@ static struct pci_driver e1000_driver = { };
>
> /**
> - * e1000_init_module - Driver Registration Routine
> + * e1000e_init_module - Driver Registration Routine
> *
> - * e1000_init_module is the first routine called when the driver is
> + * e1000e_init_module is the first routine called when the driver is
> * loaded. All it does is register with the PCI subsystem.
> **/
> -static int __init e1000_init_module(void)
> +static int __init e1000e_init_module(void)
> {
> pr_info("Intel(R) PRO/1000 Network Driver\n");
> pr_info("Copyright(c) 1999 - 2015 Intel Corporation.\n");
>
> return pci_register_driver(&e1000_driver);
> }
> -module_init(e1000_init_module);
> +module_init(e1000e_init_module);
>
> /**
> - * e1000_exit_module - Driver Exit Cleanup Routine
> + * e1000e_exit_module - Driver Exit Cleanup Routine
> *
> - * e1000_exit_module is called just before the driver is removed
> + * e1000e_exit_module is called just before the driver is removed
> * from memory.
> **/
> -static void __exit e1000_exit_module(void)
> +static void __exit e1000e_exit_module(void)
> {
> pci_unregister_driver(&e1000_driver);
> }
> -module_exit(e1000_exit_module);
> +module_exit(e1000e_exit_module);
>
> MODULE_DESCRIPTION("Intel(R) PRO/1000 Network Driver");
> MODULE_LICENSE("GPL v2");
> --
> 2.55.0

Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@xxxxxxxxx>