From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 1/2] examples/ethtool: remove query of default config Date: Wed, 20 Mar 2019 15:05:17 +0100 Message-ID: <2057184.VXND6Oo4BK@xps> References: <20190313100910.31260-1-thomas@monjalon.net> <21004029.OMB2mdimH3@xps> <9dfea583-50f8-fbbc-c83e-0ae4419dce68@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Rami Rosen , Bruce Richardson , dev@dpdk.org To: Ferruh Yigit Return-path: Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by dpdk.org (Postfix) with ESMTP id 3E3717CDA for ; Wed, 20 Mar 2019 15:05:21 +0100 (CET) In-Reply-To: <9dfea583-50f8-fbbc-c83e-0ae4419dce68@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 20/03/2019 14:56, Ferruh Yigit: > On 3/20/2019 1:51 PM, Thomas Monjalon wrote: > > 20/03/2019 14:46, Ferruh Yigit: > >> Applied to dpdk-next-net/master, thanks. > >> > >> (only this patch merged, not series. Other patch seems waiting for comment.) > > > > Applying half of patchset makes tracking of patches less obvious. > > I prefer we avoid such practice. > > Generally I agree, this case the patches in patchset looks can be independent > and merged one was trivial, I tend to this for similar cases. Even if it is independent (it is not really in this case), it looks weird to see half of patchset pending in patchwork. We may spend some time to understand what happened. > Do you prefer me drop the applied patch? You can keep it now it's applied.