]> Git Repo - linux.git/commitdiff
drm/amdgpu: drop the amdgpu_device argument from amdgpu_ib_free
authorPierre-Eric Pelloux-Prayer <[email protected]>
Fri, 6 Dec 2024 12:16:14 +0000 (13:16 +0100)
committerAlex Deucher <[email protected]>
Wed, 18 Dec 2024 17:17:32 +0000 (12:17 -0500)
It's unused.

Signed-off-by: Pierre-Eric Pelloux-Prayer <[email protected]>
Reviewed-by: Alex Deucher <[email protected]>
Reviewed-by: Christian König <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
27 files changed:
drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
drivers/gpu/drm/amd/amdgpu/amdgpu_object.h
drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h
drivers/gpu/drm/amd/amdgpu/amdgpu_sa.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c
drivers/gpu/drm/amd/amdgpu/cik_sdma.c
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v11_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v12_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
drivers/gpu/drm/amd/amdgpu/gfx_v7_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_2.c
drivers/gpu/drm/amd/amdgpu/gfx_v9_4_3.c
drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
drivers/gpu/drm/amd/amdgpu/sdma_v4_4_2.c
drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c
drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
drivers/gpu/drm/amd/amdgpu/sdma_v6_0.c
drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c
drivers/gpu/drm/amd/amdgpu/si_dma.c

index 8b512dc28df8384861be0385e8cbdb292406421e..d751995dc13126d478762efc8552de2b350e8ead 100644 (file)
@@ -89,16 +89,14 @@ int amdgpu_ib_get(struct amdgpu_device *adev, struct amdgpu_vm *vm,
 /**
  * amdgpu_ib_free - free an IB (Indirect Buffer)
  *
- * @adev: amdgpu_device pointer
  * @ib: IB object to free
  * @f: the fence SA bo need wait on for the ib alloation
  *
  * Free an IB (all asics).
  */
-void amdgpu_ib_free(struct amdgpu_device *adev, struct amdgpu_ib *ib,
-                   struct dma_fence *f)
+void amdgpu_ib_free(struct amdgpu_ib *ib, struct dma_fence *f)
 {
-       amdgpu_sa_bo_free(adev, &ib->sa_bo, f);
+       amdgpu_sa_bo_free(&ib->sa_bo, f);
 }
 
 /**
index a21c510c408ee2cf9f30eec4fff3ca66112614ad..77d3ca857cb9b4e5976259d3f6c51a7cc575e0e9 100644 (file)
@@ -267,7 +267,7 @@ void amdgpu_job_free_resources(struct amdgpu_job *job)
                f = NULL;
 
        for (i = 0; i < job->num_ibs; ++i)
-               amdgpu_ib_free(NULL, &job->ibs[i], f);
+               amdgpu_ib_free(&job->ibs[i], f);
 }
 
 static void amdgpu_job_free_cb(struct drm_sched_job *s_job)
index ce3314152d20f838097193742458ad2cc752406a..ab3fe7b42da7a390923148c755bfb5e29879851a 100644 (file)
@@ -342,8 +342,7 @@ int amdgpu_sa_bo_manager_start(struct amdgpu_device *adev,
 int amdgpu_sa_bo_new(struct amdgpu_sa_manager *sa_manager,
                     struct drm_suballoc **sa_bo,
                     unsigned int size);
-void amdgpu_sa_bo_free(struct amdgpu_device *adev,
-                      struct drm_suballoc **sa_bo,
+void amdgpu_sa_bo_free(struct drm_suballoc **sa_bo,
                       struct dma_fence *fence);
 #if defined(CONFIG_DEBUG_FS)
 void amdgpu_sa_bo_dump_debug_info(struct amdgpu_sa_manager *sa_manager,
index 36fc9578c53c03433226bd36f6c0d9af20777bd5..dee5a1b4e572136a8afd973f569e8c2bbd90da65 100644 (file)
@@ -462,8 +462,7 @@ int amdgpu_ib_get(struct amdgpu_device *adev, struct amdgpu_vm *vm,
                  unsigned size,
                  enum amdgpu_ib_pool_type pool,
                  struct amdgpu_ib *ib);
-void amdgpu_ib_free(struct amdgpu_device *adev, struct amdgpu_ib *ib,
-                   struct dma_fence *f);
+void amdgpu_ib_free(struct amdgpu_ib *ib, struct dma_fence *f);
 int amdgpu_ib_schedule(struct amdgpu_ring *ring, unsigned num_ibs,
                       struct amdgpu_ib *ibs, struct amdgpu_job *job,
                       struct dma_fence **f);
index 10df731998b22f873611092942008560ec67394d..39070b2a4c04fdda0ea29c4329192d453815908f 100644 (file)
@@ -93,8 +93,7 @@ int amdgpu_sa_bo_new(struct amdgpu_sa_manager *sa_manager,
        return 0;
 }
 
-void amdgpu_sa_bo_free(struct amdgpu_device *adev, struct drm_suballoc **sa_bo,
-                      struct dma_fence *fence)
+void amdgpu_sa_bo_free(struct drm_suballoc **sa_bo, struct dma_fence *fence)
 {
        if (sa_bo == NULL || *sa_bo == NULL) {
                return;
index 65387f6943b4c51b65bae3f5f7dcc98d77314910..b9060bcd48064d659bb930c57ed3730b4328b2f0 100644 (file)
@@ -503,7 +503,7 @@ static int amdgpu_vce_get_create_msg(struct amdgpu_ring *ring, uint32_t handle,
                ib->ptr[i] = 0x0;
 
        r = amdgpu_job_submit_direct(job, ring, &f);
-       amdgpu_ib_free(ring->adev, &ib_msg, f);
+       amdgpu_ib_free(&ib_msg, f);
        if (r)
                goto err;
 
index 96da27b86811bd363f2e86a874fde4315f4ebfb4..83faf6e6788a212bbe536fc26d2f8fd7fe0a61e5 100644 (file)
@@ -586,7 +586,7 @@ static int amdgpu_vcn_dec_send_msg(struct amdgpu_ring *ring,
        if (r)
                goto err_free;
 
-       amdgpu_ib_free(adev, ib_msg, f);
+       amdgpu_ib_free(ib_msg, f);
 
        if (fence)
                *fence = dma_fence_get(f);
@@ -597,7 +597,7 @@ static int amdgpu_vcn_dec_send_msg(struct amdgpu_ring *ring,
 err_free:
        amdgpu_job_free(job);
 err:
-       amdgpu_ib_free(adev, ib_msg, f);
+       amdgpu_ib_free(ib_msg, f);
        return r;
 }
 
@@ -779,7 +779,7 @@ static int amdgpu_vcn_dec_sw_send_msg(struct amdgpu_ring *ring,
        if (r)
                goto err_free;
 
-       amdgpu_ib_free(adev, ib_msg, f);
+       amdgpu_ib_free(ib_msg, f);
 
        if (fence)
                *fence = dma_fence_get(f);
@@ -790,7 +790,7 @@ static int amdgpu_vcn_dec_sw_send_msg(struct amdgpu_ring *ring,
 err_free:
        amdgpu_job_free(job);
 err:
-       amdgpu_ib_free(adev, ib_msg, f);
+       amdgpu_ib_free(ib_msg, f);
        return r;
 }
 
@@ -1020,7 +1020,7 @@ int amdgpu_vcn_enc_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = 0;
 
 error:
-       amdgpu_ib_free(adev, &ib, fence);
+       amdgpu_ib_free(&ib, fence);
        dma_fence_put(fence);
 
        return r;
index 2c12840ea4443b0c5ab8b38575e83ce46879d17e..121ee17b522bd4fb1ed2cf25c9c8d22b27a86f8c 100644 (file)
@@ -834,7 +834,7 @@ static int vpe_ring_test_ib(struct amdgpu_ring *ring, long timeout)
        ret = (le32_to_cpu(adev->wb.wb[index]) == test_pattern) ? 0 : -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
index a5cd950c94be4509f8147c4069d85cbbb68f94bf..d9bd8f3f17e27b4e7e1e713ef2748e976a85ba8e 100644 (file)
@@ -698,7 +698,7 @@ static int cik_sdma_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
index c4e15418e187ea9d8623c008a9534ad5413a3106..9484f3b5a9b7b2c54e408697360ac7243c04ee33 100644 (file)
@@ -4036,7 +4036,7 @@ static int gfx_v10_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
        else
                r = -EINVAL;
 err2:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err1:
        amdgpu_device_wb_free(adev, index);
index 37c552e06e5b52b9846d6189a056668a51ec5624..fe5d7cd814f7d5b09fc5160848f9dc3160af29a9 100644 (file)
@@ -615,7 +615,7 @@ static int gfx_v11_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 err2:
        if (!ring->is_mes_queue)
-               amdgpu_ib_free(adev, &ib, NULL);
+               amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err1:
        if (!ring->is_mes_queue)
index 50dca041d5891002a4e5fd7f454545769f9f8f8f..6ca9d0461e4c152641b8bad48517aefb2aa5aadc 100644 (file)
@@ -513,7 +513,7 @@ static int gfx_v12_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 err2:
        if (!ring->is_mes_queue)
-               amdgpu_ib_free(adev, &ib, NULL);
+               amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err1:
        if (!ring->is_mes_queue)
index 1b4c0dcee7e18ed9618f9b478709c715d1359cc7..f26e2cdec07a2fffce9789a7fca4acc114699bdc 100644 (file)
@@ -1910,7 +1910,7 @@ static int gfx_v6_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 error:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
        return r;
 }
index 17a07e1adffa7fc32926c7b64b484b84c06d13eb..84745b2453abeb1e1839489a817b1fe6d2bdb61c 100644 (file)
@@ -2330,7 +2330,7 @@ static int gfx_v7_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 error:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
        return r;
 }
index 576d21858c9436569e235e1d1463ee2a368a7e65..af73f85527b7a91ea5a570a224cd38f44574a4d2 100644 (file)
@@ -914,7 +914,7 @@ static int gfx_v8_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err2:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err1:
        amdgpu_device_wb_free(adev, index);
@@ -1656,7 +1656,7 @@ static int gfx_v8_0_do_edc_gpr_workarounds(struct amdgpu_device *adev)
                RREG32(sec_ded_counter_registers[i]);
 
 fail:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 
        return r;
index 30d9b6dacb6ef5bca604b453369d51a55190e521..4b5006dc3d347d4936c59f3be3773c19d4497ea5 100644 (file)
@@ -1243,7 +1243,7 @@ static int gfx_v9_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err2:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err1:
        amdgpu_device_wb_free(adev, index);
@@ -4788,7 +4788,7 @@ static int gfx_v9_0_do_edc_gpr_workarounds(struct amdgpu_device *adev)
        }
 
 fail:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 
        return r;
index 3f4fd2f08163def8957dea468e2b0eec025483d0..d81449f9d8225737be04b174ae2d217e933e23b3 100644 (file)
@@ -412,7 +412,7 @@ static int gfx_v9_4_2_run_shader(struct amdgpu_device *adev,
        r = amdgpu_ib_schedule(ring, 1, ib, NULL, fence_ptr);
        if (r) {
                dev_err(adev->dev, "ib submit failed (%d).\n", r);
-               amdgpu_ib_free(adev, ib, NULL);
+               amdgpu_ib_free(ib, NULL);
        }
        return r;
 }
@@ -611,16 +611,16 @@ static int gfx_v9_4_2_do_sgprs_init(struct amdgpu_device *adev)
        }
 
 disp2_failed:
-       amdgpu_ib_free(adev, &disp_ibs[2], NULL);
+       amdgpu_ib_free(&disp_ibs[2], NULL);
        dma_fence_put(fences[2]);
 disp1_failed:
-       amdgpu_ib_free(adev, &disp_ibs[1], NULL);
+       amdgpu_ib_free(&disp_ibs[1], NULL);
        dma_fence_put(fences[1]);
 disp0_failed:
-       amdgpu_ib_free(adev, &disp_ibs[0], NULL);
+       amdgpu_ib_free(&disp_ibs[0], NULL);
        dma_fence_put(fences[0]);
 pro_end:
-       amdgpu_ib_free(adev, &wb_ib, NULL);
+       amdgpu_ib_free(&wb_ib, NULL);
 
        if (r)
                dev_info(adev->dev, "Init SGPRS Failed\n");
@@ -687,10 +687,10 @@ static int gfx_v9_4_2_do_vgprs_init(struct amdgpu_device *adev)
        }
 
 disp_failed:
-       amdgpu_ib_free(adev, &disp_ib, NULL);
+       amdgpu_ib_free(&disp_ib, NULL);
        dma_fence_put(fence);
 pro_end:
-       amdgpu_ib_free(adev, &wb_ib, NULL);
+       amdgpu_ib_free(&wb_ib, NULL);
 
        if (r)
                dev_info(adev->dev, "Init VGPRS Failed\n");
index cc038f300a5634f2b5b768f3a9b34aae27376439..a5bdcaf7a0814fd0ed9a35c306258c912cc86470 100644 (file)
@@ -503,7 +503,7 @@ static int gfx_v9_4_3_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err2:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err1:
        amdgpu_device_wb_free(adev, index);
index 269bf1e3337b654e67f10f1ca714895b0d532a5d..135c5099bfb8e977a0cd58bbaa89688ffa36cf5b 100644 (file)
@@ -633,7 +633,7 @@ static int sdma_v2_4_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
index c9ad9ec48688ecb90133cbe65785d2d3aa076046..c611328671ed130899adae12989fc95a04bf6dcb 100644 (file)
@@ -906,7 +906,7 @@ static int sdma_v3_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
        else
                r = -EINVAL;
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
index defabd163d1714b8918e47196bbaf2b947d3b1aa..b48d9c0b2e1c3bfcb392c39acd74ecce0977311a 100644 (file)
@@ -1565,7 +1565,7 @@ static int sdma_v4_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
index 5e1cb1c2c0f80dccab78b093eab2777964e18fb4..4c8308b2878b6e2e6866de33d955500b292ee610 100644 (file)
@@ -1130,7 +1130,7 @@ static int sdma_v4_4_2_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
index 23599a5d4a124b69b03a6f0c4b96766f549ca73e..b764550834a07133c23cdb955ea6f2b6ab7d1147 100644 (file)
@@ -1194,7 +1194,7 @@ static int sdma_v5_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        if (!ring->is_mes_queue)
index 10352cfddca5c3e12ea41ddeaa82829476c0f627..b1818e87889a20c9173fbb7099e22390b5f53520 100644 (file)
@@ -1050,7 +1050,7 @@ static int sdma_v5_2_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        if (!ring->is_mes_queue)
index b14b6d344acecb3b5985244b64b0e0ea10ddc3fe..1a023b45f0be89503fc496beebaa7a2457229a1d 100644 (file)
@@ -1063,7 +1063,7 @@ static int sdma_v6_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        if (!ring->is_mes_queue)
index dc1a59246de5574d9cd2cd28e761851bf5833639..9c17df2cf37b82e5ade898c859f3cb93702b4165 100644 (file)
@@ -1108,7 +1108,7 @@ static int sdma_v7_0_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        if (!ring->is_mes_queue)
index 9f62b2b7fe0ea9162ca04d3103be230bc1bd756d..dbd78d5345a4279c1b5b640ec2bf8c1e59402368 100644 (file)
@@ -286,7 +286,7 @@ static int si_dma_ring_test_ib(struct amdgpu_ring *ring, long timeout)
                r = -EINVAL;
 
 err1:
-       amdgpu_ib_free(adev, &ib, NULL);
+       amdgpu_ib_free(&ib, NULL);
        dma_fence_put(f);
 err0:
        amdgpu_device_wb_free(adev, index);
This page took 0.077367 seconds and 4 git commands to generate.