On Thu, 19 Nov 2015, Jani Nikula <jani.nikula@xxxxxxxxx> wrote: > On Thu, 19 Nov 2015, han.lu@xxxxxxxxx wrote: >> From: "Lu, Han" <han.lu@xxxxxxxxx> >> >> Signed-off-by: Lu, Han <han.lu@xxxxxxxxx> >> >> diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c >> index 63d4706..8310bf3 100644 >> --- a/drivers/gpu/drm/i915/intel_audio.c >> +++ b/drivers/gpu/drm/i915/intel_audio.c >> @@ -591,7 +591,8 @@ static void i915_audio_component_codec_wake_override(struct device *dev, >> struct drm_i915_private *dev_priv = dev_to_i915(dev); >> u32 tmp; >> >> - if (!IS_SKYLAKE(dev_priv) && !IS_KABYLAKE(dev_priv)) >> + if (!IS_SKYLAKE(dev_priv) && !IS_BROXTON(dev_priv) && >> + !IS_KABYLAKE(dev_priv)) > > How about if (INTEL_INFO(dev_priv)->gen < 9)? Oh, and I guess we also wonder why this is only relevant for the latest platforms? > > BR, > Jani. > >> return; >> >> /* -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx