Re: [PATCH 14/14] drm/i915: Fix cursor visibility checks also for the right/bottom screen edges

Ville Syrjälä
2013-09-17T09:56:30Z

On Mon, Sep 16, 2013 at 11:52:17PM +0200, Daniel Vetter wrote:
> On Wed, Sep 04, 2013 at 06:25:31PM +0300, ville.syrjala@linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >
> > First of all we should not be looking at fb->{width,height} as those do
> > not tell us what the actual pipe size is. Second of all we need to use
> > >= for the comparison.
> >
> > So fix the comparison, and make use of the new pipe_src_{w,h} to
> > determine the real pipe source dimensions.
> >
> > Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Merged the entire series to dinq, thanks for patches&review. I've also
> made a small note that we should have an igt testcase for these cursor
> corner-cases somewhere ...

Which reminds me that I started to write one. Need to finish it up...

--
Ville Syrjälä
Intel OTC