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@xxxxxxxxx>
Reviewed-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
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.
--
~Randy
