/* hold data for each frame */
-typedef struct {
+struct frame_info {
        u32 addr;               /* address of the buffer with the frame */
        u32 tag;                /* unique number for the frame */
        struct timeval time;    /* time that capture took place */
-} frame_info_t;
+};
 
 /*
  * Structure for interrupt and buffer handling.
 struct dt3155_fbuffer_s {
        int    nbuffers;
 
-       frame_info_t frame_info[BOARD_MAX_BUFFS];
+       struct frame_info frame_info[BOARD_MAX_BUFFS];
 
        int empty_buffers[BOARD_MAX_BUFFS];     /* indexes empty frames */
        int empty_len;                          /* Number of empty buffers */
        u32 frame_seq;
        u32 state;
 
-       frame_info_t frame_info;
+       struct frame_info frame_info;
 } dt3155_read_t;
 
 #endif /* _DT3155_inc */
 
   int          minor = MINOR(filep->f_dentry->d_inode->i_rdev);
   u32          offset;
   int          frame_index;
-  frame_info_t *frame_info_p;
+  struct frame_info    *frame_info;
 
   /* TODO: this should check the error flag and */
   /*   return an error on hardware failures */
        }
     }
 
-  frame_info_p = &dt3155_status[minor].fbuffer.frame_info[frame_index];
+  frame_info = &dt3155_status[minor].fbuffer.frame_info[frame_index];
 
   /* make this an offset */
-  offset = frame_info_p->addr - dt3155_status[minor].mem_addr;
+  offset = frame_info->addr - dt3155_status[minor].mem_addr;
 
   put_user(offset, (unsigned int *) buf);
   buf += sizeof(u32);
   buf += sizeof(u32);
   put_user(dt3155_status[minor].state, (unsigned int *) buf);
   buf += sizeof(u32);
-  if (copy_to_user(buf, frame_info_p, sizeof(frame_info_t)))
+  if (copy_to_user(buf, frame_info, sizeof(*frame_info)))
       return -EFAULT;
 
   return sizeof(dt3155_read_t);