From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH] bnx2x: use the right constant Date: Wed, 06 Jun 2018 14:01:35 -0400 (EDT) Message-ID: <20180606.140135.1343207157647295023.davem@davemloft.net> References: <1528290202-32454-1-git-send-email-Julia.Lawall@lip6.fr> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: ariel.elior@cavium.com, kernel-janitors@vger.kernel.org, everest-linux-l2@cavium.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Julia.Lawall@lip6.fr Return-path: Received: from shards.monkeyblade.net ([184.105.139.130]:45444 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752643AbeFFSBi (ORCPT ); Wed, 6 Jun 2018 14:01:38 -0400 In-Reply-To: <1528290202-32454-1-git-send-email-Julia.Lawall@lip6.fr> Sender: netdev-owner@vger.kernel.org List-ID: From: Julia Lawall Date: Wed, 6 Jun 2018 15:03:22 +0200 > Nearby code that also tests port suggests that the P0 constant should be > used when port is zero. > > The semantic match that finds this problem is as follows: > (http://coccinelle.lip6.fr/) > > // > @@ > expression e,e1; > @@ > > * e ? e1 : e1 > // > > Fixes: 6c3218c6f7e5 ("bnx2x: Adjust ETS to 578xx") > Signed-off-by: Julia Lawall This definitely looks correct. Applied and queued up for -stable, thanks!