On Sun, 19 Aug 2012 21:12:48 +0200 Daniel Vetter <daniel.vet...@ffwll.ch> wrote:
> Atm we can only check the connector state after a dpms call - while > doing modeset with the copy&pasted crtc helper code things are too > ill-defined for proper checking. But the idea is very much to call > this check from the modeset code, too. > > v2: Fix dpms check and don't presume that if the hw isn't on that it > must not be linked up with an encoder (it could simply be switched off > with the dpms state). Nice. Only comment is that we may want to call it assert_state() to match our other assertions. But no biggie. Reviewed-by: Jesse Barnes <jbar...@virtuousgeek.org> -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
