OSDN Git Service

IB/qib: Remove set-but-not-used variables
authorBart Van Assche <bart.vanassche@wdc.com>
Wed, 11 Oct 2017 17:49:22 +0000 (10:49 -0700)
committerDoug Ledford <dledford@redhat.com>
Sun, 15 Oct 2017 00:47:07 +0000 (20:47 -0400)
Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Tested-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Acked-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/hw/qib/qib_diag.c
drivers/infiniband/hw/qib/qib_file_ops.c
drivers/infiniband/hw/qib/qib_iba6120.c
drivers/infiniband/hw/qib/qib_iba7220.c
drivers/infiniband/hw/qib/qib_iba7322.c
drivers/infiniband/hw/qib/qib_mad.c
drivers/infiniband/hw/qib/qib_pcie.c
drivers/infiniband/hw/qib/qib_sd7220.c
drivers/infiniband/hw/qib/qib_tx.c
drivers/infiniband/hw/qib/qib_verbs.c

index 775018b..a9377ee 100644 (file)
@@ -761,7 +761,6 @@ static ssize_t qib_diag_read(struct file *fp, char __user *data,
 {
        struct qib_diag_client *dc = fp->private_data;
        struct qib_devdata *dd = dc->dd;
-       void __iomem *kreg_base;
        ssize_t ret;
 
        if (dc->pid != current->pid) {
@@ -769,8 +768,6 @@ static ssize_t qib_diag_read(struct file *fp, char __user *data,
                goto bail;
        }
 
-       kreg_base = dd->kregbase;
-
        if (count == 0)
                ret = 0;
        else if ((count % 4) || (*off % 4))
@@ -838,7 +835,6 @@ static ssize_t qib_diag_write(struct file *fp, const char __user *data,
 {
        struct qib_diag_client *dc = fp->private_data;
        struct qib_devdata *dd = dc->dd;
-       void __iomem *kreg_base;
        ssize_t ret;
 
        if (dc->pid != current->pid) {
@@ -846,8 +842,6 @@ static ssize_t qib_diag_write(struct file *fp, const char __user *data,
                goto bail;
        }
 
-       kreg_base = dd->kregbase;
-
        if (count == 0)
                ret = 0;
        else if ((count % 4) || (*off % 4))
index 9396c18..2d6a191 100644 (file)
@@ -696,15 +696,8 @@ static void qib_clean_part_key(struct qib_ctxtdata *rcd,
                               struct qib_devdata *dd)
 {
        int i, j, pchanged = 0;
-       u64 oldpkey;
        struct qib_pportdata *ppd = rcd->ppd;
 
-       /* for debugging only */
-       oldpkey = (u64) ppd->pkeys[0] |
-               ((u64) ppd->pkeys[1] << 16) |
-               ((u64) ppd->pkeys[2] << 32) |
-               ((u64) ppd->pkeys[3] << 48);
-
        for (i = 0; i < ARRAY_SIZE(rcd->pkeys); i++) {
                if (!rcd->pkeys[i])
                        continue;
@@ -1817,7 +1810,6 @@ static int qib_close(struct inode *in, struct file *fp)
        struct qib_devdata *dd;
        unsigned long flags;
        unsigned ctxt;
-       pid_t pid;
 
        mutex_lock(&qib_mutex);
 
@@ -1859,7 +1851,6 @@ static int qib_close(struct inode *in, struct file *fp)
        spin_lock_irqsave(&dd->uctxt_lock, flags);
        ctxt = rcd->ctxt;
        dd->rcd[ctxt] = NULL;
-       pid = rcd->pid;
        rcd->pid = 0;
        spin_unlock_irqrestore(&dd->uctxt_lock, flags);
 
index 3c5adfa..3a0a28e 100644 (file)
@@ -1903,7 +1903,6 @@ static void qib_6120_put_tid_2(struct qib_devdata *dd, u64 __iomem *tidptr,
                               u32 type, unsigned long pa)
 {
        u32 __iomem *tidp32 = (u32 __iomem *)tidptr;
-       u32 tidx;
 
        if (!dd->kregbase)
                return;
@@ -1927,7 +1926,6 @@ static void qib_6120_put_tid_2(struct qib_devdata *dd, u64 __iomem *tidptr,
                else /* for now, always full 4KB page */
                        pa |= 2 << 29;
        }
-       tidx = tidptr - dd->egrtidbase;
        writel(pa, tidp32);
        mmiowb();
 }
index ceb2259..d66d68e 100644 (file)
@@ -3514,7 +3514,6 @@ static void autoneg_7220_work(struct work_struct *work)
 {
        struct qib_pportdata *ppd;
        struct qib_devdata *dd;
-       u64 startms;
        u32 i;
        unsigned long flags;
 
@@ -3522,8 +3521,6 @@ static void autoneg_7220_work(struct work_struct *work)
                            autoneg_work.work)->pportdata;
        dd = ppd->dd;
 
-       startms = jiffies_to_msecs(jiffies);
-
        /*
         * Busy wait for this first part, it should be at most a
         * few hundred usec, since we scheduled ourselves for 2msec.
index 77217fa..b0413f9 100644 (file)
@@ -3431,7 +3431,6 @@ try_intx:
        for (i = 0; msixnum < dd->cspec->num_msix_entries; i++) {
                irq_handler_t handler;
                void *arg;
-               u64 val;
                int lsb, reg, sh;
 #ifdef CONFIG_INFINIBAND_QIB_DCA
                int dca = 0;
@@ -3502,8 +3501,8 @@ try_intx:
                        mask &= ~(1ULL << lsb);
                        redirect[reg] |= ((u64) msixnum) << sh;
                }
-               val = qib_read_kreg64(dd, 2 * msixnum + 1 +
-                       (QIB_7322_MsixTable_OFFS / sizeof(u64)));
+               qib_read_kreg64(dd, 2 * msixnum + 1 +
+                               (QIB_7322_MsixTable_OFFS / sizeof(u64)));
                if (firstcpu < nr_cpu_ids &&
                        zalloc_cpumask_var(
                                &dd->cspec->msix_entries[msixnum].mask,
@@ -5358,16 +5357,11 @@ static void try_7322_autoneg(struct qib_pportdata *ppd)
 static void autoneg_7322_work(struct work_struct *work)
 {
        struct qib_pportdata *ppd;
-       struct qib_devdata *dd;
-       u64 startms;
        u32 i;
        unsigned long flags;
 
        ppd = container_of(work, struct qib_chippport_specific,
                            autoneg_work.work)->ppd;
-       dd = ppd->dd;
-
-       startms = jiffies_to_msecs(jiffies);
 
        /*
         * Busy wait for this first part, it should be at most a
@@ -7807,13 +7801,12 @@ static void ibsd_wr_allchans(struct qib_pportdata *ppd, int addr, unsigned data,
 {
        struct qib_devdata *dd = ppd->dd;
        int chan;
-       u32 rbc;
 
        for (chan = 0; chan < SERDES_CHANS; ++chan) {
                ahb_mod(dd, IBSD(ppd->hw_pidx), (chan + (chan >> 1)), addr,
                        data, mask);
-               rbc = ahb_mod(dd, IBSD(ppd->hw_pidx), (chan + (chan >> 1)),
-                             addr, 0, 0);
+               ahb_mod(dd, IBSD(ppd->hw_pidx), (chan + (chan >> 1)), addr,
+                       0, 0);
        }
 }
 
index 04367e0..841312f 100644 (file)
@@ -280,7 +280,7 @@ static int subn_get_nodeinfo(struct ib_smp *smp, struct ib_device *ibdev,
 {
        struct ib_node_info *nip = (struct ib_node_info *)&smp->data;
        struct qib_devdata *dd = dd_from_ibdev(ibdev);
-       u32 vendor, majrev, minrev;
+       u32 majrev, minrev;
        unsigned pidx = port - 1; /* IB number port from 1, hdw from 0 */
 
        /* GUID 0 is illegal */
@@ -303,7 +303,6 @@ static int subn_get_nodeinfo(struct ib_smp *smp, struct ib_device *ibdev,
        minrev = dd->minrev;
        nip->revision = cpu_to_be32((majrev << 16) | minrev);
        nip->local_port_num = port;
-       vendor = dd->vendorid;
        nip->vendor_id[0] = QIB_SRC_OUI_1;
        nip->vendor_id[1] = QIB_SRC_OUI_2;
        nip->vendor_id[2] = QIB_SRC_OUI_3;
index 6f4cc26..5ac7b31 100644 (file)
@@ -397,7 +397,6 @@ MODULE_PARM_DESC(pcie_coalesce, "tune PCIe colescing on some Intel chipsets");
  */
 static void qib_tune_pcie_coalesce(struct qib_devdata *dd)
 {
-       int r;
        struct pci_dev *parent;
        u16 devid;
        u32 mask, bits, val;
@@ -452,7 +451,7 @@ static void qib_tune_pcie_coalesce(struct qib_devdata *dd)
        pci_read_config_dword(parent, 0x48, &val);
        val &= ~mask;
        val |= bits;
-       r = pci_write_config_dword(parent, 0x48, val);
+       pci_write_config_dword(parent, 0x48, val);
 }
 
 /*
index c72775f..ad52f7a 100644 (file)
@@ -755,7 +755,6 @@ static int qib_sd7220_ram_xfer(struct qib_devdata *dd, int sdnum, u32 loc,
        int addr;
        int ret;
        unsigned long flags;
-       const char *op;
 
        /* Pick appropriate transaction reg and "Chip select" for this serdes */
        switch (sdnum) {
@@ -775,7 +774,6 @@ static int qib_sd7220_ram_xfer(struct qib_devdata *dd, int sdnum, u32 loc,
                return -1;
        }
 
-       op = rd_notwr ? "Rd" : "Wr";
        spin_lock_irqsave(&dd->cspec->sdepb_lock, flags);
 
        owned = epb_access(dd, sdnum, 1);
index eface3b..c9537cf 100644 (file)
@@ -179,8 +179,6 @@ void qib_disarm_piobufs_set(struct qib_devdata *dd, unsigned long *mask,
                pppd[i] = NULL;
 
        for (i = 0; i < cnt; i++) {
-               int which;
-
                if (!test_bit(i, mask))
                        continue;
                /*
@@ -201,9 +199,7 @@ void qib_disarm_piobufs_set(struct qib_devdata *dd, unsigned long *mask,
                    (!test_bit(i << 1, dd->pioavailkernel) &&
                     find_ctxt(dd, i))) {
                        __set_bit(i, dd->pio_need_disarm);
-                       which = 0;
                } else {
-                       which = 1;
                        dd->f_sendctrl(dd->pport, QIB_SENDCTRL_DISARM_BUF(i));
                }
                spin_unlock_irqrestore(&dd->pioavail_lock, flags);
index 9d92aeb..46e2da0 100644 (file)
@@ -701,7 +701,7 @@ void qib_put_txreq(struct qib_verbs_txreq *tx)
  */
 void qib_verbs_sdma_desc_avail(struct qib_pportdata *ppd, unsigned avail)
 {
-       struct rvt_qp *qp, *nqp;
+       struct rvt_qp *qp;
        struct qib_qp_priv *qpp, *nqpp;
        struct rvt_qp *qps[20];
        struct qib_ibdev *dev;
@@ -714,7 +714,6 @@ void qib_verbs_sdma_desc_avail(struct qib_pportdata *ppd, unsigned avail)
        /* Search wait list for first QP wanting DMA descriptors. */
        list_for_each_entry_safe(qpp, nqpp, &dev->dmawait, iowait) {
                qp = qpp->owner;
-               nqp = nqpp->owner;
                if (qp->port_num != ppd->port)
                        continue;
                if (n == ARRAY_SIZE(qps))