drm/i915: Move the vblank pre/post modeset to the common crtc_mode_set.
Signed-off-by: Eric Anholt <eric@anholt.net> Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
This commit is contained in:
committed by
Keith Packard
parent
f564048e20
commit
0b701d27b3
@@ -4543,8 +4543,6 @@ static int i9xx_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
u32 lvds_sync = 0;
|
u32 lvds_sync = 0;
|
||||||
int target_clock;
|
int target_clock;
|
||||||
|
|
||||||
drm_vblank_pre_modeset(dev, pipe);
|
|
||||||
|
|
||||||
list_for_each_entry(encoder, &mode_config->encoder_list, base.head) {
|
list_for_each_entry(encoder, &mode_config->encoder_list, base.head) {
|
||||||
if (encoder->base.crtc != crtc)
|
if (encoder->base.crtc != crtc)
|
||||||
continue;
|
continue;
|
||||||
@@ -4601,7 +4599,6 @@ static int i9xx_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
ok = limit->find_pll(limit, crtc, adjusted_mode->clock, refclk, &clock);
|
ok = limit->find_pll(limit, crtc, adjusted_mode->clock, refclk, &clock);
|
||||||
if (!ok) {
|
if (!ok) {
|
||||||
DRM_ERROR("Couldn't find PLL settings for mode!\n");
|
DRM_ERROR("Couldn't find PLL settings for mode!\n");
|
||||||
drm_vblank_post_modeset(dev, pipe);
|
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -5159,8 +5156,6 @@ static int i9xx_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
|
|
||||||
intel_update_watermarks(dev);
|
intel_update_watermarks(dev);
|
||||||
|
|
||||||
drm_vblank_post_modeset(dev, pipe);
|
|
||||||
|
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -5191,8 +5186,6 @@ static int ironlake_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
u32 lvds_sync = 0;
|
u32 lvds_sync = 0;
|
||||||
int target_clock;
|
int target_clock;
|
||||||
|
|
||||||
drm_vblank_pre_modeset(dev, pipe);
|
|
||||||
|
|
||||||
list_for_each_entry(encoder, &mode_config->encoder_list, base.head) {
|
list_for_each_entry(encoder, &mode_config->encoder_list, base.head) {
|
||||||
if (encoder->base.crtc != crtc)
|
if (encoder->base.crtc != crtc)
|
||||||
continue;
|
continue;
|
||||||
@@ -5249,7 +5242,6 @@ static int ironlake_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
ok = limit->find_pll(limit, crtc, adjusted_mode->clock, refclk, &clock);
|
ok = limit->find_pll(limit, crtc, adjusted_mode->clock, refclk, &clock);
|
||||||
if (!ok) {
|
if (!ok) {
|
||||||
DRM_ERROR("Couldn't find PLL settings for mode!\n");
|
DRM_ERROR("Couldn't find PLL settings for mode!\n");
|
||||||
drm_vblank_post_modeset(dev, pipe);
|
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -5807,8 +5799,6 @@ static int ironlake_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
|
|
||||||
intel_update_watermarks(dev);
|
intel_update_watermarks(dev);
|
||||||
|
|
||||||
drm_vblank_post_modeset(dev, pipe);
|
|
||||||
|
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -5820,11 +5810,17 @@ static int intel_crtc_mode_set(struct drm_crtc *crtc,
|
|||||||
{
|
{
|
||||||
struct drm_device *dev = crtc->dev;
|
struct drm_device *dev = crtc->dev;
|
||||||
struct drm_i915_private *dev_priv = dev->dev_private;
|
struct drm_i915_private *dev_priv = dev->dev_private;
|
||||||
|
struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
|
||||||
|
int pipe = intel_crtc->pipe;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
|
drm_vblank_pre_modeset(dev, pipe);
|
||||||
|
|
||||||
ret = dev_priv->display.crtc_mode_set(crtc, mode, adjusted_mode,
|
ret = dev_priv->display.crtc_mode_set(crtc, mode, adjusted_mode,
|
||||||
x, y, old_fb);
|
x, y, old_fb);
|
||||||
|
|
||||||
|
drm_vblank_post_modeset(dev, pipe);
|
||||||
|
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user