On Tue, Oct 9, 2012 at 3:05 AM, Rafa? Mi?ecki <zajec5 at gmail.com> wrote:
> 2012/10/8  <alexdeucher at 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

Reply via email to