Commit 04d66968 authored by Ian Abbott's avatar Ian Abbott Committed by Greg Kroah-Hartman

staging: comedi: amplc_pc263: Remove thisboard and devpriv macros

The 'thisboard' and 'devpriv' macros rely on a local variable having a
specific name and yield pointers derived from that local variable.
Replace the macros with local variables wherever they occur.
Signed-off-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7ac75ba4
...@@ -93,11 +93,6 @@ static const struct pc263_board pc263_boards[] = { ...@@ -93,11 +93,6 @@ static const struct pc263_board pc263_boards[] = {
#endif #endif
}; };
/*
* Useful for shorthand access to the particular board structure
*/
#define thisboard ((const struct pc263_board *)dev->board_ptr)
/* this structure is for data unique to this hardware driver. If /* this structure is for data unique to this hardware driver. If
several hardware drivers keep similar information in this structure, several hardware drivers keep similar information in this structure,
feel free to suggest moving the variable to the struct comedi_device struct. feel free to suggest moving the variable to the struct comedi_device struct.
...@@ -107,8 +102,6 @@ struct pc263_private { ...@@ -107,8 +102,6 @@ struct pc263_private {
struct pci_dev *pci_dev; struct pci_dev *pci_dev;
}; };
#define devpriv ((struct pc263_private *)dev->private)
/* /*
* This function looks for a PCI device matching the requested board name, * This function looks for a PCI device matching the requested board name,
* bus and slot. * bus and slot.
...@@ -116,6 +109,7 @@ struct pc263_private { ...@@ -116,6 +109,7 @@ struct pc263_private {
static struct pci_dev * static struct pci_dev *
pc263_find_pci(struct comedi_device *dev, int bus, int slot) pc263_find_pci(struct comedi_device *dev, int bus, int slot)
{ {
const struct pc263_board *thisboard = comedi_board(dev);
struct pci_dev *pci_dev = NULL; struct pci_dev *pci_dev = NULL;
/* Look for matching PCI device. */ /* Look for matching PCI device. */
...@@ -139,6 +133,7 @@ pc263_find_pci(struct comedi_device *dev, int bus, int slot) ...@@ -139,6 +133,7 @@ pc263_find_pci(struct comedi_device *dev, int bus, int slot)
if (pci_dev->device == pc263_boards[i].devid) { if (pci_dev->device == pc263_boards[i].devid) {
/* Change board_ptr to matched board. */ /* Change board_ptr to matched board. */
dev->board_ptr = &pc263_boards[i]; dev->board_ptr = &pc263_boards[i];
thisboard = comedi_board(dev);
break; break;
} }
} }
...@@ -200,6 +195,8 @@ static int pc263_do_insn_bits(struct comedi_device *dev, ...@@ -200,6 +195,8 @@ static int pc263_do_insn_bits(struct comedi_device *dev,
static void pc263_report_attach(struct comedi_device *dev) static void pc263_report_attach(struct comedi_device *dev)
{ {
const struct pc263_board *thisboard = comedi_board(dev);
struct pc263_private *devpriv = dev->private;
char tmpbuf[40]; char tmpbuf[40];
if (IS_ENABLED(CONFIG_COMEDI_AMPLC_PC263_ISA) && if (IS_ENABLED(CONFIG_COMEDI_AMPLC_PC263_ISA) &&
...@@ -222,6 +219,7 @@ static void pc263_report_attach(struct comedi_device *dev) ...@@ -222,6 +219,7 @@ static void pc263_report_attach(struct comedi_device *dev)
*/ */
static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it) static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it)
{ {
const struct pc263_board *thisboard = comedi_board(dev);
struct comedi_subdevice *s; struct comedi_subdevice *s;
unsigned long iobase = 0; unsigned long iobase = 0;
int ret; int ret;
...@@ -237,6 +235,7 @@ static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it) ...@@ -237,6 +235,7 @@ static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it)
return ret; return ret;
} else if (IS_ENABLED(CONFIG_COMEDI_AMPLC_PC263_PCI) && } else if (IS_ENABLED(CONFIG_COMEDI_AMPLC_PC263_PCI) &&
thisboard->bustype == pci_bustype) { thisboard->bustype == pci_bustype) {
struct pc263_private *devpriv;
struct pci_dev *pci_dev = NULL; struct pci_dev *pci_dev = NULL;
int bus, slot; int bus, slot;
...@@ -245,11 +244,13 @@ static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it) ...@@ -245,11 +244,13 @@ static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it)
dev_err(dev->class_dev, "error! out of memory!\n"); dev_err(dev->class_dev, "error! out of memory!\n");
return ret; return ret;
} }
devpriv = dev->private;
bus = it->options[0]; bus = it->options[0];
slot = it->options[1]; slot = it->options[1];
pci_dev = pc263_find_pci(dev, bus, slot); pci_dev = pc263_find_pci(dev, bus, slot);
if (pci_dev == NULL) if (pci_dev == NULL)
return -EIO; return -EIO;
thisboard = comedi_board(dev); /* replaced wildcard board */
devpriv->pci_dev = pci_dev; devpriv->pci_dev = pci_dev;
ret = comedi_pci_enable(pci_dev, PC263_DRIVER_NAME); ret = comedi_pci_enable(pci_dev, PC263_DRIVER_NAME);
if (ret < 0) { if (ret < 0) {
...@@ -290,6 +291,8 @@ static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it) ...@@ -290,6 +291,8 @@ static int pc263_attach(struct comedi_device *dev, struct comedi_devconfig *it)
static void pc263_detach(struct comedi_device *dev) static void pc263_detach(struct comedi_device *dev)
{ {
struct pc263_private *devpriv = dev->private;
if (IS_ENABLED(CONFIG_COMEDI_AMPLC_PC263_PCI) && devpriv && if (IS_ENABLED(CONFIG_COMEDI_AMPLC_PC263_PCI) && devpriv &&
devpriv->pci_dev) { devpriv->pci_dev) {
if (dev->iobase) if (dev->iobase)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment