Staging: comedi: Add spaces around parens as requested by checkpatch.pl
authorBill Pemberton <wfp5p@virginia.edu>
Tue, 17 Mar 2009 02:03:45 +0000 (22:03 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 3 Apr 2009 21:53:54 +0000 (14:53 -0700)
Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/comedi/drivers/mite.h
drivers/staging/comedi/drivers/s626.c
drivers/staging/comedi/drivers/s626.h
drivers/staging/comedi/kcomedilib/kcomedilib_main.c
drivers/staging/comedi/proc.c

index 37f397b340c5d49faf84bdb219b1aa38ac20737f..bca9f3af7adb6cab2acbb5ce94a8a81e91820229 100644 (file)
@@ -33,7 +33,7 @@
 #define PCIMIO_COMPAT
 
 #ifdef DEBUG_MITE
-#define MDPRINTK(format, args...)      printk(format , ## args )
+#define MDPRINTK(format, args...)      printk(format , ## args)
 #else
 #define MDPRINTK(format, args...)
 #endif
index a3763ade9c96b114fb9bbd539b2e4da187e4fd2c..71309d3e12278dd98cf42f67544eba90bd8be76c 100644 (file)
@@ -345,8 +345,8 @@ static void CountersInit(comedi_device *dev);
 /*  Counter objects constructor. */
 
 /*  Counter overflow/index event flag masks for RDMISC2. */
-#define INDXMASK(C)            ( 1 << ( ( (C) > 2 ) ? ( (C) * 2 - 1 ) : ( (C) * 2 +  4 ) ) )
-#define OVERMASK(C)            ( 1 << ( ( (C) > 2 ) ? ( (C) * 2 + 5 ) : ( (C) * 2 + 10 ) ) )
+#define INDXMASK(C)            (1 << (((C) > 2) ? ((C) * 2 - 1) : ((C) * 2 +  4)))
+#define OVERMASK(C)            (1 << (((C) > 2) ? ((C) * 2 + 5) : ((C) * 2 + 10)))
 #define EVBITS(C)              { 0, OVERMASK(C), INDXMASK(C), OVERMASK(C) | INDXMASK(C) }
 
 /*  Translation table to map IntSrc into equivalent RDMISC2 event flag  bits. */
@@ -454,11 +454,11 @@ static enc_private enc_private_data[] = {
 
 /*  enab/disable a function or test status bit(s) that are accessed */
 /*  through Main Control Registers 1 or 2. */
-#define MC_ENABLE( REGADRS, CTRLWORD ) writel(  ( (uint32_t)( CTRLWORD ) << 16 ) | (uint32_t)( CTRLWORD ), devpriv->base_addr+( REGADRS ) )
+#define MC_ENABLE(REGADRS, CTRLWORD)   writel(((uint32_t)(CTRLWORD) << 16) | (uint32_t)(CTRLWORD), devpriv->base_addr+(REGADRS))
 
-#define MC_DISABLE( REGADRS, CTRLWORD )        writel(  (uint32_t)( CTRLWORD ) << 16 , devpriv->base_addr+( REGADRS ) )
+#define MC_DISABLE(REGADRS, CTRLWORD)  writel((uint32_t)(CTRLWORD) << 16 , devpriv->base_addr+(REGADRS))
 
-#define MC_TEST( REGADRS, CTRLWORD )   ( ( readl(devpriv->base_addr+( REGADRS )) & CTRLWORD ) != 0 )
+#define MC_TEST(REGADRS, CTRLWORD)     ((readl(devpriv->base_addr+(REGADRS)) & CTRLWORD) != 0)
 
 /* #define WR7146(REGARDS,CTRLWORD)
     writel(CTRLWORD,(uint32_t)(devpriv->base_addr+(REGARDS))) */
@@ -468,16 +468,16 @@ static enc_private enc_private_data[] = {
     readl((uint32_t)(devpriv->base_addr+(REGARDS))) */
 #define RR7146(REGARDS)                readl(devpriv->base_addr+(REGARDS))
 
-#define BUGFIX_STREG(REGADRS)   ( REGADRS - 4 )
+#define BUGFIX_STREG(REGADRS)   (REGADRS - 4)
 
 /*  Write a time slot control record to TSL2. */
-#define VECTPORT( VECTNUM )            (P_TSL2 + ( (VECTNUM) << 2 ))
-#define SETVECT( VECTNUM, VECTVAL )    WR7146(VECTPORT( VECTNUM ), (VECTVAL))
+#define VECTPORT(VECTNUM)              (P_TSL2 + ((VECTNUM) << 2))
+#define SETVECT(VECTNUM, VECTVAL)      WR7146(VECTPORT(VECTNUM), (VECTVAL))
 
 /*  Code macros used for constructing I2C command bytes. */
-#define I2C_B2(ATTR, VAL)      ( ( (ATTR) << 6 ) | ( (VAL) << 24 ) )
-#define I2C_B1(ATTR, VAL)      ( ( (ATTR) << 4 ) | ( (VAL) << 16 ) )
-#define I2C_B0(ATTR, VAL)      ( ( (ATTR) << 2 ) | ( (VAL) <<  8 ) )
+#define I2C_B2(ATTR, VAL)      (((ATTR) << 6) | ((VAL) << 24))
+#define I2C_B1(ATTR, VAL)      (((ATTR) << 4) | ((VAL) << 16))
+#define I2C_B0(ATTR, VAL)      (((ATTR) << 2) | ((VAL) <<  8))
 
 static const comedi_lrange s626_range_table = { 2, {
                        RANGE(-5, 5),
@@ -2353,7 +2353,7 @@ static void s626_timer_load(comedi_device *dev, enc_private *k, int tick)
 /* ***********  DAC FUNCTIONS *********** */
 
 /*  Slot 0 base settings. */
-#define VECT0  ( XSD2 | RSD3 | SIB_A2 )
+#define VECT0  (XSD2 | RSD3 | SIB_A2)
 /*  Slot 0 always shifts in  0xFF and store it to  FB_BUFFER2. */
 
 /*  TrimDac LogicalChan-to-PhysicalChan mapping table. */
index c158110aa47c70ae4498b903ea0eae77ed7f976d..c48957be2e1c176be4f460511233bf32e9735f8c 100644 (file)
 
 /*  RPS clock parameters. */
 #define RPSCLK_SCALAR          8       /*  This is apparent ratio of PCI/RPS clks (undocumented!!). */
-#define RPSCLK_PER_US          ( 33 / RPSCLK_SCALAR )  /*  Number of RPS clocks in one microsecond. */
+#define RPSCLK_PER_US          (33 / RPSCLK_SCALAR)    /*  Number of RPS clocks in one microsecond. */
 
 /*  Event counter source addresses. */
 #define SBA_RPS_A0             0x27    /*  Time of RPS0 busy, in PCI clocks. */
 
 #if PLATFORM == INTEL          /* Base ACON1 config: always run A1 based
                                 * on TSL1. */
-#define ACON1_BASE             ( WS_MODES | A1_RUN )
+#define ACON1_BASE             (WS_MODES | A1_RUN)
 #elif PLATFORM == MOTOROLA
-#define ACON1_BASE             ( WS_MODES | A1_RUN | A1_SWAP | A2_SWAP )
+#define ACON1_BASE             (WS_MODES | A1_RUN | A1_SWAP | A2_SWAP)
 #endif
 
 #define ACON1_ADCSTART         ACON1_BASE      /* Start ADC: run A1
                                                 *  based on TSL1. */
-#define ACON1_DACSTART         ( ACON1_BASE | A2_RUN )
+#define ACON1_DACSTART         (ACON1_BASE | A2_RUN)
 /* Start transmit to DAC: run A2 based on TSL2. */
 #define ACON1_DACSTOP          ACON1_BASE      /*  Halt A2. */
 
 #define ACON2_XORMASK          0x000C0000      /*  XOR mask for ACON2 */
                                                /*  active-low bits. */
 
-#define ACON2_INIT             ( ACON2_XORMASK ^ ( A1_CLKSRC_BCLK1 | A2_CLKSRC_X2 | INVERT_BCLK2 | BCLK2_OE ) )
+#define ACON2_INIT             (ACON2_XORMASK ^ (A1_CLKSRC_BCLK1 | A2_CLKSRC_X2 | INVERT_BCLK2 | BCLK2_OE))
 
 /*  Bit masks for timeslot records. */
 #define WS1                    0x40000000      /*  WS output to assert. */
 /*  I2C manifest constants. */
 
 /*  Max retries to wait for EEPROM write. */
-#define I2C_RETRIES            ( I2C_WRTIME * I2C_BITRATE / 9.0 )
+#define I2C_RETRIES            (I2C_WRTIME * I2C_BITRATE / 9.0)
 #define I2C_ERR                        0x0002  /*  I2C control/status */
                                                /*  flag ERROR. */
 #define I2C_BUSY               0x0001  /*  I2C control/status */
 #define I2C_ATTRNOP            0x0     /*  I2C attribute NOP. */
 
 /*  I2C read command  | EEPROM address. */
-#define I2CR                   ( devpriv->I2CAdrs | 1 )
+#define I2CR                   (devpriv->I2CAdrs | 1)
 
 /*  I2C write command | EEPROM address. */
-#define I2CW                   ( devpriv->I2CAdrs )
+#define I2CW                   (devpriv->I2CAdrs)
 
 /*  Code macros used for constructing I2C command bytes. */
-#define I2C_B2(ATTR, VAL)      ( ( (ATTR) << 6 ) | ( (VAL) << 24 ) )
-#define I2C_B1(ATTR, VAL)      ( ( (ATTR) << 4 ) | ( (VAL) << 16 ) )
-#define I2C_B0(ATTR, VAL)      ( ( (ATTR) << 2 ) | ( (VAL) <<  8 ) )
+#define I2C_B2(ATTR, VAL)      (((ATTR) << 6) | ((VAL) << 24))
+#define I2C_B1(ATTR, VAL)      (((ATTR) << 4) | ((VAL) << 16))
+#define I2C_B0(ATTR, VAL)      (((ATTR) << 2) | ((VAL) <<  8))
 
 /* oldest */
 #define P_DEBICFGq              0x007C /*  DEBI configuration. */
 #define DEBI_PAGE_DISABLEQ     0x00000000      /*  paging disable */
 
 /*  DEBI command constants. */
-#define DEBI_CMD_SIZE16                ( 2 << 17 )     /*  Transfer size is */
+#define DEBI_CMD_SIZE16                (2 << 17)       /*  Transfer size is */
                                                /*  always 2 bytes. */
 #define DEBI_CMD_READ          0x00010000      /*  Read operation. */
 #define DEBI_CMD_WRITE         0x00000000      /*  Write operation. */
 
 /*  Read immediate 2 bytes. */
-#define DEBI_CMD_RDWORD                ( DEBI_CMD_READ  | DEBI_CMD_SIZE16 )
+#define DEBI_CMD_RDWORD                (DEBI_CMD_READ  | DEBI_CMD_SIZE16)
 
 /*  Write immediate 2 bytes. */
-#define DEBI_CMD_WRWORD                ( DEBI_CMD_WRITE | DEBI_CMD_SIZE16 )
+#define DEBI_CMD_WRWORD                (DEBI_CMD_WRITE | DEBI_CMD_SIZE16)
 
 /*  DEBI configuration constants. */
 #define DEBI_CFG_XIRQ_EN       0x80000000      /*  enab external */
 
 /*  Bit field masks for CRA and CRB. */
 
-#define CRAMSK_INDXSRC_B       ( (uint16_t)( 3 << CRABIT_INDXSRC_B) )
-#define CRAMSK_CLKSRC_B                ( (uint16_t)( 3 << CRABIT_CLKSRC_B) )
-#define CRAMSK_INDXPOL_A       ( (uint16_t)( 1 << CRABIT_INDXPOL_A) )
-#define CRAMSK_LOADSRC_A       ( (uint16_t)( 3 << CRABIT_LOADSRC_A) )
-#define CRAMSK_CLKMULT_A       ( (uint16_t)( 3 << CRABIT_CLKMULT_A) )
-#define CRAMSK_INTSRC_A                ( (uint16_t)( 3 << CRABIT_INTSRC_A) )
-#define CRAMSK_CLKPOL_A                ( (uint16_t)( 3 << CRABIT_CLKPOL_A) )
-#define CRAMSK_INDXSRC_A       ( (uint16_t)( 3 << CRABIT_INDXSRC_A) )
-#define CRAMSK_CLKSRC_A                ( (uint16_t)( 3 << CRABIT_CLKSRC_A) )
-
-#define CRBMSK_INTRESETCMD     ( (uint16_t)( 1 << CRBBIT_INTRESETCMD) )
-#define CRBMSK_INTRESET_B      ( (uint16_t)( 1 << CRBBIT_INTRESET_B) )
-#define CRBMSK_INTRESET_A      ( (uint16_t)( 1 << CRBBIT_INTRESET_A) )
-#define CRBMSK_CLKENAB_A       ( (uint16_t)( 1 << CRBBIT_CLKENAB_A) )
-#define CRBMSK_INTSRC_B                ( (uint16_t)( 3 << CRBBIT_INTSRC_B) )
-#define CRBMSK_LATCHSRC                ( (uint16_t)( 3 << CRBBIT_LATCHSRC) )
-#define CRBMSK_LOADSRC_B       ( (uint16_t)( 3 << CRBBIT_LOADSRC_B) )
-#define CRBMSK_CLKMULT_B       ( (uint16_t)( 3 << CRBBIT_CLKMULT_B) )
-#define CRBMSK_CLKENAB_B       ( (uint16_t)( 1 << CRBBIT_CLKENAB_B) )
-#define CRBMSK_INDXPOL_B       ( (uint16_t)( 1 << CRBBIT_INDXPOL_B) )
-#define CRBMSK_CLKPOL_B                ( (uint16_t)( 1 << CRBBIT_CLKPOL_B) )
-
-#define CRBMSK_INTCTRL         ( CRBMSK_INTRESETCMD | CRBMSK_INTRESET_A | CRBMSK_INTRESET_B )  /*  Interrupt reset control bits. */
+#define CRAMSK_INDXSRC_B       ((uint16_t)(3 << CRABIT_INDXSRC_B))
+#define CRAMSK_CLKSRC_B                ((uint16_t)(3 << CRABIT_CLKSRC_B))
+#define CRAMSK_INDXPOL_A       ((uint16_t)(1 << CRABIT_INDXPOL_A))
+#define CRAMSK_LOADSRC_A       ((uint16_t)(3 << CRABIT_LOADSRC_A))
+#define CRAMSK_CLKMULT_A       ((uint16_t)(3 << CRABIT_CLKMULT_A))
+#define CRAMSK_INTSRC_A                ((uint16_t)(3 << CRABIT_INTSRC_A))
+#define CRAMSK_CLKPOL_A                ((uint16_t)(3 << CRABIT_CLKPOL_A))
+#define CRAMSK_INDXSRC_A       ((uint16_t)(3 << CRABIT_INDXSRC_A))
+#define CRAMSK_CLKSRC_A                ((uint16_t)(3 << CRABIT_CLKSRC_A))
+
+#define CRBMSK_INTRESETCMD     ((uint16_t)(1 << CRBBIT_INTRESETCMD))
+#define CRBMSK_INTRESET_B      ((uint16_t)(1 << CRBBIT_INTRESET_B))
+#define CRBMSK_INTRESET_A      ((uint16_t)(1 << CRBBIT_INTRESET_A))
+#define CRBMSK_CLKENAB_A       ((uint16_t)(1 << CRBBIT_CLKENAB_A))
+#define CRBMSK_INTSRC_B                ((uint16_t)(3 << CRBBIT_INTSRC_B))
+#define CRBMSK_LATCHSRC                ((uint16_t)(3 << CRBBIT_LATCHSRC))
+#define CRBMSK_LOADSRC_B       ((uint16_t)(3 << CRBBIT_LOADSRC_B))
+#define CRBMSK_CLKMULT_B       ((uint16_t)(3 << CRBBIT_CLKMULT_B))
+#define CRBMSK_CLKENAB_B       ((uint16_t)(1 << CRBBIT_CLKENAB_B))
+#define CRBMSK_INDXPOL_B       ((uint16_t)(1 << CRBBIT_INDXPOL_B))
+#define CRBMSK_CLKPOL_B                ((uint16_t)(1 << CRBBIT_CLKPOL_B))
+
+#define CRBMSK_INTCTRL         (CRBMSK_INTRESETCMD | CRBMSK_INTRESET_A | CRBMSK_INTRESET_B)    /*  Interrupt reset control bits. */
 
 /*  Bit field positions for standardized SETUP structure. */
 
 
 /*  Bit field masks for standardized SETUP structure. */
 
-#define STDMSK_INTSRC          ( (uint16_t)( 3 << STDBIT_INTSRC   ) )
-#define STDMSK_LATCHSRC                ( (uint16_t)( 3 << STDBIT_LATCHSRC ) )
-#define STDMSK_LOADSRC         ( (uint16_t)( 3 << STDBIT_LOADSRC  ) )
-#define STDMSK_INDXSRC         ( (uint16_t)( 1 << STDBIT_INDXSRC  ) )
-#define STDMSK_INDXPOL         ( (uint16_t)( 1 << STDBIT_INDXPOL  ) )
-#define STDMSK_CLKSRC          ( (uint16_t)( 3 << STDBIT_CLKSRC   ) )
-#define STDMSK_CLKPOL          ( (uint16_t)( 1 << STDBIT_CLKPOL   ) )
-#define STDMSK_CLKMULT         ( (uint16_t)( 3 << STDBIT_CLKMULT  ) )
-#define STDMSK_CLKENAB         ( (uint16_t)( 1 << STDBIT_CLKENAB  ) )
+#define STDMSK_INTSRC          ((uint16_t)(3 << STDBIT_INTSRC))
+#define STDMSK_LATCHSRC                ((uint16_t)(3 << STDBIT_LATCHSRC))
+#define STDMSK_LOADSRC         ((uint16_t)(3 << STDBIT_LOADSRC))
+#define STDMSK_INDXSRC         ((uint16_t)(1 << STDBIT_INDXSRC))
+#define STDMSK_INDXPOL         ((uint16_t)(1 << STDBIT_INDXPOL))
+#define STDMSK_CLKSRC          ((uint16_t)(3 << STDBIT_CLKSRC))
+#define STDMSK_CLKPOL          ((uint16_t)(1 << STDBIT_CLKPOL))
+#define STDMSK_CLKMULT         ((uint16_t)(3 << STDBIT_CLKMULT))
+#define STDMSK_CLKENAB         ((uint16_t)(1 << STDBIT_CLKENAB))
 
 
 /* typedef struct indexCounter */
index e758e4e8c91b8a3c393040767234c663dc2f57d7..7133f2f2607c6906d749fd3d86886e5bb20a8f05 100644 (file)
@@ -57,11 +57,11 @@ comedi_t *comedi_open(const char *filename)
                return NULL;
 
        dev_file_info = comedi_get_device_file_info(minor);
-       if(dev_file_info == NULL)
+       if (dev_file_info == NULL)
                return NULL;
        dev = dev_file_info->device;
 
-       if(dev == NULL || !dev->attached)
+       if (dev == NULL || !dev->attached)
                return NULL;
 
        if (!try_module_get(dev->driver->module))
@@ -79,11 +79,11 @@ comedi_t *comedi_open_old(unsigned int minor)
                return NULL;
 
        dev_file_info = comedi_get_device_file_info(minor);
-       if(dev_file_info == NULL)
+       if (dev_file_info == NULL)
                return NULL;
        dev = dev_file_info->device;
 
-       if(dev == NULL || !dev->attached)
+       if (dev == NULL || !dev->attached)
                return NULL;
 
        return (comedi_t *) dev;
index 031004ebc6ec4445b7e41d8227a961f2c6baab09..1008773625772860c3876a808786054fa8d03a89 100644 (file)
@@ -55,7 +55,7 @@ int comedi_read_procmem(char *buf, char **start, off_t offset, int len,
                struct comedi_device_file_info *dev_file_info = comedi_get_device_file_info(i);
                comedi_device *dev;
 
-               if(dev_file_info == NULL) continue;
+               if (dev_file_info == NULL) continue;
                dev = dev_file_info->device;
 
                if (dev->attached) {