Staging: comedi: hwdrv_apci1032.c: Checkpatch cleanup

ERROR: do not initialise externals to 0 or NULL
+unsigned int ui_InterruptStatus = 0;

WARNING: braces {} are not necessary for single statement blocks
+		else {
+			outl(0x6, devpriv->iobase + APCI1032_DIGITAL_IP_IRQ);
+		}		/* else if(data[1] == ADDIDATA_OR) */

WARNING: braces {} are not necessary for single statement blocks
+		if (data[1] == 1) {
+			*data = ui_InterruptStatus;
+		}		/* if(data[1]==1) */

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

1 files changed, 5 insertions, 5 deletions

diff --git a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1032.c b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1032.c
index 27c68aac32d..fe06789699f 100644
--- a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1032.c
+++ b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1032.c
@@ -54,7 +54,7 @@ You should also find the complete GPL in the COPYING file accompanying this sour
 #include "hwdrv_apci1032.h"
 #include <linux/delay.h>
 /* Global variables */
-unsigned int ui_InterruptStatus = 0;
+unsigned int ui_InterruptStatus;
 
 /*
 +----------------------------------------------------------------------------+
@@ -108,9 +108,9 @@ int i_APCI1032_ConfigDigitalInput(struct comedi_device *dev, struct comedi_subde
                         ui_TmpValue =
                                 inl(devpriv->iobase + APCI1032_DIGITAL_IP_IRQ);
                 }               /* if (data[1] == ADDIDATA_OR) */
-                else {
+                else
                         outl(0x6, devpriv->iobase + APCI1032_DIGITAL_IP_IRQ);
-                }               /* else if(data[1] == ADDIDATA_OR) */
+                                /* else if(data[1] == ADDIDATA_OR) */
         }                       /*  if( data[0] == ADDIDATA_ENABLE) */
         else {
                 ul_Command1 = ul_Command1 & 0xFFFF0000;
@@ -221,9 +221,9 @@ int i_APCI1032_ReadMoreDigitalInput(struct comedi_device *dev, struct comedi_sub
                 }               /* switch(ui_NoOfChannels) */
         }                       /* if(data[1]==0) */
         else {
-                if (data[1] == 1) {
+                if (data[1] == 1)
                         *data = ui_InterruptStatus;
-                }               /* if(data[1]==1) */
+                                /* if(data[1]==1) */
         }                       /* else if(data[1]==0) */
         return insn->n;
 }