Commit 5599b214 authored by Liao Yuanhong's avatar Liao Yuanhong Committed by Alex Deucher
Browse files

drm/amd/display: Use swap() to simplify code 



Replace the original swapping logic with swap() to improve readability and
remove temporary variables

Signed-off-by: default avatarLiao Yuanhong <liaoyuanhong@vivo.com>
Reviewed-by: default avatarAlex Hung <alex.hung@amd.com>
Tested-by: default avatarDan Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 8a614ddf

drivers/gpu/drm/amd/display/dc/dml2/dml2_dc_resource_mgmt.c

+2 −4
Original line number Diff line number Diff line
@@ -473,7 +473,6 @@ static void sort_pipes_for_splitting(struct dc_plane_pipe_pool *pipes) 
{

	bool sorted, swapped;

	unsigned int cur_index;

	unsigned int temp;

	int odm_slice_index;



	for (odm_slice_index = 0; odm_slice_index < pipes->num_pipes_assigned_to_plane_for_odm_combine; odm_slice_index++) {
@@ -489,9 +488,8 @@ static void sort_pipes_for_splitting(struct dc_plane_pipe_pool *pipes) 
		swapped = false;

		while (!sorted) {

			if (pipes->pipes_assigned_to_plane[odm_slice_index][cur_index] > pipes->pipes_assigned_to_plane[odm_slice_index][cur_index + 1]) {

				temp = pipes->pipes_assigned_to_plane[odm_slice_index][cur_index];

				pipes->pipes_assigned_to_plane[odm_slice_index][cur_index] = pipes->pipes_assigned_to_plane[odm_slice_index][cur_index + 1];

				pipes->pipes_assigned_to_plane[odm_slice_index][cur_index + 1] = temp;

				swap(pipes->pipes_assigned_to_plane[odm_slice_index][cur_index + 1],

					 pipes->pipes_assigned_to_plane[odm_slice_index][cur_index]);



				swapped = true;

			}