<div class="gmail_quote">On Wed, Mar 28, 2012 at 17:37, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">> > My latest branch reports gives `grep IS_GEN * | wc -l` = 112; and if we<br>
> > look for recent chipsets, we have `grep IS_GEN[67] *` = 47. And most of<br>
> > those checks have sub-checks as well for specific chip features or names.<br>
> > So if we drop the IS_GEN macros, and just use the specific feature or GPU<br>
> > name checks instead we should improve both the readability and decrease the<br>
> > code complexity I think.<br>
><br>
> A grep for IS_GEN will be significantly under-counting the number of<br>
> places that gen numbers are used, given that Mesa uses intel->gen, and<br>
> the 2d driver tends to use INTEL_INFO(intel)->gen.<br>
<br>
</div></div>Actually the kernel uses INTEL_INFO(dev)->gen, too, mostly where we want<br>
to case switch on the gen number. And that happens at a fair amount of<br>
places.<br></blockquote><div><br></div><div>Indeed, you are right, my grep skills failed on me here.</div><div></div></div><div><br></div>After looking on the way gen info is used in Mesa, yeah, I think it is not really a solution to abandon it just yet. I came to think on the possible feature checks to replace them, but I don't think it would be worth the effort just to replace all the Gen checks with a different switch just for the sake of doing it.<div>

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