On Thu, 7 Jul 2011 22:14:18 +0100, Chris Wilson <[email protected]> wrote: > Upon review, all path share the same dependencies for updating the > registers and so we can benefit from sharing the code and checking > early. > > This removes the unsightly intel_wait_for_vblank() from the lowlevel > functions and upon further analysis the only path that will require a > wait is if we are performing an instantaneous transition between two > valid FBC configurations. The page-flip path itself will have disabled > FBC registers and will have waited for at least one vblank before > finishing the flip and attempting to re-enable FBC. > > Signed-off-by: Chris Wilson <[email protected]> > --- > > Respun without the blocking wait. Noting instead that since we delay > enabling FBC by 50ms we can assume that a vblank passes after disabling > FBC and before modifying the control registers. > -Chris
Looks good to me. Reviewed-by: Keith Packard <[email protected]> -- [email protected]
pgpC8jcs9a8u1.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
