On Thu, 7 Feb 2008, Christoph Hellwig wrote: ... > > +/* > > + * drivers/s390/net/ctcm_dbug.c > > + * > > + * Copyright IBM Corp. 2001, 2007 > > + * Authors: Peter Tiedemann (ptiedem@xxxxxxxxxx) > > + * > > + */ > > Please don't mention filenames in the top of file comments. > On the other hand a little description of what this file > does would be quite useful. (This comment applies to all > files in this patch) Better descriptions, yes. Of course. But are you saying to NOT name the file in its comments? I am surprised. If I am reading correctly, then I disagree. Is the exclusion of the name of the source file from its comments a stylistic decision the other kernel developers have agreed to? Or is it tied to automation? Or have I (hopefully) misunderstood? -- R; <>< () ascii ribbon campaign - against html e-mail /\ www.asciiribbon.org - against proprietary attachments - To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html