drm/i915: Add i810/i815 pci-ids for completeness

Submitted by Chris Wilson on March 13, 2017, 11:28 a.m.

Details

Message ID 20170313112810.4202-1-chris@chris-wilson.co.uk
State Accepted
Commit 92a0256e9acf20ee3f0add61b5aa774bfdf214cc
Headers show
Series "drm/i915: Add i810/i815 pci-ids for completeness" ( rev: 1 ) in Intel GFX

Not browsing as part of any series.

Commit Message

Chris Wilson March 13, 2017, 11:28 a.m.
To improve our historical record and to simplify userspace that wants to
include i915_pciids.h as its canonical breakdown of PCI IDs and their
respective generations, include the gen1 ids for i810 and i815.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 include/drm/i915_pciids.h | 8 ++++++++
 1 file changed, 8 insertions(+)

Patch hide | download patch | download mbox

diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
index 466c71592a6f..27e0dbaa6c0e 100644
--- a/include/drm/i915_pciids.h
+++ b/include/drm/i915_pciids.h
@@ -47,6 +47,14 @@ 
 	0x030000, 0xff0000,			\
 	(unsigned long) info }
 
+#define INTEL_I810_IDS(info)					\
+	INTEL_VGA_DEVICE(0x7121, info), /* I810 */		\
+	INTEL_VGA_DEVICE(0x7123, info), /* I810_DC100 */	\
+	INTEL_VGA_DEVICE(0x7125, info)  /* I810_E */
+
+#define INTEL_I815_IDS(info)					\
+	INTEL_VGA_DEVICE(0x1132, info)  /* I815*/
+
 #define INTEL_I830_IDS(info)				\
 	INTEL_VGA_DEVICE(0x3577, info)
 

Comments

On Mon, 13 Mar 2017, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> To improve our historical record and to simplify userspace that wants to
> include i915_pciids.h as its canonical breakdown of PCI IDs and their
> respective generations, include the gen1 ids for i810 and i815.

Is this how you start sneaking in support for them in i915? ;)

Acked-by: Jani Nikula <jani.nikula@intel.com>


>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  include/drm/i915_pciids.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h
> index 466c71592a6f..27e0dbaa6c0e 100644
> --- a/include/drm/i915_pciids.h
> +++ b/include/drm/i915_pciids.h
> @@ -47,6 +47,14 @@
>  	0x030000, 0xff0000,			\
>  	(unsigned long) info }
>  
> +#define INTEL_I810_IDS(info)					\
> +	INTEL_VGA_DEVICE(0x7121, info), /* I810 */		\
> +	INTEL_VGA_DEVICE(0x7123, info), /* I810_DC100 */	\
> +	INTEL_VGA_DEVICE(0x7125, info)  /* I810_E */
> +
> +#define INTEL_I815_IDS(info)					\
> +	INTEL_VGA_DEVICE(0x1132, info)  /* I815*/
> +
>  #define INTEL_I830_IDS(info)				\
>  	INTEL_VGA_DEVICE(0x3577, info)
On Mon, Mar 13, 2017 at 03:04:21PM +0200, Jani Nikula wrote:
> On Mon, 13 Mar 2017, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> > To improve our historical record and to simplify userspace that wants to
> > include i915_pciids.h as its canonical breakdown of PCI IDs and their
> > respective generations, include the gen1 ids for i810 and i815.
> 
> Is this how you start sneaking in support for them in i915? ;)

Sssh. :p
-Chris
On Mon, Mar 13, 2017 at 03:04:21PM +0200, Jani Nikula wrote:
> On Mon, 13 Mar 2017, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> > To improve our historical record and to simplify userspace that wants to
> > include i915_pciids.h as its canonical breakdown of PCI IDs and their
> > respective generations, include the gen1 ids for i810 and i815.
> 
> Is this how you start sneaking in support for them in i915? ;)
> 
> Acked-by: Jani Nikula <jani.nikula@intel.com>

Ok, pulled the trigger on this so I can use the single set of i915
pciids from userspace.
-Chris