On 29 November 2016 at 14:13, <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Looks like we're only initializing dev_priv->atomic_cdclk_freq > at resume and commit times, not at init time. Let's do that as > well. > > We're now hitting the 'WARN_ON(intel_state->cdclk == 0)' in > hsw_compute_linetime_wm() on account of populating > intel_state->cdclk from dev_priv->atomic_cdclk_freq. > Previously we were mispopulating intel_state->cdclk with > dev_priv->cdclk_freq which always had a proper value at init > time and hence the WARN_ON() didn't trigger. > > Cc: stable@xxxxxxxxxxxxxxx > Cc: Matthew Auld <matthew.auld@xxxxxxxxx> > Reported-by: Matthew Auld <matthew.auld@xxxxxxxxx> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98902 > Fixes: e0ca7a6be38c ("drm/i915: Fix cdclk vs. dev_cdclk mess when not recomputing things") > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Tested-by: Matthew Auld <matthew.auld@xxxxxxxxx> Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html