Re: [PATCH] drm/i915: fix fastboot pfit disable hack to update pipe w/h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux