From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Date: Fri, 06 Feb 2015 06:49:01 +0000 Subject: Re: [1/2] cw1200: Delete an unnecessary check before the function call "release_firmware" Message-Id: <20150206064901.1EFAE140CB9@smtp.codeaurora.org> List-Id: In-Reply-To: <54D24D1F.50001@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: SF Markus Elfring Cc: Solomon Peachy , netdev@vger.kernel.org, linux-wireless@vger.kernel.org, LKML , kernel-janitors@vger.kernel.org, Julia Lawall > From: Markus Elfring > Date: Wed, 4 Feb 2015 16:32:15 +0100 > > The release_firmware() 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 Thanks, 2 patches applied to wireless-drivers-next.git: df970d39b90e cw1200: Delete an unnecessary check before the function call "release_firmware" ee4ddad82356 cw1200: Less function calls in cw1200_load_firmware_cw1200() after error detection Kalle Valo