Message ID | 1442018576-19981-3-git-send-email-rasesh.mody@qlogic.com (mailing list archive) |
---|---|
State | Changes Requested, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 8169E8D86; Sat, 12 Sep 2015 02:43:35 +0200 (CEST) Received: from mx0b-0016ce01.pphosted.com (mx0b-0016ce01.pphosted.com [67.231.156.153]) by dpdk.org (Postfix) with ESMTP id DDD298E74 for <dev@dpdk.org>; Sat, 12 Sep 2015 02:43:34 +0200 (CEST) Received: from pps.filterd (m0085408.ppops.net [127.0.0.1]) by mx0b-0016ce01.pphosted.com (8.15.0.59/8.15.0.59) with SMTP id t8C0gmU7024740 for <dev@dpdk.org>; Fri, 11 Sep 2015 17:43:34 -0700 Received: from avcashub1.qlogic.com (avcashub2.qlogic.com [198.70.193.116]) by mx0b-0016ce01.pphosted.com with ESMTP id 1wqycnhdd8-1 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NOT) for <dev@dpdk.org>; Fri, 11 Sep 2015 17:43:33 -0700 Received: from avluser05.qlc.com (10.1.113.115) by qlc.com (10.1.4.191) with Microsoft SMTP Server id 14.3.235.1; Fri, 11 Sep 2015 17:43:32 -0700 Received: (from rmody@localhost) by avluser05.qlc.com (8.14.4/8.14.4/Submit) id t8C0hWZF020859; Fri, 11 Sep 2015 17:43:32 -0700 X-Authentication-Warning: avluser05.qlc.com: rmody set sender to rasesh.mody@qlogic.com using -f From: Rasesh Mody <rasesh.mody@qlogic.com> To: <dev@dpdk.org> Date: Fri, 11 Sep 2015 17:42:53 -0700 Message-ID: <1442018576-19981-3-git-send-email-rasesh.mody@qlogic.com> X-Mailer: git-send-email 1.7.10.3 In-Reply-To: <1442018576-19981-1-git-send-email-rasesh.mody@qlogic.com> References: <1442018576-19981-1-git-send-email-rasesh.mody@qlogic.com> MIME-Version: 1.0 Content-Type: text/plain X-Proofpoint-Virus-Version: vendor=nai engine=5700 definitions=7921 signatures=670636 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=1 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1507310000 definitions=main-1509120011 Cc: sony.chacko@qlogic.com Subject: [dpdk-dev] [PATCH 2/5] bnx2x: Fix x86_64-native-linuxapp-clang build error X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Rasesh Mody
Sept. 12, 2015, 12:42 a.m. UTC
From: Harish Patil <harish.patil@qlogic.com> Signed-off-by: Harish Patil <harish.patil@qlogic.com> --- drivers/net/bnx2x/ecore_hsi.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Comments
2015-09-11 17:42, Rasesh Mody: > #define SHMEM_EEE_ADV_STATUS_MASK 0x00f00000 > #define SHMEM_EEE_100M_ADV (1<<0) > - #define SHMEM_EEE_1G_ADV (1<<1) > + #define SHMEM_EEE_1G_ADV (1U<<1) > #define SHMEM_EEE_10G_ADV (1<<2) Why other constants are not changed? Please put at least the build error in the message.
> >2015-09-11 17:42, Rasesh Mody: >> #define SHMEM_EEE_ADV_STATUS_MASK 0x00f00000 >> #define SHMEM_EEE_100M_ADV (1<<0) >> - #define SHMEM_EEE_1G_ADV (1<<1) >> + #define SHMEM_EEE_1G_ADV (1U<<1) >> #define SHMEM_EEE_10G_ADV (1<<2) > >Why other constants are not changed? I only addressed the build error which is against SHMEM_EEE_1G_ADV. Is that okay? > >Please put at least the build error in the message. > Sure, will include the build error and submit new patch. drivers/net/bnx2x/elink.c:10384:41: error: shifting a negative signed value is undefined [-Werror,-Wshift-negative-value] vars->eee_status &= ~SHMEM_EEE_1G_ADV << ~~~~~~~~~~~~~~~~~ ^ 1 error generated. make[6]: *** [elink.o] Error 1 make[5]: *** [bnx2x] Error 2 make[4]: *** [net] Error 2 make[3]: *** [drivers] Error 2 make[2]: *** [all] Error 2 make[1]: *** [x86_64-native-linuxapp-clang_install] Error 2 make: *** [install] Error 2 ________________________________ This message and any attached documents contain information from the sending company or its parent company(s), subsidiaries, divisions or branch offices that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.
2015-10-23 16:36, Harish Patil: > > > >2015-09-11 17:42, Rasesh Mody: > >> #define SHMEM_EEE_ADV_STATUS_MASK 0x00f00000 > >> #define SHMEM_EEE_100M_ADV (1<<0) > >> - #define SHMEM_EEE_1G_ADV (1<<1) > >> + #define SHMEM_EEE_1G_ADV (1U<<1) > >> #define SHMEM_EEE_10G_ADV (1<<2) > > > >Why other constants are not changed? > I only addressed the build error which is against SHMEM_EEE_1G_ADV. > Is that okay? I would say no but you have the ownership on this code. Please think which code quality you are expecting. > This message and any attached documents contain information from the sending company or its parent company(s), subsidiaries, divisions or branch offices that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message. Please remove this footer, irrelevant on a mailing list.
>2015-10-23 16:36, Harish Patil: >> > >> >2015-09-11 17:42, Rasesh Mody: >> >> #define SHMEM_EEE_ADV_STATUS_MASK 0x00f00000 >> >> #define SHMEM_EEE_100M_ADV (1<<0) >> >> - #define SHMEM_EEE_1G_ADV (1<<1) >> >> + #define SHMEM_EEE_1G_ADV (1U<<1) >> >> #define SHMEM_EEE_10G_ADV (1<<2) >> > >> >Why other constants are not changed? >> I only addressed the build error which is against SHMEM_EEE_1G_ADV. >> Is that okay? > >I would say no but you have the ownership on this code. >Please think which code quality you are expecting. [Harish] I checked out internally. This is a header file based on regspecs which would be difficult to change/test everywhere. So I would just have to do this point fix. > >> This message and any attached documents contain information from the >>sending company or its parent company(s), subsidiaries, divisions or >>branch offices that may be confidential. If you are not the intended >>recipient, you may not read, copy, distribute, or use this information. >>If you have received this transmission in error, please notify the >>sender immediately by reply e-mail and then delete this message. > >Please remove this footer, irrelevant on a mailing list. > ________________________________ This message and any attached documents contain information from the sending company or its parent company(s), subsidiaries, divisions or branch offices that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.
diff --git a/drivers/net/bnx2x/ecore_hsi.h b/drivers/net/bnx2x/ecore_hsi.h index a4ed9b5..fe72938 100644 --- a/drivers/net/bnx2x/ecore_hsi.h +++ b/drivers/net/bnx2x/ecore_hsi.h @@ -2529,7 +2529,7 @@ struct shmem2_region { #define SHMEM_EEE_SUPPORTED_SHIFT 16 #define SHMEM_EEE_ADV_STATUS_MASK 0x00f00000 #define SHMEM_EEE_100M_ADV (1<<0) - #define SHMEM_EEE_1G_ADV (1<<1) + #define SHMEM_EEE_1G_ADV (1U<<1) #define SHMEM_EEE_10G_ADV (1<<2) #define SHMEM_EEE_ADV_STATUS_SHIFT 20 #define SHMEM_EEE_LP_ADV_STATUS_MASK 0x0f000000