On Thu, Dec 5, 2013 at 1:38 PM, Paulo Zanoni <przanoni@xxxxxxxxx> wrote: > Even with that patch applied, I think we should keep this current path > as-is since it runs before the code that checks for 2 encoders. ->compute_config is called much later after stage_output_state. So I'm a bit confused why we need both ... If you want we could smash a WARN or so into compute_config for paranoia, but since we have a proper igt testcase that's imo not required at all. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx