[PATCH] drm/amd/display: add basic atomic check for cursor plane

[Simon Ser]

Hi,

On Monday, March 30, 2020 5:23 PM, Simon Ser <contact at emersion.fr> wrote:

> On Monday, March 30, 2020 5:18 PM, Kazlauskas, Nicholas nicholas.kazlauskas at amd.com wrote:
>
> > On 2020-03-30 9:13 a.m., Simon Ser wrote:
> >
> > > On Monday, March 30, 2020 3:11 PM, Kazlauskas, Nicholas nicholas.kazlauskas at amd.com wrote:
> > >
> > > > On 2020-03-30 9:02 a.m., Simon Ser wrote:
> > > >
> > > > > On Monday, March 30, 2020 2:59 PM, Kazlauskas, Nicholas nicholas.kazlauskas at amd.com wrote:
> > > > >
> > > > > > We've been doing these checks for position before but I don't think we
> > > > > > really need them. DC should be disabling the cursor when we ask for a
> > > > > > position completely off the screen.
> > > > > > I think that's better than rejecting the commit entirely at least.
> > > > >
> > > > > I agree DC should be disabling the cursor in this case, however that's
> > > > > not yet implemented right? I think implementing this feature is
> > > > > orthogonal and should be done in a separate patch.
> > > > > This patch simply copies over the cursor checks in the atomic check
> > > > > function.
> > > >
> > > > It's implemented on DCN but I don't remember if we're doing it on DCE.
> > > > I guess the drop can be in a separate patch.
> > > >
> > > > Reviewed-by: Nicholas Kazlauskas nicholas.kazlauskas at amd.com
> > >
> > > Thanks for the review. I'll try to figure out whether we can drop this
> > > check (from both the atomic check and the other existing check).
> >
> > Oh, this was actually the checks for crtc_w/crtc_h. Not the x/y, my bad.
> > We probably can't drop this from here, but we can drop it from
> > get_cursor_position after this patch - since it's now in the atomic check.
>
> Hmm, sorry I think I missed something. This patch does copy over the
> x/y checks. We need to keep the w/h checks right?
>
> Yeah, we can probably drop get_cursor_position checks indeed.

Gentle ping: any chance to get this patch merged?

Thanks,

Simon