[PATCH] Staging: android: fix brace coding style issue in sync_debug.c

Oliver Graff oliver.e.graff at gmail.com
Mon Feb 15 01:02:38 UTC 2016


Sorry, do you mean the patch won't apply or you were the wrong person
to bother with this? Either way I'm so sorry if I wasted any of your
time.

Thanks,
Oliver

On Sun, Feb 14, 2016 at 7:35 PM, Greg KH <gregkh at linuxfoundation.org> wrote:
> On Fri, Feb 12, 2016 at 03:52:09AM -0500, Oliver Graff wrote:
>> Remove scope braces that were generating a warning in sync_debug.c since
>> they were scoping a single statement
>>
>> Signed-off-by: Oliver Graff <oliver.e.graff at 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) {
>
> Doesn't apply to my tree at all :(
>


More information about the devel mailing list