2024-01-21 09:26:34

by Randy Dunlap

[permalink] [raw]
Subject: [PATCH] dmaengine: at_hdmac: fix some kernel-doc warnings

Fix some kernel-doc format warnings:

at_hdmac.c:243: warning: Excess struct member 'sg_len' description in 'at_desc'
at_hdmac.c:252: warning: cannot understand function prototype: 'enum atc_status '
ez
at_hdmac.c:351: warning: Excess struct member 'atdma_devtype' description in 'at_dma'
at_hdmac.c:351: warning: Excess struct member 'ch_regs' description in 'at_dma'
at_hdmac.c:664: warning: contents before sections

Signed-off-by: Randy Dunlap <[email protected]>
Cc: Ludovic Desroches <[email protected]>
Cc: Tudor Ambarus <[email protected]>
Cc: [email protected]
Cc: Vinod Koul <[email protected]>
Cc: [email protected]
---
drivers/dma/at_hdmac.c | 21 +++++++++++----------
1 file changed, 11 insertions(+), 10 deletions(-)

diff -- a/drivers/dma/at_hdmac.c b/drivers/dma/at_hdmac.c
--- a/drivers/dma/at_hdmac.c
+++ b/drivers/dma/at_hdmac.c
@@ -222,7 +222,7 @@ struct atdma_sg {
* @vd: pointer to the virtual dma descriptor.
* @atchan: pointer to the atmel dma channel.
* @total_len: total transaction byte count
- * @sg_len: number of sg entries.
+ * @sglen: number of sg entries.
* @sg: array of sgs.
*/
struct at_desc {
@@ -245,7 +245,7 @@ struct at_desc {
/*-- Channels --------------------------------------------------------*/

/**
- * atc_status - information bits stored in channel status flag
+ * enum atc_status - information bits stored in channel status flag
*
* Manipulated with atomic operations.
*/
@@ -328,8 +328,7 @@ static inline u8 convert_buswidth(enum d
/**
* struct at_dma - internal representation of an Atmel HDMA Controller
* @dma_device: dmaengine dma_device object members
- * @atdma_devtype: identifier of DMA controller compatibility
- * @ch_regs: memory mapped register base
+ * @regs: memory mapped register base
* @clk: dma controller clock
* @save_imr: interrupt mask register that is saved on suspend/resume cycle
* @all_chan_mask: all channels availlable in a mask
@@ -626,6 +625,9 @@ static inline u32 atc_calc_bytes_left(u3

/**
* atc_get_llis_residue - Get residue for a hardware linked list transfer
+ * @atchan: pointer to an atmel hdmac channel.
+ * @desc: pointer to the descriptor for which the residue is calculated.
+ * @residue: residue to be set to dma_tx_state.
*
* Calculate the residue by removing the length of the Linked List Item (LLI)
* already transferred from the total length. To get the current LLI we can use
@@ -661,10 +663,8 @@ static inline u32 atc_calc_bytes_left(u3
* two DSCR values are different, we read again the CTRLA then the DSCR till two
* consecutive read values from DSCR are equal or till the maximum trials is
* reach. This algorithm is very unlikely not to find a stable value for DSCR.
- * @atchan: pointer to an atmel hdmac channel.
- * @desc: pointer to the descriptor for which the residue is calculated.
- * @residue: residue to be set to dma_tx_state.
- * Returns 0 on success, -errno otherwise.
+ *
+ * Returns: %0 on success, -errno otherwise.
*/
static int atc_get_llis_residue(struct at_dma_chan *atchan,
struct at_desc *desc, u32 *residue)
@@ -731,7 +731,8 @@ static int atc_get_llis_residue(struct a
* @chan: DMA channel
* @cookie: transaction identifier to check status of
* @residue: residue to be updated.
- * Return 0 on success, -errono otherwise.
+ *
+ * Return: %0 on success, -errno otherwise.
*/
static int atc_get_residue(struct dma_chan *chan, dma_cookie_t cookie,
u32 *residue)
@@ -1710,7 +1711,7 @@ static void atc_issue_pending(struct dma
* atc_alloc_chan_resources - allocate resources for DMA channel
* @chan: allocate descriptor resources for this channel
*
- * return - the number of allocated descriptors
+ * Return: the number of allocated descriptors
*/
static int atc_alloc_chan_resources(struct dma_chan *chan)
{


2024-01-22 17:22:20

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH] dmaengine: at_hdmac: fix some kernel-doc warnings


On Sat, 20 Jan 2024 23:00:21 -0800, Randy Dunlap wrote:
> Fix some kernel-doc format warnings:
>
> at_hdmac.c:243: warning: Excess struct member 'sg_len' description in 'at_desc'
> at_hdmac.c:252: warning: cannot understand function prototype: 'enum atc_status '
> ez
> at_hdmac.c:351: warning: Excess struct member 'atdma_devtype' description in 'at_dma'
> at_hdmac.c:351: warning: Excess struct member 'ch_regs' description in 'at_dma'
> at_hdmac.c:664: warning: contents before sections
>
> [...]

Applied, thanks!

[1/1] dmaengine: at_hdmac: fix some kernel-doc warnings
commit: e4cec073b7755a78030f30cf627141c759035b50

Best regards,
--
~Vinod