Re: [PATCH] staging: greybus: camera: Fix function declaration alignment

[S Tarun Kumar Lywait]

> That does not look correct at all, what tool asked you to make this type
> of change?

I used the checkpatch script itself (./scripts/checkpatch.pl). At first, it showed me the following issue:

CHECK: Alignment should match open parenthesis
#267: FILE: drivers/staging/greybus/camera.c:267:
+static const int gb_camera_configure_streams_validate_response(struct gb_camera *gcam,
+               struct gb_camera_configure_streams_response *resp,

So I fixed the alignment as follows:

static const int gb_camera_configure_streams_validate_response(struct gb_camera *gcam,
                                                               struct gb_camera_configure_streams_response *resp,
                                                               unsigned int nstreams)

But after making this change, I ran the script again and found another issue popping up, replacing the earlier issue:

CHECK: line length of 113 exceeds 100 columns
#267: FILE: drivers/staging/greybus/camera.c:267:
+                                                             struct gb_camera_configure_streams_response *resp,

So, I finally ended up with the current patch, that didn't raise any issue pertaining to this, when running the checkpatch script.

Please let me know in case of further queries.

Thanks,
Tarun Lywait