linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] mmc: dw_mmc: fix warning reported by kernel-doc
@ 2016-03-09  2:33 Shawn Lin
  2016-03-29  1:18 ` Jaehoon Chung
  0 siblings, 1 reply; 2+ messages in thread
From: Shawn Lin @ 2016-03-09  2:33 UTC (permalink / raw)
  To: Jaehoon Chung, Ulf Hansson; +Cc: linux-mmc, linux-kernel, Shawn Lin

Try to fix the warning reported by:
scripts/kernel-doc -man -v include/linux/mmc/dw_mmc.h > /dev/null

warning: No description found for parameter 'irq_lock'
warning: No description found for parameter 'stop_abort'
warning: No description found for parameter 'prev_blksz'
warning: No description found for parameter 'timing'
warning: No description found for parameter 'ring_size'
warning: No description found for parameter 'dms'
warning: No description found for parameter 'phy_regs'
warning: No description found for parameter 'fifoth_val'
warning: No description found for parameter 'vqmmc_enabled'
warning: No description found for parameter 'cmd11_timer'
warning: Excess struct/union/enum/typedef member 'card_tasklet'
description in 'dw_mci'

Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
---

 include/linux/mmc/dw_mmc.h | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/include/linux/mmc/dw_mmc.h b/include/linux/mmc/dw_mmc.h
index 7b41c6d..c8b6a4d 100644
--- a/include/linux/mmc/dw_mmc.h
+++ b/include/linux/mmc/dw_mmc.h
@@ -55,6 +55,7 @@ struct dw_mci_dma_slave {
 /**
  * struct dw_mci - MMC controller state shared between all slots
  * @lock: Spinlock protecting the queue and associated data.
+ * @irq_lock: Spinlock protecting the INTMASK setting.
  * @regs: Pointer to MMIO registers.
  * @fifo_reg: Pointer to MMIO registers for data FIFO
  * @sg: Scatterlist entry currently being processed by PIO code, if any.
@@ -65,6 +66,9 @@ struct dw_mci_dma_slave {
  * @cmd: The command currently being sent to the card, or NULL.
  * @data: The data currently being transferred, or NULL if no data
  *	transfer is in progress.
+ * @stop_abort: The command currently prepared for stoping transfer.
+ * @prev_blksz: The former transfer blksz record.
+ * @timing: Record of current ios timing.
  * @use_dma: Whether DMA channel is initialized or not.
  * @using_dma: Whether DMA is in use for the current transfer.
  * @dma_64bit_address: Whether DMA supports 64-bit address mode or not.
@@ -72,7 +76,10 @@ struct dw_mci_dma_slave {
  * @sg_cpu: Virtual address of DMA buffer.
  * @dma_ops: Pointer to platform-specific DMA callbacks.
  * @cmd_status: Snapshot of SR taken upon completion of the current
+ * @ring_size: Buffer size for idma descriptors.
  *	command. Only valid when EVENT_CMD_COMPLETE is pending.
+ * @dms: structure of slave-dma private data.
+ * @phy_regs: physical address of controller's register map
  * @data_status: Snapshot of SR taken upon completion of the current
  *	data transfer. Only valid when EVENT_DATA_COMPLETE or
  *	EVENT_DATA_ERROR is pending.
@@ -80,7 +87,6 @@ struct dw_mci_dma_slave {
  *	to be sent.
  * @dir_status: Direction of current transfer.
  * @tasklet: Tasklet running the request state machine.
- * @card_tasklet: Tasklet handling card detect.
  * @pending_events: Bitmask of events flagged by the interrupt handler
  *	to be processed by the tasklet.
  * @completed_events: Bitmask of events which the state machine has
@@ -91,6 +97,7 @@ struct dw_mci_dma_slave {
  *	rate and timeout calculations.
  * @current_speed: Configured rate of the controller.
  * @num_slots: Number of slots available.
+ * @fifoth_val: The value of FIFOTH register.
  * @verid: Denote Version ID.
  * @dev: Device associated with the MMC controller.
  * @pdata: Platform data associated with the MMC controller.
@@ -107,9 +114,11 @@ struct dw_mci_dma_slave {
  * @push_data: Pointer to FIFO push function.
  * @pull_data: Pointer to FIFO pull function.
  * @quirks: Set of quirks that apply to specific versions of the IP.
+ * @vqmmc_enabled: Status of vqmmc, should be true or false.
  * @irq_flags: The flags to be passed to request_irq.
  * @irq: The irq value to be passed to request_irq.
  * @sdio_id0: Number of slot0 in the SDIO interrupt registers.
+ * @cmd11_timer: Timer for SD3.0 voltage switch over scheme.
  * @dto_timer: Timer for broken data transfer over scheme.
  *
  * Locking
-- 
2.3.7

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/2] mmc: dw_mmc: fix warning reported by kernel-doc
  2016-03-09  2:33 [PATCH 1/2] mmc: dw_mmc: fix warning reported by kernel-doc Shawn Lin
@ 2016-03-29  1:18 ` Jaehoon Chung
  0 siblings, 0 replies; 2+ messages in thread
From: Jaehoon Chung @ 2016-03-29  1:18 UTC (permalink / raw)
  To: Shawn Lin, Ulf Hansson; +Cc: linux-mmc, linux-kernel

Hi Shawn,

Sorry for applying late.
Applied on my repository.

Best Regards,
Jaehoon Chung

On 03/09/2016 11:33 AM, Shawn Lin wrote:
> Try to fix the warning reported by:
> scripts/kernel-doc -man -v include/linux/mmc/dw_mmc.h > /dev/null
> 
> warning: No description found for parameter 'irq_lock'
> warning: No description found for parameter 'stop_abort'
> warning: No description found for parameter 'prev_blksz'
> warning: No description found for parameter 'timing'
> warning: No description found for parameter 'ring_size'
> warning: No description found for parameter 'dms'
> warning: No description found for parameter 'phy_regs'
> warning: No description found for parameter 'fifoth_val'
> warning: No description found for parameter 'vqmmc_enabled'
> warning: No description found for parameter 'cmd11_timer'
> warning: Excess struct/union/enum/typedef member 'card_tasklet'
> description in 'dw_mci'
> 
> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> ---
> 
>  include/linux/mmc/dw_mmc.h | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/include/linux/mmc/dw_mmc.h b/include/linux/mmc/dw_mmc.h
> index 7b41c6d..c8b6a4d 100644
> --- a/include/linux/mmc/dw_mmc.h
> +++ b/include/linux/mmc/dw_mmc.h
> @@ -55,6 +55,7 @@ struct dw_mci_dma_slave {
>  /**
>   * struct dw_mci - MMC controller state shared between all slots
>   * @lock: Spinlock protecting the queue and associated data.
> + * @irq_lock: Spinlock protecting the INTMASK setting.
>   * @regs: Pointer to MMIO registers.
>   * @fifo_reg: Pointer to MMIO registers for data FIFO
>   * @sg: Scatterlist entry currently being processed by PIO code, if any.
> @@ -65,6 +66,9 @@ struct dw_mci_dma_slave {
>   * @cmd: The command currently being sent to the card, or NULL.
>   * @data: The data currently being transferred, or NULL if no data
>   *	transfer is in progress.
> + * @stop_abort: The command currently prepared for stoping transfer.
> + * @prev_blksz: The former transfer blksz record.
> + * @timing: Record of current ios timing.
>   * @use_dma: Whether DMA channel is initialized or not.
>   * @using_dma: Whether DMA is in use for the current transfer.
>   * @dma_64bit_address: Whether DMA supports 64-bit address mode or not.
> @@ -72,7 +76,10 @@ struct dw_mci_dma_slave {
>   * @sg_cpu: Virtual address of DMA buffer.
>   * @dma_ops: Pointer to platform-specific DMA callbacks.
>   * @cmd_status: Snapshot of SR taken upon completion of the current
> + * @ring_size: Buffer size for idma descriptors.
>   *	command. Only valid when EVENT_CMD_COMPLETE is pending.
> + * @dms: structure of slave-dma private data.
> + * @phy_regs: physical address of controller's register map
>   * @data_status: Snapshot of SR taken upon completion of the current
>   *	data transfer. Only valid when EVENT_DATA_COMPLETE or
>   *	EVENT_DATA_ERROR is pending.
> @@ -80,7 +87,6 @@ struct dw_mci_dma_slave {
>   *	to be sent.
>   * @dir_status: Direction of current transfer.
>   * @tasklet: Tasklet running the request state machine.
> - * @card_tasklet: Tasklet handling card detect.
>   * @pending_events: Bitmask of events flagged by the interrupt handler
>   *	to be processed by the tasklet.
>   * @completed_events: Bitmask of events which the state machine has
> @@ -91,6 +97,7 @@ struct dw_mci_dma_slave {
>   *	rate and timeout calculations.
>   * @current_speed: Configured rate of the controller.
>   * @num_slots: Number of slots available.
> + * @fifoth_val: The value of FIFOTH register.
>   * @verid: Denote Version ID.
>   * @dev: Device associated with the MMC controller.
>   * @pdata: Platform data associated with the MMC controller.
> @@ -107,9 +114,11 @@ struct dw_mci_dma_slave {
>   * @push_data: Pointer to FIFO push function.
>   * @pull_data: Pointer to FIFO pull function.
>   * @quirks: Set of quirks that apply to specific versions of the IP.
> + * @vqmmc_enabled: Status of vqmmc, should be true or false.
>   * @irq_flags: The flags to be passed to request_irq.
>   * @irq: The irq value to be passed to request_irq.
>   * @sdio_id0: Number of slot0 in the SDIO interrupt registers.
> + * @cmd11_timer: Timer for SD3.0 voltage switch over scheme.
>   * @dto_timer: Timer for broken data transfer over scheme.
>   *
>   * Locking
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-29  1:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-09  2:33 [PATCH 1/2] mmc: dw_mmc: fix warning reported by kernel-doc Shawn Lin
2016-03-29  1:18 ` Jaehoon Chung

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).