Re: [PATCH] usb: host: ehci-sched: silence checkpatch warning

[Geyslan G. Bem]

Hello,

2015-12-12 13:19 GMT-03:00 Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>:
> On Fri, 11 Dec 2015, Geyslan G. Bem wrote:
>
>> This patch moves comment into the else branch avoiding the following
>> misleading warning.
>>
>> "braces {} are not necessary for single statement blocks"
>>
>> Caught by checkpatch.
>>
>> Signed-off-by: Geyslan G. Bem <geyslan@xxxxxxxxx>
>> Suggested-by: Joe Perches <joe@xxxxxxxxxxx>
>> ---
>>  drivers/usb/host/ehci-sched.c | 7 ++-----
>>  1 file changed, 2 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/usb/host/ehci-sched.c b/drivers/usb/host/ehci-sched.c
>> index f6e828a..b2af0b3 100644
>> --- a/drivers/usb/host/ehci-sched.c
>> +++ b/drivers/usb/host/ehci-sched.c
>> @@ -1560,13 +1560,10 @@ iso_stream_schedule(
>>                                       (stream->ps.bw_period - 1);
>>                       stream->ps.phase_uf = start & 7;
>>                       reserve_release_iso_bandwidth(ehci, stream, 1);
>> -             }
>> -
>> -             /* New stream is already scheduled; use the upcoming slot */
>> -             else {
>> +             } else {
>> +                     /* New stream is already scheduled; use the upcoming slot */
>
> Please don't do this.  The comment at the start of the "if" part of
> this statement comes before the "if" line, so the commend at the start
> of the "else" part should come before the "else" line.

I see. It's particular comment style.
>
> If you want to silence the warning about unnecessary braces, get rid of
> the braces.
Well, this is a particular case of checkpatch due it think that the if
block has a single statement. But it's not true.

Please see:
[BUG] checkpatch warning default switch case
https://lkml.org/lkml/2015/12/11/836

So removing the braces breaks the coding style.

>
>>                       start = (stream->ps.phase << 3) + stream->ps.phase_uf;
>>               }
>> -
>
> Don't do this.
>
>>               stream->next_uframe = start;
>>               new_stream = true;
>>       }
>
> Alan Stern
>



-- 
Regards,

Geyslan G. Bem
hackingbits.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/