Re: [PATCH] lustre:include:linux:libcfs:libcfs_private.h:fixes the duplicate case value issue.

From: Jeremiah Mahler
Date: Sat Jan 31 2015 - 15:40:56 EST


On Fri, Jan 30, 2015 at 03:55:49PM +0530, chowdegowda.ms@xxxxxxxxxxxxxxx wrote:
> From: Chowdegowda <Chowdegowda.MS@xxxxxxxxxxxxxxx>
>

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