diff options
| author | Devendra Naga <devendra.aaru@gmail.com> | 2012-10-30 15:18:40 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-10-30 12:45:38 -0700 |
| commit | 958bcf5b36f5bbe7b5a3bb423181f8b745bfe23b (patch) | |
| tree | fb3bc72cb9af0ea2d383505dcb4eeff46c94de8d /drivers/staging/csr | |
| parent | ffe6d91cdb3ff26d8287286e2af41ccb1302dfbf (diff) | |
staging: csr: os.c: remove col variable
theres no point in checking the col variable,
its always zero, and wont print a new line, actually
its better to have a new line after a set of characters
printed, instead of confusing with the concatenated
characters when called multiple times at a time.
Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/csr')
| -rw-r--r-- | drivers/staging/csr/os.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/csr/os.c b/drivers/staging/csr/os.c index 35dc9087d79..0d9842a894c 100644 --- a/drivers/staging/csr/os.c +++ b/drivers/staging/csr/os.c @@ -459,7 +459,7 @@ dump16(void *mem, u16 len) void dump_str(void *mem, u16 len) { - int i, col = 0; + int i; unsigned char *pdata = (unsigned char *)mem; #ifdef ANDROID_TIMESTAMP printk("timestamp %s \n", print_time()); @@ -467,9 +467,7 @@ dump_str(void *mem, u16 len) for (i = 0; i < len; i++) { printk("%c", pdata[i]); } - if (col) { - printk("\n"); - } + printk("\n"); } /* dump_str() */ #endif /* CSR_ONLY_NOTES */ |