Message ID | 2f01722d-bb57-4645-8995-4bfebafe41ea@gmail.com |
---|---|
State | Accepted |
Headers | show |
Series | i2c: i801: simplify module boilerplate code | expand |
On Mon, 18 Sep 2023 14:16:58 +0200, Heiner Kallweit wrote: > Simplify the module boilerplate code. > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > --- > 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@yahoo.com>"); > @@ -1864,5 +1859,4 @@ MODULE_AUTHOR("Jean Delvare <jdelvare@suse.de>"); > 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@suse.de> Thanks,
Hi Heiner, On Mon, Sep 18, 2023 at 02:16:58PM +0200, Heiner Kallweit wrote: > Simplify the module boilerplate code. > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> Acked-by: Andi Shyti <andi.shyti@kernel.org> Thanks, Andi
On Mon, Sep 18, 2023 at 02:16:58PM +0200, Heiner Kallweit wrote: > Simplify the module boilerplate code. > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> Applied to for-next, thanks!
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@yahoo.com>"); @@ -1864,5 +1859,4 @@ MODULE_AUTHOR("Jean Delvare <jdelvare@suse.de>"); 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);
Simplify the module boilerplate code. Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> --- drivers/i2c/busses/i2c-i801.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-)