Subject: Re: [PATCH 04/10] PTI Makefile and Kconfig additions.
From: J Freyensee <james_p_freyensee@linux.intel.com>
Reply-To: james_p_freyensee@linux.intel.com
To: Arnd Bergmann <arnd@arndb.de>
Cc: gregkh@suse.de, linux-kernel@vger.kernel.org, suhail.ahmed@intel.com,
        christophe.guerard@intel.com
In-Reply-To: <201103011237.50400.arnd@arndb.de>
References: <james_p_freyensee@linux.intel.com>
	 <201102281031.42402.arnd@arndb.de> <1298915373.3156.34.camel@localhost>
	 <201103011237.50400.arnd@arndb.de>
Content-Type: text/plain; charset="ISO-8859-1"
Date: Tue, 01 Mar 2011 09:37:45 -0800
Message-ID: <1299001065.3156.35.camel@localhost>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1052
Lines: 32

On Tue, 2011-03-01 at 12:37 +0100, Arnd Bergmann wrote:
> On Monday 28 February 2011, J Freyensee wrote:
> > On Mon, 2011-02-28 at 10:31 +0100, Arnd Bergmann wrote:
> >
> > > The Kconfig description is a nice text, how about copying it to the
> > > changeset so that reviewers know what the driver is about?
> > > 
> > 
> > I thought patch 02/10, 'Kernel documentation', would had been first and
> > met this need?  
> 
> Documentation in source files is not the same as documentation in the
> changelog, you need both. The changeset comments are usually just one
> sentence in your series, and a little more would certainly be helpful.
> 

Okay, I can do that.

Thanks,
Jay

> Also, if you have one changeset with the documentation, that does
> not help anyone who looks at another changeset.
> 
> 	Arnd


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/