Re: [PATCH] drm/radeon: check if pcie gen 2 is already enabled

On Tue, Oct 9, 2012 at 3:05 AM, Rafał Miłecki zajec5@gmail.com wrote:

2012/10/8 alexdeucher@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