diff options
| author | Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> | 2015-02-18 15:47:27 +0200 |
|---|---|---|
| committer | Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> | 2015-03-03 16:16:11 +0200 |
| commit | 845f46356ba490b654194b8d5c26032841719a78 (patch) | |
| tree | b243b4e4149ecfd49bac22be4a7ad556b4b163b6 /drivers/gpu/drm/rcar-du/rcar_du_plane.c | |
| parent | 920888a2d56f0ef7117bf3456cacb49c6801d8de (diff) | |
| download | linux-stable-845f46356ba490b654194b8d5c26032841719a78.tar.gz linux-stable-845f46356ba490b654194b8d5c26032841719a78.tar.bz2 linux-stable-845f46356ba490b654194b8d5c26032841719a78.zip | |
drm: rcar-du: Handle primary plane config through atomic plane ops
Use the new CRTC atomic transitional helpers drm_helper_crtc_mode_set()
and drm_helper_crtc_mode_set_base() to implement the CRTC .mode_set and
.mode_set_base operations. This delegates primary plane configuration to
the plane .atomic_update and .atomic_disable operations, removing
duplicate code from the CRTC implementation.
There is now no code path available to the driver in which to drop the
reference to the CRTC acquired in the .prepare() operation if an error
then occurs. The driver thus now leaks a reference if an error occurs
during mode set. So be it, this will be fixed in a further step of the
atomic update transition.
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Diffstat (limited to 'drivers/gpu/drm/rcar-du/rcar_du_plane.c')
| -rw-r--r-- | drivers/gpu/drm/rcar-du/rcar_du_plane.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/rcar-du/rcar_du_plane.c index 03995c15bbe3..d4682ac7db03 100644 --- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c +++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c @@ -78,8 +78,8 @@ static int rcar_du_plane_reserve_check(struct rcar_du_plane *plane, return ret; } -int rcar_du_plane_reserve(struct rcar_du_plane *plane, - const struct rcar_du_format_info *format) +static int rcar_du_plane_reserve(struct rcar_du_plane *plane, + const struct rcar_du_format_info *format) { struct rcar_du_group *rgrp = plane->group; unsigned int i; @@ -112,7 +112,7 @@ done: return ret; } -void rcar_du_plane_release(struct rcar_du_plane *plane) +static void rcar_du_plane_release(struct rcar_du_plane *plane) { struct rcar_du_group *rgrp = plane->group; @@ -363,7 +363,6 @@ static void rcar_du_plane_disable(struct rcar_du_plane *rplane) mutex_lock(&rplane->group->planes.lock); rplane->enabled = false; - rcar_du_crtc_update_planes(rplane->crtc); mutex_unlock(&rplane->group->planes.lock); rcar_du_plane_release(rplane); @@ -411,7 +410,6 @@ static void rcar_du_plane_atomic_update(struct drm_plane *plane, mutex_lock(&rplane->group->planes.lock); rplane->enabled = true; - rcar_du_crtc_update_planes(rplane->crtc); mutex_unlock(&rplane->group->planes.lock); } |