Commit a738bd9b authored by Lee Jones's avatar Lee Jones Committed by Martin K. Petersen

scsi: lpfc: lpfc_attr: Fix-up a bunch of kernel-doc misdemeanours

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

 from drivers/scsi/lpfc/lpfc_attr.c:26:
 inlined from ‘lpfc_stat_data_ctrl_store’ at drivers/scsi/lpfc/lpfc_attr.c:4165:3:
 drivers/scsi/lpfc/lpfc_attr.c:2537: warning: Function parameter or member 'attr' not described in 'lpfc_vport_param_init'
 drivers/scsi/lpfc/lpfc_attr.c:2537: warning: Function parameter or member 'default' not described in 'lpfc_vport_param_init'
 drivers/scsi/lpfc/lpfc_attr.c:2537: warning: Function parameter or member 'minval' not described in 'lpfc_vport_param_init'
 drivers/scsi/lpfc/lpfc_attr.c:2537: warning: Function parameter or member 'maxval' not described in 'lpfc_vport_param_init'
 drivers/scsi/lpfc/lpfc_attr.c:2537: warning: Excess function parameter 'phba' description in 'lpfc_vport_param_init'
 drivers/scsi/lpfc/lpfc_attr.c:2537: warning: Excess function parameter 'val' description in 'lpfc_vport_param_init'
 drivers/scsi/lpfc/lpfc_attr.c:2792: warning: Function parameter or member 'dev' not described in 'lpfc_soft_wwpn_store'
 drivers/scsi/lpfc/lpfc_attr.c:2886: warning: Function parameter or member 'dev' not described in 'lpfc_soft_wwnn_store'
 drivers/scsi/lpfc/lpfc_attr.c:2886: warning: Function parameter or member 'attr' not described in 'lpfc_soft_wwnn_store'
 drivers/scsi/lpfc/lpfc_attr.c:2886: warning: Excess function parameter 'cdev' description in 'lpfc_soft_wwnn_store'
 drivers/scsi/lpfc/lpfc_attr.c:3224: warning: Function parameter or member 'vpt_wwpn' not described in 'lpfc_oas_lun_state_set'
 drivers/scsi/lpfc/lpfc_attr.c:3224: warning: Function parameter or member 'tgt_wwpn' not described in 'lpfc_oas_lun_state_set'
 drivers/scsi/lpfc/lpfc_attr.c:3224: warning: Function parameter or member 'pri' not described in 'lpfc_oas_lun_state_set'
 drivers/scsi/lpfc/lpfc_attr.c:3224: warning: Excess function parameter 'ndlp' description in 'lpfc_oas_lun_state_set'
 drivers/scsi/lpfc/lpfc_attr.c:3264: warning: Function parameter or member 'lun_pri' not described in 'lpfc_oas_lun_get_next'
 drivers/scsi/lpfc/lpfc_attr.c:3302: warning: Function parameter or member 'pri' not described in 'lpfc_oas_lun_state_change'
 drivers/scsi/lpfc/lpfc_attr.c:3376: warning: Function parameter or member 'count' not described in 'lpfc_oas_lun_store'
 drivers/scsi/lpfc/lpfc_attr.c:3835: warning: Function parameter or member 'vport' not described in 'lpfc_tgt_queue_depth_set'
 drivers/scsi/lpfc/lpfc_attr.c:3835: warning: Excess function parameter 'phba' description in 'lpfc_tgt_queue_depth_set'
 drivers/scsi/lpfc/lpfc_attr.c:4026: warning: Function parameter or member 'dev' not described in 'lpfc_topology_store'
 drivers/scsi/lpfc/lpfc_attr.c:4026: warning: Function parameter or member 'attr' not described in 'lpfc_topology_store'
 drivers/scsi/lpfc/lpfc_attr.c:4026: warning: Function parameter or member 'buf' not described in 'lpfc_topology_store'
 drivers/scsi/lpfc/lpfc_attr.c:4026: warning: Function parameter or member 'count' not described in 'lpfc_topology_store'
 drivers/scsi/lpfc/lpfc_attr.c:4026: warning: Excess function parameter 'phba' description in 'lpfc_topology_store'
 drivers/scsi/lpfc/lpfc_attr.c:4026: warning: Excess function parameter 'val' description in 'lpfc_topology_store'
 drivers/scsi/lpfc/lpfc_attr.c:4148: warning: Function parameter or member 'attr' not described in 'lpfc_stat_data_ctrl_store'
 drivers/scsi/lpfc/lpfc_attr.c:4302: warning: Function parameter or member 'attr' not described in 'lpfc_stat_data_ctrl_show'
 drivers/scsi/lpfc/lpfc_attr.c:4383: warning: Function parameter or member 'buf' not described in 'sysfs_drvr_stat_data_read'
 drivers/scsi/lpfc/lpfc_attr.c:4383: warning: Excess function parameter 'buff' description in 'sysfs_drvr_stat_data_read'
 drivers/scsi/lpfc/lpfc_attr.c:4476: warning: Function parameter or member 'dev' not described in 'lpfc_link_speed_store'
 drivers/scsi/lpfc/lpfc_attr.c:4476: warning: Function parameter or member 'attr' not described in 'lpfc_link_speed_store'
 drivers/scsi/lpfc/lpfc_attr.c:4476: warning: Function parameter or member 'buf' not described in 'lpfc_link_speed_store'
 drivers/scsi/lpfc/lpfc_attr.c:4476: warning: Function parameter or member 'count' not described in 'lpfc_link_speed_store'
 drivers/scsi/lpfc/lpfc_attr.c:4476: warning: Excess function parameter 'phba' description in 'lpfc_link_speed_store'
 drivers/scsi/lpfc/lpfc_attr.c:4476: warning: Excess function parameter 'val' description in 'lpfc_link_speed_store'
 drivers/scsi/lpfc/lpfc_attr.c:7117: warning: Function parameter or member 'verbose' not described in 'lpfc_hba_log_verbose_init'

Link: https://lore.kernel.org/r/20201102142359.561122-6-lee.jones@linaro.org
Cc: James Smart <james.smart@broadcom.com>
Cc: Dick Kennedy <dick.kennedy@broadcom.com>
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 9176ad27
...@@ -2771,7 +2771,7 @@ lpfc_soft_wwpn_show(struct device *dev, struct device_attribute *attr, ...@@ -2771,7 +2771,7 @@ lpfc_soft_wwpn_show(struct device *dev, struct device_attribute *attr,
/** /**
* lpfc_soft_wwpn_store - Set the ww port name of the adapter * lpfc_soft_wwpn_store - Set the ww port name of the adapter
* @dev class device that is converted into a Scsi_host. * @dev: class device that is converted into a Scsi_host.
* @attr: device attribute, not used. * @attr: device attribute, not used.
* @buf: contains the wwpn in hexadecimal. * @buf: contains the wwpn in hexadecimal.
* @count: number of wwpn bytes in buf * @count: number of wwpn bytes in buf
...@@ -2868,7 +2868,8 @@ lpfc_soft_wwnn_show(struct device *dev, struct device_attribute *attr, ...@@ -2868,7 +2868,8 @@ lpfc_soft_wwnn_show(struct device *dev, struct device_attribute *attr,
/** /**
* lpfc_soft_wwnn_store - sets the ww node name of the adapter * lpfc_soft_wwnn_store - sets the ww node name of the adapter
* @cdev: class device that is converted into a Scsi_host. * @dev: class device that is converted into a Scsi_host.
* @attr: device attribute, not used.
* @buf: contains the ww node name in hexadecimal. * @buf: contains the ww node name in hexadecimal.
* @count: number of wwnn bytes in buf. * @count: number of wwnn bytes in buf.
* *
...@@ -3204,9 +3205,11 @@ static DEVICE_ATTR(lpfc_xlane_lun_status, S_IRUGO, ...@@ -3204,9 +3205,11 @@ static DEVICE_ATTR(lpfc_xlane_lun_status, S_IRUGO,
* lpfc_oas_lun_state_set - enable or disable a lun for Optimized Access Storage * lpfc_oas_lun_state_set - enable or disable a lun for Optimized Access Storage
* (OAS) operations. * (OAS) operations.
* @phba: lpfc_hba pointer. * @phba: lpfc_hba pointer.
* @ndlp: pointer to fcp target node. * @vpt_wwpn: wwpn of the vport associated with the returned lun
* @tgt_wwpn: wwpn of the target associated with the returned lun
* @lun: the fc lun for setting oas state. * @lun: the fc lun for setting oas state.
* @oas_state: the oas state to be set to the lun. * @oas_state: the oas state to be set to the lun.
* @pri: priority
* *
* Returns: * Returns:
* SUCCESS : 0 * SUCCESS : 0
...@@ -3244,6 +3247,7 @@ lpfc_oas_lun_state_set(struct lpfc_hba *phba, uint8_t vpt_wwpn[], ...@@ -3244,6 +3247,7 @@ lpfc_oas_lun_state_set(struct lpfc_hba *phba, uint8_t vpt_wwpn[],
* @vpt_wwpn: wwpn of the vport associated with the returned lun * @vpt_wwpn: wwpn of the vport associated with the returned lun
* @tgt_wwpn: wwpn of the target associated with the returned lun * @tgt_wwpn: wwpn of the target associated with the returned lun
* @lun_status: status of the lun returned lun * @lun_status: status of the lun returned lun
* @lun_pri: priority of the lun returned lun
* *
* Returns the first or next lun enabled for OAS operations for the vport/target * Returns the first or next lun enabled for OAS operations for the vport/target
* specified. If a lun is found, its vport wwpn, target wwpn and status is * specified. If a lun is found, its vport wwpn, target wwpn and status is
...@@ -3282,6 +3286,7 @@ lpfc_oas_lun_get_next(struct lpfc_hba *phba, uint8_t vpt_wwpn[], ...@@ -3282,6 +3286,7 @@ lpfc_oas_lun_get_next(struct lpfc_hba *phba, uint8_t vpt_wwpn[],
* @tgt_wwpn: target wwpn by reference. * @tgt_wwpn: target wwpn by reference.
* @lun: the fc lun for setting oas state. * @lun: the fc lun for setting oas state.
* @oas_state: the oas state to be set to the oas_lun. * @oas_state: the oas state to be set to the oas_lun.
* @pri: priority
* *
* This routine enables (OAS_LUN_ENABLE) or disables (OAS_LUN_DISABLE) * This routine enables (OAS_LUN_ENABLE) or disables (OAS_LUN_DISABLE)
* a lun for OAS operations. * a lun for OAS operations.
...@@ -3356,6 +3361,7 @@ lpfc_oas_lun_show(struct device *dev, struct device_attribute *attr, ...@@ -3356,6 +3361,7 @@ lpfc_oas_lun_show(struct device *dev, struct device_attribute *attr,
* @dev: class device that is converted into a Scsi_host. * @dev: class device that is converted into a Scsi_host.
* @attr: device attribute, not used. * @attr: device attribute, not used.
* @buf: buffer for passing information. * @buf: buffer for passing information.
* @count: size of the formatting string
* *
* This function sets the OAS state for lun. Before this function is called, * This function sets the OAS state for lun. Before this function is called,
* the vport wwpn, target wwpn, and oas state need to be set. * the vport wwpn, target wwpn, and oas state need to be set.
...@@ -3817,7 +3823,7 @@ lpfc_vport_param_init(tgt_queue_depth, LPFC_MAX_TGT_QDEPTH, ...@@ -3817,7 +3823,7 @@ lpfc_vport_param_init(tgt_queue_depth, LPFC_MAX_TGT_QDEPTH,
/** /**
* lpfc_tgt_queue_depth_store: Sets an attribute value. * lpfc_tgt_queue_depth_store: Sets an attribute value.
* @phba: pointer the the adapter structure. * @vport: lpfc vport structure pointer.
* @val: integer attribute value. * @val: integer attribute value.
* *
* Description: Sets the parameter to the new value. * Description: Sets the parameter to the new value.
...@@ -4002,8 +4008,10 @@ LPFC_ATTR(topology, 0, 0, 6, ...@@ -4002,8 +4008,10 @@ LPFC_ATTR(topology, 0, 0, 6,
/** /**
* lpfc_topology_set - Set the adapters topology field * lpfc_topology_set - Set the adapters topology field
* @phba: lpfc_hba pointer. * @dev: class device that is converted into a scsi_host.
* @val: topology value. * @attr:device attribute, not used.
* @buf: buffer for passing information.
* @count: size of the data buffer.
* *
* Description: * Description:
* If val is in a valid range then set the adapter's topology field and * If val is in a valid range then set the adapter's topology field and
...@@ -4122,6 +4130,7 @@ static DEVICE_ATTR_RO(lpfc_static_vport); ...@@ -4122,6 +4130,7 @@ static DEVICE_ATTR_RO(lpfc_static_vport);
/** /**
* lpfc_stat_data_ctrl_store - write call back for lpfc_stat_data_ctrl sysfs file * lpfc_stat_data_ctrl_store - write call back for lpfc_stat_data_ctrl sysfs file
* @dev: Pointer to class device. * @dev: Pointer to class device.
* @attr: Unused.
* @buf: Data buffer. * @buf: Data buffer.
* @count: Size of the data buffer. * @count: Size of the data buffer.
* *
...@@ -4285,7 +4294,8 @@ lpfc_stat_data_ctrl_store(struct device *dev, struct device_attribute *attr, ...@@ -4285,7 +4294,8 @@ lpfc_stat_data_ctrl_store(struct device *dev, struct device_attribute *attr,
/** /**
* lpfc_stat_data_ctrl_show - Read function for lpfc_stat_data_ctrl sysfs file * lpfc_stat_data_ctrl_show - Read function for lpfc_stat_data_ctrl sysfs file
* @dev: Pointer to class device object. * @dev: Pointer to class device.
* @attr: Unused.
* @buf: Data buffer. * @buf: Data buffer.
* *
* This function is the read call back function for * This function is the read call back function for
...@@ -4364,7 +4374,7 @@ static DEVICE_ATTR_RW(lpfc_stat_data_ctrl); ...@@ -4364,7 +4374,7 @@ static DEVICE_ATTR_RW(lpfc_stat_data_ctrl);
* @filp: sysfs file * @filp: sysfs file
* @kobj: Pointer to the kernel object * @kobj: Pointer to the kernel object
* @bin_attr: Attribute object * @bin_attr: Attribute object
* @buff: Buffer pointer * @buf: Buffer pointer
* @off: File offset * @off: File offset
* @count: Buffer size * @count: Buffer size
* *
...@@ -4451,8 +4461,10 @@ static struct bin_attribute sysfs_drvr_stat_data_attr = { ...@@ -4451,8 +4461,10 @@ static struct bin_attribute sysfs_drvr_stat_data_attr = {
*/ */
/** /**
* lpfc_link_speed_set - Set the adapters link speed * lpfc_link_speed_set - Set the adapters link speed
* @phba: lpfc_hba pointer. * @dev: Pointer to class device.
* @val: link speed value. * @attr: Unused.
* @buf: Data buffer.
* @count: Size of the data buffer.
* *
* Description: * Description:
* If val is in a valid range then set the adapter's link speed field and * If val is in a valid range then set the adapter's link speed field and
...@@ -7111,6 +7123,7 @@ lpfc_set_vport_symbolic_name(struct fc_vport *fc_vport) ...@@ -7111,6 +7123,7 @@ lpfc_set_vport_symbolic_name(struct fc_vport *fc_vport)
/** /**
* lpfc_hba_log_verbose_init - Set hba's log verbose level * lpfc_hba_log_verbose_init - Set hba's log verbose level
* @phba: Pointer to lpfc_hba struct. * @phba: Pointer to lpfc_hba struct.
* @verbose: Verbose level to set.
* *
* This function is called by the lpfc_get_cfgparam() routine to set the * This function is called by the lpfc_get_cfgparam() routine to set the
* module lpfc_log_verbose into the @phba cfg_log_verbose for use with * module lpfc_log_verbose into the @phba cfg_log_verbose for use with
......
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