On Thu, 10 Oct 2019 16:11:59 +0300
Ville Syrjala <ville.syrj...@linux.intel.com> wrote:

> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> @swap@
> identifier TEMP;
> expression A,B;
> @@
> - TEMP = A;
> - A = B;
> - B = TEMP;
> + swap(A, B);
> 
> @@
> type T;
> identifier swap.TEMP;
> @@
> (
> - T TEMP;
> |
> - T TEMP = {...};
> )
> ... when != TEMP
> 
> Cc: Sam Ravnborg <s...@ravnborg.org>
> Cc: Boris Brezillon <bbrezil...@kernel.org>
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

Reviewed-by: Boris Brezillon <boris.brezil...@collabora.com>

> ---
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c 
> b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> index 89f5a756fa37..034f202dfe8f 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> @@ -601,7 +601,6 @@ static int atmel_hlcdc_plane_atomic_check(struct 
> drm_plane *p,
>       struct drm_framebuffer *fb = state->base.fb;
>       const struct drm_display_mode *mode;
>       struct drm_crtc_state *crtc_state;
> -     unsigned int tmp;
>       int ret;
>       int i;
>  
> @@ -694,9 +693,7 @@ static int atmel_hlcdc_plane_atomic_check(struct 
> drm_plane *p,
>        * Swap width and size in case of 90 or 270 degrees rotation
>        */
>       if (drm_rotation_90_or_270(state->base.rotation)) {
> -             tmp = state->src_w;
> -             state->src_w = state->src_h;
> -             state->src_h = tmp;
> +             swap(state->src_w, state->src_h);
>       }
>  
>       if (!desc->layout.size &&

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to