<div class="gmail_quote">On Fri, Nov 11, 2011 at 06:31, Zhigang Gong <<a href="mailto:zhigang.gong@linux.intel.com" target="_blank">zhigang.gong@linux.intel.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Added one configuration option --enable-glamor to control whether use glamor. Added one new file intel_glamor.c to wrap glamor egl API for intel driver's usage. This commit doesn't really change the driver's control path. It just adds necessary files for glamor and change some configuration. </blockquote><div> For the series, I've reviewed the patches and they seem OK for me. I also like the idea of having Glamor available in main ddx branch, for easier testing. So: Reviewed-by: Eugeni Dodonov <<a href="mailto:eugeni.dodonov@intel.com" target="_blank">eugeni.dodonov@intel.com</a>> (However, I think Chris will have more comments, he is The Master of the DDX). </div></div>-- Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"> </a>