From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 0227F282 for ; Fri, 9 Dec 2016 17:08:08 +0100 (CET) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP; 09 Dec 2016 08:08:07 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,324,1477983600"; d="scan'208";a="200879393" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.29]) ([10.237.220.29]) by fmsmga004.fm.intel.com with ESMTP; 09 Dec 2016 08:08:06 -0800 To: Jingjing Wu , dev@dpdk.org References: <1480727953-92137-1-git-send-email-jingjing.wu@intel.com> <1481294364-83505-1-git-send-email-jingjing.wu@intel.com> Cc: helin.zhang@intel.com From: Ferruh Yigit Message-ID: <47da5edd-488b-49d8-6067-bd42ea6304f7@intel.com> Date: Fri, 9 Dec 2016 16:08:06 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <1481294364-83505-1-git-send-email-jingjing.wu@intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 00/31] net/i40e: base code update X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 09 Dec 2016 16:08:09 -0000 On 12/9/2016 2:38 PM, Jingjing Wu wrote: > i40e base code upate. The main changes are: > - add clause22 and clause45 implementation for PHY registers accessing > - replace existing legacy memcpy() calls with i40e_memcpy() calls. > - use BIT() macro instead of bit fields > - add clear all WoL filters implementation > - add ERROR state for NVM update state machine > - add broadcast promiscuous control per VLAN > - remove unused X722_SUPPORT and I40E_NDIS_SUPPORT MARCOs > > v2 changes: > - comments rework > - complie issue fix > - rebase to dpdk-next-net > > Jingjing Wu (31): > net/i40e/base: add encap csum VF offload flag > net/i40e/base: preserve extended PHY type field > net/i40e/base: remove unnecessary code > net/i40e/base: fix bit test mask > net/i40e/base: group base mode VF offload flags > net/i40e/base: fix long link down notification time > net/i40e/base: add media type detection for 25G link > net/i40e/base: add clause22 and clause45 implementation > net/i40e/base: add bus number info > net/i40e/base: add protocols when discover capabilities > net/i40e/base: pass unknown PHY type for unknown PHYs > net/i40e/base: replace memcpy > net/i40e/base: deprecating unused macro > net/i40e/base: remove FPK HyperV VF device ID > net/i40e/base: add FEC bits to PHY capabilities > net/i40e/base: use BIT() macro instead of bit fields > net/i40e/base: adjust 25G PHY type values > net/i40e/base: implement clear all WoL filters > net/i40e/base: implement set VSI full promisc mode > net/i40e/base: add defines for new aq command > net/i40e/base: save link FEC info from link up event > net/i40e/base: acquire NVM lock before reads on all devices > net/i40e/base: change shift values to hex > net/i40e/base: comment that udp port must be in Host order > net/i40e/base: remove duplicate definitions > net/i40e/base: add ERROR state for NVM update state machine > net/i40e/base: add broadcast promiscuous control per VLAN > net/i40e/base: avoid division by zero > net/i40e/base: fix byte order > net/i40e/base: remove unused macro > net/i40e: remove unused macro from PMD > Overall patches looks good, thanks. But according comment logs, following patches can be for fixing something broken in the existing code, can you please check, if so can you please update commit logs according [1]? [PATCH v2 02/31] net/i40e/base: preserve extended PHY type field [PATCH v2 11/31] net/i40e/base: pass unknown PHY type for unknown PHYs [PATCH v2 20/31] net/i40e/base: add defines for new aq command [PATCH v2 22/31] net/i40e/base: acquire NVM lock before reads on all devices [PATCH v2 28/31] net/i40e/base: avoid division by zero Thanks, ferruh [1] Updating patch title to point the fix, adding fixes tag and CC: stable tree.