On Mon, Nov 03, 2014 at 01:45:56PM -0500, Sean Paul wrote: > On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding <thierry.reding at gmail.com> > wrote: > > From: Thierry Reding <treding at nvidia.com> > > > > When the CRTC is enabled, make sure the VBLANK machinery is enabled. > > Failure to do so will cause drm_vblank_get() to not enable the VBLANK on > > the CRTC and VBLANK-synchronized page-flips won't work. > > > > While at it, get rid of the legacy drm_vblank_pre_modeset() and > > drm_vblank_post_modeset() calls that are replaced by drm_vblank_on() > > and drm_vblank_off(). > > > > Reported-by: Alexandre Courbot <acourbot at nvidia.com> > > Signed-off-by: Thierry Reding <treding at nvidia.com> > > --- > > drivers/gpu/drm/tegra/dc.c | 7 ++----- > > 1 file changed, 2 insertions(+), 5 deletions(-) > > > > diff --git a/drivers/gpu/drm/tegra/dc.c b/drivers/gpu/drm/tegra/dc.c > > index 4a015232e2e8..4da366a4d78a 100644 > > --- a/drivers/gpu/drm/tegra/dc.c > > +++ b/drivers/gpu/drm/tegra/dc.c > > @@ -739,7 +739,6 @@ static const struct drm_crtc_funcs tegra_crtc_funcs = { > > > > static void tegra_crtc_disable(struct drm_crtc *crtc) > > { > > - struct tegra_dc *dc = to_tegra_dc(crtc); > > struct drm_device *drm = crtc->dev; > > struct drm_plane *plane; > > > > @@ -755,7 +754,7 @@ static void tegra_crtc_disable(struct drm_crtc *crtc) > > } > > } > > > > - drm_vblank_off(drm, dc->pipe); > > + drm_crtc_vblank_off(crtc); > > } > > > > static bool tegra_crtc_mode_fixup(struct drm_crtc *crtc, > > @@ -844,8 +843,6 @@ static int tegra_crtc_mode_set(struct drm_crtc *crtc, > > u32 value; > > int err; > > > > - drm_vblank_pre_modeset(crtc->dev, dc->pipe); > > - > > Should you replace this with a call to drm_crtc_vblank_off in > prepare()? AFAICT, crtc_funcs->disable() isn't guaranteed to be called > before modeset, and it's unclear to me whether the vblank counter will > be reset in prepare.
Done. Thanks, Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20141104/9650b2f1/attachment.sig>