Commit e1ba78fc authored by Sean Paul's avatar Sean Paul Committed by Rob Clark

drm/msm: dpu: Remove dpu_plane_enabled()

plane->state->visible encompasses all of these checks and more, so we
can just check visible.

Changes in v2:
- None
Reviewed-by: default avatarJeykumar Sankaran <jsanka@codeaurora.org>
Signed-off-by: default avatarSean Paul <seanpaul@chromium.org>
Signed-off-by: default avatarRob Clark <robdclark@gmail.com>
parent 31ab6bff
...@@ -135,11 +135,6 @@ static struct dpu_kms *_dpu_plane_get_kms(struct drm_plane *plane) ...@@ -135,11 +135,6 @@ static struct dpu_kms *_dpu_plane_get_kms(struct drm_plane *plane)
return to_dpu_kms(priv->kms); return to_dpu_kms(priv->kms);
} }
static bool dpu_plane_enabled(struct drm_plane_state *state)
{
return state && state->fb && state->crtc;
}
/** /**
* _dpu_plane_calc_fill_level - calculate fill level of the given source format * _dpu_plane_calc_fill_level - calculate fill level of the given source format
* @plane: Pointer to drm plane * @plane: Pointer to drm plane
...@@ -165,7 +160,7 @@ static inline int _dpu_plane_calc_fill_level(struct drm_plane *plane, ...@@ -165,7 +160,7 @@ static inline int _dpu_plane_calc_fill_level(struct drm_plane *plane,
fixed_buff_size = pdpu->pipe_sblk->common->pixel_ram_size; fixed_buff_size = pdpu->pipe_sblk->common->pixel_ram_size;
list_for_each_entry(tmp, &pdpu->mplane_list, mplane_list) { list_for_each_entry(tmp, &pdpu->mplane_list, mplane_list) {
if (!dpu_plane_enabled(tmp->base.state)) if (!tmp->base.state->visible)
continue; continue;
DPU_DEBUG("plane%d/%d src_width:%d/%d\n", DPU_DEBUG("plane%d/%d src_width:%d/%d\n",
pdpu->base.base.id, tmp->base.base.id, pdpu->base.base.id, tmp->base.base.id,
...@@ -1395,7 +1390,7 @@ static void dpu_plane_atomic_update(struct drm_plane *plane, ...@@ -1395,7 +1390,7 @@ static void dpu_plane_atomic_update(struct drm_plane *plane,
DPU_DEBUG_PLANE(pdpu, "\n"); DPU_DEBUG_PLANE(pdpu, "\n");
if (!dpu_plane_enabled(state)) { if (!state->visible) {
_dpu_plane_atomic_disable(plane, old_state); _dpu_plane_atomic_disable(plane, old_state);
} else { } else {
int ret; int 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