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 <hswee...@visionengravers.com>
Reviewed-by: Ian Abbott <abbo...@mev.co.uk>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/staging/comedi/drivers/ni_mio_common.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/comedi/drivers/ni_mio_common.c 
b/drivers/staging/comedi/drivers/ni_mio_common.c
index 8a8ba26..655a1a1 100644
--- a/drivers/staging/comedi/drivers/ni_mio_common.c
+++ b/drivers/staging/comedi/drivers/ni_mio_common.c
@@ -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;
 }
-- 
2.6.3

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to