From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wes1-so2.wedos.net (wes1-so2.wedos.net [46.28.106.16]) by dpdk.org (Postfix) with ESMTP id 5AB80567C for ; Tue, 8 Dec 2015 22:17:58 +0100 (CET) Received: from jvn (dynamic-109-81-211-171.ipv4.broadband.iol.cz [109.81.211.171]) by wes1-so2.wedos.net (Postfix) with ESMTPSA id 3pFZ9L0H21zTB; Tue, 8 Dec 2015 22:17:58 +0100 (CET) Date: Tue, 8 Dec 2015 22:17:51 +0100 From: Jan Viktorin To: Thomas Monjalon Message-ID: <20151208221751.597585c8@jvn> In-Reply-To: <2962709.1B5sjFu2MI@xps13> References: <1449602993-6047-1-git-send-email-viktorin@rehivetech.com> <12400151.dpjLSRpzoz@xps13> <20151208215504.3c2de0c8@jvn> <2962709.1B5sjFu2MI@xps13> Organization: RehiveTech X-Mailer: Claws Mail 3.13.0 (GTK+ 2.24.28; x86_64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [[RFC PATCH]] lib/ether: fix 16-bit unaligned access 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, 08 Dec 2015 21:17:58 -0000 On Tue, 08 Dec 2015 21:57:33 +0100 Thomas Monjalon wrote: > 2015-12-08 21:55, Jan Viktorin: > > On Tue, 08 Dec 2015 21:30:03 +0100 > > Thomas Monjalon wrote: > > > > > 2015-12-08 20:29, Jan Viktorin: > > > > (I considered to not add the cover-letter as this is just a single small patch. > > > > I hope it does not matter a lot. Is there any convention how to do this?) > > > > > > The main interest of splitting patches or adding a cover letter it to have > > > a place to explain the changes. When you have several changes, they deserve > > > several patches to provide an accurate explanation. In such case, you may > > > need a cover letter to describe the global idea of the series. A cover letter > > > is also helpful for mail threading and acking all the series. > > > When you have only one change, one email is enough. > > > > That's good. Should there be a way how to separate the "inline-cover" > > and the commit? > > Do you mean how add a comment which won't be in the git tree? > You can put whatever between the 3 dashes and the diff. > Exactly. When I was writing this message, I decided to put the "cover" first and the commit log after the 3 dashes. It was more logical to me...