[PATCHv3 23/30] drm/omap: fix plane update warning when crtc is disabled
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Wed Mar 29 10:30:30 UTC 2017
Hi Tomi,
Thank you for the patch.
On Tuesday 28 Mar 2017 16:08:09 Tomi Valkeinen wrote:
> If omap_plane_atomic_update() is called when the crtc is disabled, and
> the timings are zero, we'll see the following warning:
>
> omapdss DISPC error: cannot calculate scaling settings: pclk is zero
> omapdrm omapdrm.0: Failed to setup plane vid2
>
> It shouldn't cause any issues, as the crtc is disabled so the plane is
> not used by the HW.
>
> To remove the warning, check whether the crtc is enabled at the
> beginning of omap_plane_atomic_update(), and bail out if not.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
> ---
> drivers/gpu/drm/omapdrm/omap_plane.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c
> b/drivers/gpu/drm/omapdrm/omap_plane.c index 78a92422ede1..27c1902415de
> 100644
> --- a/drivers/gpu/drm/omapdrm/omap_plane.c
> +++ b/drivers/gpu/drm/omapdrm/omap_plane.c
> @@ -86,6 +86,11 @@ static void omap_plane_atomic_update(struct drm_plane
> *plane,
>
> DBG("%s, crtc=%p fb=%p", omap_plane->name, state->crtc, state->fb);
>
> + if (!state->crtc->state->enable) {
> + priv->dispc_ops->ovl_enable(omap_plane->id, false);
> + return;
> + }
I'm a bit puzzled, why is omap_plane_atomic_update() called for a plane on a
disabled CRTC ?
> memset(&info, 0, sizeof(info));
> info.rotation_type = OMAP_DSS_ROT_DMA;
> info.rotation = OMAP_DSS_ROT_0;
--
Regards,
Laurent Pinchart
More information about the dri-devel
mailing list