On Tue, Jan 7, 2014 at 12:31 AM, Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote: > On Thu, 19 Dec 2013 10:48:01 -0800 > Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote: > >> When fastbooting, we read out the pipe timings early on, and then in a >> panel fitted config, disable the fitter later. But we weren't updating >> the pipe src h/w, which meant the mouse cursor was clipped to the >> pfitted size rather than the native size set later. Fix that up so the >> cursor is visible in the new mode. >> >> Signed-off-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> >> --- >> drivers/gpu/drm/i915/intel_display.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c >> index 4d1357a..7e46d75 100644 >> --- a/drivers/gpu/drm/i915/intel_display.c >> +++ b/drivers/gpu/drm/i915/intel_display.c >> @@ -2382,6 +2382,8 @@ intel_pipe_set_base(struct drm_crtc *crtc, int x, int y, >> I915_WRITE(PF_WIN_POS(intel_crtc->pipe), 0); >> I915_WRITE(PF_WIN_SZ(intel_crtc->pipe), 0); >> } >> + intel_crtc->config.pipe_src_w = adjusted_mode->crtc_hdisplay; >> + intel_crtc->config.pipe_src_h = adjusted_mode->crtc_vdisplay; >> } >> >> ret = dev_priv->display.update_plane(crtc, fb, x, y); > > References: https://bugzilla.kernel.org/show_bug.cgi?id=67591 The bug report doesn't say anything about the pipe config state checker being unhappy. Are we missing a call to the checker for fastboot somewhere to catch this stuff? -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
