Re: [PATCH] staging : panel : fix checkpatch warnings (linux-next)

From: Dan Carpenter
Date: Thu Apr 03 2014 - 13:53:27 EST


On Thu, Apr 03, 2014 at 06:59:09PM +0200, armand.bastien@xxxxxxxxxxx wrote:
> This patch fixes "Missing a blank line after declarations" checkpatch warning in panel.c.
>
>
> Signed-off-by: "Bastien Armand <armand.bastien@xxxxxxxxxxx>"

Fix your email client to use the proper From header. The signed-off-by
format is wrong as well (extra quotes). The subject should be:

[PATCH] staging: panel: add blank lines after declarations

The original had extra space characters. Don't put "(linux-next)" in
the permanent log. Be as specific as you can in roughly 52 characters.

Otherwise it looks ok.

regards,
dan carpenter

--
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/