Commit dda70217 authored by Tvrtko Ursulin's avatar Tvrtko Ursulin Committed by Alex Deucher
Browse files

drm/amdgpu: Simplify sorting of the bo list



Sort function only cares about the sign so we can replace the conditionals
with a single subtraction.

Signed-off-by: default avatarTvrtko Ursulin <tvrtko.ursulin@igalia.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent f7e06786
Loading
Loading
Loading
Loading
+3 −5
Original line number Diff line number Diff line
@@ -60,11 +60,9 @@ static int amdgpu_bo_list_entry_cmp(const void *_a, const void *_b)
{
	const struct amdgpu_bo_list_entry *a = _a, *b = _b;

	if (a->priority > b->priority)
		return 1;
	if (a->priority < b->priority)
		return -1;
	return 0;
	BUILD_BUG_ON(AMDGPU_BO_LIST_MAX_PRIORITY >= INT_MAX);

	return (int)a->priority - (int)b->priority;
}

int amdgpu_bo_list_create(struct amdgpu_device *adev, struct drm_file *filp,