<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 20, 2016 at 12:08 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</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">On Sat 11 Jun 2016, Jason Ekstrand wrote:<br>
> ---<br>
>  src/intel/genxml/genX_pack.h  | 10 +++++++++-<br>
>  src/intel/genxml/gen_macros.h | 15 ++++++++++++++-<br>
>  2 files changed, 23 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/intel/genxml/genX_pack.h b/src/intel/genxml/genX_pack.h<br>
> index 7967c29..0c25c4e 100644<br>
> --- a/src/intel/genxml/genX_pack.h<br>
> +++ b/src/intel/genxml/genX_pack.h<br>
> @@ -27,7 +27,15 @@<br>
>  #  error "The GEN_VERSIONx10 macro must be defined"<br>
>  #endif<br>
><br>
> -#if (GEN_VERSIONx10 == 70)<br>
> +#if (GEN_VERSIONx10 == 40)<br>
> +#  include "genxml/gen4_pack.h"<br>
> +#elif (GEN_VERSIONx10 == 45)<br>
> +#  include "genxml/gen45_pack.h"<br>
> +#elif (GEN_VERSIONx10 == 50)<br>
> +#  include "genxml/gen5_pack.h"<br>
> +#elif (GEN_VERSIONx10 == 60)<br>
> +#  include "genxml/gen6_pack.h"<br>
> +#elif (GEN_VERSIONx10 == 70)<br>
>  #  include "genxml/gen7_pack.h"<br>
>  #elif (GEN_VERSIONx10 == 75)<br>
>  #  include "genxml/gen75_pack.h"<br>
> diff --git a/src/intel/genxml/gen_macros.h b/src/intel/genxml/gen_macros.h<br>
> index 868bc22..1d591fa 100644<br>
> --- a/src/intel/genxml/gen_macros.h<br>
> +++ b/src/intel/genxml/gen_macros.h<br>
> @@ -57,9 +57,22 @@<br>
><br>
>  #define GEN_GEN ((GEN_VERSIONx10) / 10)<br>
>  #define GEN_IS_HASWELL ((GEN_VERSIONx10) == 75)<br>
> +#define GEN_IS_G4X ((GEN_VERSIONx10) == 45)<br>
<br>
</div></div>I expected the new macro name to be GEN_IS_G45. Why is it G4X?<br>
</blockquote></div><br></div><div class="gmail_extra">i started with IS_G45 and then realized that the brw_device_info field is is_g4x so I decided to be consistent.  Chatting around the cube a bit, it sounds like g44 and g43 were also things.  Ken would be happy to rename it all to is_g45.  I don't think anyone cares that much.<br></div></div>