From: Gustavo A. R. Silva Date: Fri, 10 Nov 2017 22:38:13 +0000 (-0600) Subject: drm/amd/display/dc/core/dc_resource: use swap macro in rect_swap_helper X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=99676f20dc913672166ab3b22fe29c20a2c27a82;p=openwrt%2Fstaging%2Fblogic.git drm/amd/display/dc/core/dc_resource: use swap macro in rect_swap_helper Make use of the swap macro instead of _manually_ swapping values and remove unnecessary variable temp. This makes the code easier to read and maintain. This code was detected with the help of Coccinelle. Signed-off-by: Gustavo A. R. Silva Signed-off-by: Alex Deucher --- diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c index 15e518aba916..9c5e879f18b3 100644 --- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c +++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c @@ -426,15 +426,8 @@ static enum pixel_format convert_pixel_format_to_dalsurface( static void rect_swap_helper(struct rect *rect) { - uint32_t temp = 0; - - temp = rect->height; - rect->height = rect->width; - rect->width = temp; - - temp = rect->x; - rect->x = rect->y; - rect->y = temp; + swap(rect->height, rect->width); + swap(rect->x, rect->y); } static void calculate_viewport(struct pipe_ctx *pipe_ctx)