On 12/18/2017 10:00 PM, Thomas Monjalon wrote:

18/12/2017 16:52, Bruce Richardson:

quoted

On Mon, Dec 18, 2017 at 06:09:02PM +0530, Hemant Agrawal wrote:

quoted

--- a/GNUmakefile
+++ b/GNUmakefile

@@ -1,33 +1,6 @@
-#   BSD LICENSE
+# SPDX-License-Identifier: BSD-3-Clause
 #
 #   Copyright(c) 2010-2015 Intel Corporation. All rights reserved.

Actually, minor comment, can we remove the blank line between the SPDX
identifier and the copyright? Given that in the majority of cases, we
will only have one line each, having a space between them is a) probably
unnecessary, and b) looks weird IMHO.

I'm currently working on a patchset to update the meson.build files, and
I think having:

# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2017 Intel Corporation

looks better as a header than two one-line paragraphs.

+1, I agree with Bruce

I will fix it.
However,  I will suggest to not enforced it as a policy. Let the 
copyright owners decide, how they want to place the copyright.