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 <ben at 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
