Messages in this thread | | | From | David Howells <> | Subject | Re: [PATCH 2/2] staging:android:fix line over 80 characters issue in binders.c this patch fixes line over 80 characters warning that was found using checkpatch.pl tool Signed-off-by:Anirudh Bhat <> | Date | Thu, 19 Apr 2012 09:22:58 +0100 |
| |
anirudh bhat <abhat38@gmail.com> wrote:
> - printk(KERN_ERR "binder: %d: binder_alloc_buf failed to " > - "map pages in userspace, no vma\n", proc->pid); > + printk(KERN_ERR "binder: %d: binder_alloc_buf failed to map pages in userspace, no vma\n", > + proc->pid);
Are these the wrong way round? Surely these are introducing checkpatch warnings rather than curing them?
If checkpatch is complaining about:
printk(KERN_ERR "binder: %d: binder_alloc_buf failed to " "map pages in userspace, no vma\n", proc->pid);
being a string split over multiple lines, then *checkpatch* needs to be fixed.
David
| |