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;
Date:   Mon, 14 May 2018 14:52:16 +0100 (BST)
From:   Justin Skists <justin.skists@juzza.co.uk>
To:     Dan Carpenter <dan.carpenter@oracle.com>,
        Nathan Chancellor <natechancellor@gmail.com>
Cc:     Todd Kjos <tkjos@android.com>,
        =?UTF-8?Q?Arve_Hj=C3=B8nnev=C3=A5g?= <arve@android.com>,
        devel@driverdev.osuosl.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-kernel@vger.kernel.org, Martijn Coenen <maco@android.com>
Message-ID: <121696744.206121.1526305937025@email.1and1.co.uk>
In-Reply-To: <20180514132945.jofe4rb3ml4muusa@mwanda>
References: <20180507011328.11982-1-natechancellor@gmail.com>
 <20180507011328.11982-2-natechancellor@gmail.com>
 <20180514132945.jofe4rb3ml4muusa@mwanda>
Subject: Re: [PATCH 2/6] staging: android: Clean up license identifiers
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Importance: Medium
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


> On 14 May 2018 at 14:29 Dan Carpenter <dan.carpenter@oracle.com> wrote:
> 
> 
> On Sun, May 06, 2018 at 06:13:24PM -0700, Nathan Chancellor wrote:
> > Add the identifiers when missing and fix the ones already present
> > according to checkpatch.pl.
> > 
> > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
> > ---
> >  drivers/staging/android/ashmem.h        |  6 +-----
> >  drivers/staging/android/uapi/ashmem.h   |  6 +-----
> >  drivers/staging/android/uapi/vsoc_shm.h | 10 +---------
> >  drivers/staging/android/vsoc.c          | 11 +----------
> >  4 files changed, 4 insertions(+), 29 deletions(-)
> > 
> > diff --git a/drivers/staging/android/ashmem.h b/drivers/staging/android/ashmem.h
> > index 60d7208f110a..1a478173cd21 100644
> > --- a/drivers/staging/android/ashmem.h
> > +++ b/drivers/staging/android/ashmem.h
> > @@ -1,13 +1,9 @@
> > -// SPDX-License-Identifier: (GPL-2.0 OR Apache-2.0)
> > +/* SPDX-License-Identifier: GPL-2.0 OR Apache-2.0 */
> 
> 
> // was correct for SPDX headers.

Sorry, header files use the /* ... */ format. :)

https://elixir.bootlin.com/linux/v4.17-rc5/source/Documentation/process/license-rules.rst


Justin.