Staging: dt3155: dt3155_isr.c: Fix checkpatch.pl issues.
authorNanakos Chrysostomos <nanakos@wired-net.gr>
Thu, 27 May 2010 19:20:41 +0000 (19:20 +0000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 18 Jun 2010 23:43:15 +0000 (16:43 -0700)
Fix resolves checkpatch.pl issues.

Signed-off-by: Nanakos Chrysostomos <nanakos@wired-net.gr>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/dt3155/dt3155_isr.c

index 33ddc9c057ffd0ec0317adeffe9cf215e01447b0..59b7bfd535d6023dcedd02d0c61ad7c101a41565 100644 (file)
@@ -153,10 +153,12 @@ void push_ready(int m, int index)
  *****************************************************/
 static int get_tail(int m)
 {
-  return (dt3155_fbuffer[m]->ready_head -
-          dt3155_fbuffer[m]->ready_len +
-          dt3155_fbuffer[m]->nbuffers)%
-         (dt3155_fbuffer[m]->nbuffers);
+       int ncount;
+       ncount = (dt3155_fbuffer[m]->ready_head -
+                       dt3155_fbuffer[m]->ready_len +
+                       dt3155_fbuffer[m]->nbuffers)%
+               (dt3155_fbuffer[m]->nbuffers);
+       return ncount;
 }
 
 
@@ -191,23 +193,23 @@ void printques(int m)
 
   tail = get_tail(m);
 
-  printk("\n R:");
+  printk(KERN_INFO "\n R:");
     for (index = tail; index != head; index++, index = index % (num)) {
        frame_index = dt3155_fbuffer[m]->ready_que[index];
        printk(" %d ", frame_index);
     }
 
-  printk("\n E:");
+  printk(KERN_INFO "\n E:");
     for (index = 0; index < dt3155_fbuffer[m]->empty_len; index++) {
        frame_index = dt3155_fbuffer[m]->empty_buffers[index];
        printk(" %d ", frame_index);
     }
 
   frame_index = dt3155_fbuffer[m]->active_buf;
-  printk("\n A: %d", frame_index);
+  printk(KERN_INFO "\n A: %d", frame_index);
 
   frame_index = dt3155_fbuffer[m]->locked_buf;
-  printk("\n L: %d\n", frame_index);
+  printk(KERN_INFO "\n L: %d\n", frame_index);
 
 }
 
@@ -263,19 +265,21 @@ void allocate_buffers(u32 *buf_addr, u32* total_size_kbs,
 #endif
   size_kbs = full_size_kbs;
   *buf_addr = 0;
-  printk("DT3155: We would like to get: %d KB\n", full_size_kbs);
-  printk("DT3155: ...but need at least: %d KB\n", min_size_kbs);
-  printk("DT3155: ...the allocator has: %d KB\n", allocator_max);
+  printk(KERN_INFO "DT3155: We would like to get: %d KB\n", full_size_kbs);
+  printk(KERN_INFO "DT3155: ...but need at least: %d KB\n", min_size_kbs);
+  printk(KERN_INFO "DT3155: ...the allocator has: %d KB\n", allocator_max);
   size_kbs = (full_size_kbs <= allocator_max ? full_size_kbs : allocator_max);
     if (size_kbs > min_size_kbs) {
-       if ((*buf_addr = allocator_allocate_dma(size_kbs, GFP_KERNEL)) != 0) {
-               printk("DT3155:  Managed to allocate: %d KB\n", size_kbs);
+               *buf_addr = allocator_allocate_dma(size_kbs, GFP_KERNEL);
+       if (*buf_addr != 0) {
+               printk(KERN_INFO "DT3155:  Managed to allocate: %d KB\n",
+                               size_kbs);
                *total_size_kbs = size_kbs;
                return;
        }
     }
   /* If we got here, the allocation failed */
-  printk("DT3155: Allocator failed!\n");
+  printk(KERN_INFO "DT3155: Allocator failed!\n");
   *buf_addr = 0;
   *total_size_kbs = 0;
   return;
@@ -324,9 +328,9 @@ u32 dt3155_setup_buffers(u32 *allocatorAddr)
 
   /* allocate a large contiguous chunk of RAM */
   allocate_buffers(&rambuff_addr, &rambuff_size, bufsize);
-  printk("DT3155: mem info\n");
-  printk("  - rambuf_addr = 0x%x\n", rambuff_addr);
-  printk("  - length (kb) = %u\n", rambuff_size);
+  printk(KERN_INFO "DT3155: mem info\n");
+  printk(KERN_INFO "  - rambuf_addr = 0x%x\n", rambuff_addr);
+  printk(KERN_INFO "  - length (kb) = %u\n", rambuff_size);
     if (rambuff_addr == 0) {
        printk(KERN_INFO
            "DT3155: Error setup_buffers() allocator dma failed\n");
@@ -339,7 +343,8 @@ u32 dt3155_setup_buffers(u32 *allocatorAddr)
      are so we can give an equal number to each device */
   rambuff_acm = rambuff_addr;
     for (index = 0; index < MAXBUFFERS; index++) {
-       rambuff_acm = adjust_4MB(rambuff_acm, bufsize);/*avoid spanning 4MB bdry*/
+       /*avoid spanning 4MB bdry*/
+       rambuff_acm = adjust_4MB(rambuff_acm, bufsize);
        if (rambuff_acm + bufsize > rambuff_end)
                break;
        rambuff_acm += bufsize;
@@ -347,7 +352,7 @@ u32 dt3155_setup_buffers(u32 *allocatorAddr)
   /* Following line is OK, will waste buffers if index
    * not evenly divisible by ndevices -NJC*/
   numbufs = index / ndevices;
-  printk("  - numbufs = %u\n", numbufs);
+  printk(KERN_INFO "  - numbufs = %u\n", numbufs);
     if (numbufs < 2) {
        printk(KERN_INFO
        "DT3155: Error setup_buffers() couldn't allocate 2 bufs/board\n");
@@ -367,7 +372,7 @@ u32 dt3155_setup_buffers(u32 *allocatorAddr)
                rambuff_acm = adjust_4MB(rambuff_acm, bufsize);
                if (rambuff_acm + bufsize > rambuff_end) {
                        /* Should never happen */
-                       printk("DT3155 PROGRAM ERROR (GCS)\n"
+                       printk(KERN_INFO "DT3155 PROGRAM ERROR (GCS)\n"
                        "Error distributing allocated buffers\n");
                        return -ENOMEM;
                }
@@ -395,7 +400,7 @@ u32 dt3155_setup_buffers(u32 *allocatorAddr)
        /* setup the ready queue */
        dt3155_fbuffer[m]->ready_head = 0;
        dt3155_fbuffer[m]->ready_len = 0;
-       printk("Available buffers for device %d: %d\n",
+       printk(KERN_INFO "Available buffers for device %d: %d\n",
            m, dt3155_fbuffer[m]->nbuffers);
     }