> On Oct 18, 2018, at 3:45 PM, Bart Van Assche <bvanass...@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.madh...@cavium.com> > Signed-off-by: Bart Van Assche <bvanass...@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.madh...@cavium.com> Thanks, - Himanshu
