drm/amdgpu: Drop redundant parameter in amdgpu_gfx_kiq_init_ring
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>
This commit is contained in:
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
|
||||
|
||||
@@ -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;
|
||||
}
|
||||
|
||||
@@ -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;
|
||||
}
|
||||
|
||||
@@ -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;
|
||||
|
||||
|
||||
@@ -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;
|
||||
|
||||
|
||||
@@ -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;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user