On Fri, Jan 30, 2015 at 03:55:49PM +0530, [email protected] wrote:
> From: Chowdegowda <[email protected]>
> 

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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to