staging: udlfb: fix checkpatch and style
Fix warnings detected by tools
Signed-off-by: Bernie Thompson <bernie@plugable.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
diff --git a/drivers/staging/udlfb/udlfb.c b/drivers/staging/udlfb/udlfb.c
index 082468a..5969e84 100644
--- a/drivers/staging/udlfb/udlfb.c
+++ b/drivers/staging/udlfb/udlfb.c
@@ -27,7 +27,6 @@
#include <linux/slab.h>
#include <linux/delay.h>
-
#include "udlfb.h"
static struct fb_fix_screeninfo dlfb_fix = {
@@ -256,6 +255,7 @@
urb = dlfb_get_urb(dev);
if (!urb)
return -ENOMEM;
+
buf = (char *) urb->transfer_buffer;
/*
@@ -288,13 +288,14 @@
unsigned long offset = vma->vm_pgoff << PAGE_SHIFT;
unsigned long page, pos;
- dl_notice("MMAP: %lu %u\n", offset + size, info->fix.smem_len);
-
if (offset + size > info->fix.smem_len)
return -EINVAL;
pos = (unsigned long)info->fix.smem_start + offset;
+ dl_notice("mmap() framebuffer addr:%lu size:%lu\n",
+ pos, size);
+
while (size > 0) {
page = vmalloc_to_pfn((void *)pos);
if (remap_pfn_range(vma, start, page, PAGE_SIZE, PAGE_SHARED))
@@ -310,7 +311,6 @@
vma->vm_flags |= VM_RESERVED; /* avoid to swap out this VMA */
return 0;
-
}
/*
@@ -372,13 +372,13 @@
* A single command can transmit a maximum of 256 pixels,
* regardless of the compression ratio (protocol design limit).
* To the hardware, 0 for a size byte means 256
- *
+ *
* Rather than 256 pixel commands which are either rl or raw encoded,
* the rlx command simply assumes alternating raw and rl spans within one cmd.
* This has a slightly larger header overhead, but produces more even results.
* It also processes all data (read and write) in a single pass.
* Performance benchmarks of common cases show it having just slightly better
- * compression than 256 pixel raw -or- rle commands, with similar CPU consumpion.
+ * compression than 256 pixel raw or rle commands, with similar CPU consumpion.
* But for very rl friendly data, will compress not quite as well.
*/
static void dlfb_compress_hline(
@@ -570,7 +570,7 @@
if (dlfb_render_hline(dev, &urb,
(char *) dev->info->fix.smem_start,
- &cmd, byte_offset, width * BPP,
+ &cmd, byte_offset, width * BPP,
&bytes_identical, &bytes_sent))
goto error;
}
@@ -1536,8 +1536,8 @@
struct usb_device *usbdev;
struct dlfb_data *dev = 0;
struct fb_info *info = 0;
- int i;
int retval = -ENOMEM;
+ int i;
/* usb initialization */