Staging: dt3155: coding style cleanups for the .h files
authorGreg Kroah-Hartman <gregkh@suse.de>
Tue, 22 Dec 2009 23:19:21 +0000 (15:19 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 4 Mar 2010 00:42:33 +0000 (16:42 -0800)
This cleans up some of the coding style issues in the .h files.

More remains to be done.

Cc: Scott Smedley <ss@aao.gov.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/dt3155/dt3155.h
drivers/staging/dt3155/dt3155_drv.h
drivers/staging/dt3155/dt3155_io.h
drivers/staging/dt3155/dt3155_isr.h

index 2e158151a7551e011a6a642c4c330aa5cb7ee3ed..2a01e017d5fe49fb814d0b6233fc4496a1aee4b9 100644 (file)
@@ -1,7 +1,7 @@
 /*
 
 Copyright 1996,2002,2005 Gregory D. Hager, Alfred A. Rizzi, Noah J. Cowan,
-                         Jason Lapenta, Scott Smedley
+                        Jason Lapenta, Scott Smedley
 
 This file is part of the DT3155 Device Driver.
 
@@ -20,8 +20,6 @@ along with the DT3155 Device Driver; if not, write to the Free
 Software Foundation, Inc., 59 Temple Place, Suite 330, Boston,
 MA 02111-1307 USA
 
-  $Id: dt3155.h,v 1.11 2005/08/09 06:08:51 ssmedley Exp $
-
 -- Changes --
 
   Date     Programmer  Description of changes made
@@ -56,120 +54,118 @@ MA 02111-1307 USA
 /* Can be 1 or 2 */
 #define MAXBOARDS 1
 
-#define BOARD_MAX_BUFFS 3
-#define MAXBUFFERS BOARD_MAX_BUFFS*MAXBOARDS
+#define BOARD_MAX_BUFFS        3
+#define MAXBUFFERS     (BOARD_MAX_BUFFS*MAXBOARDS)
 
-#define PCI_PAGE_SIZE (1 << 12)
+#define PCI_PAGE_SIZE  (1 << 12)
 
 #ifdef CCIR
-#define DT3155_MAX_ROWS 576
-#define DT3155_MAX_COLS 768
-#define FORMAT50HZ TRUE
+#define DT3155_MAX_ROWS        576
+#define DT3155_MAX_COLS        768
+#define FORMAT50HZ     TRUE
 #else
-#define DT3155_MAX_ROWS 480
-#define DT3155_MAX_COLS 640
-#define FORMAT50HZ FALSE
+#define DT3155_MAX_ROWS        480
+#define DT3155_MAX_COLS        640
+#define FORMAT50HZ     FALSE
 #endif
 
 /* Configuration structure */
 struct dt3155_config_s {
-  u_int acq_mode;
-  u_int cols, rows;
-  u_int continuous;
+       u_int acq_mode;
+       u_int cols, rows;
+       u_int continuous;
 };
 
 
 /* hold data for each frame */
-typedef struct
-{
-  u_long addr;          /* address of the buffer with the frame */
-  u_long tag;           /* unique number for the frame */
-  struct timeval time;  /* time that capture took place */
+typedef struct {
+       u_long addr;            /* address of the buffer with the frame */
+       u_long tag;             /* unique number for the frame */
+       struct timeval time;    /* time that capture took place */
 } frame_info_t;
 
-/* Structure for interrupt and buffer handling.  */
-/* This is the setup for 1 card                  */
+/*
+ * Structure for interrupt and buffer handling.
+ * This is the setup for 1 card
+ */
 struct dt3155_fbuffer_s {
-  int    nbuffers;
-
-  frame_info_t frame_info[ BOARD_MAX_BUFFS ];
+       int    nbuffers;
 
-  int empty_buffers[ BOARD_MAX_BUFFS ];    /* indexes empty frames */
-  int empty_len;                /* Number of empty buffers  */
-                                /* Zero means empty */
+       frame_info_t frame_info[BOARD_MAX_BUFFS];
 
-  int active_buf;               /* Where data is currently dma'ing */
-  int locked_buf;               /* Buffers used by user */
+       int empty_buffers[BOARD_MAX_BUFFS];     /* indexes empty frames */
+       int empty_len;                          /* Number of empty buffers */
+                                               /* Zero means empty */
 
-  int ready_que[ BOARD_MAX_BUFFS ];
-  u_long ready_head;  /* The most recent buffer located here */
-  u_long ready_len;   /* The number of ready buffers */
+       int active_buf;                 /* Where data is currently dma'ing */
+       int locked_buf;                 /* Buffers used by user */
 
-  int even_happened;
-  int even_stopped;
+       int ready_que[BOARD_MAX_BUFFS];
+       u_long ready_head;      /* The most recent buffer located here */
+       u_long ready_len;       /* The number of ready buffers */
 
-  int stop_acquire;             /* Flag to stop interrupts */
-  u_long frame_count;           /* Counter for frames acquired by this card */
+       int even_happened;
+       int even_stopped;
 
+       int stop_acquire;       /* Flag to stop interrupts */
+       u_long frame_count;     /* Counter for frames acquired by this card */
 };
 
 
 
-#define DT3155_MODE_FRAME 1
-#define DT3155_MODE_FIELD 2
+#define DT3155_MODE_FRAME      1
+#define DT3155_MODE_FIELD      2
 
-#define DT3155_SNAP 1
-#define DT3155_ACQ  2
+#define DT3155_SNAP            1
+#define DT3155_ACQ             2
 
 /* There is one status structure for each card. */
-typedef struct dt3155_status_s
-{
-  int fixed_mode;              /* if 1, we are in fixed frame mode */
-  u_long reg_addr;             /* Register address for a single card */
-  u_long mem_addr;             /* Buffer start addr for this card */
-  u_long mem_size;             /* This is the amount of mem available  */
-  u_int  irq;                  /* this card's irq */
-  struct dt3155_config_s config;  /* configuration struct */
-  struct dt3155_fbuffer_s fbuffer;/* frame buffer state struct */
-  u_long state;                /* this card's state */
-  u_int device_installed;      /* Flag if installed. 1=installed */
+typedef struct dt3155_status_s {
+       int fixed_mode;         /* if 1, we are in fixed frame mode */
+       u_long reg_addr;        /* Register address for a single card */
+       u_long mem_addr;        /* Buffer start addr for this card */
+       u_long mem_size;        /* This is the amount of mem available  */
+       u_int irq;              /* this card's irq */
+       struct dt3155_config_s config;          /* configuration struct */
+       struct dt3155_fbuffer_s fbuffer;        /* frame buffer state struct */
+       u_long state;           /* this card's state */
+       u_int device_installed; /* Flag if installed. 1=installed */
 } dt3155_status_t;
 
 /* Reference to global status structure */
 extern struct dt3155_status_s dt3155_status[MAXBOARDS];
 
-#define DT3155_STATE_IDLE    0x00
-#define DT3155_STATE_FRAME   0x01
-#define DT3155_STATE_FLD     0x02
-#define DT3155_STATE_STOP   0x100
-#define DT3155_STATE_ERROR  0x200
-#define DT3155_STATE_MODE   0x0ff
+#define DT3155_STATE_IDLE      0x00
+#define DT3155_STATE_FRAME     0x01
+#define DT3155_STATE_FLD       0x02
+#define DT3155_STATE_STOP      0x100
+#define DT3155_STATE_ERROR     0x200
+#define DT3155_STATE_MODE      0x0ff
 
-#define DT3155_IOC_MAGIC     '!'
+#define DT3155_IOC_MAGIC       '!'
 
-#define DT3155_SET_CONFIG  _IOW( DT3155_IOC_MAGIC, 1, struct dt3155_config_s )
-#define DT3155_GET_CONFIG  _IOR( DT3155_IOC_MAGIC, 2, struct dt3155_status_s )
-#define DT3155_STOP        _IO(  DT3155_IOC_MAGIC, 3 )
-#define DT3155_START       _IO(  DT3155_IOC_MAGIC, 4 )
-#define DT3155_FLUSH       _IO(  DT3155_IOC_MAGIC, 5 )
-#define DT3155_IOC_MAXNR   5
+#define DT3155_SET_CONFIG      _IOW(DT3155_IOC_MAGIC, 1, struct dt3155_config_s)
+#define DT3155_GET_CONFIG      _IOR(DT3155_IOC_MAGIC, 2, struct dt3155_status_s)
+#define DT3155_STOP            _IO(DT3155_IOC_MAGIC, 3)
+#define DT3155_START           _IO(DT3155_IOC_MAGIC, 4)
+#define DT3155_FLUSH           _IO(DT3155_IOC_MAGIC, 5)
+#define DT3155_IOC_MAXNR       5
 
 /* Error codes */
 
-#define DT_ERR_NO_BUFFERS   0x10000 /* not used but it might be one day - SS */
-#define DT_ERR_CORRUPT      0x20000
-#define DT_ERR_OVERRUN      0x30000
-#define DT_ERR_I2C_TIMEOUT  0x40000
-#define DT_ERR_MASK         0xff0000/* not used but it might be one day - SS */
+#define DT_ERR_NO_BUFFERS      0x10000 /* not used but it might be one day */
+#define DT_ERR_CORRUPT         0x20000
+#define DT_ERR_OVERRUN         0x30000
+#define DT_ERR_I2C_TIMEOUT     0x40000
+#define DT_ERR_MASK            0xff0000/* not used but it might be one day */
 
 /* User code will probably want to declare one of these for each card */
-typedef struct dt3155_read_s
-{
-  u_long offset;
-  u_long frame_seq;
-  u_long state;
+typedef struct dt3155_read_s {
+       u_long offset;
+       u_long frame_seq;
+       u_long state;
 
-  frame_info_t frame_info;
+       frame_info_t frame_info;
 } dt3155_read_t;
 
 #endif /* _DT3155_inc */
index 92a92dc2736cf27b3bf231d1aa27eea588378c1d..dc110b76e4637baafc9a6a0a96843b9dad92b760 100644 (file)
@@ -1,7 +1,7 @@
 /*
 
 Copyright 1996,2002 Gregory D. Hager, Alfred A. Rizzi, Noah J. Cowan,
-                    Scott Smedley
+                   Scott Smedley
 
 This file is part of the DT3155 Device Driver.
 
@@ -25,7 +25,7 @@ MA 02111-1307 USA
 #define DT3155_DRV_INC
 
 /* kernel logical address of the frame grabbers */
-extern u_char *dt3155_lbase[ MAXBOARDS ];
+extern u_char *dt3155_lbase[MAXBOARDS];
 
 /* kernel logical address of ram buffer */
 extern u_char *dt3155_bbase;
@@ -35,7 +35,7 @@ extern u_char *dt3155_bbase;
 
 #include <linux/version.h>     /* need access to LINUX_VERSION_CODE */
 /* wait queue for reads */
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,3,1)
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 3, 1)
 extern wait_queue_head_t dt3155_read_wait_queue[MAXBOARDS];
 #else
 extern struct wait_queue *dt3155_read_wait_queue[MAXBOARDS];
index c1174ab7eac5b89ae75eb0cdd599f4f9a8b63e3b..67df27fc6a8a8f7073767f27b8cf00e8e22a006d 100644 (file)
@@ -1,7 +1,7 @@
 /*
 
 Copyright 1996,2002 Gregory D. Hager, Alfred A. Rizzi, Noah J. Cowan,
-                    Jason Lapenta, Scott Smedley
+                   Jason Lapenta, Scott Smedley
 
 This file is part of the DT3155 Device Driver.
 
@@ -36,8 +36,8 @@ MA 02111-1307 USA
 
 /* macros to access registers */
 
-#define   WriteMReg(Address, Data)        * ((u_long *) (Address)) = Data
-#define   ReadMReg(Address, Data)         Data = * ((u_long *) (Address))
+#define WriteMReg(Address, Data)       (*((u_long *)(Address)) = Data)
+#define ReadMReg(Address, Data)                (Data = *((u_long *)(Address)))
 
 /***************** 32 bit register globals  **************/
 
@@ -240,12 +240,11 @@ typedef union iic_csr2_tag {
  * dma_upper_lmt_tag
  */
 typedef union dma_upper_lmt_tag   {
-  u_long   reg;
-  struct
-  {
-    u_long   DMA_UPPER_LMT_VAL       :   24;
-    u_long                           :   8;
-  } fld;
+       u_long reg;
+       struct {
+               u_long DMA_UPPER_LMT_VAL:24;
+               u_long :8;
+       } fld;
 } DMA_UPPER_LMT_R;
 
 
@@ -253,12 +252,12 @@ typedef union dma_upper_lmt_tag   {
  * Global declarations of local copies
  * of boards' 32 bit registers
  ***************************************/
-extern u_long            even_dma_start_r;   /*  bit 0 should always be 0 */
-extern u_long            odd_dma_start_r;    /*               .. */
-extern u_long            even_dma_stride_r;  /*  bits 0&1 should always be 0 */
-extern u_long            odd_dma_stride_r;   /*               .. */
-extern u_long            even_pixel_fmt_r;
-extern u_long            odd_pixel_fmt_r;
+extern u_long even_dma_start_r;   /*  bit 0 should always be 0 */
+extern u_long odd_dma_start_r;    /*               .. */
+extern u_long even_dma_stride_r;  /*  bits 0&1 should always be 0 */
+extern u_long odd_dma_stride_r;   /*               .. */
+extern u_long even_pixel_fmt_r;
+extern u_long odd_pixel_fmt_r;
 
 extern FIFO_TRIGGER_R      fifo_trigger_r;
 extern XFER_MODE_R         xfer_mode_r;
@@ -266,8 +265,8 @@ extern CSR1_R              csr1_r;
 extern RETRY_WAIT_CNT_R    retry_wait_cnt_r;
 extern INT_CSR_R           int_csr_r;
 
-extern u_long              even_fld_mask_r;
-extern u_long              odd_fld_mask_r;
+extern u_long even_fld_mask_r;
+extern u_long odd_fld_mask_r;
 
 extern MASK_LENGTH_R       mask_length_r;
 extern FIFO_FLAG_CNT_R     fifo_flag_cnt_r;
@@ -301,28 +300,26 @@ extern DMA_UPPER_LMT_R     odd_dma_upper_lmt_r;
 /******** Assignments and Typedefs for 8 bit I2C Registers********************/
 
 typedef union i2c_csr2_tag {
-  u_char   reg;
-  struct
-  {
-    u_char    CHROM_FIL         :   1;
-    u_char    SYNC_SNTL         :   1;
-    u_char    HZ50              :   1;
-    u_char    SYNC_PRESENT      :   1;
-    u_char    BUSY_EVE          :   1;
-    u_char    BUSY_ODD          :   1;
-    u_char    DISP_PASS         :   1;
-  } fld;
+       u_char reg;
+       struct {
+               u_char CHROM_FIL:1;
+               u_char SYNC_SNTL:1;
+               u_char HZ50:1;
+               u_char SYNC_PRESENT:1;
+               u_char BUSY_EVE:1;
+               u_char BUSY_ODD:1;
+               u_char DISP_PASS:1;
+       } fld;
 } I2C_CSR2;
 
 typedef union i2c_even_csr_tag {
-  u_char    reg;
-  struct
-  {
-    u_char    DONE_EVE       :   1;
-    u_char    SNGL_EVE       :   1;
-    u_char    ERROR_EVE      :   1;
-    u_char                   :   5;
-  } fld;
+       u_char    reg;
+       struct {
+               u_char DONE_EVE:1;
+               u_char SNGL_EVE:1;
+               u_char ERROR_EVE:1;
+               u_char :5;
+       } fld;
 } I2C_EVEN_CSR;
 
 typedef union i2c_odd_csr_tag {
@@ -394,7 +391,7 @@ extern u_char                 i2c_pm_lut_data;
 
 /* access 8-bit IIC registers */
 
-extern int ReadI2C (u_char * lpReg, u_short wIregIndex, u_char * byVal);
-extern int WriteI2C (u_char * lpReg, u_short wIregIndex, u_char byVal);
+extern int ReadI2C(u_char *lpReg, u_short wIregIndex, u_char *byVal);
+extern int WriteI2C(u_char *lpReg, u_short wIregIndex, u_char byVal);
 
 #endif
index b117b504e27730dde5a79c01ed5b4de18f00cf2f..c77a1008c09cd959b0fa1cafa85ad470d1fff2c9 100644 (file)
@@ -1,7 +1,7 @@
 /*
 
 Copyright 1996,2002 Gregory D. Hager, Alfred A. Rizzi, Noah J. Cowan,
-                    Jason Lapenta, Scott Smedley
+                   Jason Lapenta, Scott Smedley
 
 This file is part of the DT3155 Device Driver.
 
@@ -33,8 +33,8 @@ MA 02111-1307 USA
 
 */
 
-#ifndef  DT3155_ISR_H
-#define  DT3155_ISR_H
+#ifndef DT3155_ISR_H
+#define DT3155_ISR_H
 
 extern struct dt3155_fbuffer_s *dt3155_fbuffer[MAXBOARDS];
 
@@ -62,16 +62,16 @@ int dt3155_flush(int minor);
  * Simple array based que struct
  **********************************/
 
-bool are_empty_buffers( int minor );
-void push_empty( int index, int minor );
+bool are_empty_buffers(int minor);
+void push_empty(int index, int minor);
 
-int  pop_empty( int minor );
+int  pop_empty(int minor);
 
-bool is_ready_buf_empty( int minor );
-bool is_ready_buf_full( int minor );
+bool is_ready_buf_empty(int minor);
+bool is_ready_buf_full(int minor);
 
-void push_ready( int minor, int index );
-int  pop_ready( int minor );
+void push_ready(int minor, int index);
+int  pop_ready(int minor);
 
 
 #endif