From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757320AbcK3Nts (ORCPT ); Wed, 30 Nov 2016 08:49:48 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:52480 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757200AbcK3Nte (ORCPT ); Wed, 30 Nov 2016 08:49:34 -0500 Date: Wed, 30 Nov 2016 14:49:42 +0100 From: "gregkh@linuxfoundation.org" To: Atul Raj Cc: Viresh Kumar , "mathias.nyman@linux.intel.com" , "stern@rowland.harvard.edu" , "baolu.lu@linux.intel.com" , "oneukum@suse.com" , "hansy@nvidia.com" , "alcooperx@gmail.com" , "atulraj.nith@gmail.com" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , Pankaj Singh Subject: Re: Re: [PATCH] usb:hub: readibility and consistency in coding style Message-ID: <20161130134942.GB937@kroah.com> References: <20161130063238epcms5p5b82d469df3d562bdfb1b6ca53ea1e226@epcms5p5> <20161130115139epcms5p3fb0520cf6cabcf69e470be63463a46d1@epcms5p3> <20161130123521epcms5p3aa0bd43d60fa69c7c7fbc1bbe7fa0099@epcms5p3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161130123521epcms5p3aa0bd43d60fa69c7c7fbc1bbe7fa0099@epcms5p3> User-Agent: Mutt/1.7.1 (2016-10-04) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A: http://en.wikipedia.org/wiki/Top_post Q: Were do I find info about this thing called top-posting? A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Wed, Nov 30, 2016 at 12:35:21PM +0000, Atul Raj wrote: > Hi Greg, > > I am so happy to get a mail from you and I am really sorry Greg that I disappointed you. > I will take utmost care in future. > I will send you correct patch in sometime. > for your ques > "Also, why are you including a ton of people who have nothing to do with > usb patches on this one?" > > I got the name of people from get_maintainer.pl When using get_maintainer.pl on this patch it does not include the dri-devel list, so I don't think you used it correctly. > from now I will include only you. > is it ok?? No, do it correctly. > one more thing in another mail you replied to me as > On Wed, Nov 30, 2016 at 11:40:28AM +0000, Atul Raj wrote: > > changes in v2 > > - added braces as suggested. > > Why is this here? > > > > > drivers/usb/core/hub.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > do you disagree with viresh comment of braces??? I disagree that this is a valid changelog comment, please read Documentation/SubmittingPatches for the correct way to do this. Again, if you don't have experience doing this, please work in the drivers/staging/ directory first, not in the "core" kernel. thanks, greg k-h