[Intel-gfx] [PATCH i-g-t 1/2] lib/i915_pciids.h: Update to latest version wich includes GLK ids
Petri Latvala
petri.latvala at intel.com
Tue Jan 3 08:57:28 UTC 2017
On Mon, Jan 02, 2017 at 04:33:03PM +0200, Ander Conselvan De Oliveira wrote:
> Hi Petri,
>
> I'm not sure what are the i-g-t rules regarding R-b tags nowadays. Does this require one or can I push it?
The series is
Reviewed-by: Petri Latvala <petri.latvala at intel.com>
Some of the comments got misaligned but that's also present in the kernel.
R-b/A-b for all commits is preferred, for the record.
--
Petri Latvala
>
> Thanks,
> Ander
>
> On Tue, 2016-12-20 at 14:33 +0200, Ander Conselvan de Oliveira wrote:
> > Copy the include/drm/i915_pciids.h file from following kernel commit,
> > which includes Geminilake PCI IDs.
> >
> > commit 8363e3c3947d0e22955f94a6a87e4f17ce5087b4
> > Author: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira at intel.com>
> > Date: Thu Nov 10 17:23:08 2016 +0200
> >
> > drm/i915/glk: Add Geminilake PCI IDs
> >
> > Signed-off-by: Ander Conselvan de Oliveira
> > ---
> > lib/i915_pciids.h | 42 ++++++++++++++++++------------------------
> > lib/intel_device_info.c | 27 +++------------------------
> > overlay/igfx.c | 9 +++------
> > tests/gen7_forcewake_mt.c | 3 +--
> > 4 files changed, 25 insertions(+), 56 deletions(-)
> >
> > diff --git a/lib/i915_pciids.h b/lib/i915_pciids.h
> > index 33466bf..540be9f 100644
> > --- a/lib/i915_pciids.h
> > +++ b/lib/i915_pciids.h
> > @@ -134,7 +134,7 @@
> > #define INTEL_IVB_Q_IDS(info) \
> > INTEL_QUANTA_VGA_DEVICE(info) /* Quanta transcode */
> >
> > -#define INTEL_HSW_D_IDS(info) \
> > +#define INTEL_HSW_IDS(info) \
> > INTEL_VGA_DEVICE(0x0402, info), /* GT1 desktop */ \
> > INTEL_VGA_DEVICE(0x0412, info), /* GT2 desktop */ \
> > INTEL_VGA_DEVICE(0x0422, info), /* GT3 desktop */ \
> > @@ -179,9 +179,7 @@
> > INTEL_VGA_DEVICE(0x0D2B, info), /* CRW GT3 reserved */ \
> > INTEL_VGA_DEVICE(0x0D0E, info), /* CRW GT1 reserved */ \
> > INTEL_VGA_DEVICE(0x0D1E, info), /* CRW GT2 reserved */ \
> > - INTEL_VGA_DEVICE(0x0D2E, info) /* CRW GT3 reserved */ \
> > -
> > -#define INTEL_HSW_M_IDS(info) \
> > + INTEL_VGA_DEVICE(0x0D2E, info), /* CRW GT3 reserved */ \
> > INTEL_VGA_DEVICE(0x0406, info), /* GT1 mobile */ \
> > INTEL_VGA_DEVICE(0x0416, info), /* GT2 mobile */ \
> > INTEL_VGA_DEVICE(0x0426, info), /* GT2 mobile */ \
> > @@ -198,17 +196,15 @@
> > INTEL_VGA_DEVICE(0x0D16, info), /* CRW GT2 mobile */ \
> > INTEL_VGA_DEVICE(0x0D26, info) /* CRW GT3 mobile */
> >
> > -#define INTEL_VLV_M_IDS(info) \
> > +#define INTEL_VLV_IDS(info) \
> > INTEL_VGA_DEVICE(0x0f30, info), \
> > INTEL_VGA_DEVICE(0x0f31, info), \
> > INTEL_VGA_DEVICE(0x0f32, info), \
> > INTEL_VGA_DEVICE(0x0f33, info), \
> > - INTEL_VGA_DEVICE(0x0157, info)
> > -
> > -#define INTEL_VLV_D_IDS(info) \
> > + INTEL_VGA_DEVICE(0x0157, info), \
> > INTEL_VGA_DEVICE(0x0155, info)
> >
> > -#define INTEL_BDW_GT12M_IDS(info) \
> > +#define INTEL_BDW_GT12_IDS(info) \
> > INTEL_VGA_DEVICE(0x1602, info), /* GT1 ULT */ \
> > INTEL_VGA_DEVICE(0x1606, info), /* GT1 ULT */ \
> > INTEL_VGA_DEVICE(0x160B, info), /* GT1 Iris */ \
> > @@ -216,21 +212,17 @@
> > INTEL_VGA_DEVICE(0x1612, info), /* GT2 Halo */ \
> > INTEL_VGA_DEVICE(0x1616, info), /* GT2 ULT */ \
> > INTEL_VGA_DEVICE(0x161B, info), /* GT2 ULT */ \
> > - INTEL_VGA_DEVICE(0x161E, info) /* GT2 ULX */
> > -
> > -#define INTEL_BDW_GT12D_IDS(info) \
> > + INTEL_VGA_DEVICE(0x161E, info), /* GT2 ULX */ \
> > INTEL_VGA_DEVICE(0x160A, info), /* GT1 Server */ \
> > INTEL_VGA_DEVICE(0x160D, info), /* GT1 Workstation */ \
> > INTEL_VGA_DEVICE(0x161A, info), /* GT2 Server */ \
> > INTEL_VGA_DEVICE(0x161D, info) /* GT2 Workstation */
> >
> > -#define INTEL_BDW_GT3M_IDS(info) \
> > +#define INTEL_BDW_GT3_IDS(info) \
> > INTEL_VGA_DEVICE(0x1622, info), /* ULT */ \
> > INTEL_VGA_DEVICE(0x1626, info), /* ULT */ \
> > INTEL_VGA_DEVICE(0x162B, info), /* Iris */ \
> > - INTEL_VGA_DEVICE(0x162E, info) /* ULX */
> > -
> > -#define INTEL_BDW_GT3D_IDS(info) \
> > + INTEL_VGA_DEVICE(0x162E, info), /* ULX */\
> > INTEL_VGA_DEVICE(0x162A, info), /* Server */ \
> > INTEL_VGA_DEVICE(0x162D, info) /* Workstation */
> >
> > @@ -244,14 +236,12 @@
> > INTEL_VGA_DEVICE(0x163A, info), /* Server */ \
> > INTEL_VGA_DEVICE(0x163D, info) /* Workstation */
> >
> > -#define INTEL_BDW_M_IDS(info) \
> > - INTEL_BDW_GT12M_IDS(info), \
> > - INTEL_BDW_GT3M_IDS(info), \
> > - INTEL_BDW_RSVDM_IDS(info)
> > -
> > -#define INTEL_BDW_D_IDS(info) \
> > - INTEL_BDW_GT12D_IDS(info), \
> > - INTEL_BDW_GT3D_IDS(info), \
> > +#define INTEL_BDW_IDS(info) \
> > + INTEL_BDW_GT12_IDS(info), \
> > + INTEL_BDW_GT3_IDS(info), \
> > + INTEL_BDW_RSVDM_IDS(info), \
> > + INTEL_BDW_GT12_IDS(info), \
> > + INTEL_BDW_GT3_IDS(info), \
> > INTEL_BDW_RSVDD_IDS(info)
> >
> > #define INTEL_CHV_IDS(info) \
> > @@ -302,6 +292,10 @@
> > INTEL_VGA_DEVICE(0x5A84, info), /* APL HD Graphics 505 */ \
> > INTEL_VGA_DEVICE(0x5A85, info) /* APL HD Graphics 500 */
> >
> > +#define INTEL_GLK_IDS(info) \
> > + INTEL_VGA_DEVICE(0x3184, info), \
> > + INTEL_VGA_DEVICE(0x3185, info)
> > +
> > #define INTEL_KBL_GT1_IDS(info) \
> > INTEL_VGA_DEVICE(0x5913, info), /* ULT GT1.5 */ \
> > INTEL_VGA_DEVICE(0x5915, info), /* ULX GT1.5 */ \
> > diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> > index 5aab684..5805b5e 100644
> > --- a/lib/intel_device_info.c
> > +++ b/lib/intel_device_info.c
> > @@ -138,36 +138,18 @@ static const struct intel_device_info intel_valleyview_info = {
> > .is_valleyview = true,
> > .codename = "valleyview"
> > };
> > -static const struct intel_device_info intel_valleyview_m_info = {
> > - .gen = BIT(6),
> > - .is_mobile = true,
> > - .is_valleyview = true,
> > - .codename = "valleyview"
> > -};
> >
> > static const struct intel_device_info intel_haswell_info = {
> > .gen = BIT(6),
> > .is_haswell = true,
> > .codename = "haswell"
> > };
> > -static const struct intel_device_info intel_haswell_m_info = {
> > - .gen = BIT(6),
> > - .is_mobile = true,
> > - .is_haswell = true,
> > - .codename = "haswell"
> > -};
> >
> > static const struct intel_device_info intel_broadwell_info = {
> > .gen = BIT(7),
> > .is_broadwell = true,
> > .codename = "broadwell"
> > };
> > -static const struct intel_device_info intel_broadwell_m_info = {
> > - .gen = BIT(7),
> > - .is_mobile = true,
> > - .is_broadwell = true,
> > - .codename = "broadwell"
> > -};
> >
> > static const struct intel_device_info intel_cherryview_info = {
> > .gen = BIT(7),
> > @@ -222,14 +204,11 @@ static const struct pci_id_match intel_device_match[] = {
> > INTEL_IVB_D_IDS(&intel_ivybridge_info),
> > INTEL_IVB_M_IDS(&intel_ivybridge_m_info),
> >
> > - INTEL_HSW_D_IDS(&intel_haswell_info),
> > - INTEL_HSW_M_IDS(&intel_haswell_m_info),
> > + INTEL_HSW_IDS(&intel_haswell_info),
> >
> > - INTEL_VLV_D_IDS(&intel_valleyview_info),
> > - INTEL_VLV_M_IDS(&intel_valleyview_m_info),
> > + INTEL_VLV_IDS(&intel_valleyview_info),
> >
> > - INTEL_BDW_D_IDS(&intel_broadwell_info),
> > - INTEL_BDW_M_IDS(&intel_broadwell_m_info),
> > + INTEL_BDW_IDS(&intel_broadwell_info),
> >
> > INTEL_CHV_IDS(&intel_cherryview_info),
> >
> > diff --git a/overlay/igfx.c b/overlay/igfx.c
> > index 4a9181c..6d82bc8 100644
> > --- a/overlay/igfx.c
> > +++ b/overlay/igfx.c
> > @@ -145,14 +145,11 @@ static const struct pci_id_match match[] = {
> > INTEL_IVB_D_IDS(&ivybridge_info),
> > INTEL_IVB_M_IDS(&ivybridge_info),
> >
> > - INTEL_HSW_D_IDS(&haswell_info),
> > - INTEL_HSW_M_IDS(&haswell_info),
> > + INTEL_HSW_IDS(&haswell_info),
> >
> > - INTEL_VLV_D_IDS(&valleyview_info),
> > - INTEL_VLV_M_IDS(&valleyview_info),
> > + INTEL_VLV_IDS(&valleyview_info),
> >
> > - INTEL_BDW_D_IDS(&broadwell_info),
> > - INTEL_BDW_M_IDS(&broadwell_info),
> > + INTEL_BDW_IDS(&broadwell_info),
> >
> > INTEL_CHV_IDS(&cherryview_info),
> >
> > diff --git a/tests/gen7_forcewake_mt.c b/tests/gen7_forcewake_mt.c
> > index dea1e8a..07320ef 100644
> > --- a/tests/gen7_forcewake_mt.c
> > +++ b/tests/gen7_forcewake_mt.c
> > @@ -53,8 +53,7 @@ static const struct pci_id_match match[] = {
> > INTEL_IVB_D_IDS(NULL),
> > INTEL_IVB_M_IDS(NULL),
> >
> > - INTEL_HSW_D_IDS(NULL),
> > - INTEL_HSW_M_IDS(NULL),
> > + INTEL_HSW_IDS(NULL),
> >
> > { 0, 0, 0 },
> > };
More information about the Intel-gfx
mailing list