<div class="gmail_quote">On Tue, Mar 20, 2012 at 17:13, Jesse Barnes <span dir="ltr"><<a href="mailto:jbarnes@virtuousgeek.org">jbarnes@virtuousgeek.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">> > Thoughts?  It may also make sense to split some of our port specific<br></div><div class="im">
> > files where they differ enough from previous platforms.  E.g. g4x DP vs<br>
> > ironlake+...<br>
><br>
> I've just talked about this a bit with Eugeni in the context of Haswell,<br>
> and I think we might want to hold of for that code until we move output<br>
> stuff all over the place.<br>
<br>
</div>Yeah I don't want to make HSW any harder than necessary; we can put off<br>
splits there.<br></blockquote><div><br></div><div>(I suspect those 2 paragraphs above are enough for a new phoronix article on HSW already :)).</div><div><br></div><div>Yeah, I've been through the same issues as Jesse, and we were talking with Daniel about this. Our intel_display.c is huge and unnecessarily complex, and we need to split it at some point if we want to keep our mental sanity. At least the split of it into intel_display.c and intel_pch_display.c would simplify the things a lot; and maybe we could add intel_display_gen6.c, intel_display_gen7.c, intel_display_pch_cpt.c and intel_display_pch_lpt.c and move those hardware-specific stuff there for simplifying out tasks in the future.</div>

<div><br></div><div>I was also thinking on adding a intel_workarounds.c for handling all the W/As we have in one place, and having some support for calling them via callbacks to run specific WAs during the modeset, after suspend/resume cycle and so on.</div>

</div><div><br></div><div>For i915_regs.h, I don't think there is a need to split is for now. It is a nice way to have all the registers in just one place, and at least I never had a problem with it so far.</div><div>

<br></div>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br></a><br>