On Thu, Nov 17, 2016 at 10:53:04AM +0200, David Weinehall wrote:
> On Tue, Nov 15, 2016 at 02:21:01PM -0800, Rodrigo Vivi wrote:
> > This commit adding all known marketing names for latest gen9 platforms.
> >
> > Cc: Chris Wilson <ch...@chris-wilson.co.uk>
> > Signed-off-by: Rodrigo Vivi <rodrigo.v...@intel.com>
> > ---
> > README | 2 +-
> > man/intel.man | 2 +-
> > src/intel_module.c | 29 ++++++++++++++++++++++++++++-
> > 3 files changed, 30 insertions(+), 3 deletions(-)
> >
> > diff --git a/README b/README
> > index cf4d88d..e58477d 100644
> > --- a/README
> > +++ b/README
> > @@ -15,7 +15,7 @@ Intel graphics chipsets including:
> > G/Q33,G/Q35,G41,G/Q43,G/GM/Q45
> > PineView-M (Atom N400 series)
> > PineView-D (Atom D400/D500 series)
> > - Intel(R) HD Graphics: 2000-6000,
> > + Intel(R) HD Graphics: 500-6000/P530/P555/P580,
> > Intel(R) Iris(TM) Graphics: 5100/6100, and
> > Intel(R) Iris(TM) Pro Graphics: 5200/6200/P6300.
> >
> > diff --git a/man/intel.man b/man/intel.man
> > index 8da496e..16cc5d9 100644
> > --- a/man/intel.man
> > +++ b/man/intel.man
> > @@ -27,7 +27,7 @@ supports the i810, i810-DC100, i810e, i815, i830M, 845G,
> > 852GM, 855GM,
> > 865G, 915G, 915GM, 945G, 945GM, 965G, 965Q, 946GZ, 965GM, 945GME,
> > G33, Q33, Q35, G35, GM45, G45, Q45, G43, G41 chipsets, Pineview-M in
> > Atom N400 series, Pineview-D in Atom D400/D500 series,
> > -Intel(R) HD Graphics: 2000-6000,
> > +Intel(R) HD Graphics: 500-6000/P530/P555/P580,
> > Intel(R) Iris(TM) Graphics: 5100/6100, and
> > Intel(R) Iris(TM) Pro Graphics: 5200/6200/P6300.
> >
> > diff --git a/src/intel_module.c b/src/intel_module.c
> > index e443c9e..86b4aae 100644
> > --- a/src/intel_module.c
> > +++ b/src/intel_module.c
> > @@ -272,6 +272,33 @@ static const SymTabRec intel_chipsets[] = {
> > {0x22b2, "HD Graphics"},
> > {0x22b3, "HD Graphics"},
> >
> > + /* Skylake */
> > + {0x1902, "HD Graphics 510"},
> > + {0x1906, "HD Graphics 510"},
> > + {0x190B, "HD Graphics 510"},
> > + {0x1912, "HD Graphics 530"},
> > + {0x1916, "HD Graphics 520"},
> > + {0x191B, "HD Graphics 530"},
> > + {0x191D, "HD Graphics P530"},
> > + {0x191E, "HD Graphics 515"},
> > + {0x1921, "HD Graphics 520"},
> > + {0x1926, "Iris Graphics 540"},
> > + {0x1927, "Iris Graphics 550"},
> > + {0x192B, "Iris Graphics 555"},
> > + {0x192D, "Iris Graphics P555"},
> > + {0x1932, "Iris Pro Graphics 580"},
> > + {0x193A, "Iris Pro Graphics P580"},
> > + {0x193B, "Iris Pro Graphics 580"},
> > + {0x193D, "Iris Pro Graphics P580"},
> > +
> > + /* Broxton (Apollolake) */
> > + {0x5A84, "HD Graphics 505"},
> > + {0x5A85, "HD Graphics 500"},
> > +
> > + /* Kabylake */
> > + {0x5916, "HD Graphics 620"},
> > + {0x591E, "HD Graphics 615"},
> > +
> > /* When adding new identifiers, also update:
> > * 1. intel_identify()
> > * 2. man/intel.man
> > @@ -465,7 +492,7 @@ static void intel_identify(int flags)
> > if (unique != stack)
> > free(unique);
> >
> > - xf86Msg(X_INFO, INTEL_NAME ": Driver for Intel(R) HD Graphics:
> > 2000-6000\n");
> > + xf86Msg(X_INFO, INTEL_NAME ": Driver for Intel(R) HD Graphics:
> > 500-6000/P530/P555/P580\n");
> > xf86Msg(X_INFO, INTEL_NAME ": Driver for Intel(R) Iris(TM) Graphics:
> > 5100, 6100\n");
> > xf86Msg(X_INFO, INTEL_NAME ": Driver for Intel(R) Iris(TM) Pro
> > Graphics: 5200, 6200, P6300\n");
>
> You missed the Iris & Iris Pro models.
Thanks
>
> Also, might it make sense to use 5xx, P5xx instead?
What about ont those generic lists we just kill all numbers and let just:
"
Intel(R) HD Graphics.
Intel(R) Iris(TM) Graphics.
Intel(R) Iris(TM) Pro Graphics.
'
?
Chris?
>
>
> Kind regards, David
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
