On Mon, Sep 22, 2014 at 07:23:09PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan <[email protected]> > > Now that universal planes are in place we don't need this plane unref on > failures. > > Suggested-by: Ville Syrjälä <[email protected]> > Signed-off-by: Gustavo Padovan <[email protected]> > --- > drivers/gpu/drm/i915/intel_display.c | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index bef37dc..2ef1836 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -8362,8 +8362,7 @@ static bool cursor_size_ok(struct drm_device *dev, > /* > * intel_crtc_cursor_set_obj - Set cursor to specified GEM object > * > - * Note that the object's reference will be consumed if the update fails. If > - * the update succeeds, the reference of the old object (if any) will be > + * If the update succeeds, the reference of the old object (if any) will be > * consumed.
The rest of this comment should be removed as well. With universal planes the fb holds the gem obj reference so we don't frob with gem obj references at all here. With that fixed this patch is Reviewed-by: Ville Syrjälä <[email protected]> > */ > static int intel_crtc_cursor_set_obj(struct drm_crtc *crtc, > @@ -8395,8 +8394,7 @@ static int intel_crtc_cursor_set_obj(struct drm_crtc > *crtc, > stride = roundup_pow_of_two(width) * 4; > if (obj->base.size < stride * height) { > DRM_DEBUG_KMS("buffer is too small\n"); > - ret = -ENOMEM; > - goto fail; > + return -ENOMEM; > } > > /* we only need to pin inside GTT if cursor is non-phy */ > @@ -8485,8 +8483,6 @@ fail_unpin: > i915_gem_object_unpin_from_display_plane(obj); > fail_locked: > mutex_unlock(&dev->struct_mutex); > -fail: > - drm_gem_object_unreference_unlocked(&obj->base); > return ret; > } > > -- > 1.9.3 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
