Commit c8af99b6 authored by Laurent Pinchart's avatar Laurent Pinchart

drm: rcar-du: Consider plane to CRTC associations in the plane allocator

Hardware planes are driven by the timing generator of the CRTC they are
associated to. Changing the association requires restarting the CRTC
group that the plane belongs to, resulting in flicker on the other CRTC.

To avoid flicker as much as possible, try to allocate planes first from
the free planes already associated with the target CRTC. If allocation
fails then fall back to allocation from all free planes.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
parent 2a57e9b5
...@@ -383,6 +383,8 @@ static int rcar_du_atomic_check(struct drm_device *dev, ...@@ -383,6 +383,8 @@ static int rcar_du_atomic_check(struct drm_device *dev,
for (i = 0; i < dev->mode_config.num_total_plane; ++i) { for (i = 0; i < dev->mode_config.num_total_plane; ++i) {
struct rcar_du_plane_state *plane_state; struct rcar_du_plane_state *plane_state;
struct rcar_du_plane *plane; struct rcar_du_plane *plane;
unsigned int crtc_planes;
unsigned int free;
int idx; int idx;
if (!state->planes[i]) if (!state->planes[i])
...@@ -401,8 +403,21 @@ static int rcar_du_atomic_check(struct drm_device *dev, ...@@ -401,8 +403,21 @@ static int rcar_du_atomic_check(struct drm_device *dev,
!rcar_du_plane_needs_realloc(plane, plane_state)) !rcar_du_plane_needs_realloc(plane, plane_state))
continue; continue;
/* Try to allocate the plane from the free planes currently
* associated with the target CRTC to avoid restarting the CRTC
* group and thus minimize flicker. If it fails fall back to
* allocating from all free planes.
*/
crtc_planes = to_rcar_crtc(plane_state->state.crtc)->index % 2
? plane->group->dptsr_planes
: ~plane->group->dptsr_planes;
free = group_free_planes[plane->group->index];
idx = rcar_du_plane_hwalloc(plane_state->format->planes, idx = rcar_du_plane_hwalloc(plane_state->format->planes,
group_free_planes[plane->group->index]); free & crtc_planes);
if (idx < 0)
idx = rcar_du_plane_hwalloc(plane_state->format->planes,
free);
if (idx < 0) { if (idx < 0) {
dev_dbg(rcdu->dev, "%s: no available hardware plane\n", dev_dbg(rcdu->dev, "%s: no available hardware plane\n",
__func__); __func__);
...@@ -749,6 +764,11 @@ int rcar_du_modeset_init(struct rcar_du_device *rcdu) ...@@ -749,6 +764,11 @@ int rcar_du_modeset_init(struct rcar_du_device *rcdu)
rgrp->mmio_offset = mmio_offsets[i]; rgrp->mmio_offset = mmio_offsets[i];
rgrp->index = i; rgrp->index = i;
/* Pre-associate all hardware planes with the first CRTC in the
* group.
*/
rgrp->dptsr_planes = 0;
ret = rcar_du_planes_init(rgrp); ret = rcar_du_planes_init(rgrp);
if (ret < 0) if (ret < 0)
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment