On Mon, 18 Sep 2023 14:16:58 +0200, Heiner Kallweit wrote: > Simplify the module boilerplate code. > > Signed-off-by: Heiner Kallweit <hkallweit1@xxxxxxxxx> > --- > drivers/i2c/busses/i2c-i801.c | 12 +++--------- > 1 file changed, 3 insertions(+), 9 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c > index 811541797..6d02a8b88 100644 > --- a/drivers/i2c/busses/i2c-i801.c > +++ b/drivers/i2c/busses/i2c-i801.c > @@ -1847,16 +1847,11 @@ static struct pci_driver i801_driver = { > }, > }; > > -static int __init i2c_i801_init(void) > +static int __init i2c_i801_init(struct pci_driver *drv) > { > if (dmi_name_in_vendors("FUJITSU")) > input_apanel_init(); > - return pci_register_driver(&i801_driver); > -} > - > -static void __exit i2c_i801_exit(void) > -{ > - pci_unregister_driver(&i801_driver); > + return pci_register_driver(drv); > } > > MODULE_AUTHOR("Mark D. Studebaker <mdsxyz123@xxxxxxxxx>"); > @@ -1864,5 +1859,4 @@ MODULE_AUTHOR("Jean Delvare <jdelvare@xxxxxxx>"); > MODULE_DESCRIPTION("I801 SMBus driver"); > MODULE_LICENSE("GPL"); > > -module_init(i2c_i801_init); > -module_exit(i2c_i801_exit); > +module_driver(i801_driver, i2c_i801_init, pci_unregister_driver); Reviewed-by: Jean Delvare <jdelvare@xxxxxxx> Thanks, -- Jean Delvare SUSE L3 Support