diff options
author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-04-10 10:03:59 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-04-11 12:52:35 -0700 |
commit | 8fdbc75b70967f78cbd508e33c4a1da9e90b09f8 (patch) | |
tree | 11455d5e253fa2a26c35e0123ff78ab9a79aeb97 | |
parent | 5b365a8ace90bd027a34f0db4c1c28f2ba433910 (diff) |
staging: comedi: das1800: use __comedi_request_region()
Use __comedi_request_region() to request the additional I/O region
used by this driver.
Remove the error message when the request_region() fails,
comedi_request_reqion() will output the error message if necessary.
For aesthetic reasons, rename the local variable 'retval' to
simply 'ret'.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/comedi/drivers/das1800.c | 31 |
1 files changed, 14 insertions, 17 deletions
diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c index cf265c1be7d..76762010202 100644 --- a/drivers/staging/comedi/drivers/das1800.c +++ b/drivers/staging/comedi/drivers/das1800.c @@ -1514,18 +1514,17 @@ static int das1800_attach(struct comedi_device *dev, unsigned int irq = it->options[1]; unsigned int dma0 = it->options[2]; unsigned int dma1 = it->options[3]; - unsigned long iobase2; int board; - int retval; + int ret; devpriv = kzalloc(sizeof(*devpriv), GFP_KERNEL); if (!devpriv) return -ENOMEM; dev->private = devpriv; - retval = comedi_request_region(dev, it->options[0], DAS1800_SIZE); - if (retval) - return retval; + ret = comedi_request_region(dev, it->options[0], DAS1800_SIZE); + if (ret) + return ret; board = das1800_probe(dev); if (board < 0) { @@ -1539,13 +1538,11 @@ static int das1800_attach(struct comedi_device *dev, /* if it is an 'ao' board with fancy analog out then we need extra io ports */ if (thisboard->ao_ability == 2) { - iobase2 = dev->iobase + IOBASE2; - if (!request_region(iobase2, DAS1800_SIZE, dev->board_name)) { + unsigned long iobase2 = dev->iobase + IOBASE2; + + ret = __comedi_request_region(dev, iobase2, DAS1800_SIZE); + if (ret) { release_region(dev->iobase, DAS1800_SIZE); - dev_warn(dev->class_dev, - "%s: I/O port conflict (%#lx,%d)\n", - dev->board_name, - iobase2, DAS1800_SIZE); dev->iobase = 0; return -EIO; } @@ -1591,9 +1588,9 @@ static int das1800_attach(struct comedi_device *dev, break; } - retval = das1800_init_dma(dev, dma0, dma1); - if (retval < 0) - return retval; + ret = das1800_init_dma(dev, dma0, dma1); + if (ret < 0) + return ret; if (devpriv->ai_buf0 == NULL) { devpriv->ai_buf0 = @@ -1602,9 +1599,9 @@ static int das1800_attach(struct comedi_device *dev, return -ENOMEM; } - retval = comedi_alloc_subdevices(dev, 4); - if (retval) - return retval; + ret = comedi_alloc_subdevices(dev, 4); + if (ret) + return ret; /* analog input subdevice */ s = &dev->subdevices[0]; |