Re: [PATCH] staging: greybus: remove blank line after an open brace '{'.

From: Johan Hovold
Date: Thu Sep 05 2019 - 03:17:55 EST


On Wed, Sep 04, 2019 at 08:55:58PM +0000, joahannes wrote:
> Fix checkpatch error
> "CHECK: Blank lines aren't necessary after an open brace '{'"
> in loopback_test.c:742.

Please fix up all of the blank lines before/after closing/opening brace
checkpatch CHECKs in one go. There appears to be many of them.

> Signed-off-by: joahannes <joahannes@xxxxxxxxx>

We need your full name here and in the From line.

> ---
> drivers/staging/greybus/tools/loopback_test.c | 1 -

Also, please include the component your modifying in the subject line
even if you need to shorten the description somehow, for example,

staging: greybus: loopback_test: remove unnecessary blank lines

> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/greybus/tools/loopback_test.c b/drivers/staging/greybus/tools/loopback_test.c
> index ba6f905f2..251b05710 100644
> --- a/drivers/staging/greybus/tools/loopback_test.c
> +++ b/drivers/staging/greybus/tools/loopback_test.c
> @@ -739,7 +739,6 @@ static int wait_for_complete(struct loopback_test *t)
> ts = &t->poll_timeout;
>
> while (1) {
> -
> ret = ppoll(t->fds, t->poll_count, ts, &mask_old);
> if (ret <= 0) {
> stop_tests(t);

Johan