lkml.org 
[lkml]   [2015]   [Jan]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] lustre:include:linux:libcfs:libcfs_private.h:fixes the duplicate case value issue.
On Fri, Jan 30, 2015 at 03:55:49PM +0530, chowdegowda.ms@smartplayin.com wrote:
> From: Chowdegowda <Chowdegowda.MS@smartplayin.com>
>

There are a lot of problems with this patch. First, is "Chowdegowda"
your full name that you would use to sign legal documents?

In your Subject line

Subject: [PATCH] lustre:include:linux:libcfs:libcfs_private.h:fixes the

you are using far too many tags. The path to the file is not necessary
since this can be deduced from the patch. In this case I might use
"staging: lustre:".

If you want a crash course in submitting patches I recommend watching
Greg Kroah-Hartman's presentation on how to write and submit your first
Linux kernel patch [1].

[1]: https://www.youtube.com/watch?v=LLBrBBImJt4

--
- Jeremiah Mahler


\
 
 \ /
  Last update: 2015-01-31 22:01    [W:0.061 / U:1.296 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site