On Wed, Apr 27, 2016 at 06:10:47PM -0700, tom.orourke@xxxxxxxxx wrote: > From: Tom O'Rourke <Tom.O'Rourke@xxxxxxxxx> > > The SLPC interface has changed and could continue to > change. Only GuC versions known to be compatible are > supported here. > > On Skylake, GuC firmware v6 is supported. Other > platforms and versions can be added here later. > > This patch also adds has_slpc to skylake info. > > v2: Move slpc_version_check to intel_guc_ucode_init > v3: fix whitespace (Sagar) > --- > drivers/gpu/drm/i915/i915_drv.c | 1 + > drivers/gpu/drm/i915/intel_guc_loader.c | 14 ++++++++++++++ > 2 files changed, 15 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index d37c0a6..cc22fa0 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -334,6 +334,7 @@ static const struct intel_device_info intel_skylake_info = { > BDW_FEATURES, > .is_skylake = 1, > .gen = 9, > + .has_slpc = 1, > }; > > static const struct intel_device_info intel_skylake_gt3_info = { > diff --git a/drivers/gpu/drm/i915/intel_guc_loader.c b/drivers/gpu/drm/i915/intel_guc_loader.c > index 876e5da..87702cd 100644 > --- a/drivers/gpu/drm/i915/intel_guc_loader.c > +++ b/drivers/gpu/drm/i915/intel_guc_loader.c > @@ -116,6 +116,18 @@ static void direct_interrupts_to_guc(struct drm_i915_private *dev_priv) > I915_WRITE(GUC_WD_VECS_IER, ~irqs); > } > > +static void slpc_version_check(struct drm_device *dev, > + struct intel_guc_fw *guc_fw) > +{ > + struct drm_i915_private *dev_priv = dev->dev_private; > + struct intel_device_info *info; > + > + if (IS_SKYLAKE(dev) && (guc_fw->guc_fw_major_found != 6)) { > + info = (struct intel_device_info *) &dev_priv->info; > + info->has_slpc = 0; dev_priv->info.has_slpc = 0; And pass around dev_priv. Nothing here uses dev. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx