Skip to content

Commit

Permalink
staging: comedi: refactor ii_pci20kc driver and use module_comedi_driver
Browse files Browse the repository at this point in the history
Move the module_init/module_exit routines and the associated
struct comedi_drive to the end of the source. This is more
typical of how other drivers are written and removes the need
for the forward declarations.

Convert the driver to use the module_comedi_driver() macro
which makes the code smaller and a bit simpler.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
H Hartley Sweeten authored and Greg Kroah-Hartman committed May 16, 2012
1 parent cb4c516 commit 2396472
Showing 1 changed file with 7 additions and 23 deletions.
30 changes: 7 additions & 23 deletions drivers/staging/comedi/drivers/ii_pci20kc.c
Original file line number Diff line number Diff line change
Expand Up @@ -159,17 +159,6 @@ struct pci20xxx_private {
#define devpriv ((struct pci20xxx_private *)dev->private)
#define CHAN (CR_CHAN(it->chanlist[0]))

static int pci20xxx_attach(struct comedi_device *dev,
struct comedi_devconfig *it);
static int pci20xxx_detach(struct comedi_device *dev);

static struct comedi_driver driver_pci20xxx = {
.driver_name = "ii_pci20kc",
.module = THIS_MODULE,
.attach = pci20xxx_attach,
.detach = pci20xxx_detach,
};

static int pci20006_init(struct comedi_device *dev, struct comedi_subdevice *s,
int opt0, int opt1);
static int pci20341_init(struct comedi_device *dev, struct comedi_subdevice *s,
Expand Down Expand Up @@ -666,18 +655,13 @@ static unsigned int pci20xxx_di(struct comedi_device *dev,
}
#endif

static int __init driver_pci20xxx_init_module(void)
{
return comedi_driver_register(&driver_pci20xxx);
}

static void __exit driver_pci20xxx_cleanup_module(void)
{
comedi_driver_unregister(&driver_pci20xxx);
}

module_init(driver_pci20xxx_init_module);
module_exit(driver_pci20xxx_cleanup_module);
static struct comedi_driver pci20xxx_driver = {
.driver_name = "ii_pci20kc",
.module = THIS_MODULE,
.attach = pci20xxx_attach,
.detach = pci20xxx_detach,
};
module_comedi_driver(pci20xxx_driver);

MODULE_AUTHOR("Comedi http://www.comedi.org");
MODULE_DESCRIPTION("Comedi low-level driver");
Expand Down

0 comments on commit 2396472

Please sign in to comment.