On 14/12/16 02:27, Laurent Pinchart wrote: > The vblank interrupt is disabled after one occurrence, preventing the > atomic update event from being processed twice. However, this also > prevents the software frame counter from being updated correctly that > would require vblank interrupts to be kept enabled while the CRTC is > active. > > In preparation for vblank interrupt fixes, make sure that the atomic > update event will be processed once only when the vblank interrupt will > be kept enabled. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > Changes since v3: > > - Don't release and reacquire the spinlock for just one return > - Remove unneeded (and unbalanced) drm_crtc_vblank_get() > - Store the event in the atomic flush handler to avoid race condition > - Use spin_lock_irq instead of spin_lock_irsave in flush handler > --- > drivers/gpu/drm/omapdrm/omap_crtc.c | 23 +++++++++++++++-------- > 1 file changed, 15 insertions(+), 8 deletions(-) Reviewed-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx> Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
