Staging: comedi: cb_pcimdas.c: Checkpatch cleanup

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Linux Kernel Mailing List
Date: Sunday, March 7, 2010 - 6:03 pm

Gitweb:     http://git.kernel.org/linus/7c2f1946a5d41da3cfc1696944aaa420db5b5145
Commit:     7c2f1946a5d41da3cfc1696944aaa420db5b5145
Parent:     e3a21d0ac2ee4f6c1be5314e76939867a2d0b470
Author:     Andrea Gelmini <andrea.gelmini@gelma.net>
AuthorDate: Fri Feb 26 10:15:02 2010 +0100
Committer:  Greg Kroah-Hartman <gregkh@suse.de>
CommitDate: Wed Mar 3 16:43:06 2010 -0800

    Staging: comedi: cb_pcimdas.c: Checkpatch cleanup
    
    WARNING: braces {} are not necessary for any arm of this statement
    +	if (thisboard->has_dio) {
    [...]
    +	} else {
    [...]
    
    WARNING: braces {} are not necessary for single statement blocks
    +			if (devpriv->BADR0) {
    +				comedi_pci_disable(devpriv->pci_dev);
    +			}
    
    Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---
 drivers/staging/comedi/drivers/cb_pcimdas.c |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/comedi/drivers/cb_pcimdas.c b/drivers/staging/comedi/drivers/cb_pcimdas.c
index cbbca05..2e61727 100644
--- a/drivers/staging/comedi/drivers/cb_pcimdas.c
+++ b/drivers/staging/comedi/drivers/cb_pcimdas.c
@@ -330,11 +330,10 @@ found:
 
 	s = dev->subdevices + 2;
 	/* digital i/o subdevice */
-	if (thisboard->has_dio) {
+	if (thisboard->has_dio)
 		subdev_8255_init(dev, s, NULL, devpriv->BADR4);
-	} else {
+	else
 		s->type = COMEDI_SUBD_UNUSED;
-	}
 
 	printk("attached\n");
 
@@ -365,9 +364,8 @@ static int cb_pcimdas_detach(struct comedi_device *dev)
 		free_irq(dev->irq, dev);
 	if (devpriv) {
 		if (devpriv->pci_dev) {
-			if (devpriv->BADR0) {
+			if (devpriv->BADR0)
 				comedi_pci_disable(devpriv->pci_dev);
-			}
 			pci_dev_put(devpriv->pci_dev);
 		}
 	}
--
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Staging: comedi: cb_pcimdas.c: Checkpatch cleanup, Linux Kernel Mailing ..., (Sun Mar 7, 6:03 pm)