<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Nov 8, 2016 at 1:53 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Nov 8, 2016 at 1:21 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Most of the 3-D engine Kaby Lake is identical to Sky Lake.  However, there<br>
<br>
</span>While hyphenating 3D looks a little odd to me, Skylake is definitely<br>
just a single word. (Strangely, Kaby Lake is indeed two words)<br>
<span class=""><br>
> are a few small differences that we need to be able to detect.<br>
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
> ---<br>
>  src/intel/common/gen_device_<wbr>info.c | 14 +++++++++-----<br>
>  src/intel/common/gen_device_<wbr>info.h |  1 +<br>
>  2 files changed, 10 insertions(+), 5 deletions(-)<br>
><br>
> diff --git a/src/intel/common/gen_device_<wbr>info.c b/src/intel/common/gen_device_<wbr>info.c<br>
> index 30df0b2..3ff98f0 100644<br>
> --- a/src/intel/common/gen_device_<wbr>info.c<br>
> +++ b/src/intel/common/gen_device_<wbr>info.c<br>
> @@ -427,6 +427,10 @@ static const struct gen_device_info gen_device_info_bxt_2x6 = {<br>
>   * There's no KBL entry. Using the default SKL (GEN9) GS entries value.<br>
>   */<br>
><br>
> +#define KBL_FEATURES \<br>
<br>
</span>We don't have subgen FEATURES #defines for anything else. bxt, for<br>
instance just sets .is_broxton in its couple of fields. Not wrong, but<br>
doesn't seem particularly necessary for a single field.<br></blockquote><div><br></div><div>We do for Haswell which is why I did it this way.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'd prefer to just put .is_kabylake in the KBL structs, unless you've<br>
got further plans.<br></blockquote><div><br></div><div>Sure.  I don't care much either way.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With that fixed, both patches are<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br></blockquote><div><br></div><div>Thanks <br></div></div><br></div></div>