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 :)

-- 
Rafa?

Reply via email to