staging: comedi: contec_pci_dio: cleanup the "find pci device" code
PATCH 20/30 staging: comedi: contec_pci_dio: cleanup the "find pci device" code
Cleanup the "find pci device" code to follow the format of the other comedi pci drivers.
Create local variables for the bus and slot options to clarify the code. Change the pci ven/dev test to reduce the indent level. Change the printk into a dev_warn.
Signed-off-by H Hartley Sweeten <[email protected]>
Cc Ian Abbott <[email protected]>
Cc Greg Kroah-Hartman <[email protected]>
--- drivers/staging/comedi/drivers/contec_pci_dio.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-)
drivers/staging/comedi/drivers/contec_pci_dio.c +12 -12
@@ -101,22 +101,22 @@ static struct pci_dev *contec_find_pci_dev(struct comedi_device *dev, struct comedi_devconfig *it) {struct pci_dev *pcidev =NULL ; +int bus = it->options[0 ]; +int slot = it->options[1 ];for_each_pci_dev (pcidev) { -if (pcidev->vendor ==PCI_VENDOR_ID_CONTEC && - pcidev->device ==PCI_DEVICE_ID_PIO1616L ) { -if (it->options[0 ] || it->options[1 ]) { -/* Check bus and slot. */ -if (it->options[0 ] != pcidev->bus->number || - it->options[1 ] !=PCI_SLOT (pcidev->devfn)) { -continue ; - } - } - dev->board_ptr = contec_boards +0 ; -return pcidev; +if (pcidev->vendor !=PCI_VENDOR_ID_CONTEC || + pcidev->device !=PCI_DEVICE_ID_PIO1616L ) +continue ; +if (bus || slot) { +if (bus != pcidev->bus->number || + slot !=PCI_SLOT (pcidev->devfn)) +continue ; } + dev->board_ptr = contec_boards +0 ; +return pcidev; } -printk ("card not present!\n" ); +dev_warn (dev->class_dev,"card not present!\n" );return NULL ; }
Signature · 1 line
1.7.11