On Wed, Oct 24, 2012 at 1:38 PM, Daniel Vetter <[email protected]> wrote:
> On Tue, Oct 23, 2012 at 06:29:58PM -0200, Paulo Zanoni wrote:
>> From: Paulo Zanoni <[email protected]>
>>
>> We need to check if any of the pipes is using TRANSCODER_EDP.
>>
>> Signed-off-by: Paulo Zanoni <[email protected]>
>
> One thing that irks me is that you add new state readout code here, but
> don't call the same code in the modeset_check_state code

Isn't what the previous patch introduce? (look at the
intel_ddi_get_hw_state() hunk in the patch 07/18 of the series).
AFAICS we're are already checking if the pipe returned by the encoder
is what we think it is.

-- 
Damien
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to