Hi Vinod,
Thank you for the patch.
On Sat, Jul 18, 2020 at 07:22:01PM +0530, Vinod Koul wrote:
> xilinx_dpdma_chan structure documents 'desc' members, but that leads
> to warnings, so split that up and describe members
>
> drivers/dma/xilinx/xilinx_dpdma.c:241: warning: Function parameter or
> member 'desc' not described in 'xilinx_dpdma_chan'
>
> Signed-off-by: Vinod Koul <vkoul@xxxxxxxxxx>
> ---
> drivers/dma/xilinx/xilinx_dpdma.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/dma/xilinx/xilinx_dpdma.c b/drivers/dma/xilinx/xilinx_dpdma.c
> index 430f3714f6a3..d94c75a842f8 100644
> --- a/drivers/dma/xilinx/xilinx_dpdma.c
> +++ b/drivers/dma/xilinx/xilinx_dpdma.c
> @@ -214,8 +214,8 @@ struct xilinx_dpdma_tx_desc {
> * @lock: lock to access struct xilinx_dpdma_chan
> * @desc_pool: descriptor allocation pool
> * @err_task: error IRQ bottom half handler
> - * @desc.pending: Descriptor schedule to the hardware, pending execution
> - * @desc.active: Descriptor being executed by the hardware
> + * @desc: pending: Descriptor schedule to the hardware, pending execution
> + * active: Descriptor being executed by the hardware
According to
https://www.kernel.org/doc/html/latest/doc-guide/kernel-doc.html#nested-structs-unions,
the existing syntax is supposed to be valid. Where does the above
warning come from ?
> * @xdev: DPDMA device
> */
> struct xilinx_dpdma_chan {
--
Regards,
Laurent Pinchart
