2019-06-15 18:46:48

by Saiyam Doshi

Subject: [PATCH v1] staging: android: fix style problem

checkpatch reported "WARNING: line over 80 characters".
This patch fixes it by aligning function arguments.

Signed-off-by: Saiyam Doshi <[email protected]>
---
Changes in v1:
* Updated as per review comment. Now function arguments
uses two lines, one less line than previous submission.

drivers/staging/android/ion/ion_chunk_heap.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/android/ion/ion_chunk_heap.c b/drivers/staging/android/ion/ion_chunk_heap.c
index 3cdde9c1a717..1e869f4bad45 100644
--- a/drivers/staging/android/ion/ion_chunk_heap.c
+++ b/drivers/staging/android/ion/ion_chunk_heap.c
@@ -107,7 +107,8 @@ static struct ion_heap_ops chunk_heap_ops = {
.unmap_kernel = ion_heap_unmap_kernel,
};

-struct ion_heap *ion_chunk_heap_create(phys_addr_t base, size_t size, size_t chunk_size)
+struct ion_heap *ion_chunk_heap_create(phys_addr_t base, size_t size,
+ size_t chunk_size)
{
struct ion_chunk_heap *chunk_heap;
int ret;
--
2.20.1

2019-06-15 18:52:56

by Greg Kroah-Hartman

Subject: Re: [PATCH v1] staging: android: fix style problem

On Sun, Jun 16, 2019 at 12:16:05AM +0530, Saiyam Doshi wrote:
> checkpatch reported "WARNING: line over 80 characters".
> This patch fixes it by aligning function arguments.
>
> Signed-off-by: Saiyam Doshi <[email protected]>
> ---
> Changes in v1:
> * Updated as per review comment. Now function arguments
> uses two lines, one less line than previous submission.

It's really "v2", but that's fine :)