staging: comedi: ni_mio_common: remove 'fudge_factor_80_to_20Mhz'
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 14 Apr 2016 16:58:04 +0000 (09:58 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 29 Apr 2016 05:17:28 +0000 (22:17 -0700)
For aesthetics, remove this local variable to fix the checkpatch.pl issue:
CHECK: Avoid CamelCase: <fudge_factor_80_to_20Mhz>

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/ni_mio_common.c

index 8a8ba26ac222b144a43ee8586166baf6fef0fda1..655a1a1d0c61b52a61ddfe3f077772ad606c48e7 100644 (file)
@@ -4831,7 +4831,6 @@ static int ni_mseries_get_pll_parameters(unsigned int reference_period_ns,
         * divided by 4 to 20 MHz for most timing clocks
         */
        static const unsigned int target_picosec = 12500;
-       static const unsigned int fudge_factor_80_to_20Mhz = 4;
        int best_period_picosec = 0;
 
        for (div = 1; div <= NI_M_PLL_MAX_DIVISOR; ++div) {
@@ -4851,8 +4850,8 @@ static int ni_mseries_get_pll_parameters(unsigned int reference_period_ns,
 
        *freq_divider = best_div;
        *freq_multiplier = best_mult;
-       *actual_period_ns = DIV_ROUND_CLOSEST(best_period_picosec *
-                                             fudge_factor_80_to_20Mhz,
+       /* return the actual period (* fudge factor for 80 to 20 MHz) */
+       *actual_period_ns = DIV_ROUND_CLOSEST(best_period_picosec * 4,
                                              pico_per_nano);
        return 0;
 }