diff mbox series

fpga: dfl: more kernel-doc corrections

Message ID 20230114155444.794712-1-yilun.xu@intel.com (mailing list archive)
State New
Headers show
Series fpga: dfl: more kernel-doc corrections | expand

Commit Message

Xu Yilun Jan. 14, 2023, 3:54 p.m. UTC
Fix more kernel-doc warnings in drivers/fpga/:

$ ./scripts/kernel-doc -none drivers/fpga/*
drivers/fpga/dfl-afu.h:61: warning: expecting prototype for struct fpga_afu_dma_region. Prototype was for struct dfl_afu_dma_region instead
drivers/fpga/dfl-afu-region.c:52: warning: Function parameter or member 'pdata' not described in 'afu_mmio_region_add'
drivers/fpga/dfl-fme-perf.c:161: warning: Function parameter or member 'node' not described in 'fme_perf_priv'
drivers/fpga/dfl-fme-pr.h:70: warning: expecting prototype for struct dfl_fme_bridge_pdata. Prototype was for struct dfl_fme_br_pdata instead
drivers/fpga/dfl.h:256: warning: Function parameter or member 'revision' not described in 'dfl_feature'

Signed-off-by: Xu Yilun <yilun.xu@intel.com>
---
 drivers/fpga/dfl-afu-region.c | 1 +
 drivers/fpga/dfl-afu.h        | 2 +-
 drivers/fpga/dfl-fme-perf.c   | 2 +-
 drivers/fpga/dfl-fme-pr.h     | 2 +-
 drivers/fpga/dfl.h            | 1 +
 5 files changed, 5 insertions(+), 3 deletions(-)

Comments

Randy Dunlap Jan. 14, 2023, 5 p.m. UTC | #1
On 1/14/23 07:54, Xu Yilun wrote:
> Fix more kernel-doc warnings in drivers/fpga/:
> 
> $ ./scripts/kernel-doc -none drivers/fpga/*
> drivers/fpga/dfl-afu.h:61: warning: expecting prototype for struct fpga_afu_dma_region. Prototype was for struct dfl_afu_dma_region instead
> drivers/fpga/dfl-afu-region.c:52: warning: Function parameter or member 'pdata' not described in 'afu_mmio_region_add'
> drivers/fpga/dfl-fme-perf.c:161: warning: Function parameter or member 'node' not described in 'fme_perf_priv'
> drivers/fpga/dfl-fme-pr.h:70: warning: expecting prototype for struct dfl_fme_bridge_pdata. Prototype was for struct dfl_fme_br_pdata instead
> drivers/fpga/dfl.h:256: warning: Function parameter or member 'revision' not described in 'dfl_feature'
> 
> Signed-off-by: Xu Yilun <yilun.xu@intel.com>

Reviewed-by: Randy Dunlap <rdunlap@infradead.org>

Thanks.

> ---
>  drivers/fpga/dfl-afu-region.c | 1 +
>  drivers/fpga/dfl-afu.h        | 2 +-
>  drivers/fpga/dfl-fme-perf.c   | 2 +-
>  drivers/fpga/dfl-fme-pr.h     | 2 +-
>  drivers/fpga/dfl.h            | 1 +
>  5 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/fpga/dfl-afu-region.c b/drivers/fpga/dfl-afu-region.c
> index 0804b7a0c298..2e7b41629406 100644
> --- a/drivers/fpga/dfl-afu-region.c
> +++ b/drivers/fpga/dfl-afu-region.c
> @@ -39,6 +39,7 @@ static struct dfl_afu_mmio_region *get_region_by_index(struct dfl_afu *afu,
>  /**
>   * afu_mmio_region_add - add a mmio region to given feature dev.
>   *
> + * @pdata: afu platform device's pdata.
>   * @region_index: region index.
>   * @region_size: region size.
>   * @phys: region's physical address of this region.
> diff --git a/drivers/fpga/dfl-afu.h b/drivers/fpga/dfl-afu.h
> index e5020e2b1f3d..674e9772f0ea 100644
> --- a/drivers/fpga/dfl-afu.h
> +++ b/drivers/fpga/dfl-afu.h
> @@ -41,7 +41,7 @@ struct dfl_afu_mmio_region {
>  };
>  
>  /**
> - * struct fpga_afu_dma_region - afu DMA region data structure
> + * struct dfl_afu_dma_region - afu DMA region data structure
>   *
>   * @user_addr: region userspace virtual address.
>   * @length: region length.
> diff --git a/drivers/fpga/dfl-fme-perf.c b/drivers/fpga/dfl-fme-perf.c
> index 587c82be12f7..7422d2bc6f37 100644
> --- a/drivers/fpga/dfl-fme-perf.c
> +++ b/drivers/fpga/dfl-fme-perf.c
> @@ -141,7 +141,7 @@
>   * @fab_port_id: used to indicate current working mode of fabric counters.
>   * @fab_lock: lock to protect fabric counters working mode.
>   * @cpu: active CPU to which the PMU is bound for accesses.
> - * @cpuhp_node: node for CPU hotplug notifier link.
> + * @node: node for CPU hotplug notifier link.
>   * @cpuhp_state: state for CPU hotplug notification;
>   */
>  struct fme_perf_priv {
> diff --git a/drivers/fpga/dfl-fme-pr.h b/drivers/fpga/dfl-fme-pr.h
> index 096a699089d3..761f80f63312 100644
> --- a/drivers/fpga/dfl-fme-pr.h
> +++ b/drivers/fpga/dfl-fme-pr.h
> @@ -58,7 +58,7 @@ struct dfl_fme_bridge {
>  };
>  
>  /**
> - * struct dfl_fme_bridge_pdata - platform data for FME bridge platform device.
> + * struct dfl_fme_br_pdata - platform data for FME bridge platform device.
>   *
>   * @cdev: container device.
>   * @port_id: port id.
> diff --git a/drivers/fpga/dfl.h b/drivers/fpga/dfl.h
> index 06cfcd5e84bb..5fc424d505c8 100644
> --- a/drivers/fpga/dfl.h
> +++ b/drivers/fpga/dfl.h
> @@ -231,6 +231,7 @@ struct dfl_feature_irq_ctx {
>   *
>   * @dev: ptr to pdev of the feature device which has the sub feature.
>   * @id: sub feature id.
> + * @revision: revision value of this sub feature.
>   * @resource_index: each sub feature has one mmio resource for its registers.
>   *		    this index is used to find its mmio resource from the
>   *		    feature dev (platform device)'s resources.
diff mbox series

Patch

diff --git a/drivers/fpga/dfl-afu-region.c b/drivers/fpga/dfl-afu-region.c
index 0804b7a0c298..2e7b41629406 100644
--- a/drivers/fpga/dfl-afu-region.c
+++ b/drivers/fpga/dfl-afu-region.c
@@ -39,6 +39,7 @@  static struct dfl_afu_mmio_region *get_region_by_index(struct dfl_afu *afu,
 /**
  * afu_mmio_region_add - add a mmio region to given feature dev.
  *
+ * @pdata: afu platform device's pdata.
  * @region_index: region index.
  * @region_size: region size.
  * @phys: region's physical address of this region.
diff --git a/drivers/fpga/dfl-afu.h b/drivers/fpga/dfl-afu.h
index e5020e2b1f3d..674e9772f0ea 100644
--- a/drivers/fpga/dfl-afu.h
+++ b/drivers/fpga/dfl-afu.h
@@ -41,7 +41,7 @@  struct dfl_afu_mmio_region {
 };
 
 /**
- * struct fpga_afu_dma_region - afu DMA region data structure
+ * struct dfl_afu_dma_region - afu DMA region data structure
  *
  * @user_addr: region userspace virtual address.
  * @length: region length.
diff --git a/drivers/fpga/dfl-fme-perf.c b/drivers/fpga/dfl-fme-perf.c
index 587c82be12f7..7422d2bc6f37 100644
--- a/drivers/fpga/dfl-fme-perf.c
+++ b/drivers/fpga/dfl-fme-perf.c
@@ -141,7 +141,7 @@ 
  * @fab_port_id: used to indicate current working mode of fabric counters.
  * @fab_lock: lock to protect fabric counters working mode.
  * @cpu: active CPU to which the PMU is bound for accesses.
- * @cpuhp_node: node for CPU hotplug notifier link.
+ * @node: node for CPU hotplug notifier link.
  * @cpuhp_state: state for CPU hotplug notification;
  */
 struct fme_perf_priv {
diff --git a/drivers/fpga/dfl-fme-pr.h b/drivers/fpga/dfl-fme-pr.h
index 096a699089d3..761f80f63312 100644
--- a/drivers/fpga/dfl-fme-pr.h
+++ b/drivers/fpga/dfl-fme-pr.h
@@ -58,7 +58,7 @@  struct dfl_fme_bridge {
 };
 
 /**
- * struct dfl_fme_bridge_pdata - platform data for FME bridge platform device.
+ * struct dfl_fme_br_pdata - platform data for FME bridge platform device.
  *
  * @cdev: container device.
  * @port_id: port id.
diff --git a/drivers/fpga/dfl.h b/drivers/fpga/dfl.h
index 06cfcd5e84bb..5fc424d505c8 100644
--- a/drivers/fpga/dfl.h
+++ b/drivers/fpga/dfl.h
@@ -231,6 +231,7 @@  struct dfl_feature_irq_ctx {
  *
  * @dev: ptr to pdev of the feature device which has the sub feature.
  * @id: sub feature id.
+ * @revision: revision value of this sub feature.
  * @resource_index: each sub feature has one mmio resource for its registers.
  *		    this index is used to find its mmio resource from the
  *		    feature dev (platform device)'s resources.