From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965236Ab2CBMdY (ORCPT ); Fri, 2 Mar 2012 07:33:24 -0500 Received: from mga09.intel.com ([134.134.136.24]:19648 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756115Ab2CBMdW (ORCPT ); Fri, 2 Mar 2012 07:33:22 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.67,352,1309762800"; d="scan'208";a="113815062" Date: Fri, 2 Mar 2012 14:32:15 +0200 From: Andrei Emeltchenko To: santosh prasad nayak Cc: marcel@holtmann.org, padovan@profusion.mobi, davem@davemloft.net, linux-bluetooth@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: Resend: [PATCH] Bluetooth: Fix Endian Bug. Message-ID: <20120302123210.GA4741@aemeltch-MOBL1> Mail-Followup-To: Andrei Emeltchenko , santosh prasad nayak , marcel@holtmann.org, padovan@profusion.mobi, davem@davemloft.net, linux-bluetooth@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org References: <1330599561-20392-1-git-send-email-santoshprasadnayak@gmail.com> <20120301123540.GE25627@aemeltch-MOBL1> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Santosh, On Thu, Mar 01, 2012 at 07:34:58PM +0530, santosh prasad nayak wrote: > Andrei, > > I could not get you. > > Do you want me to resend the patch with subject, > "[PATCH] Bluetooth: Fix network to host endian conversion for L2CAP > chan id." ? No, the main concern was your commit message which is changelog and should come after "---" as it is shown in the example below. > > On Thu, Mar 01, 2012 at 04:29:21PM +0530, santosh nayak wrote: > >> From: Santosh Nayak > >> > >> Fix for endian bug. > >> > >> Fix for null dereferenced removed as it is already > >> submitted by Andrei. > >> > >> Signed-off-by: Santosh Nayak > >> --- > >>  net/bluetooth/l2cap_sock.c |    5 +++-- > >>  1 files changed, 3 insertions(+), 2 deletions(-) > > > > We prefer following commit log (this is the example) > > > > <------8<---------------------------------------------------------- > > |  From: Santosh Nayak > > | > > |  Fix network to host endian conversion for L2CAP chan id. > > | > > |  Signed-off-by: Santosh Nayak > > | > > |  --- > > |          * v2 Fix for null dereferenced removed as it is already > > |          submitted by Andrei. > > | > > |    net/bluetooth/l2cap_sock.c |    5 +++-- > > |    1 files changed, 3 insertions(+), 2 deletions(-) > > | > > <------8<---------------------------------------------------------- Best regards Andrei Emeltchenko