diff mbox series

async_tx: Fix some kernel-doc warnings

Message ID 20221111064441.2926-1-liubo03@inspur.com
State New
Headers show
Series async_tx: Fix some kernel-doc warnings | expand

Commit Message

Bo Liu Nov. 11, 2022, 6:44 a.m. UTC
Fixes the following W=1 kernel build warning(s):
  crypto/async_tx/async_tx.c:136: warning: cannot understand function prototype: 'enum submit_disposition '
  crypto/async_tx/async_tx.c:264: warning: Function parameter or member 'tx' not described in 'async_tx_quiesce'

Signed-off-by: Bo Liu <liubo03@inspur.com>
---
 crypto/async_tx/async_tx.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Randy Dunlap Nov. 11, 2022, 9:05 p.m. UTC | #1
Hi--

On 11/10/22 22:44, Bo Liu wrote:
> Fixes the following W=1 kernel build warning(s):
>   crypto/async_tx/async_tx.c:136: warning: cannot understand function prototype: 'enum submit_disposition '
>   crypto/async_tx/async_tx.c:264: warning: Function parameter or member 'tx' not described in 'async_tx_quiesce'
> 
> Signed-off-by: Bo Liu <liubo03@inspur.com>
> ---
>  crypto/async_tx/async_tx.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/crypto/async_tx/async_tx.c b/crypto/async_tx/async_tx.c
> index 9256934312d7..8840731cefbb 100644
> --- a/crypto/async_tx/async_tx.c
> +++ b/crypto/async_tx/async_tx.c
> @@ -33,7 +33,7 @@ static void __exit async_tx_exit(void)
>  module_init(async_tx_init);
>  module_exit(async_tx_exit);
>  
> -/**
> +/*

This change is not described and is not needed.

>   * __async_tx_find_channel - find a channel to carry out the operation or let
>   *	the transaction execute synchronously
>   * @submit: transaction dependency and submission modifiers
> @@ -55,7 +55,7 @@ EXPORT_SYMBOL_GPL(__async_tx_find_channel);
>  #endif
>  
>  
> -/**
> +/*
>   * async_tx_channel_switch - queue an interrupt descriptor with a dependency
>   * 	pre-attached.

Same as above.

>   * @depend_tx: the operation that must finish before the new operation runs
> @@ -123,7 +123,7 @@ async_tx_channel_switch(struct dma_async_tx_descriptor *depend_tx,
>  }
>  
>  
> -/**
> +/*
>   * submit_disposition - flags for routing an incoming operation

Just change to
 * enum submit_disposition - flags for routing an incoming operation

i.e. insert 'enum.


>   * @ASYNC_TX_SUBMITTED: we were able to append the new operation under the lock
>   * @ASYNC_TX_CHANNEL_SWITCH: when the lock is dropped schedule a channel switch
> @@ -209,7 +209,7 @@ async_tx_submit(struct dma_chan *chan, struct dma_async_tx_descriptor *tx,
>  }
>  EXPORT_SYMBOL_GPL(async_tx_submit);
>  
> -/**
> +/*

Not described, not needed.

>   * async_trigger_callback - schedules the callback function to be run
>   * @submit: submission and completion parameters
>   *
> @@ -256,9 +256,9 @@ async_trigger_callback(struct async_submit_ctl *submit)
>  }
>  EXPORT_SYMBOL_GPL(async_trigger_callback);
>  
> -/**
> +/*
>   * async_tx_quiesce - ensure tx is complete and freeable upon return
> - * @tx - transaction to quiesce
> + * @*tx - transaction to quiesce

Just change to
 * @tx: transaction to quiesce

i.e. s/ -/:/

>   */
>  void async_tx_quiesce(struct dma_async_tx_descriptor **tx)
>  {
diff mbox series

Patch

diff --git a/crypto/async_tx/async_tx.c b/crypto/async_tx/async_tx.c
index 9256934312d7..8840731cefbb 100644
--- a/crypto/async_tx/async_tx.c
+++ b/crypto/async_tx/async_tx.c
@@ -33,7 +33,7 @@  static void __exit async_tx_exit(void)
 module_init(async_tx_init);
 module_exit(async_tx_exit);
 
-/**
+/*
  * __async_tx_find_channel - find a channel to carry out the operation or let
  *	the transaction execute synchronously
  * @submit: transaction dependency and submission modifiers
@@ -55,7 +55,7 @@  EXPORT_SYMBOL_GPL(__async_tx_find_channel);
 #endif
 
 
-/**
+/*
  * async_tx_channel_switch - queue an interrupt descriptor with a dependency
  * 	pre-attached.
  * @depend_tx: the operation that must finish before the new operation runs
@@ -123,7 +123,7 @@  async_tx_channel_switch(struct dma_async_tx_descriptor *depend_tx,
 }
 
 
-/**
+/*
  * submit_disposition - flags for routing an incoming operation
  * @ASYNC_TX_SUBMITTED: we were able to append the new operation under the lock
  * @ASYNC_TX_CHANNEL_SWITCH: when the lock is dropped schedule a channel switch
@@ -209,7 +209,7 @@  async_tx_submit(struct dma_chan *chan, struct dma_async_tx_descriptor *tx,
 }
 EXPORT_SYMBOL_GPL(async_tx_submit);
 
-/**
+/*
  * async_trigger_callback - schedules the callback function to be run
  * @submit: submission and completion parameters
  *
@@ -256,9 +256,9 @@  async_trigger_callback(struct async_submit_ctl *submit)
 }
 EXPORT_SYMBOL_GPL(async_trigger_callback);
 
-/**
+/*
  * async_tx_quiesce - ensure tx is complete and freeable upon return
- * @tx - transaction to quiesce
+ * @*tx - transaction to quiesce
  */
 void async_tx_quiesce(struct dma_async_tx_descriptor **tx)
 {