From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id B8A012BCF for ; Tue, 22 Mar 2016 12:21:29 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP; 22 Mar 2016 04:21:28 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,376,1455004800"; d="scan'208";a="929314476" Received: from irsmsx110.ger.corp.intel.com ([163.33.3.25]) by fmsmga001.fm.intel.com with ESMTP; 22 Mar 2016 04:21:28 -0700 Received: from irsmsx112.ger.corp.intel.com (10.108.20.5) by irsmsx110.ger.corp.intel.com (163.33.3.25) with Microsoft SMTP Server (TLS) id 14.3.248.2; Tue, 22 Mar 2016 11:21:26 +0000 Received: from irsmsx103.ger.corp.intel.com ([169.254.3.239]) by irsmsx112.ger.corp.intel.com ([169.254.1.234]) with mapi id 14.03.0248.002; Tue, 22 Mar 2016 11:21:26 +0000 From: "Richardson, Bruce" To: Rasesh Mody , "thomas.monjalon@6wind.com" CC: "dev@dpdk.org" , "ameen.rahman@qlogic.com" , "harish.patil@qlogic.com" , "sony.chacko@qlogic.com" Thread-Topic: [PATCH v3 00/10] qede: Add qede PMD Thread-Index: AQHRgXnKVvbEhXwJpUq42bcDSp52SZ9lVeBw Date: Tue, 22 Mar 2016 11:21:25 +0000 Message-ID: <59AF69C657FD0841A61C55336867B5B0359F0E37@IRSMSX103.ger.corp.intel.com> References: <1458348805-32648-1-git-send-email-rasesh.mody@qlogic.com> In-Reply-To: <1458348805-32648-1-git-send-email-rasesh.mody@qlogic.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMDNhNjcyNmItMTUwMS00NTBhLWFlMjgtOWIwYWY1NjliNzIyIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IktQVFhkdDB3YmhzakxnK3RtWExVUytmU2U5SHY0YTJ0ZFZYWnN0c2o3YkU9In0= x-ctpclassification: CTP_IC x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v3 00/10] qede: Add qede PMD X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Mar 2016 11:21:30 -0000 I've had a quick scan over this patchset, and as you've probably seen I've = made some public comments on it. General comments on the whole patchset are= : * Please run checkpatch on the patchset and clear up as many issues as you = can. There are a number of typos called out which especially must be fixed.= Both myself and Thomas always run checkpatch against patches before applyi= ng them. [I suggest using Thomas's checkpatches.sh script to do the checks = as it disables many unnecessary warnings from checkpatch] * Please put in commit descriptions for all patches bar those doing trivial= things. The first three patches probably don't need a commit message, but = the rest do. /Bruce > -----Original Message----- > From: Rasesh Mody [mailto:rasesh.mody@qlogic.com] > Sent: Saturday, March 19, 2016 12:53 AM > To: thomas.monjalon@6wind.com; Richardson, Bruce > > Cc: dev@dpdk.org; ameen.rahman@qlogic.com; harish.patil@qlogic.com; > sony.chacko@qlogic.com; Rasesh Mody > Subject: [PATCH v3 00/10] qede: Add qede PMD >=20 > Submitting v3 patch series for QEDE PMD. There is no code change from v2 > series except PMD version change. Earlier we had generated and tested the > v2 series against dpdk tree then latest. >=20 > The v3 series includes: > - Patches generated and tested against latest dpdk-next-net > - Reworked MAINTAINERS patch to make it apply cleanly > - Incorporated Overview.rst update in the documentation patch >=20 > Please Apply. >=20 > Thanks! > Rasesh >=20 > Rasesh Mody (10): > qede: Add maintainers > qede: Add documentation > qede: Add license file > qede: Add base driver > qede: Add core driver > qede: Add L2 support > qede: Add SRIOV support > qede: Add attention support > qede: Add DCBX support > qede: Enable PMD build >=20 > MAINTAINERS | 7 + > config/common_base | 14 + > doc/guides/nics/index.rst | 1 + > doc/guides/nics/overview.rst | 78 +- > doc/guides/nics/qede.rst | 340 + > drivers/net/Makefile | 1 + > drivers/net/qede/LICENSE.qede_pmd | 28 + > drivers/net/qede/Makefile | 95 + > drivers/net/qede/base/bcm_osal.c | 178 + > drivers/net/qede/base/bcm_osal.h | 395 + > drivers/net/qede/base/common_hsi.h | 714 ++ > drivers/net/qede/base/ecore.h | 746 ++ > drivers/net/qede/base/ecore_attn_values.h |13287 > +++++++++++++++++++++++++++ > drivers/net/qede/base/ecore_chain.h | 724 ++ > drivers/net/qede/base/ecore_cxt.c | 1961 ++++ > drivers/net/qede/base/ecore_cxt.h | 157 + > drivers/net/qede/base/ecore_cxt_api.h | 79 + > drivers/net/qede/base/ecore_dcbx.c | 887 ++ > drivers/net/qede/base/ecore_dcbx.h | 55 + > drivers/net/qede/base/ecore_dcbx_api.h | 160 + > drivers/net/qede/base/ecore_dev.c | 3578 ++++++++ > drivers/net/qede/base/ecore_dev_api.h | 497 + > drivers/net/qede/base/ecore_gtt_reg_addr.h | 42 + > drivers/net/qede/base/ecore_gtt_values.h | 33 + > drivers/net/qede/base/ecore_hsi_common.h | 1912 ++++ > drivers/net/qede/base/ecore_hsi_eth.h | 1912 ++++ > drivers/net/qede/base/ecore_hsi_tools.h | 1081 +++ > drivers/net/qede/base/ecore_hw.c | 992 ++ > drivers/net/qede/base/ecore_hw.h | 269 + > drivers/net/qede/base/ecore_hw_defs.h | 49 + > drivers/net/qede/base/ecore_init_fw_funcs.c | 1275 +++ > drivers/net/qede/base/ecore_init_fw_funcs.h | 263 + > drivers/net/qede/base/ecore_init_ops.c | 599 ++ > drivers/net/qede/base/ecore_init_ops.h | 103 + > drivers/net/qede/base/ecore_int.c | 2225 +++++ > drivers/net/qede/base/ecore_int.h | 234 + > drivers/net/qede/base/ecore_int_api.h | 277 + > drivers/net/qede/base/ecore_iov_api.h | 933 ++ > drivers/net/qede/base/ecore_iro.h | 115 + > drivers/net/qede/base/ecore_iro_values.h | 59 + > drivers/net/qede/base/ecore_l2.c | 1798 ++++ > drivers/net/qede/base/ecore_l2.h | 151 + > drivers/net/qede/base/ecore_l2_api.h | 401 + > drivers/net/qede/base/ecore_mcp.c | 1928 ++++ > drivers/net/qede/base/ecore_mcp.h | 304 + > drivers/net/qede/base/ecore_mcp_api.h | 611 ++ > drivers/net/qede/base/ecore_proto_if.h | 28 + > drivers/net/qede/base/ecore_rt_defs.h | 446 + > drivers/net/qede/base/ecore_sp_api.h | 42 + > drivers/net/qede/base/ecore_sp_commands.c | 525 ++ > drivers/net/qede/base/ecore_sp_commands.h | 137 + > drivers/net/qede/base/ecore_spq.c | 944 ++ > drivers/net/qede/base/ecore_spq.h | 284 + > drivers/net/qede/base/ecore_sriov.c | 3422 +++++++ > drivers/net/qede/base/ecore_sriov.h | 390 + > drivers/net/qede/base/ecore_status.h | 30 + > drivers/net/qede/base/ecore_utils.h | 31 + > drivers/net/qede/base/ecore_vf.c | 1322 +++ > drivers/net/qede/base/ecore_vf.h | 415 + > drivers/net/qede/base/ecore_vf_api.h | 186 + > drivers/net/qede/base/ecore_vfpf_if.h | 590 ++ > drivers/net/qede/base/eth_common.h | 526 ++ > drivers/net/qede/base/mcp_public.h | 1195 +++ > drivers/net/qede/base/nvm_cfg.h | 919 ++ > drivers/net/qede/base/reg_addr.h | 1107 +++ > drivers/net/qede/qede_eth_if.c | 456 + > drivers/net/qede/qede_eth_if.h | 176 + > drivers/net/qede/qede_ethdev.c | 986 ++ > drivers/net/qede/qede_ethdev.h | 157 + > drivers/net/qede/qede_if.h | 164 + > drivers/net/qede/qede_logs.h | 93 + > drivers/net/qede/qede_main.c | 601 ++ > drivers/net/qede/qede_rxtx.c | 1364 +++ > drivers/net/qede/qede_rxtx.h | 187 + > drivers/net/qede/rte_pmd_qede_version.map | 4 + > mk/rte.app.mk | 2 + > scripts/test-build.sh | 1 + > 77 files changed, 58239 insertions(+), 39 deletions(-) create mode > 100644 doc/guides/nics/qede.rst create mode 100644 > drivers/net/qede/LICENSE.qede_pmd create mode 100644 > drivers/net/qede/Makefile create mode 100644 > drivers/net/qede/base/bcm_osal.c create mode 100644 > drivers/net/qede/base/bcm_osal.h create mode 100644 > drivers/net/qede/base/common_hsi.h > create mode 100644 drivers/net/qede/base/ecore.h create mode 100644 > drivers/net/qede/base/ecore_attn_values.h > create mode 100644 drivers/net/qede/base/ecore_chain.h > create mode 100644 drivers/net/qede/base/ecore_cxt.c create mode 100644 > drivers/net/qede/base/ecore_cxt.h create mode 100644 > drivers/net/qede/base/ecore_cxt_api.h > create mode 100644 drivers/net/qede/base/ecore_dcbx.c > create mode 100644 drivers/net/qede/base/ecore_dcbx.h > create mode 100644 drivers/net/qede/base/ecore_dcbx_api.h > create mode 100644 drivers/net/qede/base/ecore_dev.c create mode 100644 > drivers/net/qede/base/ecore_dev_api.h > create mode 100644 drivers/net/qede/base/ecore_gtt_reg_addr.h > create mode 100644 drivers/net/qede/base/ecore_gtt_values.h > create mode 100644 drivers/net/qede/base/ecore_hsi_common.h > create mode 100644 drivers/net/qede/base/ecore_hsi_eth.h > create mode 100644 drivers/net/qede/base/ecore_hsi_tools.h > create mode 100644 drivers/net/qede/base/ecore_hw.c create mode 100644 > drivers/net/qede/base/ecore_hw.h create mode 100644 > drivers/net/qede/base/ecore_hw_defs.h > create mode 100644 drivers/net/qede/base/ecore_init_fw_funcs.c > create mode 100644 drivers/net/qede/base/ecore_init_fw_funcs.h > create mode 100644 drivers/net/qede/base/ecore_init_ops.c > create mode 100644 drivers/net/qede/base/ecore_init_ops.h > create mode 100644 drivers/net/qede/base/ecore_int.c create mode 100644 > drivers/net/qede/base/ecore_int.h create mode 100644 > drivers/net/qede/base/ecore_int_api.h > create mode 100644 drivers/net/qede/base/ecore_iov_api.h > create mode 100644 drivers/net/qede/base/ecore_iro.h create mode 100644 > drivers/net/qede/base/ecore_iro_values.h > create mode 100644 drivers/net/qede/base/ecore_l2.c create mode 100644 > drivers/net/qede/base/ecore_l2.h create mode 100644 > drivers/net/qede/base/ecore_l2_api.h > create mode 100644 drivers/net/qede/base/ecore_mcp.c create mode 100644 > drivers/net/qede/base/ecore_mcp.h create mode 100644 > drivers/net/qede/base/ecore_mcp_api.h > create mode 100644 drivers/net/qede/base/ecore_proto_if.h > create mode 100644 drivers/net/qede/base/ecore_rt_defs.h > create mode 100644 drivers/net/qede/base/ecore_sp_api.h > create mode 100644 drivers/net/qede/base/ecore_sp_commands.c > create mode 100644 drivers/net/qede/base/ecore_sp_commands.h > create mode 100644 drivers/net/qede/base/ecore_spq.c create mode 100644 > drivers/net/qede/base/ecore_spq.h create mode 100644 > drivers/net/qede/base/ecore_sriov.c > create mode 100644 drivers/net/qede/base/ecore_sriov.h > create mode 100644 drivers/net/qede/base/ecore_status.h > create mode 100644 drivers/net/qede/base/ecore_utils.h > create mode 100644 drivers/net/qede/base/ecore_vf.c create mode 100644 > drivers/net/qede/base/ecore_vf.h create mode 100644 > drivers/net/qede/base/ecore_vf_api.h > create mode 100644 drivers/net/qede/base/ecore_vfpf_if.h > create mode 100644 drivers/net/qede/base/eth_common.h > create mode 100644 drivers/net/qede/base/mcp_public.h > create mode 100644 drivers/net/qede/base/nvm_cfg.h create mode 100644 > drivers/net/qede/base/reg_addr.h create mode 100644 > drivers/net/qede/qede_eth_if.c create mode 100644 > drivers/net/qede/qede_eth_if.h create mode 100644 > drivers/net/qede/qede_ethdev.c create mode 100644 > drivers/net/qede/qede_ethdev.h create mode 100644 > drivers/net/qede/qede_if.h create mode 100644 > drivers/net/qede/qede_logs.h create mode 100644 > drivers/net/qede/qede_main.c create mode 100644 > drivers/net/qede/qede_rxtx.c create mode 100644 > drivers/net/qede/qede_rxtx.h create mode 100644 > drivers/net/qede/rte_pmd_qede_version.map >=20 > -- > 1.7.10.3