Date: Mon, 4 Dec 2017 22:57:55 +0000 (UTC)
From: Hin-Tak Leung <htl10@users.sourceforge.net>
Reply-To: Hin-Tak Leung <htl10@users.sourceforge.net>
To: Larry Finger <Larry.Finger@lwfinger.net>,
        Kalle Valo <kvalo@codeaurora.org>, <linux-wireless@vger.kernel.org>,
        <netdev@vger.kernel.org>, Colin King <colin.king@canonical.com>
Cc: <kernel-janitors@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Message-ID: <734417985.2789125.1512428275673@mail.yahoo.com>
Subject: Re: [PATCH] rtlwifi: rtl818x: remove redundant check for cck_power
 > 15
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
References: <734417985.2789125.1512428275673.ref@mail.yahoo.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 562
Lines: 20


--------------------------------------------
On Tue, 14/11/17, Colin King <colin.king@canonical.com> wrote:

 
> From: Colin Ian King <colin.king@canonical.com>
 
> cck_poweri cannot be greated than 15 as
> is derived from the bottom 4 bits
> from riv->channels[channel -
> 1].hw_value & 0xf.  Hence the check for it
> being greater than 15 is redundant and
> can be removed.
 
> Detected by CoverityScan, CID#744303
> ("Logically dead code")
 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
 
Acked-by: Hin-Tak Leung <htl10@users.sourceforge.net>