[PATCH] drm/amdgpu: Fix missing parameter descriptions in ih_v7_0.c
Christian König
ckoenig.leichtzumerken at gmail.com
Thu Feb 15 14:49:05 UTC 2024
Am 15.02.24 um 13:38 schrieb Srinivasan Shanmugam:
> Rectifies kdoc warnings related to the 'ih' parameter in the
> 'ih_v7_0_get_wptr', 'ih_v7_0_irq_rearm', and 'ih_v7_0_set_rptr'
> functions within the 'ih_v7_0.c' file.
>
> Fixes the below with gcc W=1:
> drivers/gpu/drm/amd/amdgpu/ih_v7_0.c:392: warning: Function parameter or member 'ih' not described in 'ih_v7_0_get_wptr'
> drivers/gpu/drm/amd/amdgpu/ih_v7_0.c:432: warning: Function parameter or member 'ih' not described in 'ih_v7_0_irq_rearm'
> drivers/gpu/drm/amd/amdgpu/ih_v7_0.c:458: warning: Function parameter or member 'ih' not described in 'ih_v7_0_set_rptr'
>
> Fixes: b6ba7a165b13 ("drm/amdgpu: Add ih v7_0 ip block support")
> Cc: Likun Gao <Likun.Gao at amd.com>
> Cc: Hawking Zhang <Hawking.Zhang at amd.com>
> Cc: Christian König <christian.koenig at amd.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
Reviewed-by: Christian König <christian.koenig at amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/ih_v7_0.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/ih_v7_0.c b/drivers/gpu/drm/amd/amdgpu/ih_v7_0.c
> index 236806797b23..16fe428c0722 100644
> --- a/drivers/gpu/drm/amd/amdgpu/ih_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/ih_v7_0.c
> @@ -378,9 +378,10 @@ static void ih_v7_0_irq_disable(struct amdgpu_device *adev)
> }
>
> /**
> - * ih_v7_0_get_wptr - get the IH ring buffer wptr
> + * ih_v7_0_get_wptr() - get the IH ring buffer wptr
> *
> * @adev: amdgpu_device pointer
> + * @ih: IH ring buffer to fetch wptr
> *
> * Get the IH ring buffer wptr from either the register
> * or the writeback memory buffer. Also check for
> @@ -425,6 +426,7 @@ static u32 ih_v7_0_get_wptr(struct amdgpu_device *adev,
> * ih_v7_0_irq_rearm - rearm IRQ if lost
> *
> * @adev: amdgpu_device pointer
> + * @ih: IH ring to match
> *
> */
> static void ih_v7_0_irq_rearm(struct amdgpu_device *adev,
> @@ -450,8 +452,7 @@ static void ih_v7_0_irq_rearm(struct amdgpu_device *adev,
> * ih_v7_0_set_rptr - set the IH ring buffer rptr
> *
> * @adev: amdgpu_device pointer
> - *
> - * Set the IH ring buffer rptr.
> + * @ih: IH ring buffer to set rptr
> */
> static void ih_v7_0_set_rptr(struct amdgpu_device *adev,
> struct amdgpu_ih_ring *ih)
More information about the amd-gfx
mailing list