staging: comedi: amplc_dio200: absorb dio200_subdev_timer_init()

This function is only called by amplc_dio200_common_attach() and it
can never fail. For aesthetics, absorb it into that function.

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>

diff --git a/drivers/staging/comedi/drivers/amplc_dio200_common.c b/drivers/staging/comedi/drivers/amplc_dio200_common.c
index 5835e27c18422020e96a68aaa68be0160fdd722a..ba449029e16d6fa261aa30562394aaf545db4c34 100644 (file)
--- a/drivers/staging/comedi/drivers/amplc_dio200_common.c
+++ b/drivers/staging/comedi/drivers/amplc_dio200_common.c
@@ -979,18 +979,6 @@ static int dio200_subdev_timer_config(struct comedi_device *dev,
        return ret < 0 ? ret : insn->n;
 }
 
-static int dio200_subdev_timer_init(struct comedi_device *dev,
-                                   struct comedi_subdevice *s)
-{
-       s->type = COMEDI_SUBD_TIMER;
-       s->subdev_flags = SDF_READABLE | SDF_LSAMPL;
-       s->n_chan = 1;
-       s->maxdata = 0xFFFFFFFF;
-       s->insn_read = dio200_subdev_timer_read;
-       s->insn_config = dio200_subdev_timer_config;
-       return 0;
-}
-
 void amplc_dio200_set_enhance(struct comedi_device *dev, unsigned char val)
 {
        dio200_write8(dev, DIO200_ENHANCE, val);
@@ -1040,9 +1028,12 @@ int amplc_dio200_common_attach(struct comedi_device *dev, unsigned int irq,
                        }
                        break;
                case sd_timer:
-                       ret = dio200_subdev_timer_init(dev, s);
-                       if (ret < 0)
-                               return ret;
+                       s->type         = COMEDI_SUBD_TIMER;
+                       s->subdev_flags = SDF_READABLE | SDF_LSAMPL;
+                       s->n_chan       = 1;
+                       s->maxdata      = 0xffffffff;
+                       s->insn_read    = dio200_subdev_timer_read;
+                       s->insn_config  = dio200_subdev_timer_config;
                        break;
                default:
                        s->type = COMEDI_SUBD_UNUSED;