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 <[email protected]>
Signed-off-by: Qian Cai <[email protected]>
---
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
*
--
1.8.3.1
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 <[email protected]>
> Signed-off-by: Qian Cai <[email protected]>
> ---
>
> 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
> *
Hi Qian,
On 2019/07/12, Qian Cai wrote:
> 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.
>
AFAICT there are a couple of reasons why this has gone unnoticed:
- summary is the pretty ambiguous
- commit does two thigs
Another thing to consider is that this patch touches a single file,
while the exact same issue is also present in many other files.
Quick look at the following lists:
head -n2 drivers/gpu/drm/drm_*[ch] | less
drm_agpsupport.c
drm_dma.c
drm_legacy_misc.c
drm_lock.c
drm_memory.c
drm_scatter.c
drm_vm.c
If you can fixup the s|/**|/*| in the above, I'd gladly merge the patch.
On the topic of SPDX - no objection on my end, but it should be a
separate patch, which replaces the explicit verbose license text with
the tag.
Thanks
Emil