drm/sun4i: backend: Move the coord function in the shared part

The function supposed to update a plane's coordinates is called in both
branches of our function. Let's move it out the if statement.

Acked-by: Chen-Yu Tsai <wens@csie.org>
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Link: https://patchwork.freedesktop.org/patch/msgid/2cd57bcf13652109da7bd5bbe12fa1d29429f02f.1516617243.git-series.maxime.ripard@free-electrons.com

diff --git a/drivers/gpu/drm/sun4i/sun4i_layer.c b/drivers/gpu/drm/sun4i/sun4i_layer.c
index f03da16..c448cb6 100644 (file)
--- a/drivers/gpu/drm/sun4i/sun4i_layer.c
+++ b/drivers/gpu/drm/sun4i/sun4i_layer.c
@@ -106,14 +106,13 @@ static void sun4i_backend_layer_atomic_update(struct drm_plane *plane,
                                              DRM_FORMAT_ARGB8888);
                sun4i_backend_update_layer_frontend(backend, layer->id,
                                                    DRM_FORMAT_ARGB8888);
-               sun4i_backend_update_layer_coord(backend, layer->id, plane);
                sun4i_frontend_enable(frontend);
        } else {
-               sun4i_backend_update_layer_coord(backend, layer->id, plane);
                sun4i_backend_update_layer_formats(backend, layer->id, plane);
                sun4i_backend_update_layer_buffer(backend, layer->id, plane);
        }
 
+       sun4i_backend_update_layer_coord(backend, layer->id, plane);
        sun4i_backend_layer_enable(backend, layer->id, true);
 }