On Tue, Oct 9, 2012 at 3:05 AM, Rafał Miłecki <zaj...@gmail.com> wrote:
> 2012/10/8  <alexdeuc...@gmail.com>:
>> +       if (speed_cntl & LC_CURRENT_DATA_RATE) {
>> +               DRM_INFO("PCIE gen 2 link speeds already enabled\n");
>> +               return;
>> +       } else
>> +               DRM_INFO("enabling PCIE gen 2 link speeds, disable with 
>> radeon.pcie_gen2=0\n");
>> +
>
> See documentation/CodingStyle:
>
>> This does not apply if only one branch of a conditional statement is a single
>> statement; in the latter case use braces in both branches:
>
> Your else should use braces :)

I'll fix that up.

Alex
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to