On Wed, Oct 07, 2015 at 04:26:03PM +0200, Daniel Vetter wrote: > On Tue, Oct 06, 2015 at 09:26:31AM -0700, Matt Roper wrote: > > intel_mode_from_pipe_config() fills in a mode structure from the CRTC > > state that was read out of the hardware, but does not set the > > .crtc_clock field (it only sets the .clock). This causes the subsequent > > call to drm_calc_timestamping_constants() to complain with messages like > > "*ERROR* crtc 21: Can't calculate constants, dotclock = 0!" Ensuring > > .crtc_clock is set as well eliminates this error. > > > > Signed-off-by: Matt Roper <matthew.d.roper@xxxxxxxxx> > > Is this fixing the bug Paulo reported? Why is he not on CC? Why is there > no citation of the commit which broke this? > -Daniel This isn't related to the watermark issue he reported, so no (unless this is a different issue he brought up elsewhere that I haven't seen). Ville already pointed out that this might already be fixed properly on nightly; I think I might have got my trees mixed up while building, so this might not be necessary at all (I'll need to double check today). But my original thinking (possibly wrong) was that this was just something we never initialized from day 1, so there was no specific commit to cite. Anyway, I think you can probably disregard this patch for now. Matt > > > --- > > drivers/gpu/drm/i915/intel_display.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > > index bbeb6d3..4e481e3 100644 > > --- a/drivers/gpu/drm/i915/intel_display.c > > +++ b/drivers/gpu/drm/i915/intel_display.c > > @@ -7752,6 +7752,7 @@ void intel_mode_from_pipe_config(struct drm_display_mode *mode, > > mode->type = DRM_MODE_TYPE_DRIVER; > > > > mode->clock = pipe_config->base.adjusted_mode.crtc_clock; > > + mode->crtc_clock = pipe_config->base.adjusted_mode.crtc_clock; > > mode->flags |= pipe_config->base.adjusted_mode.flags; > > > > mode->hsync = drm_mode_hsync(mode); > > -- > > 2.1.4 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch -- Matt Roper Graphics Software Engineer IoTG Platform Enabling & Development Intel Corporation (916) 356-2795 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
