On Tue, Oct 9, 2012 at 3:05 AM, Rafał Miłecki <zajec5@xxxxxxxxx> wrote:
> 2012/10/8 <alexdeucher@xxxxxxxxx>:
>> + 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@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
