staging: comedi: remove this_board macro in the pcmad driver
authorH Hartley Sweeten <hartleys@visionengravers.com>
Wed, 23 May 2012 00:39:22 +0000 (17:39 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 5 Jun 2012 03:30:02 +0000 (20:30 -0700)
The 'this_board' macro depends on having a local variable with
a magic name. The CodingStyle document suggests not doing this
to avoid confusion. Remove the macro and use the comedi_board()
inline helper to get the dev->board_ptr information.

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

index 1ec7d5cb346aadab594c4229f7d3de5945b82b35..2ec0b2e63c351c1b8e9e0c56849a4fb9b32eddc7 100644 (file)
@@ -58,8 +58,6 @@ struct pcmad_board_struct {
        int n_ai_bits;
 };
 
-#define this_board ((const struct pcmad_board_struct *)(dev->board_ptr))
-
 struct pcmad_priv_struct {
        int differential;
        int twos_comp;
@@ -72,6 +70,7 @@ static int pcmad_ai_insn_read(struct comedi_device *dev,
                              struct comedi_subdevice *s,
                              struct comedi_insn *insn, unsigned int *data)
 {
+       const struct pcmad_board_struct *board = comedi_board(dev);
        int i;
        int chan;
        int n;
@@ -89,7 +88,7 @@ static int pcmad_ai_insn_read(struct comedi_device *dev,
                data[n] |= (inb(dev->iobase + PCMAD_MSB) << 8);
 
                if (devpriv->twos_comp)
-                       data[n] ^= (1 << (this_board->n_ai_bits - 1));
+                       data[n] ^= (1 << (board->n_ai_bits - 1));
        }
 
        return n;
@@ -104,6 +103,7 @@ static int pcmad_ai_insn_read(struct comedi_device *dev,
  */
 static int pcmad_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 {
+       const struct pcmad_board_struct *board = comedi_board(dev);
        int ret;
        struct comedi_subdevice *s;
        unsigned long iobase;
@@ -125,7 +125,7 @@ static int pcmad_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        if (ret < 0)
                return ret;
 
-       dev->board_name = this_board->name;
+       dev->board_name = board->name;
 
        s = dev->subdevices + 0;
        s->type = COMEDI_SUBD_AI;
@@ -133,7 +133,7 @@ static int pcmad_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        s->n_chan = 16;         /* XXX */
        s->len_chanlist = 1;
        s->insn_read = pcmad_ai_insn_read;
-       s->maxdata = (1 << this_board->n_ai_bits) - 1;
+       s->maxdata = (1 << board->n_ai_bits) - 1;
        s->range_table = &range_unknown;
 
        return 0;