ANDROID: binder: make binder_alloc_new_buf_locked static and indent its arguments
authorXiongwei Song <sxwjean@gmail.com>
Thu, 14 Dec 2017 04:15:42 +0000 (12:15 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 19 Dec 2017 09:12:57 +0000 (10:12 +0100)
The function binder_alloc_new_buf_locked() is only used in this file, so
make it static. Also clean up sparse warning:

drivers/android/binder_alloc.c:330:23: warning: no previous prototype
for ‘binder_alloc_new_buf_locked’ [-Wmissing-prototypes]

In addition, the line of the function name exceeds 80 characters when
add static for this function, hence indent its arguments anew.

Signed-off-by: Xiongwei Song <sxwjean@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/android/binder_alloc.c

index fdf9d9f..07b866a 100644 (file)
@@ -327,11 +327,12 @@ err_no_vma:
        return vma ? -ENOMEM : -ESRCH;
 }
 
-struct binder_buffer *binder_alloc_new_buf_locked(struct binder_alloc *alloc,
-                                                 size_t data_size,
-                                                 size_t offsets_size,
-                                                 size_t extra_buffers_size,
-                                                 int is_async)
+static struct binder_buffer *binder_alloc_new_buf_locked(
+                               struct binder_alloc *alloc,
+                               size_t data_size,
+                               size_t offsets_size,
+                               size_t extra_buffers_size,
+                               int is_async)
 {
        struct rb_node *n = alloc->free_buffers.rb_node;
        struct binder_buffer *buffer;