Re: [PATCH 00/12] PSR clean-up, new vfuncs and more use of HW tracking.

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

 



Sorry for the delayed review, this series needs to rebased as we are
passing crtc_state to a lot of these functions now. I don't see any
issues based on my cursory review.


On Wed, 2017-07-12 at 12:20 -0700, Rodrigo Vivi wrote:
> Most of patches on this series is only a clean up with
> no functional changes. The goal is to allow split VLV PSR
> implementation better. In a way that would get more cleaner
> when we use or not the hw tracking.
> 
> The ones with actual functional changes are:
> 0002-drm-i915-psr-Avoid-any-PSR-stuff-on-platforms-withou.patch
> that protects PSR of running anything on the platforms without support.
> 
> and
> 0012-drm-i915-psr-Use-more-PSR-HW-tracking.patch
> that allows us to use more HW tracking and consequently
> a proper PSR2 with partial screen updates.
> 
> Rodrigo Vivi (12):
>   drm/i915/psr: Remove vlv_is_active function.
>   drm/i915/psr: Avoid any PSR stuff on platforms without support.
>   drm/i915/psr: vfunc for disabling source.
>   drm/i915/psr: hsw_psr_activate.
>   drm/i915/psr: Add activate vfunc.
>   drm/i915/psr: Unify VSC setup functions.
>   drm/i915/psr: Re-create a hsw_psr_enable_source.
>   drm/i915/psr: Re-org Activate after enable
>   drm/i915/psr: Add setup VSC vfunc.
>   drm/i915/psr: Add enable_sink vfunc.
>   drm/i915/psr: Add enable_source vfunc.
>   drm/i915/psr: Use more PSR HW tracking.
> 
>  drivers/gpu/drm/i915/i915_drv.h          |   7 +
>  drivers/gpu/drm/i915/intel_drv.h         |   3 +-
>  drivers/gpu/drm/i915/intel_frontbuffer.c |   2 +-
>  drivers/gpu/drm/i915/intel_psr.c         | 267 ++++++++++++++++---------------
>  4 files changed, 150 insertions(+), 129 deletions(-)
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://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