<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 16, 2014 at 3:21 AM, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, May 15, 2014 at 08:13:04PM -0400, Rodrigo Vivi wrote:<br>
> Also do not cache aux info. That info could be related to another panel.<br>
<br>
</div>You should kill the bool sink_support then. There are other places that<br>
it used that could be invalid. </blockquote><div><br></div><div>the point here was to keep the info for debugfs in a simple way.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
I'm not sure though that we can cope with eDP panels being swapped at<br>
runtime.<br></blockquote><div><br></div><div>the point wan't swap, but the possibility of having more than one eDP in the future, but anyway other parts of the could should be changed to support this.</div><div><br></div>
<div>so maybe just get back to the origial version keeping it on sink_support...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888">-Chris<br>
<br>
--<br>
Chris Wilson, Intel Open Source Technology Centre<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div></div>