diff mbox series

[03/13] ti-vpe/cal.h: fix kernel-doc formatting

Message ID 20210311102022.96954-4-hverkuil-cisco@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series media platform: fix kernel-doc formatting | expand

Commit Message

Hans Verkuil March 11, 2021, 10:20 a.m. UTC
Fix inline struct member documentation, was missing the
'@memb:' prefix.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Cc: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
---
 drivers/media/platform/ti-vpe/cal.h | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

Comments

Tomi Valkeinen March 16, 2021, 3:03 p.m. UTC | #1
On 11/03/2021 12:20, Hans Verkuil wrote:
> Fix inline struct member documentation, was missing the
> '@memb:' prefix.
> 
> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
> Cc: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
> ---
>   drivers/media/platform/ti-vpe/cal.h | 19 +++++++++----------
>   1 file changed, 9 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/media/platform/ti-vpe/cal.h b/drivers/media/platform/ti-vpe/cal.h
> index d471b7f82519..7aaea82c6b42 100644
> --- a/drivers/media/platform/ti-vpe/cal.h
> +++ b/drivers/media/platform/ti-vpe/cal.h
> @@ -84,35 +84,34 @@ struct cal_buffer {
>   
>   /**
>    * struct cal_dmaqueue - Queue of DMA buffers
> - * @active: Buffer being DMA'ed to for the current frame
>    */
>   struct cal_dmaqueue {
>   	/**
> -	 * Protects all fields in the cal_dmaqueue.
> +	 * @lock: Protects all fields in the cal_dmaqueue.
>   	 */
>   	spinlock_t		lock;
>   
>   	/**
> -	 * Buffers queued to the driver and waiting for DMA processing.
> +	 * @queue: Buffers queued to the driver and waiting for DMA processing.
>   	 * Buffers are added to the list by the vb2 .buffer_queue() operation,
>   	 * and move to @pending when they are scheduled for the next frame.
>   	 */
>   	struct list_head	queue;
>   	/**
> -	 * Buffer provided to the hardware to DMA the next frame. Will move to
> -	 * @active at the end of the current frame.
> +	 * @pending: Buffer provided to the hardware to DMA the next frame.
> +	 * Will move to @active at the end of the current frame.
>   	 */
>   	struct cal_buffer	*pending;
>   	/**
> -	 * Buffer being DMA'ed to for the current frame. Will be retired and
> -	 * given back to vb2 at the end of the current frame if a @pending
> -	 * buffer has been scheduled to replace it.
> +	 * @active: Buffer being DMA'ed to for the current frame. Will be
> +	 * retired and given back to vb2 at the end of the current frame if
> +	 * a @pending buffer has been scheduled to replace it.
>   	 */
>   	struct cal_buffer	*active;
>   
> -	/** State of the DMA engine. */
> +	/** @state: State of the DMA engine. */
>   	enum cal_dma_state	state;
> -	/** Wait queue to signal a @state transition to CAL_DMA_STOPPED. */
> +	/** @wait: Wait queue to signal a @state transition to CAL_DMA_STOPPED. */
>   	struct wait_queue_head	wait;
>   };
>   
> 

Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>

  Tomi
diff mbox series

Patch

diff --git a/drivers/media/platform/ti-vpe/cal.h b/drivers/media/platform/ti-vpe/cal.h
index d471b7f82519..7aaea82c6b42 100644
--- a/drivers/media/platform/ti-vpe/cal.h
+++ b/drivers/media/platform/ti-vpe/cal.h
@@ -84,35 +84,34 @@  struct cal_buffer {
 
 /**
  * struct cal_dmaqueue - Queue of DMA buffers
- * @active: Buffer being DMA'ed to for the current frame
  */
 struct cal_dmaqueue {
 	/**
-	 * Protects all fields in the cal_dmaqueue.
+	 * @lock: Protects all fields in the cal_dmaqueue.
 	 */
 	spinlock_t		lock;
 
 	/**
-	 * Buffers queued to the driver and waiting for DMA processing.
+	 * @queue: Buffers queued to the driver and waiting for DMA processing.
 	 * Buffers are added to the list by the vb2 .buffer_queue() operation,
 	 * and move to @pending when they are scheduled for the next frame.
 	 */
 	struct list_head	queue;
 	/**
-	 * Buffer provided to the hardware to DMA the next frame. Will move to
-	 * @active at the end of the current frame.
+	 * @pending: Buffer provided to the hardware to DMA the next frame.
+	 * Will move to @active at the end of the current frame.
 	 */
 	struct cal_buffer	*pending;
 	/**
-	 * Buffer being DMA'ed to for the current frame. Will be retired and
-	 * given back to vb2 at the end of the current frame if a @pending
-	 * buffer has been scheduled to replace it.
+	 * @active: Buffer being DMA'ed to for the current frame. Will be
+	 * retired and given back to vb2 at the end of the current frame if
+	 * a @pending buffer has been scheduled to replace it.
 	 */
 	struct cal_buffer	*active;
 
-	/** State of the DMA engine. */
+	/** @state: State of the DMA engine. */
 	enum cal_dma_state	state;
-	/** Wait queue to signal a @state transition to CAL_DMA_STOPPED. */
+	/** @wait: Wait queue to signal a @state transition to CAL_DMA_STOPPED. */
 	struct wait_queue_head	wait;
 };