On Sun, 19 Aug 2012 21:12:48 +0200 Daniel Vetter <[email protected]> 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 <[email protected]> -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
