From: H Hartley Sweeten Date: Mon, 9 Dec 2013 22:30:50 +0000 (-0700) Subject: staging: comedi: pcmmio: tidy up the ao subdevice init X-Git-Tag: upstream/snapshot3+hdmi~3491^2~336 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5d4997bbfa3bb7ed4786bd96bfcc756b02f9551a;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git staging: comedi: pcmmio: tidy up the ao subdevice init Add some whitespace to the analog output subdevice init and reorder the init to follow the "norm" in comedi drivers. Remove the init of len_chanlist. This member is only used for subdevices the support async commands. The core will default the value correctly. Signed-off-by: H Hartley Sweeten Reviewed-by: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/comedi/drivers/pcmmio.c b/drivers/staging/comedi/drivers/pcmmio.c index 9338f0c..39dff77 100644 --- a/drivers/staging/comedi/drivers/pcmmio.c +++ b/drivers/staging/comedi/drivers/pcmmio.c @@ -1016,16 +1016,15 @@ static int pcmmio_attach(struct comedi_device *dev, struct comedi_devconfig *it) outb(0, dev->iobase + PCMMIO_AI_2ND_ADC_OFFSET + PCMMIO_AI_RESOURCE_REG); - /* Next, AO */ + /* Analog Output subdevice */ s = &dev->subdevices[1]; - s->maxdata = 0xffff; - s->range_table = &pcmmio_ao_ranges; - s->subdev_flags = SDF_READABLE; - s->type = COMEDI_SUBD_AO; - s->n_chan = 8; - s->len_chanlist = s->n_chan; - s->insn_read = pcmmio_ao_insn_read; - s->insn_write = pcmmio_ao_insn_write; + s->type = COMEDI_SUBD_AO; + s->subdev_flags = SDF_READABLE; + s->n_chan = 8; + s->maxdata = 0xffff; + s->range_table = &pcmmio_ao_ranges; + s->insn_read = pcmmio_ao_insn_read; + s->insn_write = pcmmio_ao_insn_write; /* initialize the resource enable register by clearing it */ outb(0, dev->iobase + PCMMIO_AO_RESOURCE_ENA_REG);