Daniel Vetter writes:
>
> Hm, I think to address Chris' concern we should split this into two
> pieces: connector_break_all links which only breaks connector->encoder
> stuff, used in both places as is. And a new encoder_break_all links which
> we'll use in sanitize_crtc in a new encoder loop.
I've got a different solution, although it requires a bit more
code: Instead of having a single break_all_links() function I
move the link breaking code into the two functions where it
is used (ie sanitize_encoder() and sanitize_crtc()).
This gives one a bit more flexibility in implementing what is
needed and makes the code a bit clearer.
I will send later - once I had the opportunity to test.
Cheers,
Egbert.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
