drm/i915/skl: Remove unnecessary () used with div_u64()
authorDamien Lespiau <damien.lespiau@intel.com>
Thu, 7 May 2015 17:38:43 +0000 (18:38 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 29 May 2015 08:15:24 +0000 (10:15 +0200)
div_u64() can be either a inline function or a define, but in either
case it's safe to provide expressions as parameters without outer ()
around them.

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 a8d3976..416f8fb 100644 (file)
@@ -1178,10 +1178,10 @@ static void skl_wrpll_params_populate(struct skl_wrpll_params *params,
         * Intermediate values are in Hz.
         * Divide by MHz to match bsepc
         */
-       params->dco_integer = div_u64(dco_freq, (24 * MHz(1)));
+       params->dco_integer = div_u64(dco_freq, 24 * MHz(1));
        params->dco_fraction =
-               div_u64(((div_u64(dco_freq, 24) -
-                         params->dco_integer * MHz(1)) * 0x8000), MHz(1));
+               div_u64((div_u64(dco_freq, 24) -
+                        params->dco_integer * MHz(1)) * 0x8000, MHz(1));
 }
 
 static bool