On 2024-11-10 21:46, Srinivasan Shanmugam wrote:
This patch adds the missing parameter descriptors to the functions
vcn_v4_0_3 _start, _stop, _set_unified_ring_funcs, and _set_irq_funcs.

Fixes the below with gcc W=1:
drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c:1104: warning: Function parameter or 
struct member 'inst' not described in 'vcn_v4_0_3_start'
drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c:1310: warning: Function parameter or 
struct member 'inst' not described in 'vcn_v4_0_3_stop'
drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c:1545: warning: Function parameter or 
struct member 'inst' not described in 'vcn_v4_0_3_set_unified_ring_funcs'
drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c:1732: warning: Function parameter or 
struct member 'inst' not described in 'vcn_v4_0_3_set_irq_funcs'

Cc: Christian König<[email protected]>
Cc: Alex Deucher<[email protected]>
Signed-off-by: Srinivasan Shanmugam<[email protected]>


Reviewed-by: Boyuan Zhang <[email protected]> <mailto:[email protected]>


---
  drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c | 4 ++++
  1 file changed, 4 insertions(+)

diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c 
b/drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c
index aa06b2fdeb7a..af351f12c17b 100644
--- a/drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c
+++ b/drivers/gpu/drm/amd/amdgpu/vcn_v4_0_3.c
@@ -1097,6 +1097,7 @@ static int vcn_v4_0_3_start_sriov(struct amdgpu_device 
*adev)
   * vcn_v4_0_3_start - VCN start
   *
   * @adev: amdgpu_device pointer
+ * @inst: VCN instance index to be started
   *
   * Start VCN block
   */
@@ -1303,6 +1304,7 @@ static int vcn_v4_0_3_stop_dpg_mode(struct amdgpu_device 
*adev, int inst_idx)
   * vcn_v4_0_3_stop - VCN stop
   *
   * @adev: amdgpu_device pointer
+ * @inst: VCN instance index to be stopped
   *
   * Stop VCN block
   */
@@ -1538,6 +1540,7 @@ static const struct amdgpu_ring_funcs 
vcn_v4_0_3_unified_ring_vm_funcs = {
   * vcn_v4_0_3_set_unified_ring_funcs - set unified ring functions
   *
   * @adev: amdgpu_device pointer
+ * @inst: instance of the VCN block for which to set the ring functions
   *
   * Set unified ring functions
   */
@@ -1725,6 +1728,7 @@ static const struct amdgpu_irq_src_funcs 
vcn_v4_0_3_irq_funcs = {
   * vcn_v4_0_3_set_irq_funcs - set VCN block interrupt irq functions
   *
   * @adev: amdgpu_device pointer
+ * @inst: instance of the VCN block for which to set the IRQ functions
   *
   * Set VCN block interrupt irq functions
   */

Reply via email to