On Wed, Mar 13, 2013 at 11:21:08AM -0700, Ben Widawsky wrote:
> FIXME: This is based on some HW being used for a demo. We should
> probably wait until we have confirmation on the IDs before upstreaming
> this patch.
> 
> Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
> ---
>  drivers/gpu/drm/i915/i915_drv.c | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index 0fa55cb..9b3bb1e 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -135,6 +135,16 @@ extern int intel_agp_enabled;
>       .subdevice = PCI_ANY_ID,                \
>       .driver_data = (unsigned long) info }
>  
> +#define INTEL_QUANTA_VGA_DEVICE(info) {              \
> +     .class = PCI_BASE_CLASS_DISPLAY << 16,  \
> +     .class_mask = 0xff0000,                 \
> +     .vendor = 0x8086,                       \
> +     .device = 0x16a,                        \
> +     .subvendor = 0x152d,                    \
> +     .subdevice = 0x8990,                    \
> +     .driver_data = (unsigned long) info }
> +
> +
>  static const struct intel_device_info intel_i830_info = {
>       .gen = 2, .is_mobile = 1, .cursor_needs_physical = 1, .num_pipes = 1,
>       .has_overlay = 1, .overlay_needs_physical = 1,
> @@ -248,6 +258,15 @@ static const struct intel_device_info 
> intel_sandybridge_m_info = {
>       .has_force_wake = 1,
>  };
>  
> +static const struct intel_device_info intel_ivybridge_q_info = {
> +     .is_ivybridge = 1, .gen = 7, .num_pipes = 0,
> +     .need_gfx_hws = 1,
> +     .has_bsd_ring = 1,
> +     .has_blt_ring = 1,
> +     .has_llc = 1,
> +     .has_force_wake = 1,

What we could start to introduce is
#define IVB_FEATURES \
  .is_ivybridge = 1, \
  .gen = 7, \
  .num_pipes = 3, \
  .need_gfx_hws = 1,\
  etc

then

 static const struct intel_device_info intel_ivybridge_d_info = {
        IVB_FEATURES,
 };
 static const struct intel_device_info intel_ivybridge_m_info = {
        IVB_FEATURES,
        .is_mobile = 1,
 };
 static const struct intel_device_info intel_ivybridge_q_info = {
        IVB_FEATURES,
        .num_pipes = 0, /* legal, last one wins */
 };
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to