On Wed, Sep 27, 2017 at 07:58:42PM +0100, Kieran Bingham wrote:
> Hi Laurent,
>
> Thankyou for the patch,
>
> This looks good, and passes the tests.
>
> On 16/08/17 00:03, Laurent Pinchart wrote:
> > There is no point in accepting fully off-screen planes as they won't be
> > displayed. Reject them in the atomic check.
> >
> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
>
> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx>
Just a note, this is different from what i915 does. Not sure you want to
intentionally be incompatible with i915 and hence defacto make this part
of atomic undefined (or break some apps, depending upon how this pans
out). I think X loves to put the cursor offscreen at least.
I think we even have igts to test this properly.
-Daniel
>
> > ---
> > drivers/gpu/drm/rcar-du/rcar_du_plane.c | 16 ++++++++++++++++
> > 1 file changed, 16 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_plane.c b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
> > index 4f076c364f25..714e02960635 100644
> > --- a/drivers/gpu/drm/rcar-du/rcar_du_plane.c
> > +++ b/drivers/gpu/drm/rcar-du/rcar_du_plane.c
> > @@ -569,6 +569,8 @@ int __rcar_du_plane_atomic_check(struct drm_plane *plane,
> > struct drm_plane_state *state,
> > const struct rcar_du_format_info **format)
> > {
> > + const struct drm_display_mode *mode;
> > + struct drm_crtc_state *crtc_state;
> > struct drm_device *dev = plane->dev;
> >
> > if (!state->fb || !state->crtc) {
> > @@ -582,6 +584,20 @@ int __rcar_du_plane_atomic_check(struct drm_plane *plane,
> > return -EINVAL;
> > }
> >
> > + crtc_state = drm_atomic_get_crtc_state(state->state, state->crtc);
> > + if (IS_ERR(crtc_state))
> > + return PTR_ERR(crtc_state);
> > +
> > + mode = &crtc_state->adjusted_mode;
> > + if (state->crtc_x >= mode->hdisplay ||
> > + state->crtc_y >= mode->vdisplay ||
> > + state->crtc_x + (int)state->crtc_w <= 0 ||
> > + state->crtc_y + (int)state->crtc_h <= 0) {
> > + dev_dbg(dev->dev, "%s: plane can't be fully off-screen\n",
> > + __func__);
> > + return -EINVAL;
> > + }
> > +
> > *format = rcar_du_format_info(state->fb->format->format);
> > if (*format == NULL) {
> > dev_dbg(dev->dev, "%s: unsupported format %08x\n", __func__,
> >
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
