Re: [PATCH v3] gpu/drm_memory: fix a few warnings

From: Qian Cai
Date: Fri Jul 12 2019 - 15:45:41 EST


Maybe one of the non-DRM maintainers (Andrew, Thomas or Greg) who cares a bit
about SPDX can pick this up. It occurs to me none of DRM maintainers cares about
this as there is no feedback from any of them for months since v1.

On Tue, 2019-07-09 at 11:13 -0400, Qian Cai wrote:
> The opening comment mark "/**" is reserved for kernel-doc comments, so
> it will generate a warning with "make W=1".
>
> drivers/gpu/drm/drm_memory.c:2: warning: Cannot understandÂÂ* \file
> drm_memory.c
>
> Also, silence a checkpatch warning by adding a license identfiter where
> it indicates the MIT license further down in the source file.
>
> WARNING: Missing or malformed SPDX-License-Identifier tag in line 1
>
> Fix it by adding the MIT SPDX identifier without touching the boilerplate
> language.
>
> Suggested-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Signed-off-by: Qian Cai <cai@xxxxxx>
> ---
>
> v3: Keep the boilerplate language.
> v2: Remove the redundant description of the license.
>
> Âdrivers/gpu/drm/drm_memory.c | 3 ++-
> Â1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_memory.c b/drivers/gpu/drm/drm_memory.c
> index 132fef8ff1b6..683042c8ee2c 100644
> --- a/drivers/gpu/drm/drm_memory.c
> +++ b/drivers/gpu/drm/drm_memory.c
> @@ -1,4 +1,5 @@
> -/**
> +// SPDX-License-Identifier: MIT
> +/*
> Â * \file drm_memory.c
> Â * Memory management wrappers for DRM
> Â *