From: Ville Syrjälä Date: Wed, 4 Sep 2013 15:25:30 +0000 (+0300) Subject: drm/i915: Fix cursor visibility check with negative coordinates X-Git-Tag: upstream/snapshot3+hdmi~3696^2~201^2~32 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=efc9064e7282aab65b281738089245c229c2df45;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git drm/i915: Fix cursor visibility check with negative coordinates When the cursor x coordinate is exactly -cursor_width, the cursor is invisible. And obviously the same holds for the y coordinate and cursor_height. Reviewed-by: Chris Wilson Signed-off-by: Ville Syrjälä Signed-off-by: Daniel Vetter --- diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index b8d7ad6..f5e80b1 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -6937,7 +6937,7 @@ static void intel_crtc_update_cursor(struct drm_crtc *crtc, base = 0; if (x < 0) { - if (x + intel_crtc->cursor_width < 0) + if (x + intel_crtc->cursor_width <= 0) base = 0; pos |= CURSOR_POS_SIGN << CURSOR_X_SHIFT; @@ -6946,7 +6946,7 @@ static void intel_crtc_update_cursor(struct drm_crtc *crtc, pos |= x << CURSOR_X_SHIFT; if (y < 0) { - if (y + intel_crtc->cursor_height < 0) + if (y + intel_crtc->cursor_height <= 0) base = 0; pos |= CURSOR_POS_SIGN << CURSOR_Y_SHIFT;