On Fri, Jun 05, 2015 at 03:24:45PM +0300, Ville Syrjälä wrote: > On Thu, Jun 04, 2015 at 06:21:35PM +0100, Damien Lespiau wrote: > > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > > --- > > drivers/gpu/drm/i915/intel_display.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > > index a232dc9..a018465 100644 > > --- a/drivers/gpu/drm/i915/intel_display.c > > +++ b/drivers/gpu/drm/i915/intel_display.c > > @@ -5627,6 +5627,10 @@ static void skl_set_cdclk(struct drm_i915_private *dev_priv, unsigned int freq) > > mutex_unlock(&dev_priv->rps.hw_lock); > > > > intel_update_cdclk(dev); > > + > > + WARN(freq != dev_priv->cdclk_freq, > > + "cdclk requested %d kHz but got %d kHz\n", > > + freq, dev_priv->cdclk_freq); > > } > > Could you add this to all the set_cdclk() functions? Maybe > intel_check_cdclk() or something. I was thinking that we should probably introduce a low level set_core_display_clock() vfunc and a intel_set_core_display_clock() wrapper were we'd put the common code (updating the cached value, that WARN(), ...) Thoughts? -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
