Re: [PATCH 32/32] elx: efct: Tie into kernel Kconfig and build process

Randy Dunlap
2019-10-26T00:39:39Z

Hi James,

On 10/25/19 5:34 PM, kbuild test robot wrote:
> vim +/offset +818 drivers/scsi/elx/libefc_sli/sli4.c
>
> 8994bfd36daa33 James Smart 2019-10-23 795
> 8994bfd36daa33 James Smart 2019-10-23 796 /**
> 8994bfd36daa33 James Smart 2019-10-23 797 * @ingroup sli_fc
> 8994bfd36daa33 James Smart 2019-10-23 798 * @brief Write an RQ_CREATE_V2 command.
> 8994bfd36daa33 James Smart 2019-10-23 799 *
> 8994bfd36daa33 James Smart 2019-10-23 800 * @param sli4 SLI context.
> 8994bfd36daa33 James Smart 2019-10-23 801 * @param buf Destination buffer for the command.
> 8994bfd36daa33 James Smart 2019-10-23 802 * @param size Buffer size, in bytes.
> 8994bfd36daa33 James Smart 2019-10-23 803 * @param qmem DMA memory for the queue.
> 8994bfd36daa33 James Smart 2019-10-23 804 * @param cq_id Associated CQ_ID.
> 8994bfd36daa33 James Smart 2019-10-23 805 * @param buffer_size Buffer size pointed to by each RQE.
> 8994bfd36daa33 James Smart 2019-10-23 806 *
> 8994bfd36daa33 James Smart 2019-10-23 807 * @note This creates a Version 0 message
> 8994bfd36daa33 James Smart 2019-10-23 808 *
> 8994bfd36daa33 James Smart 2019-10-23 809 * @return Returns zero for success and non-zero for failure.
> 8994bfd36daa33 James Smart 2019-10-23 810 */

BTW, what is that notation? It's not kernel-doc. Please use kernel-doc notation
for documentation in the kernel.

> 8994bfd36daa33 James Smart 2019-10-23 811 static int
> 8994bfd36daa33 James Smart 2019-10-23 812 sli_cmd_rq_create_v2(struct sli4_s *sli4, u32 num_rqs,
> 8994bfd36daa33 James Smart 2019-10-23 813 struct sli4_queue_s *qs[], u32 base_cq_id,
> 8994bfd36daa33 James Smart 2019-10-23 814 u32 header_buffer_size,
> 8994bfd36daa33 James Smart 2019-10-23 815 u32 payload_buffer_size, struct efc_dma_s *dma)
> 8994bfd36daa33 James Smart 2019-10-23 816 {

thanks.
--
~Randy