Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Message-ID: <1562960735.8510.30.camel@lca.pw>
Subject: Re: [PATCH v3] gpu/drm_memory: fix a few warnings
From:   Qian Cai <cai@lca.pw>
To:     akpm@linux-foundation.org
Cc:     imirkin@alum.mit.edu, airlied@linux.ie, daniel@ffwll.ch,
        maarten.lankhorst@linux.intel.com, sean@poorly.run,
        joe@perches.com, rfontana@redhat.com, tglx@linutronix.de,
        torvalds@linux-foundation.org, corbet@lwn.net,
        gregkh@linuxfoundation.org, linux-spdx@archiver.kernel.org,
        dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Date:   Fri, 12 Jul 2019 15:45:35 -0400
In-Reply-To: <1562685190-1353-1-git-send-email-cai@lca.pw>
References: <1562685190-1353-1-git-send-email-cai@lca.pw>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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@linutronix.de>
> Signed-off-by: Qian Cai <cai@lca.pw>
> ---
> 
> 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
>   *