staging: comedi: ni_mio_common: consolidate call to ni_mseries_get_pll_parameters()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Fri, 18 Jul 2014 20:29:53 +0000 (13:29 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Jul 2014 23:08:01 +0000 (16:08 -0700)
All cases of the switch call ni_mseries_get_pll_parameters() with the same
parameters. Move the call out of the switch.

Also, move the printk from ni_mseries_get_pll_parameters() so that it can
be converted into a dev_err().

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

index efe62af..551a150 100644 (file)
@@ -4902,10 +4902,9 @@ static int ni_mseries_get_pll_parameters(unsigned reference_period_ns,
                        }
                }
        }
-       if (best_period_picosec == 0) {
-               printk("%s: bug, failed to find pll parameters\n", __func__);
+       if (best_period_picosec == 0)
                return -EIO;
-       }
+
        *freq_divider = best_div;
        *freq_multiplier = best_mult;
        *actual_period_ns =
@@ -4948,21 +4947,11 @@ static int ni_mseries_set_pll_master_clock(struct comedi_device *dev,
        case NI_MIO_PLL_PXI_STAR_TRIGGER_CLOCK:
                devpriv->clock_and_fout2 |=
                    MSeries_PLL_In_Source_Select_Star_Trigger_Bits;
-               retval = ni_mseries_get_pll_parameters(period_ns, &freq_divider,
-                                                      &freq_multiplier,
-                                                      &devpriv->clock_ns);
-               if (retval < 0)
-                       return retval;
                break;
        case NI_MIO_PLL_PXI10_CLOCK:
                /* pxi clock is 10MHz */
                devpriv->clock_and_fout2 |=
                    MSeries_PLL_In_Source_Select_PXI_Clock10;
-               retval = ni_mseries_get_pll_parameters(period_ns, &freq_divider,
-                                                      &freq_multiplier,
-                                                      &devpriv->clock_ns);
-               if (retval < 0)
-                       return retval;
                break;
        default:
                {
@@ -4981,16 +4970,19 @@ static int ni_mseries_set_pll_master_clock(struct comedi_device *dev,
                        }
                        if (rtsi_channel > max_rtsi_channel)
                                return -EINVAL;
-                       retval = ni_mseries_get_pll_parameters(period_ns,
-                                                              &freq_divider,
-                                                              &freq_multiplier,
-                                                              &devpriv->
-                                                              clock_ns);
-                       if (retval < 0)
-                               return retval;
                }
                break;
        }
+       retval = ni_mseries_get_pll_parameters(period_ns,
+                                              &freq_divider,
+                                              &freq_multiplier,
+                                              &devpriv->clock_ns);
+       if (retval < 0) {
+               dev_err(dev->class_dev,
+                       "%s: bug, failed to find pll parameters\n", __func__);
+               return retval;
+       }
+
        ni_writew(dev, devpriv->clock_and_fout2, M_Offset_Clock_and_Fout2);
        pll_control_bits |=
            MSeries_PLL_Divisor_Bits(freq_divider) |