On Wed, May 16, 2018 at 08:27:07PM +0200, Sabin Mihai Rapan wrote:
> This patch fixes the checkpatch.pl warning:
>
> WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
>
> Signed-off-by: Sabin Mihai Rapan <sabin.rapan@xxxxxxxxx>
> ---
> .../staging/lustre/include/uapi/linux/lustre/lustre_idl.h | 6 ++++--
> .../staging/lustre/include/uapi/linux/lustre/lustre_user.h | 6 ++++--
> 2 files changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/lustre/include/uapi/linux/lustre/lustre_idl.h b/drivers/staging/lustre/include/uapi/linux/lustre/lustre_idl.h
> index e6016a71ad23..f13138972be2 100644
> --- a/drivers/staging/lustre/include/uapi/linux/lustre/lustre_idl.h
> +++ b/drivers/staging/lustre/include/uapi/linux/lustre/lustre_idl.h
> @@ -1526,9 +1526,11 @@ struct mdt_rec_setattr {
> #define MDS_OPEN_HAS_OBJS 020000000000 /* Just set the EA the obj exist */
> #define MDS_OPEN_NORESTORE 0100000000000ULL /* Do not restore file at open */
> #define MDS_OPEN_NEWSTRIPE 0200000000000ULL /* New stripe needed (restripe or
> - * hsm restore) */
> + * hsm restore)
> + */
> #define MDS_OPEN_VOLATILE 0400000000000ULL /* File is volatile = created
> - unlinked */
> + * unlinked
> + */
Always run checkpatch.pl on your patches so you do not get grumpy
maintainers telling you to run checkpatch.pl on your patches :)
Also, do not create a patch to fix up a checkpatch warning, only to have
it add another one, unless you want to have an eternal number of changes
to make over time...
Please fix up and resend the whole series.
thanks,
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
