diff mbox series

[2/7] qla2xxx: Improve several kernel-doc headers

Message ID 20181018224546.189268-3-bvanassche@acm.org (mailing list archive)
State Accepted
Headers show
Series qla2xxx patches for kernel v4.20 | expand

Commit Message

Bart Van Assche Oct. 18, 2018, 10:45 p.m. UTC
This patch avoids that complaints about kernel-doc headers are reported
when building with W=1.

Cc: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
 drivers/scsi/qla2xxx/qla_iocb.c   | 4 ++--
 drivers/scsi/qla2xxx/qla_isr.c    | 6 +++---
 drivers/scsi/qla2xxx/qla_mbx.c    | 6 +++---
 drivers/scsi/qla2xxx/qla_mr.c     | 6 +++---
 drivers/scsi/qla2xxx/qla_nx.c     | 2 +-
 drivers/scsi/qla2xxx/qla_nx2.c    | 2 +-
 drivers/scsi/qla2xxx/qla_sup.c    | 2 +-
 drivers/scsi/qla2xxx/qla_target.c | 6 +++---
 8 files changed, 17 insertions(+), 17 deletions(-)

Comments

Madhani, Himanshu Oct. 30, 2018, 4:54 a.m. UTC | #1
> On Oct 18, 2018, at 3:45 PM, Bart Van Assche <bvanassche@acm.org> wrote:
> 
> External Email
> 
> This patch avoids that complaints about kernel-doc headers are reported
> when building with W=1.
> 
> Cc: Himanshu Madhani <himanshu.madhani@cavium.com>
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> drivers/scsi/qla2xxx/qla_iocb.c   | 4 ++--
> drivers/scsi/qla2xxx/qla_isr.c    | 6 +++---
> drivers/scsi/qla2xxx/qla_mbx.c    | 6 +++---
> drivers/scsi/qla2xxx/qla_mr.c     | 6 +++---
> drivers/scsi/qla2xxx/qla_nx.c     | 2 +-
> drivers/scsi/qla2xxx/qla_nx2.c    | 2 +-
> drivers/scsi/qla2xxx/qla_sup.c    | 2 +-
> drivers/scsi/qla2xxx/qla_target.c | 6 +++---
> 8 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/scsi/qla2xxx/qla_iocb.c b/drivers/scsi/qla2xxx/qla_iocb.c
> index 86fb8b21aa71..032635321ad6 100644
> --- a/drivers/scsi/qla2xxx/qla_iocb.c
> +++ b/drivers/scsi/qla2xxx/qla_iocb.c
> @@ -1195,8 +1195,8 @@ qla24xx_walk_and_build_prot_sglist(struct qla_hw_data *ha, srb_t *sp,
>  * @sp: SRB command to process
>  * @cmd_pkt: Command type 3 IOCB
>  * @tot_dsds: Total number of segments to transfer
> - * @tot_prot_dsds:
> - * @fw_prot_opts:
> + * @tot_prot_dsds: Total number of segments with protection information
> + * @fw_prot_opts: Protection options to be passed to firmware
>  */
> inline int
> qla24xx_build_scsi_crc_2_iocbs(srb_t *sp, struct cmd_type_crc_2 *cmd_pkt,
> diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c
> index d73b04e40590..30d3090842f8 100644
> --- a/drivers/scsi/qla2xxx/qla_isr.c
> +++ b/drivers/scsi/qla2xxx/qla_isr.c
> @@ -25,7 +25,7 @@ static int qla2x00_error_entry(scsi_qla_host_t *, struct rsp_que *,
> 
> /**
>  * qla2100_intr_handler() - Process interrupts for the ISP2100 and ISP2200.
> - * @irq:
> + * @irq: interrupt number
>  * @dev_id: SCSI driver HA context
>  *
>  * Called by system whenever the host adapter generates an interrupt.
> @@ -144,7 +144,7 @@ qla2x00_check_reg16_for_disconnect(scsi_qla_host_t *vha, uint16_t reg)
> 
> /**
>  * qla2300_intr_handler() - Process interrupts for the ISP23xx and ISP63xx.
> - * @irq:
> + * @irq: interrupt number
>  * @dev_id: SCSI driver HA context
>  *
>  * Called by system whenever the host adapter generates an interrupt.
> @@ -3109,7 +3109,7 @@ qla2xxx_check_risc_status(scsi_qla_host_t *vha)
> 
> /**
>  * qla24xx_intr_handler() - Process interrupts for the ISP23xx and ISP24xx.
> - * @irq:
> + * @irq: interrupt number
>  * @dev_id: SCSI driver HA context
>  *
>  * Called by system whenever the host adapter generates an interrupt.
> diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c
> index bd8c86aeccc2..38ef06d613ba 100644
> --- a/drivers/scsi/qla2xxx/qla_mbx.c
> +++ b/drivers/scsi/qla2xxx/qla_mbx.c
> @@ -3479,9 +3479,9 @@ qla8044_read_serdes_word(scsi_qla_host_t *vha, uint32_t addr, uint32_t *data)
> /**
>  * qla2x00_set_serdes_params() -
>  * @vha: HA context
> - * @sw_em_1g:
> - * @sw_em_2g:
> - * @sw_em_4g:
> + * @sw_em_1g: serial link options
> + * @sw_em_2g: serial link options
> + * @sw_em_4g: serial link options
>  *
>  * Returns
>  */
> diff --git a/drivers/scsi/qla2xxx/qla_mr.c b/drivers/scsi/qla2xxx/qla_mr.c
> index 521a51370554..2d96f3b7e3e3 100644
> --- a/drivers/scsi/qla2xxx/qla_mr.c
> +++ b/drivers/scsi/qla2xxx/qla_mr.c
> @@ -2624,7 +2624,7 @@ qlafx00_status_cont_entry(struct rsp_que *rsp, sts_cont_entry_t *pkt)
>  * qlafx00_multistatus_entry() - Process Multi response queue entries.
>  * @vha: SCSI driver HA context
>  * @rsp: response queue
> - * @pkt:
> + * @pkt: received packet
>  */
> static void
> qlafx00_multistatus_entry(struct scsi_qla_host *vha,
> @@ -2867,7 +2867,7 @@ qlafx00_async_event(scsi_qla_host_t *vha)
> /**
>  * qlafx00x_mbx_completion() - Process mailbox command completions.
>  * @vha: SCSI driver HA context
> - * @mb0:
> + * @mb0: value to be written into mailbox register 0
>  */
> static void
> qlafx00_mbx_completion(scsi_qla_host_t *vha, uint32_t mb0)
> @@ -2893,7 +2893,7 @@ qlafx00_mbx_completion(scsi_qla_host_t *vha, uint32_t mb0)
> 
> /**
>  * qlafx00_intr_handler() - Process interrupts for the ISPFX00.
> - * @irq:
> + * @irq: interrupt number
>  * @dev_id: SCSI driver HA context
>  *
>  * Called by system whenever the host adapter generates an interrupt.
> diff --git a/drivers/scsi/qla2xxx/qla_nx.c b/drivers/scsi/qla2xxx/qla_nx.c
> index de2bc78449e7..8dbe94c52f65 100644
> --- a/drivers/scsi/qla2xxx/qla_nx.c
> +++ b/drivers/scsi/qla2xxx/qla_nx.c
> @@ -2010,7 +2010,7 @@ qla82xx_mbx_completion(scsi_qla_host_t *vha, uint16_t mb0)
> 
> /**
>  * qla82xx_intr_handler() - Process interrupts for the ISP23xx and ISP63xx.
> - * @irq:
> + * @irq: interrupt number
>  * @dev_id: SCSI driver HA context
>  *
>  * Called by system whenever the host adapter generates an interrupt.
> diff --git a/drivers/scsi/qla2xxx/qla_nx2.c b/drivers/scsi/qla2xxx/qla_nx2.c
> index 3a2b0282df14..fe856b602e03 100644
> --- a/drivers/scsi/qla2xxx/qla_nx2.c
> +++ b/drivers/scsi/qla2xxx/qla_nx2.c
> @@ -3878,7 +3878,7 @@ qla8044_write_optrom_data(struct scsi_qla_host *vha, uint8_t *buf,
> #define PF_BITS_MASK           (0xF << 16)
> /**
>  * qla8044_intr_handler() - Process interrupts for the ISP8044
> - * @irq:
> + * @irq: interrupt number
>  * @dev_id: SCSI driver HA context
>  *
>  * Called by system whenever the host adapter generates an interrupt.
> diff --git a/drivers/scsi/qla2xxx/qla_sup.c b/drivers/scsi/qla2xxx/qla_sup.c
> index 4499c787165f..2a3055c799fb 100644
> --- a/drivers/scsi/qla2xxx/qla_sup.c
> +++ b/drivers/scsi/qla2xxx/qla_sup.c
> @@ -2229,7 +2229,7 @@ qla2x00_erase_flash_sector(struct qla_hw_data *ha, uint32_t addr,
> 
> /**
>  * qla2x00_get_flash_manufacturer() - Read manufacturer ID from flash chip.
> - * @ha:
> + * @ha: host adapter
>  * @man_id: Flash manufacturer ID
>  * @flash_id: Flash ID
>  */
> diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
> index 902f9c25da2e..ad7612b80d91 100644
> --- a/drivers/scsi/qla2xxx/qla_target.c
> +++ b/drivers/scsi/qla2xxx/qla_target.c
> @@ -6598,9 +6598,9 @@ static void qlt_lport_dump(struct scsi_qla_host *vha, u64 wwpn,
>  * qla_tgt_lport_register - register lport with external module
>  *
>  * @target_lport_ptr: pointer for tcm_qla2xxx specific lport data
> - * @phys_wwpn:
> - * @npiv_wwpn:
> - * @npiv_wwnn:
> + * @phys_wwpn: physical port WWPN
> + * @npiv_wwpn: NPIV WWPN
> + * @npiv_wwnn: NPIV WWNN
>  * @callback:  lport initialization callback for tcm_qla2xxx code
>  */
> int qlt_lport_register(void *target_lport_ptr, u64 phys_wwpn,
> --
> 2.19.1.568.g152ad8e336-goog
> 

Looks good. 

Acked-by: Himanshu Madhani <himanshu.madhani@cavium.com> 

Thanks,
- Himanshu
diff mbox series

Patch

diff --git a/drivers/scsi/qla2xxx/qla_iocb.c b/drivers/scsi/qla2xxx/qla_iocb.c
index 86fb8b21aa71..032635321ad6 100644
--- a/drivers/scsi/qla2xxx/qla_iocb.c
+++ b/drivers/scsi/qla2xxx/qla_iocb.c
@@ -1195,8 +1195,8 @@  qla24xx_walk_and_build_prot_sglist(struct qla_hw_data *ha, srb_t *sp,
  * @sp: SRB command to process
  * @cmd_pkt: Command type 3 IOCB
  * @tot_dsds: Total number of segments to transfer
- * @tot_prot_dsds:
- * @fw_prot_opts:
+ * @tot_prot_dsds: Total number of segments with protection information
+ * @fw_prot_opts: Protection options to be passed to firmware
  */
 inline int
 qla24xx_build_scsi_crc_2_iocbs(srb_t *sp, struct cmd_type_crc_2 *cmd_pkt,
diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c
index d73b04e40590..30d3090842f8 100644
--- a/drivers/scsi/qla2xxx/qla_isr.c
+++ b/drivers/scsi/qla2xxx/qla_isr.c
@@ -25,7 +25,7 @@  static int qla2x00_error_entry(scsi_qla_host_t *, struct rsp_que *,
 
 /**
  * qla2100_intr_handler() - Process interrupts for the ISP2100 and ISP2200.
- * @irq:
+ * @irq: interrupt number
  * @dev_id: SCSI driver HA context
  *
  * Called by system whenever the host adapter generates an interrupt.
@@ -144,7 +144,7 @@  qla2x00_check_reg16_for_disconnect(scsi_qla_host_t *vha, uint16_t reg)
 
 /**
  * qla2300_intr_handler() - Process interrupts for the ISP23xx and ISP63xx.
- * @irq:
+ * @irq: interrupt number
  * @dev_id: SCSI driver HA context
  *
  * Called by system whenever the host adapter generates an interrupt.
@@ -3109,7 +3109,7 @@  qla2xxx_check_risc_status(scsi_qla_host_t *vha)
 
 /**
  * qla24xx_intr_handler() - Process interrupts for the ISP23xx and ISP24xx.
- * @irq:
+ * @irq: interrupt number
  * @dev_id: SCSI driver HA context
  *
  * Called by system whenever the host adapter generates an interrupt.
diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c
index bd8c86aeccc2..38ef06d613ba 100644
--- a/drivers/scsi/qla2xxx/qla_mbx.c
+++ b/drivers/scsi/qla2xxx/qla_mbx.c
@@ -3479,9 +3479,9 @@  qla8044_read_serdes_word(scsi_qla_host_t *vha, uint32_t addr, uint32_t *data)
 /**
  * qla2x00_set_serdes_params() -
  * @vha: HA context
- * @sw_em_1g:
- * @sw_em_2g:
- * @sw_em_4g:
+ * @sw_em_1g: serial link options
+ * @sw_em_2g: serial link options
+ * @sw_em_4g: serial link options
  *
  * Returns
  */
diff --git a/drivers/scsi/qla2xxx/qla_mr.c b/drivers/scsi/qla2xxx/qla_mr.c
index 521a51370554..2d96f3b7e3e3 100644
--- a/drivers/scsi/qla2xxx/qla_mr.c
+++ b/drivers/scsi/qla2xxx/qla_mr.c
@@ -2624,7 +2624,7 @@  qlafx00_status_cont_entry(struct rsp_que *rsp, sts_cont_entry_t *pkt)
  * qlafx00_multistatus_entry() - Process Multi response queue entries.
  * @vha: SCSI driver HA context
  * @rsp: response queue
- * @pkt:
+ * @pkt: received packet
  */
 static void
 qlafx00_multistatus_entry(struct scsi_qla_host *vha,
@@ -2867,7 +2867,7 @@  qlafx00_async_event(scsi_qla_host_t *vha)
 /**
  * qlafx00x_mbx_completion() - Process mailbox command completions.
  * @vha: SCSI driver HA context
- * @mb0:
+ * @mb0: value to be written into mailbox register 0
  */
 static void
 qlafx00_mbx_completion(scsi_qla_host_t *vha, uint32_t mb0)
@@ -2893,7 +2893,7 @@  qlafx00_mbx_completion(scsi_qla_host_t *vha, uint32_t mb0)
 
 /**
  * qlafx00_intr_handler() - Process interrupts for the ISPFX00.
- * @irq:
+ * @irq: interrupt number
  * @dev_id: SCSI driver HA context
  *
  * Called by system whenever the host adapter generates an interrupt.
diff --git a/drivers/scsi/qla2xxx/qla_nx.c b/drivers/scsi/qla2xxx/qla_nx.c
index de2bc78449e7..8dbe94c52f65 100644
--- a/drivers/scsi/qla2xxx/qla_nx.c
+++ b/drivers/scsi/qla2xxx/qla_nx.c
@@ -2010,7 +2010,7 @@  qla82xx_mbx_completion(scsi_qla_host_t *vha, uint16_t mb0)
 
 /**
  * qla82xx_intr_handler() - Process interrupts for the ISP23xx and ISP63xx.
- * @irq:
+ * @irq: interrupt number
  * @dev_id: SCSI driver HA context
  *
  * Called by system whenever the host adapter generates an interrupt.
diff --git a/drivers/scsi/qla2xxx/qla_nx2.c b/drivers/scsi/qla2xxx/qla_nx2.c
index 3a2b0282df14..fe856b602e03 100644
--- a/drivers/scsi/qla2xxx/qla_nx2.c
+++ b/drivers/scsi/qla2xxx/qla_nx2.c
@@ -3878,7 +3878,7 @@  qla8044_write_optrom_data(struct scsi_qla_host *vha, uint8_t *buf,
 #define PF_BITS_MASK		(0xF << 16)
 /**
  * qla8044_intr_handler() - Process interrupts for the ISP8044
- * @irq:
+ * @irq: interrupt number
  * @dev_id: SCSI driver HA context
  *
  * Called by system whenever the host adapter generates an interrupt.
diff --git a/drivers/scsi/qla2xxx/qla_sup.c b/drivers/scsi/qla2xxx/qla_sup.c
index 4499c787165f..2a3055c799fb 100644
--- a/drivers/scsi/qla2xxx/qla_sup.c
+++ b/drivers/scsi/qla2xxx/qla_sup.c
@@ -2229,7 +2229,7 @@  qla2x00_erase_flash_sector(struct qla_hw_data *ha, uint32_t addr,
 
 /**
  * qla2x00_get_flash_manufacturer() - Read manufacturer ID from flash chip.
- * @ha:
+ * @ha: host adapter
  * @man_id: Flash manufacturer ID
  * @flash_id: Flash ID
  */
diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
index 902f9c25da2e..ad7612b80d91 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -6598,9 +6598,9 @@  static void qlt_lport_dump(struct scsi_qla_host *vha, u64 wwpn,
  * qla_tgt_lport_register - register lport with external module
  *
  * @target_lport_ptr: pointer for tcm_qla2xxx specific lport data
- * @phys_wwpn:
- * @npiv_wwpn:
- * @npiv_wwnn:
+ * @phys_wwpn: physical port WWPN
+ * @npiv_wwpn: NPIV WWPN
+ * @npiv_wwnn: NPIV WWNN
  * @callback:  lport initialization callback for tcm_qla2xxx code
  */
 int qlt_lport_register(void *target_lport_ptr, u64 phys_wwpn,