[PATCH] INITEST: Documentation: Explains how INITEST works.

[Amit Kumar]

On Fri, Sep 30, 2016, 1:22 PM <Valdis.Kletnieks at vt.edu> wrote:

> On Fri, 30 Sep 2016 07:49:50 +0200, Bjørn Mork said:
>
> > 1) There should be *some* body text describing the patch, even for a very
> >    basic change which is explained in full in subject.  This goes first
> >    in the body, separated from the tags with a single empty line.
>
> And to add to what he said:
>
> Explaining the *why* is better than the *how*.
>
> "Add Documentation/initest.txt" isn't that helpful - we can see from the
> diff
> that's what it does.  "Add documentation explaining how to deal with the
> Frobnizz  3000's wonky interrupts" is a lot better.
>
> Similarly, "fix off-by-one error" is OK - but extending it to "fix
> off-by-one
> error that causes the wireless card to select the wrong channel" is a lot
> better.
>
> Sell us that patch - tell us *why* we want it in the kernel, and why we
> should
> spend time reviewing it....
>
> Please, you should not bother to review this patch seriously. As I have
> written in

this patch that "this patch is a test patch", not real one. It has been
answered by Mr. Bjorn Mork
and you. This patch was merely to check whether my patch format is
acceptable to the kernel
community.
-- 

Regards,
Amit Kumar
Twitter: @freeark1
Only Numbers Can Command.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20160930/550322de/attachment.html