On 4/11/2025 7:50 PM, Alex Deucher wrote:
Not techincally wrong, but I think a semicolon was intended here. Fixes: 6cc6e61788f7 ("drm/amdgpu: use a dummy owner for sysfs triggered cleaner shaders v3") Cc: Christian König <[email protected]> Signed-off-by: Alex Deucher <[email protected]> --- drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c index f1b87b177f4e3..c206c3b371ad0 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c @@ -1460,7 +1460,7 @@ static int amdgpu_gfx_run_cleaner_shader_job(struct amdgpu_ring *ring) * the cleaner shader on each submission. The value just need to change * for each submission and is otherwise meaningless. */ - owner = (void *)(unsigned long)atomic_inc_return(&counter), + owner = (void *)(unsigned long)atomic_inc_return(&counter);r = amdgpu_job_alloc_with_ib(ring->adev, &entity, owner,64, 0, &job);
Reviewed-by: Srinivasan Shanmugam <[email protected]>
