Re: [PATCH 69/79] staging: comedi: das1800: use __comedi_request_region()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 10, 2013 at 11:09:55AM -0500, H Hartley Sweeten wrote:
> On Wednesday, April 10, 2013 6:25 AM, Dan Carpenter wrote:
> > On Tue, Apr 09, 2013 at 04:31:26PM -0700, H Hartley Sweeten wrote:
> >> 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@xxxxxxxxxxxxxxxxxxx>
> >> Cc: Ian Abbott <abbotti@xxxxxxxxx>
> >> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> >> ---
> >>  drivers/staging/comedi/drivers/das1800.c | 31 ++++++++++++++-----------------
> >>  1 file changed, 14 insertions(+), 17 deletions(-)
> >> 
> >> diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c
> >> index cf265c1..0427bf0 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);
> >
> > Was the intent to use the underscored version here?
> 
> Ugh. Fat-fingered that one.
> 
> Is a v2 of just this patch ok or should I repost the entire series?

Just send a v2.

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux