diff options
| author | Xiongwei Song <sxwjean@gmail.com> | 2017-12-13 23:15:42 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-12-19 04:12:57 -0500 |
| commit | 3f827245463a57f5ef64a665e1ca64eed0da00a5 (patch) | |
| tree | 67ca0be87a45b838b65c223ad5224861157d4b88 | |
| parent | 8a943bd283ecbaaab26b184717bd2395af1da1e4 (diff) | |
ANDROID: binder: make binder_alloc_new_buf_locked static and indent its arguments
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>
| -rw-r--r-- | drivers/android/binder_alloc.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/drivers/android/binder_alloc.c b/drivers/android/binder_alloc.c index fdf9d9f12aac..07b866afee54 100644 --- a/drivers/android/binder_alloc.c +++ b/drivers/android/binder_alloc.c | |||
| @@ -327,11 +327,12 @@ err_no_vma: | |||
| 327 | return vma ? -ENOMEM : -ESRCH; | 327 | return vma ? -ENOMEM : -ESRCH; |
| 328 | } | 328 | } |
| 329 | 329 | ||
| 330 | struct binder_buffer *binder_alloc_new_buf_locked(struct binder_alloc *alloc, | 330 | static struct binder_buffer *binder_alloc_new_buf_locked( |
| 331 | size_t data_size, | 331 | struct binder_alloc *alloc, |
| 332 | size_t offsets_size, | 332 | size_t data_size, |
| 333 | size_t extra_buffers_size, | 333 | size_t offsets_size, |
| 334 | int is_async) | 334 | size_t extra_buffers_size, |
| 335 | int is_async) | ||
| 335 | { | 336 | { |
| 336 | struct rb_node *n = alloc->free_buffers.rb_node; | 337 | struct rb_node *n = alloc->free_buffers.rb_node; |
| 337 | struct binder_buffer *buffer; | 338 | struct binder_buffer *buffer; |
