On Mon, 2015-05-04 at 11:15 -0600, Azael Avalos wrote:
> This patch removes some blank lines after comment blocks, capitalizes
> some comments first words and adds a few comments to the beggining
> of some functions.
I think blank lines after comments are generally OK.
This is now inconsistent about
/*
* short comment
*/
and
/* short comment */
Not all function blocks need introductory comments.
> diff --git a/drivers/platform/x86/toshiba_acpi.c
> b/drivers/platform/x86/toshiba_acpi.c
[]
> @@ -78,10 +78,9 @@ MODULE_LICENSE("GPL");
> * SCI stands for "System Configuration Interface" which aim is to
> * conceal differences in hardware between different models.
> */
> -
> #define TCI_WORDS 6
It's sometimes nice to have blank lines after long
block comments.
> -/* operations */
> +/* Operations */
I think these sentence case changes aren't better.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
