From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752451AbbF1Ipr (ORCPT ); Sun, 28 Jun 2015 04:45:47 -0400 Received: from mout.web.de ([212.227.17.11]:65315 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752056AbbF1Ipk (ORCPT ); Sun, 28 Jun 2015 04:45:40 -0400 Message-ID: <558FB427.80103@users.sourceforge.net> Date: Sun, 28 Jun 2015 10:45:27 +0200 From: SF Markus Elfring User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Alex Deucher , David Airlie , dri-devel@lists.freedesktop.org CC: LKML , kernel-janitors@vger.kernel.org, Julia Lawall Subject: [PATCH] drm/amdgpu: Delete an unnecessary check before the function call "kfree" References: <5307CAA2.8060406@users.sourceforge.net> <530A086E.8010901@users.sourceforge.net> <530A72AA.3000601@users.sourceforge.net> <530B5FB6.6010207@users.sourceforge.net> <530C5E18.1020800@users.sourceforge.net> <530CD2C4.4050903@users.sourceforge.net> <530CF8FF.8080600@users.sourceforge.net> <530DD06F.4090703@users.sourceforge.net> <5317A59D.4@users.sourceforge.net> In-Reply-To: <5317A59D.4@users.sourceforge.net> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:eI2c7Xq42Nh+QL9AuUnS+aSLB30ZkNvJrRc+aFM0joflUZ2DHFC Xl2sjBA6ucteJtR8PexenZYCc41Kl38MsjNE5HmN9fznZkDIeSeXaWXqn6NZp0Peh4hyNzd /Sugv33jzKr75ha8Ogy44hXs7QAflvfR7vTb86fbyyBHCwwXIyppnzxS7I/IuaHoL5CJubm fZw9s3yT4oIoGANPxaIzQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:C7CE34JXaiM=:s+rPaSeZd1MiyKwDTio5Vg jIQlFi+RZ0Hjo3sCMCC2I82+s3cuxvcxdToVWJG2HiKjvj9vY064eKwNjGIm6DA8cBTwd7gro hKkcpqqJrjAmJcgwdTBUkjs/mKBgiftkaHPdydNwp9kkSyCgDv4Zp8OoMYHDtBPgb1eh40DIU 3a0zVowRpEBWNEgyRuCJAnIMFkQGhyo3PiTKylcLK60qAuxyabR2T13e+Fc42vM0qUf2Tq3Ai u9QjBaEwA5T4afQAuJTm4plDnBrHD+WYxOAEBUgyojzC9WXWrx5FyT8DE5oLVh1O/5qml3iKK qYQWUWWIMsBQyO1ft3uPPpzY3wggJwaYnopvMR/2276k3ASrM+eb5sY/OmOr38eHaLq1V6S3s P5DlH1DRtEH2CbCvIQ4rqJHAt5SmVWMg3vT/pdxGa89/dAXgbB7rK8q/FT29j0RAvgyQzJBx7 QyxJUxFfcr9UUQhXdYTIUhcOj3NdWqhczckJovRmjecDkTcYY6CIu6ugQCduaP6c7CIppniD7 O+GxIQV1pamX+i+AyNNj6F5T8/YMoNLSg5aMslUt8L8cfFlOQhuOJJ6nyiPRmezS5W7dQU8iz VWjORhVl9CKS7kTUwK+qMAjg8SUY3HzvZYiB/s2XCWB3+FEaLH4QhRBwIKYgrEwjqjz3keVJX zk+t94z62bDb71IFVe7F8p6XQcfg7aTAvtCeI+Mr7SpW6q5wGCbL/XieeO4Ddp25IVX0= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sun, 28 Jun 2015 10:27:35 +0200 The kfree() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c index fec487d..a85cd08 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c @@ -1575,8 +1575,7 @@ void amdgpu_device_fini(struct amdgpu_device *adev) amdgpu_fence_driver_fini(adev); amdgpu_fbdev_fini(adev); r = amdgpu_fini(adev); - if (adev->ip_block_enabled) - kfree(adev->ip_block_enabled); + kfree(adev->ip_block_enabled); adev->ip_block_enabled = NULL; adev->accel_working = false; /* free i2c buses */ -- 2.4.4