From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:54748 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752346AbdCTRQm (ORCPT ); Mon, 20 Mar 2017 13:16:42 -0400 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Subject: Re: ipw2200: remove redundant check of rc < 0 From: Kalle Valo In-Reply-To: <20170227234019.19081-1-colin.king@canonical.com> References: <20170227234019.19081-1-colin.king@canonical.com> To: Colin Ian King Cc: Stanislav Yakovlev , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Message-Id: <20170320171641.6B24460C78@smtp.codeaurora.org> (sfid-20170320_181646_195102_2989FB8E) Date: Mon, 20 Mar 2017 17:16:41 +0000 (UTC) Sender: linux-wireless-owner@vger.kernel.org List-ID: Colin Ian King wrote: > From: Colin Ian King > > The check for rc < 0 is always false so the check is redundant > and can be removed. > > Detected with CoverityScan, CID#101143 ("Logically dead code") > > Signed-off-by: Colin Ian King Patch applied to wireless-drivers-next.git, thanks. 889cdff72b85 ipw2200: remove redundant check of rc < 0 -- https://patchwork.kernel.org/patch/9594495/ Documentation about submitting wireless patches and checking status from patchwork: https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches