On Mon, Jan 2, 2012 at 17:37, Eric Anholt <e...@anholt.net> wrote:
> - if (intel->gen == 6 || (intel->gen == 7 && override_version >= 30))
> + if (intel->gen == 6 ||
> + (intel->gen == 7 && intel->intelScreen->kernel_has_gen7_sol_reset))
> ctx->Extensions.EXT_transform_feedback = true;
>
If the kernel patch in previous series would return
...
value = IS_GEN7(dev);
...
the line above could be simplified to:
if (intel->gen == 6 || intel->intelScreen->kernel_has_gen7_sol_reset)
But on the other hand, it would work either way. The check for GEN7 in
kernel looks cleaner and more logically correct to me, but in any case:
Reviewed-by: Eugeni Dodonov <eugeni.dodo...@intel.com>
--
Eugeni Dodonov
<http://eugeni.dodonov.net/>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
