mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-10 15:13:44 -04:00
staging: comedi: remove this_board macro in the pcmad driver
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>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
2fbffee060
commit
449b683c6b
@@ -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;
|
||||
|
||||
Reference in New Issue
Block a user