[Intel-gfx] [PATCH 1/3] drm/i915: Show the computed dotclock in the debug logs

Chris Wilson chris at chris-wilson.co.uk
Sun Jun 9 22:58:50 CEST 2013


On Sun, Jun 09, 2013 at 09:51:28PM +0200, Daniel Vetter wrote:
> On Sun, Jun 9, 2013 at 7:23 PM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> > Reviewing the dotclock computation from the refclk and dividers is a
> > very useful step in manually checking the register configuration.
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > ---
> >  drivers/gpu/drm/i915/intel_display.c |   23 +++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> > index e2e2173..1333a56 100644
> > --- a/drivers/gpu/drm/i915/intel_display.c
> > +++ b/drivers/gpu/drm/i915/intel_display.c
> > @@ -4844,6 +4844,19 @@ static void i9xx_set_pipeconf(struct intel_crtc *intel_crtc)
> >         POSTING_READ(PIPECONF(intel_crtc->pipe));
> >  }
> >
> > +static u64 compute_dotclock(int refclk, const intel_clock_t *clock)
> > +{
> > +       u64 dotclock;
> > +
> > +       dotclock = refclk;
> > +       dotclock *= 5 * (clock->m1 + 2) + clock->m2 + 2;
> > +       do_div(dotclock, 2+clock->n);
> > +       do_div(dotclock, clock->p1);
> > +       do_div(dotclock, clock->p2);
> > +
> > +       return dotclock;
> > +}
> 
> Why is intel_clock (and pineview_clock fwiw) not good enough?

Ignorance. I was just throwing together something to check the values.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list