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@xxxxxxxxx>
---
drivers/staging/csr/os.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/csr/os.c b/drivers/staging/csr/os.c
index 35dc908..0d9842a 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 */
--
1.7.10.4
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
