Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1328646
| From | Greg KH <gregkh@linuxfoundation.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] staging: android: sync_debug: Fixed braces related coding style issue |
| Date | 2016-02-08 00:00 +0100 |
| Message-ID | <qZGtb-4zn-1@gated-at.bofh.it> (permalink) |
| References | <qUq5H-3kf-1@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Sun, Jan 24, 2016 at 03:51:32PM +0530, Tapan Prakash T wrote:
> This patch fixes checkpatch.pl warning in file sync_debug
> WARNING: braces {} are not necessary for single statement blocks
>
> Signed-off-by: Tapan Prakash T <tapanprakasht@gmail.com>
> ---
> drivers/staging/android/sync_debug.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/staging/android/sync_debug.c b/drivers/staging/android/sync_debug.c
> index f45d13c..02a1649 100644
> --- a/drivers/staging/android/sync_debug.c
> +++ b/drivers/staging/android/sync_debug.c
> @@ -158,9 +158,8 @@ static void sync_print_fence(struct seq_file *s, struct sync_fence *fence)
> seq_printf(s, "[%p] %s: %s\n", fence, fence->name,
> sync_status_str(atomic_read(&fence->status)));
>
> - for (i = 0; i < fence->num_fences; ++i) {
> + for (i = 0; i < fence->num_fences; ++i)
> sync_print_pt(s, fence->cbs[i].sync_pt, true);
> - }
>
> spin_lock_irqsave(&fence->wq.lock, flags);
> list_for_each_entry(pos, &fence->wq.task_list, task_list) {
> --
> 1.9.1
Someone sent this fix in before you, sorry.
Back to linux.kernel | Previous | Next | Find similar | Unroll thread
Re: [PATCH] staging: android: sync_debug: Fixed braces related coding style issue Greg KH <gregkh@linuxfoundation.org> - 2016-02-08 00:00 +0100
csiph-web