drm/amdgpu: Drop redundant parameter in amdgpu_gfx_kiq_init_ring
authorMa Jun <Jun.Ma2@amd.com>
Sun, 18 Feb 2024 08:47:51 +0000 (16:47 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 22 Feb 2024 15:17:45 +0000 (10:17 -0500)
Drop redundant parameters in function amdgpu_gfx_kiq_init_ring
to simplify the code

Signed-off-by: Ma Jun <Jun.Ma2@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c

index e114694d1131c76dfcb9d52599e1066ba1284ec2..4835d6d899e79119a6197b277809eea6d8eefe84 100644 (file)
@@ -304,11 +304,11 @@ static int amdgpu_gfx_kiq_acquire(struct amdgpu_device *adev,
        return -EINVAL;
 }
 
-int amdgpu_gfx_kiq_init_ring(struct amdgpu_device *adev,
-                            struct amdgpu_ring *ring,
-                            struct amdgpu_irq_src *irq, int xcc_id)
+int amdgpu_gfx_kiq_init_ring(struct amdgpu_device *adev, int xcc_id)
 {
        struct amdgpu_kiq *kiq = &adev->gfx.kiq[xcc_id];
+       struct amdgpu_irq_src *irq = &kiq->irq;
+       struct amdgpu_ring *ring = &kiq->ring;
        int r = 0;
 
        spin_lock_init(&kiq->ring_lock);
index f23bafec71c5ff5e00c7a0072037179497241324..8fcf889ddce92b86a1a724b56ef2c493f43a326e 100644 (file)
@@ -471,9 +471,7 @@ static inline u32 amdgpu_gfx_create_bitmask(u32 bit_width)
 void amdgpu_gfx_parse_disable_cu(unsigned *mask, unsigned max_se,
                                 unsigned max_sh);
 
-int amdgpu_gfx_kiq_init_ring(struct amdgpu_device *adev,
-                            struct amdgpu_ring *ring,
-                            struct amdgpu_irq_src *irq, int xcc_id);
+int amdgpu_gfx_kiq_init_ring(struct amdgpu_device *adev, int xcc_id);
 
 void amdgpu_gfx_kiq_free_ring(struct amdgpu_ring *ring);
 
index b02d63328f1cd937063cc031f553c4c44e2d4e9c..691fa40e4e0157bcf682c265ca7ec8376a4e6c45 100644 (file)
@@ -4490,7 +4490,7 @@ static int gfx_v10_0_compute_ring_init(struct amdgpu_device *adev, int ring_id,
 static int gfx_v10_0_sw_init(void *handle)
 {
        int i, j, k, r, ring_id = 0;
-       struct amdgpu_kiq *kiq;
+       int xcc_id = 0;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
        switch (amdgpu_ip_version(adev, GC_HWIP, 0)) {
@@ -4619,8 +4619,7 @@ static int gfx_v10_0_sw_init(void *handle)
                        return r;
                }
 
-               kiq = &adev->gfx.kiq[0];
-               r = amdgpu_gfx_kiq_init_ring(adev, &kiq->ring, &kiq->irq, 0);
+               r = amdgpu_gfx_kiq_init_ring(adev, xcc_id);
                if (r)
                        return r;
        }
index 2fb1342d5bd93b3659f31d7ae7207c44d3d76515..9d8ec709cd521e9aac970a7da1aae1dc99237b78 100644 (file)
@@ -1329,7 +1329,7 @@ static int gfx_v11_0_rlc_backdoor_autoload_enable(struct amdgpu_device *adev)
 static int gfx_v11_0_sw_init(void *handle)
 {
        int i, j, k, r, ring_id = 0;
-       struct amdgpu_kiq *kiq;
+       int xcc_id = 0;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
        switch (amdgpu_ip_version(adev, GC_HWIP, 0)) {
@@ -1454,8 +1454,7 @@ static int gfx_v11_0_sw_init(void *handle)
                        return r;
                }
 
-               kiq = &adev->gfx.kiq[0];
-               r = amdgpu_gfx_kiq_init_ring(adev, &kiq->ring, &kiq->irq, 0);
+               r = amdgpu_gfx_kiq_init_ring(adev, xcc_id);
                if (r)
                        return r;
        }
index ea174b76ee7008439ba3a6f03c5076a1bc2b3a77..b97ea62212b6c14f5e74aded473ac1c477759a06 100644 (file)
@@ -1900,8 +1900,8 @@ static void gfx_v8_0_sq_irq_work_func(struct work_struct *work);
 static int gfx_v8_0_sw_init(void *handle)
 {
        int i, j, k, r, ring_id;
+       int xcc_id = 0;
        struct amdgpu_ring *ring;
-       struct amdgpu_kiq *kiq;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
        switch (adev->asic_type) {
@@ -2022,8 +2022,7 @@ static int gfx_v8_0_sw_init(void *handle)
                return r;
        }
 
-       kiq = &adev->gfx.kiq[0];
-       r = amdgpu_gfx_kiq_init_ring(adev, &kiq->ring, &kiq->irq, 0);
+       r = amdgpu_gfx_kiq_init_ring(adev, xcc_id);
        if (r)
                return r;
 
index 169d45268ef6d50bd5b191b91cc4fa4bb086649f..7669f82aa1dac15700eff04c9b393010c7952ea3 100644 (file)
@@ -1997,8 +1997,8 @@ static int gfx_v9_0_compute_ring_init(struct amdgpu_device *adev, int ring_id,
 static int gfx_v9_0_sw_init(void *handle)
 {
        int i, j, k, r, ring_id;
+       int xcc_id = 0;
        struct amdgpu_ring *ring;
-       struct amdgpu_kiq *kiq;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
        unsigned int hw_prio;
 
@@ -2151,8 +2151,7 @@ static int gfx_v9_0_sw_init(void *handle)
                return r;
        }
 
-       kiq = &adev->gfx.kiq[0];
-       r = amdgpu_gfx_kiq_init_ring(adev, &kiq->ring, &kiq->irq, 0);
+       r = amdgpu_gfx_kiq_init_ring(adev, xcc_id);
        if (r)
                return r;
 
index aace4594a603b85450450293036c618f0dc776fc..ec92c3c2080bde9cd7f31f613cbdb6ed8e0e7b3a 100644 (file)
@@ -843,7 +843,6 @@ static int gfx_v9_4_3_compute_ring_init(struct amdgpu_device *adev, int ring_id,
 static int gfx_v9_4_3_sw_init(void *handle)
 {
        int i, j, k, r, ring_id, xcc_id, num_xcc;
-       struct amdgpu_kiq *kiq;
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
 
        adev->gfx.mec.num_mec = 2;
@@ -912,8 +911,7 @@ static int gfx_v9_4_3_sw_init(void *handle)
                        return r;
                }
 
-               kiq = &adev->gfx.kiq[xcc_id];
-               r = amdgpu_gfx_kiq_init_ring(adev, &kiq->ring, &kiq->irq, xcc_id);
+               r = amdgpu_gfx_kiq_init_ring(adev, xcc_id);
                if (r)
                        return r;