From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 2/9] drm/i915: trivial: remove unneed set to NULL Date: Fri, 29 Aug 2014 10:28:48 +0300 Message-ID: <87mwaniwjz.fsf@intel.com> References: <1409247613-14232-1-git-send-email-gustavo@padovan.org> <1409247613-14232-2-git-send-email-gustavo@padovan.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1409247613-14232-2-git-send-email-gustavo@padovan.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Gustavo Padovan , intel-gfx@lists.freedesktop.org Cc: Gustavo Padovan , dri-devel@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, 28 Aug 2014, Gustavo Padovan wrote: > From: Gustavo Padovan > > At this point of the code the obj var is already NULL, so we don't > need to set it again to NULL. Reviewed-by: Jani Nikula > Signed-off-by: Gustavo Padovan > --- > drivers/gpu/drm/i915/intel_display.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index b2e4eac..05937fe 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -8253,7 +8253,6 @@ static int intel_crtc_cursor_set_obj(struct drm_crtc *crtc, > if (!obj) { > DRM_DEBUG_KMS("cursor off\n"); > addr = 0; > - obj = NULL; > mutex_lock(&dev->struct_mutex); > goto finish; > } > -- > 1.9.3 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- Jani Nikula, Intel Open Source Technology Center