drm/i915/skl: Use a more idomatic early return
authorDamien Lespiau <damien.lespiau@intel.com>
Thu, 7 May 2015 17:38:41 +0000 (18:38 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 29 May 2015 08:15:23 +0000 (10:15 +0200)
We can coalesce the WARN() condition with the WARN() itself and, as we
are returning early, we can de-intent the rest of the function.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_ddi.c

index f10f55690749c06b9f05b3f87d58b5199ac5f733..c547f03904bca09e4d197ff0ee63cfa87ec51deb 100644 (file)
@@ -1193,76 +1193,73 @@ found:
                }
        }
 
-       if (min_dco_index > 2) {
-               WARN(1, "No valid parameters found for pixel clock: %dHz\n",
-                    clock);
+       if (WARN(min_dco_index > 2,
+                "No valid parameters found for pixel clock: %dHz\n", clock))
                return false;
-       } else {
-               wrpll_params->central_freq = dco_central_freq[min_dco_index];
 
-               switch (dco_central_freq[min_dco_index]) {
-               case 9600000000ULL:
-                       wrpll_params->central_freq = 0;
-                       break;
-               case 9000000000ULL:
-                       wrpll_params->central_freq = 1;
-                       break;
-               case 8400000000ULL:
-                       wrpll_params->central_freq = 3;
-               }
+       wrpll_params->central_freq = dco_central_freq[min_dco_index];
 
-               switch (candidate_p0[min_dco_index]) {
-               case 1:
-                       wrpll_params->pdiv = 0;
-                       break;
-               case 2:
-                       wrpll_params->pdiv = 1;
-                       break;
-               case 3:
-                       wrpll_params->pdiv = 2;
-                       break;
-               case 7:
-                       wrpll_params->pdiv = 4;
-                       break;
-               default:
-                       WARN(1, "Incorrect PDiv\n");
-               }
+       switch (dco_central_freq[min_dco_index]) {
+       case 9600000000ULL:
+               wrpll_params->central_freq = 0;
+               break;
+       case 9000000000ULL:
+               wrpll_params->central_freq = 1;
+               break;
+       case 8400000000ULL:
+               wrpll_params->central_freq = 3;
+       }
 
-               switch (candidate_p2[min_dco_index]) {
-               case 5:
-                       wrpll_params->kdiv = 0;
-                       break;
-               case 2:
-                       wrpll_params->kdiv = 1;
-                       break;
-               case 3:
-                       wrpll_params->kdiv = 2;
-                       break;
-               case 1:
-                       wrpll_params->kdiv = 3;
-                       break;
-               default:
-                       WARN(1, "Incorrect KDiv\n");
-               }
+       switch (candidate_p0[min_dco_index]) {
+       case 1:
+               wrpll_params->pdiv = 0;
+               break;
+       case 2:
+               wrpll_params->pdiv = 1;
+               break;
+       case 3:
+               wrpll_params->pdiv = 2;
+               break;
+       case 7:
+               wrpll_params->pdiv = 4;
+               break;
+       default:
+               WARN(1, "Incorrect PDiv\n");
+       }
 
-               wrpll_params->qdiv_ratio = candidate_p1[min_dco_index];
-               wrpll_params->qdiv_mode =
-                       (wrpll_params->qdiv_ratio == 1) ? 0 : 1;
+       switch (candidate_p2[min_dco_index]) {
+       case 5:
+               wrpll_params->kdiv = 0;
+               break;
+       case 2:
+               wrpll_params->kdiv = 1;
+               break;
+       case 3:
+               wrpll_params->kdiv = 2;
+               break;
+       case 1:
+               wrpll_params->kdiv = 3;
+               break;
+       default:
+               WARN(1, "Incorrect KDiv\n");
+       }
 
-               dco_freq = candidate_p0[min_dco_index] *
-                       candidate_p1[min_dco_index] *
-                       candidate_p2[min_dco_index] * afe_clock;
+       wrpll_params->qdiv_ratio = candidate_p1[min_dco_index];
+       wrpll_params->qdiv_mode =
+               (wrpll_params->qdiv_ratio == 1) ? 0 : 1;
 
-               /*
-                * Intermediate values are in Hz.
-                * Divide by MHz to match bsepc
-                */
-               wrpll_params->dco_integer = div_u64(dco_freq, (24 * MHz(1)));
-               wrpll_params->dco_fraction =
-                       div_u64(((div_u64(dco_freq, 24) -
-                                 wrpll_params->dco_integer * MHz(1)) * 0x8000), MHz(1));
+       dco_freq = candidate_p0[min_dco_index] *
+               candidate_p1[min_dco_index] *
+               candidate_p2[min_dco_index] * afe_clock;
 
-       }
+       /*
+        * Intermediate values are in Hz.
+        * Divide by MHz to match bsepc
+        */
+       wrpll_params->dco_integer = div_u64(dco_freq, (24 * MHz(1)));
+       wrpll_params->dco_fraction =
+               div_u64(((div_u64(dco_freq, 24) -
+                         wrpll_params->dco_integer * MHz(1)) * 0x8000), MHz(1));
 
        return true;
 }