From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [patch -next] bnxt: fix a condition Date: Sat, 09 Jul 2016 00:02:30 -0400 (EDT) Message-ID: <20160709.000230.1944126559698750228.davem@davemloft.net> References: <20160707082309.GC16742@mwanda> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: michael.chan@broadcom.com, huangjw@broadcom.com, aduyck@mirantis.com, netdev@vger.kernel.org, kernel-janitors@vger.kernel.org To: dan.carpenter@oracle.com Return-path: In-Reply-To: <20160707082309.GC16742@mwanda> Sender: kernel-janitors-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Dan Carpenter Date: Thu, 7 Jul 2016 11:23:09 +0300 > This code generates as static checker warning because htons(ETH_P_IPV6) > is always true. From the context it looks like the && was intended to > be !=. > > Fixes: 94758f8de037 ('bnxt_en: Add GRO logic for BCM5731X chips.') > Signed-off-by: Dan Carpenter Applied.