Messages in this thread | | | Subject | Re: [greybus-dev] [PATCH] staging: greybus: tools: Fix braces {} style | From | Alex Elder <> | Date | Sun, 22 Mar 2020 13:34:35 -0500 |
| |
On 3/22/20 12:30 PM, Simran Singhal wrote: > This patch fixes the check reported by checkpatch.pl > for braces {} should be used on all arms of this statement. > > Signed-off-by: Simran Singhal <singhalsimran0@gmail.com>
Looks fine to me. And I saw no other instances of this in the Greybus code. Thanks for the patch.
Reviewed-by: Alex Elder <elder@linaro.org>
> --- > drivers/staging/greybus/tools/loopback_test.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/staging/greybus/tools/loopback_test.c b/drivers/staging/greybus/tools/loopback_test.c > index ba6f905f26fa..d46721502897 100644 > --- a/drivers/staging/greybus/tools/loopback_test.c > +++ b/drivers/staging/greybus/tools/loopback_test.c > @@ -801,8 +801,9 @@ static void prepare_devices(struct loopback_test *t) > write_sysfs_val(t->devices[i].sysfs_entry, > "outstanding_operations_max", > t->async_outstanding_operations); > - } else > + } else { > write_sysfs_val(t->devices[i].sysfs_entry, "async", 0); > + } > } > } > >
| |