[Intel-gfx] [PATCH 02/12] drm: Make the vblank disable timer per-crtc
Thierry Reding
thierry.reding at gmail.com
Wed May 21 13:17:49 CEST 2014
On Wed, May 14, 2014 at 08:51:04PM +0200, Daniel Vetter wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Currently there's one per-device vblank disable timer, and it gets
> reset wheneven the vblank refcount for any crtc drops to zero. That
"whenever"
> means that one crtc could accidentally be keeping the vblank interrupts
> for other crtcs enabled even if there are no users for them. Make the
> disable timer per-crtc to avoid this issue.
Very pedantically: s/crtc/CRTC/ and maybe even s/vblank/VBLANK/. Feel
free to ignore those, though. =)
Also, and I may have asked before, why do we even need this timer? Why
not simply disable interrupts when the last vblank reference goes away?
Generally, though:
Reviewed-by: Thierry Reding <treding at nvidia.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20140521/6f0a88e7/attachment.sig>
More information about the Intel-gfx
mailing list