From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ben Widawsky <ben@bwidawsk.net>
Subject: Re: [PATCH 04/62] [v6] drm/i915/bdw: Add device IDs
Date: Mon, 4 Nov 2013 08:56:10 -0800
Message-ID: <20131104165610.GA6676@bwidawsk.net>
References: <20131103215800.GB8183@nuc-i3427.alporthouse.com>
	<1383525418-31596-1-git-send-email-benjamin.widawsky@intel.com>
	<20131104144921.GK15018@nuc-i3427.alporthouse.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
Received: from mail.bwidawsk.net (bwidawsk.net [166.78.191.112])
	by gabe.freedesktop.org (Postfix) with ESMTP id DBD1DFA88C
	for <intel-gfx@lists.freedesktop.org>;
	Mon,  4 Nov 2013 08:56:15 -0800 (PST)
Content-Disposition: inline
In-Reply-To: <20131104144921.GK15018@nuc-i3427.alporthouse.com>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Sender: intel-gfx-bounces@lists.freedesktop.org
Errors-To: intel-gfx-bounces@lists.freedesktop.org
To: Chris Wilson <chris@chris-wilson.co.uk>, Ben Widawsky <benjamin.widawsky@intel.com>, Intel GFX <intel-gfx@lists.freedesktop.org>, Daniel Vetter <daniel.vetter@ffwll.ch>
List-Id: intel-gfx@lists.freedesktop.org

On Mon, Nov 04, 2013 at 02:49:21PM +0000, Chris Wilson wrote:
> On Sun, Nov 03, 2013 at 04:36:58PM -0800, Ben Widawsky wrote:
> > v2: Squash in "drm/i915/bdw: Add BDW to the HAS_DDI check" as
> > suggested by Damien.
> > 
> > v3: Squash in VEBOX enabling from  Zhao Yakui <yakui.zhao@intel.com>
> > 
> > v4: Rebase on top of Jesse's patch to extract all pci ids to
> > include/drm/i915_pciids.h.
> > 
> > v4: Replace Halo by its marketing moniker Iris. Requested by Ben.
> > 
> > v5: Switch from info->has*ring to info->ring_mask.
> > 
> > v6: Add 0x16X2 variant (which is newer than this patch)
> > Rename to use new naming scheme (Chris)
> > Remove Simulator PCI ids. These snuck in during rebase (Chris)
> > 
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
> > Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> 
> As a paranoid question, are we sure we have all pci-ids reserved?
> 

I'll never say I am sure. This includes everything in the bspec, and
ones which aren't supposed to ship. I feel bad about adding those, since
in theory - they could end up in some other product. Since we saw it
happen on HSW though, I figured better safe than sorry.

> 
-- 
Ben Widawsky, Intel Open Source Technology Center