On Fri, Apr 03, 2020 at 11:40:03PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrj...@linux.intel.com> > > htotal*vtotal*vrefresh ~= clock. So just say "clock" when we mean it.
Glorious. Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> > > Cc: Linus Walleij <linus.wall...@linaro.org> > Cc: Sam Ravnborg <s...@ravnborg.org> > Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com> > --- > drivers/gpu/drm/mcde/mcde_dsi.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/mcde/mcde_dsi.c b/drivers/gpu/drm/mcde/mcde_dsi.c > index 52031d826f2c..c07a8e273b6f 100644 > --- a/drivers/gpu/drm/mcde/mcde_dsi.c > +++ b/drivers/gpu/drm/mcde/mcde_dsi.c > @@ -537,8 +537,7 @@ static void mcde_dsi_setup_video_mode(struct mcde_dsi *d, > * porches and sync. > */ > /* (ps/s) / (pixels/s) = ps/pixels */ > - pclk = DIV_ROUND_UP_ULL(1000000000000, > - (drm_mode_vrefresh(mode) * mode->htotal * > mode->vtotal)); > + pclk = DIV_ROUND_UP_ULL(1000000000000, mode->clock); > dev_dbg(d->dev, "picoseconds between two pixels: %llu\n", > pclk); > > -- > 2.24.1 > > _______________________________________________ > dri-devel mailing list > dri-de...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
