mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-12-22 18:44:44 +08:00
drm/i915: Kill a goto from sprite disable code
Let's not use goto when a simple if suffices. This is not error handling code or anything, so the goto looks out of place. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
parent
82284b6bec
commit
5f3fb46bbd
@ -868,7 +868,6 @@ intel_disable_plane(struct drm_plane *plane)
|
||||
struct drm_device *dev = plane->dev;
|
||||
struct intel_plane *intel_plane = to_intel_plane(plane);
|
||||
struct intel_crtc *intel_crtc;
|
||||
int ret = 0;
|
||||
|
||||
if (!plane->fb)
|
||||
return 0;
|
||||
@ -883,20 +882,18 @@ intel_disable_plane(struct drm_plane *plane)
|
||||
intel_plane->disable_plane(plane, plane->crtc);
|
||||
}
|
||||
|
||||
if (!intel_plane->obj)
|
||||
goto out;
|
||||
if (intel_plane->obj) {
|
||||
if (intel_crtc->active)
|
||||
intel_wait_for_vblank(dev, intel_plane->pipe);
|
||||
|
||||
if (intel_crtc->active)
|
||||
intel_wait_for_vblank(dev, intel_plane->pipe);
|
||||
mutex_lock(&dev->struct_mutex);
|
||||
intel_unpin_fb_obj(intel_plane->obj);
|
||||
mutex_unlock(&dev->struct_mutex);
|
||||
|
||||
mutex_lock(&dev->struct_mutex);
|
||||
intel_unpin_fb_obj(intel_plane->obj);
|
||||
mutex_unlock(&dev->struct_mutex);
|
||||
intel_plane->obj = NULL;
|
||||
}
|
||||
|
||||
intel_plane->obj = NULL;
|
||||
out:
|
||||
|
||||
return ret;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static void intel_destroy_plane(struct drm_plane *plane)
|
||||
|
Loading…
Reference in New Issue
Block a user