Re: [PATCH v2] staging: greybus: remove tabs to fix line length and merge lines

Dan Carpenter
2023-03-11T04:16:26Z

On Sat, Mar 11, 2023 at 03:18:04AM +0500, Khadija Kamran wrote:
> In file drivers/staging/greybus/arche-platform.c,
> - Length of line 181 exceeds 100 columns, fix by removing tabs from the
> line.
> - If condition and spin_unlock_...() call is split into two lines, join
> them to form a single line.
>
> Signed-off-by: Khadija Kamran <kamrankhadijadj@gmail.com>
> ---
> Changes in v2:
> - Change the subject and log message
> - Merge if condition and spin_unlock...() from two lines to one

Apply your patch and then re-run checkpatch.pl -f on the file. You will
see the problem.

regards,
dan carpenter