IB/hfi1: Remove rcvhdrsize
authorMike Marciniszyn <mike.marciniszyn@intel.com>
Mon, 4 Jun 2018 18:43:46 +0000 (11:43 -0700)
committerJason Gunthorpe <jgg@mellanox.com>
Tue, 19 Jun 2018 17:49:46 +0000 (11:49 -0600)
The field is based on a constant that can never change.

Use the define to assign the register instead.

Reviewed-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
Signed-off-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/hw/hfi1/chip.c
drivers/infiniband/hw/hfi1/hfi.h
drivers/infiniband/hw/hfi1/init.c

index b13636c..ccbdce2 100644 (file)
@@ -15216,8 +15216,6 @@ struct hfi1_devdata *hfi1_init_dd(struct pci_dev *pdev,
         */
        aspm_init(dd);
 
-       dd->rcvhdrsize = DEFAULT_RCVHDRSIZE;
-
        ret = init_pervl_scs(dd);
        if (ret)
                goto bail_cleanup;
index b30985a..ba6dd2c 100644 (file)
@@ -1136,8 +1136,6 @@ struct hfi1_devdata {
 
        /* these are the "32 bit" regs */
 
-       /* value we put in kr_rcvhdrsize */
-       u32 rcvhdrsize;
        /* number of receive contexts the chip supports */
        u32 chip_rcv_contexts;
        /* number of receive array entries */
index f3e0b89..f2a0b03 100644 (file)
@@ -1887,7 +1887,7 @@ int hfi1_create_rcvhdrq(struct hfi1_devdata *dd, struct hfi1_ctxtdata *rcd)
                        & RCV_HDR_ENT_SIZE_ENT_SIZE_MASK)
                << RCV_HDR_ENT_SIZE_ENT_SIZE_SHIFT;
        write_kctxt_csr(dd, rcd->ctxt, RCV_HDR_ENT_SIZE, reg);
-       reg = (dd->rcvhdrsize & RCV_HDR_SIZE_HDR_SIZE_MASK)
+       reg = ((u64)DEFAULT_RCVHDRSIZE & RCV_HDR_SIZE_HDR_SIZE_MASK)
                << RCV_HDR_SIZE_HDR_SIZE_SHIFT;
        write_kctxt_csr(dd, rcd->ctxt, RCV_HDR_SIZE, reg);