<div dir="ltr">Hi Daniel,<div><br></div><div>please consider to merge the first version. So we can move fwd with ddx patche followed by proper marketing names.</div><div><br></div><div>Thanks,</div><div>Rodrigo.</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 11, 2014 at 10:47 AM, Ben Widawsky <span dir="ltr"><<a href="mailto:ben@bwidawsk.net" target="_blank">ben@bwidawsk.net</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 Tue, Jun 10, 2014 at 10:41:07AM -0700, Rodrigo Vivi wrote:<br>
> These PCI IDs are reserved on BSpec and can be used at any time in the future.<br>
> So let's add this now in order to avoid issues that we already faced on previous<br>
> platforms, like finding out about new ids when user reported accelaration weren't<br>
> enabled.<br>
><br>
> v2: Reserved IDs doesn't have GT defined. So, creating a separated list. (Ben)<br>
><br>
> Cc: Ben Widawsky <<a href="mailto:ben@bwidawsk.net">ben@bwidawsk.net</a>><br>
> Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>><br>
> ---<br>
>  include/drm/i915_pciids.h | 16 ++++++++++++++--<br>
>  1 file changed, 14 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h<br>
> index 0572035..0968478 100644<br>
> --- a/include/drm/i915_pciids.h<br>
> +++ b/include/drm/i915_pciids.h<br>
> @@ -237,13 +237,25 @@<br>
>  #define INTEL_BDW_GT3D_IDS(info) \<br>
>       _INTEL_BDW_D_IDS(3, info)<br>
><br>
> +#define INTEL_BDW_RSVDM_IDS(info) \<br>
> +     INTEL_VGA_DEVICE(0x1632, info), \<br>
> +     INTEL_VGA_DEVICE(0x1636, info), \<br>
> +     INTEL_VGA_DEVICE(0x163B, info), \<br>
> +     INTEL_VGA_DEVICE(0x163A, info)<br>
> +<br>
> +#define INTEL_BDW_RSVDD_IDS(info) \<br>
> +     INTEL_VGA_DEVICE(0x163D, info), \<br>
> +     INTEL_VGA_DEVICE(0x163E, info)<br>
> +<br>
>  #define INTEL_BDW_M_IDS(info) \<br>
>       INTEL_BDW_GT12M_IDS(info), \<br>
> -     INTEL_BDW_GT3M_IDS(info)<br>
> +     INTEL_BDW_GT3M_IDS(info), \<br>
> +     INTEL_BDW_RSVDM_IDS(info)<br>
><br>
>  #define INTEL_BDW_D_IDS(info) \<br>
>       INTEL_BDW_GT12D_IDS(info), \<br>
> -     INTEL_BDW_GT3D_IDS(info)<br>
> +     INTEL_BDW_GT3D_IDS(info), \<br>
> +     INTEL_BDW_RSVDD_IDS(info)<br>
><br>
>  #define INTEL_CHV_IDS(info) \<br>
>       INTEL_VGA_DEVICE(0x22b0, info), \<br>
<br>
</div></div>I thought we saved off the GT info, but now that I actually look at the<br>
code, we do not. Therefore, I actually think v1 is a better patch.<br>
<br>
In either case, both v1 and v2 are:<br>
Reviewed-by: Ben Widawsky <<a href="mailto:ben@bwidawsk.net">ben@bwidawsk.net</a>><br>
<br>
I apologize for the extra work.<br>
<div class="im HOEnZb"><br>
--<br>
Ben Widawsky, Intel Open Source Technology Center<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div>