staging: comedi: mite: pass comedi_device to mite_setup()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 29 Jul 2014 22:01:37 +0000 (15:01 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 30 Jul 2014 23:55:15 +0000 (16:55 -0700)
For aesthetics, pass the comedi_device to mite_setup() and mite_setup2()
so that the dev->class_dev can be used in the dev_{level} message to make
the messages have consistancy with the other comedi drivers.

Remove the extra dev_warn() noise in the drivers when mite_setup() fails.

Rename the 'use_iodwbsr_1' parameter and change it to a bool.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/mite.c
drivers/staging/comedi/drivers/mite.h
drivers/staging/comedi/drivers/ni_660x.c
drivers/staging/comedi/drivers/ni_pcidio.c
drivers/staging/comedi/drivers/ni_pcimio.c

index d90063bf0279aca189ff60a06f304c196c48ebd5..9041fd1803813210dcb315dc872f359cb95d54f2 100644 (file)
@@ -94,7 +94,8 @@ static unsigned mite_fifo_size(struct mite_struct *mite, unsigned channel)
        return empty_count + full_count;
 }
 
-int mite_setup2(struct mite_struct *mite, unsigned use_iodwbsr_1)
+int mite_setup2(struct comedi_device *dev,
+               struct mite_struct *mite, bool use_win1)
 {
        unsigned long length;
        int i;
@@ -105,7 +106,7 @@ int mite_setup2(struct mite_struct *mite, unsigned use_iodwbsr_1)
 
        mite->mite_io_addr = pci_ioremap_bar(mite->pcidev, 0);
        if (!mite->mite_io_addr) {
-               dev_err(&mite->pcidev->dev,
+               dev_err(dev->class_dev,
                        "Failed to remap mite io memory address\n");
                return -ENOMEM;
        }
@@ -113,16 +114,16 @@ int mite_setup2(struct mite_struct *mite, unsigned use_iodwbsr_1)
 
        mite->daq_io_addr = pci_ioremap_bar(mite->pcidev, 1);
        if (!mite->daq_io_addr) {
-               dev_err(&mite->pcidev->dev,
+               dev_err(dev->class_dev,
                        "Failed to remap daq io memory address\n");
                return -ENOMEM;
        }
        mite->daq_phys_addr = pci_resource_start(mite->pcidev, 1);
        length = pci_resource_len(mite->pcidev, 1);
 
-       if (use_iodwbsr_1) {
+       if (use_win1) {
                writel(0, mite->mite_io_addr + MITE_IODWBSR);
-               dev_info(&mite->pcidev->dev,
+               dev_info(dev->class_dev,
                         "using I/O Window Base Size register 1\n");
                writel(mite->daq_phys_addr | WENAB |
                       MITE_IODWBSR_1_WSIZE_bits(length),
@@ -148,7 +149,7 @@ int mite_setup2(struct mite_struct *mite, unsigned use_iodwbsr_1)
        csigr_bits = readl(mite->mite_io_addr + MITE_CSIGR);
        mite->num_channels = mite_csigr_dmac(csigr_bits);
        if (mite->num_channels > MAX_MITE_DMA_CHANNELS) {
-               dev_warn(&mite->pcidev->dev,
+               dev_warn(dev->class_dev,
                         "mite: bug? chip claims to have %i dma channels. Setting to %i.\n",
                         mite->num_channels, MAX_MITE_DMA_CHANNELS);
                mite->num_channels = MAX_MITE_DMA_CHANNELS;
@@ -163,7 +164,7 @@ int mite_setup2(struct mite_struct *mite, unsigned use_iodwbsr_1)
                       mite->mite_io_addr + MITE_CHCR(i));
        }
        mite->fifo_size = mite_fifo_size(mite, 0);
-       dev_info(&mite->pcidev->dev, "fifo size is %i.\n", mite->fifo_size);
+       dev_info(dev->class_dev, "fifo size is %i.\n", mite->fifo_size);
        return 0;
 }
 EXPORT_SYMBOL_GPL(mite_setup2);
index 7c938b0452c5e5112457a1900411a7d917711c58..b15c06671e870c5d41d304f65d2a66e474b9d756 100644 (file)
@@ -65,11 +65,12 @@ struct mite_struct {
 
 struct mite_struct *mite_alloc(struct pci_dev *pcidev);
 
-int mite_setup2(struct mite_struct *mite, unsigned use_iodwbsr_1);
+int mite_setup2(struct comedi_device *, struct mite_struct *, bool use_win1);
 
-static inline int mite_setup(struct mite_struct *mite)
+static inline int mite_setup(struct comedi_device *dev,
+                            struct mite_struct *mite)
 {
-       return mite_setup2(mite, 0);
+       return mite_setup2(dev, mite, false);
 }
 
 void mite_detach(struct mite_struct *mite);
index ef4a8f0fb29cf4fe9ecfa09ca199ef664a52d2c9..fa19a8e7627d36107ae1351d06f14c6188b4b13f 100644 (file)
@@ -1078,11 +1078,9 @@ static int ni_660x_auto_attach(struct comedi_device *dev,
        if (!devpriv->mite)
                return -ENOMEM;
 
-       ret = mite_setup2(devpriv->mite, 1);
-       if (ret < 0) {
-               dev_warn(dev->class_dev, "error setting up mite\n");
+       ret = mite_setup2(dev, devpriv->mite, true);
+       if (ret < 0)
                return ret;
-       }
 
        ret = ni_660x_alloc_mite_rings(dev);
        if (ret < 0)
index 352c60fa58b6f6ed527b147fdcce4d856093fc49..002d276371352255f77ed95fbba2d963c066fb66 100644 (file)
@@ -968,11 +968,9 @@ static int nidio_auto_attach(struct comedi_device *dev,
        if (!devpriv->mite)
                return -ENOMEM;
 
-       ret = mite_setup(devpriv->mite);
-       if (ret < 0) {
-               dev_warn(dev->class_dev, "error setting up mite\n");
+       ret = mite_setup(dev, devpriv->mite);
+       if (ret < 0)
                return ret;
-       }
 
        devpriv->di_mite_ring = mite_alloc_ring(devpriv->mite);
        if (devpriv->di_mite_ring == NULL)
index 22c0e95dc17455f09dbfd79cb4d90c2ce2b18b1b..b95019b7205fe397cef3603d34c1b4d3480909cc 100644 (file)
@@ -1176,11 +1176,9 @@ static int pcimio_auto_attach(struct comedi_device *dev,
        if (board->reg_type == ni_reg_6713)
                devpriv->is_6713 = 1;
 
-       ret = mite_setup(devpriv->mite);
-       if (ret < 0) {
-               dev_warn(dev->class_dev, "error setting up mite\n");
+       ret = mite_setup(dev, devpriv->mite);
+       if (ret < 0)
                return ret;
-       }
 
        devpriv->ai_mite_ring = mite_alloc_ring(devpriv->mite);
        if (devpriv->ai_mite_ring == NULL)