From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 0BF782A5D for ; Thu, 11 Feb 2016 23:35:53 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga102.jf.intel.com with ESMTP; 11 Feb 2016 14:35:52 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,433,1449561600"; d="scan'208";a="913423148" Received: from kitchen-mobl1.ger.corp.intel.com ([10.252.11.24]) by fmsmga002.fm.intel.com with SMTP; 11 Feb 2016 14:35:49 -0800 Received: by (sSMTP sendmail emulation); Thu, 11 Feb 2016 22:35:49 +0025 Date: Thu, 11 Feb 2016 22:35:48 +0000 From: Bruce Richardson To: Wenzhuo Lu Message-ID: <20160211223548.GA19068@bricha3-MOBL3> References: <1453171693-25571-1-git-send-email-wenzhuo.lu@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1453171693-25571-1-git-send-email-wenzhuo.lu@intel.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 00/12] update ixgbe base driver 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: Thu, 11 Feb 2016 22:35:54 -0000 On Tue, Jan 19, 2016 at 10:48:01AM +0800, Wenzhuo Lu wrote: > *add new devices and mac type > *use PHY token for X550EM_a > *set VF mac address only when acked by PF > *always turn PHY power on when requested > *add definition of a register for mac SGMII busy > *support fuse for both x550em_x V1 and V2 > *remove duplicate setting for fdir no match drop > *consolidate MDIO speed-setting > *use link instead of I2C combined abstraction > > > Wenzhuo Lu (12): > ixgbe/base: update readme > ixgbe/base: add new devices and mac type > ixgeb/base: use PHY token for X550EM_a > ixgbe/base: set VF mac address only when acked by PF > ixgbe/base: always turn PHY power on when requested > ixgbe/base: add definition of a register for mac SGMII busy > ixgbe/base: support fuse for both x550em_x V1 and V2 > ixgbe/base: remove duplicate setting for fdir no match drop > ixgbe/base: consolidate MDIO speed-setting > ixgbe/base: use link instead of I2C combined abstraction > ixgbe: support new devices and mac types > doc: update release note for ixgbe base code update > Hi Wenzhuo Thanks for this patchset. Just some feedback on it. * Can you please take the new features etc. called out in the doc patch and just add the RN update as part of the patch adding the feature itself. Those additions that were part of a previous base code update (presumably part of a previous release) shouldn't go in the latest release notes. There shouldn't need to be a separate doc patch in this set. * The order of the patches could do with being changed. I'd suggest having the readme updated as the last of the base code update series, right before the ixgbe patch. * The "new devices and mac types" patch, only seems to add one new device type, so I'd call it out explicitly in the title. * The X550em_x V1/V2 patch is similarly adding new device support so should go alongside the other new device patch. I'd suggest titling both patches similarly e.g. ixgbe/base: add x550em_a device support ixgbe/base: add x550em_x V2 device support * Some of the titles are a bit long or are not a great summary of the primary purpose of the patch. For example: "always turn PHY power on when requested" might be better titled: "ignore manageability for phy power on". Can you maybe send out a V2 with the above changes taken into account - especially the release note split? Thanks, /Bruce