Commit a8a47b16 authored by Lee Jones's avatar Lee Jones Committed by Jason Gunthorpe

RDMA/hw/qib/qib_iba7322: Fix a bunch of copy/paste issues

Fixes the following W=1 kernel build warning(s):

 drivers/infiniband/hw/qib/qib_iba7322.c:2521: warning: Function parameter or member 'ppd' not described in 'qib_7322_mini_quiet_serdes'
 drivers/infiniband/hw/qib/qib_iba7322.c:2521: warning: Excess function parameter 'dd' description in 'qib_7322_mini_quiet_serdes'
 drivers/infiniband/hw/qib/qib_iba7322.c:3768: warning: Function parameter or member 'type' not described in 'qib_7322_put_tid'
 drivers/infiniband/hw/qib/qib_iba7322.c:3768: warning: Excess function parameter 'tidtype' description in 'qib_7322_put_tid'
 drivers/infiniband/hw/qib/qib_iba7322.c:3806: warning: Function parameter or member 'rcd' not described in 'qib_7322_clear_tids'
 drivers/infiniband/hw/qib/qib_iba7322.c:3806: warning: Excess function parameter 'ctxt' description in 'qib_7322_clear_tids'
 drivers/infiniband/hw/qib/qib_iba7322.c:3872: warning: Function parameter or member 'kinfo' not described in 'qib_7322_get_base_info'
 drivers/infiniband/hw/qib/qib_iba7322.c:3872: warning: Excess function parameter 'kbase' description in 'qib_7322_get_base_info'
 drivers/infiniband/hw/qib/qib_iba7322.c:4730: warning: Function parameter or member 'reg' not described in 'qib_portcntr_7322'
 drivers/infiniband/hw/qib/qib_iba7322.c:4730: warning: Excess function parameter 'creg' description in 'qib_portcntr_7322'
 drivers/infiniband/hw/qib/qib_iba7322.c:5109: warning: Function parameter or member 't' not described in 'qib_get_7322_faststats'
 drivers/infiniband/hw/qib/qib_iba7322.c:7189: warning: Function parameter or member 'pdev' not described in 'qib_init_iba7322_funcs'
 drivers/infiniband/hw/qib/qib_iba7322.c:7189: warning: Excess function parameter 'dev' description in 'qib_init_iba7322_funcs'

Link: https://lore.kernel.org/r/20210121094519.2044049-28-lee.jones@linaro.org
Cc: Dennis Dalessandro <dennis.dalessandro@cornelisnetworks.com>
Cc: Mike Marciniszyn <mike.marciniszyn@cornelisnetworks.com>
Cc: Doug Ledford <dledford@redhat.com>
Cc: Jason Gunthorpe <jgg@ziepe.ca>
Cc: linux-rdma@vger.kernel.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent cf8f5cea
...@@ -2514,7 +2514,7 @@ static int qib_7322_bringup_serdes(struct qib_pportdata *ppd) ...@@ -2514,7 +2514,7 @@ static int qib_7322_bringup_serdes(struct qib_pportdata *ppd)
/** /**
* qib_7322_quiet_serdes - set serdes to txidle * qib_7322_quiet_serdes - set serdes to txidle
* @dd: the qlogic_ib device * @ppd: the qlogic_ib device
* Called when driver is being unloaded * Called when driver is being unloaded
*/ */
static void qib_7322_mini_quiet_serdes(struct qib_pportdata *ppd) static void qib_7322_mini_quiet_serdes(struct qib_pportdata *ppd)
...@@ -3760,7 +3760,7 @@ static int qib_do_7322_reset(struct qib_devdata *dd) ...@@ -3760,7 +3760,7 @@ static int qib_do_7322_reset(struct qib_devdata *dd)
* qib_7322_put_tid - write a TID to the chip * qib_7322_put_tid - write a TID to the chip
* @dd: the qlogic_ib device * @dd: the qlogic_ib device
* @tidptr: pointer to the expected TID (in chip) to update * @tidptr: pointer to the expected TID (in chip) to update
* @tidtype: 0 for eager, 1 for expected * @type: 0 for eager, 1 for expected
* @pa: physical address of in memory buffer; tidinvalid if freeing * @pa: physical address of in memory buffer; tidinvalid if freeing
*/ */
static void qib_7322_put_tid(struct qib_devdata *dd, u64 __iomem *tidptr, static void qib_7322_put_tid(struct qib_devdata *dd, u64 __iomem *tidptr,
...@@ -3796,7 +3796,7 @@ static void qib_7322_put_tid(struct qib_devdata *dd, u64 __iomem *tidptr, ...@@ -3796,7 +3796,7 @@ static void qib_7322_put_tid(struct qib_devdata *dd, u64 __iomem *tidptr,
/** /**
* qib_7322_clear_tids - clear all TID entries for a ctxt, expected and eager * qib_7322_clear_tids - clear all TID entries for a ctxt, expected and eager
* @dd: the qlogic_ib device * @dd: the qlogic_ib device
* @ctxt: the ctxt * @rcd: the ctxt
* *
* clear all TID entries for a ctxt, expected and eager. * clear all TID entries for a ctxt, expected and eager.
* Used from qib_close(). * Used from qib_close().
...@@ -3861,7 +3861,7 @@ static void qib_7322_tidtemplate(struct qib_devdata *dd) ...@@ -3861,7 +3861,7 @@ static void qib_7322_tidtemplate(struct qib_devdata *dd)
/** /**
* qib_init_7322_get_base_info - set chip-specific flags for user code * qib_init_7322_get_base_info - set chip-specific flags for user code
* @rcd: the qlogic_ib ctxt * @rcd: the qlogic_ib ctxt
* @kbase: qib_base_info pointer * @kinfo: qib_base_info pointer
* *
* We set the PCIE flag because the lower bandwidth on PCIe vs * We set the PCIE flag because the lower bandwidth on PCIe vs
* HyperTransport can affect some user packet algorithims. * HyperTransport can affect some user packet algorithims.
...@@ -4724,7 +4724,7 @@ static void sendctrl_7322_mod(struct qib_pportdata *ppd, u32 op) ...@@ -4724,7 +4724,7 @@ static void sendctrl_7322_mod(struct qib_pportdata *ppd, u32 op)
/** /**
* qib_portcntr_7322 - read a per-port chip counter * qib_portcntr_7322 - read a per-port chip counter
* @ppd: the qlogic_ib pport * @ppd: the qlogic_ib pport
* @creg: the counter to read (not a chip offset) * @reg: the counter to read (not a chip offset)
*/ */
static u64 qib_portcntr_7322(struct qib_pportdata *ppd, u32 reg) static u64 qib_portcntr_7322(struct qib_pportdata *ppd, u32 reg)
{ {
...@@ -5096,7 +5096,7 @@ static u32 qib_read_7322portcntrs(struct qib_devdata *dd, loff_t pos, u32 port, ...@@ -5096,7 +5096,7 @@ static u32 qib_read_7322portcntrs(struct qib_devdata *dd, loff_t pos, u32 port,
/** /**
* qib_get_7322_faststats - get word counters from chip before they overflow * qib_get_7322_faststats - get word counters from chip before they overflow
* @opaque - contains a pointer to the qlogic_ib device qib_devdata * @t: contains a pointer to the qlogic_ib device qib_devdata
* *
* VESTIGIAL IBA7322 has no "small fast counters", so the only * VESTIGIAL IBA7322 has no "small fast counters", so the only
* real purpose of this function is to maintain the notion of * real purpose of this function is to maintain the notion of
...@@ -7175,7 +7175,7 @@ static int qib_7322_tempsense_rd(struct qib_devdata *dd, int regnum) ...@@ -7175,7 +7175,7 @@ static int qib_7322_tempsense_rd(struct qib_devdata *dd, int regnum)
/** /**
* qib_init_iba7322_funcs - set up the chip-specific function pointers * qib_init_iba7322_funcs - set up the chip-specific function pointers
* @dev: the pci_dev for qlogic_ib device * @pdev: the pci_dev for qlogic_ib device
* @ent: pci_device_id struct for this dev * @ent: pci_device_id struct for this dev
* *
* Also allocates, inits, and returns the devdata struct for this * Also allocates, inits, and returns the devdata struct for this
......
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