From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 29 Nov 2017 10:22:55 +0100 From: Heiko Carstens Subject: Re: [PATCH v3] s390/virtio: add BSD license to virtio-ccw References: <1511874948-27609-1-git-send-email-mst@redhat.com> <20171128145119.2866d7a7.cohuck@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171128145119.2866d7a7.cohuck@redhat.com> Message-Id: <20171129092255.GB4061@osiris> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: Cornelia Huck Cc: "Michael S. Tsirkin" , linux-kernel@vger.kernel.org, Christian Borntraeger , Martin Schwidefsky , linux-s390@vger.kernel.org, virtualization@lists.linux-foundation.org List-ID: On Tue, Nov 28, 2017 at 02:51:19PM +0100, Cornelia Huck wrote: > On Tue, 28 Nov 2017 15:17:52 +0200 > "Michael S. Tsirkin" wrote: > > > The original intent of the virtio header relicensing > > from 2008 was to make sure anyone can implement compatible > > devices/drivers. The virtio-ccw was omitted by mistake. > > > > We have an ack from the only contributor as well as the > > maintainer from IBM, so it's not too late to fix that. > > > > Make it dual-licensed with GPLv2, as the whole kernel is GPL2. > > > > Acked-by: Christian Borntraeger > > Acked-by: Cornelia Huck > > Signed-off-by: Michael S. Tsirkin > > --- > > arch/s390/include/uapi/asm/virtio-ccw.h | 6 +----- > > arch/s390/kernel/setup.c | 1 - > > 2 files changed, 1 insertion(+), 6 deletions(-) > > > > diff --git a/arch/s390/include/uapi/asm/virtio-ccw.h b/arch/s390/include/uapi/asm/virtio-ccw.h > > index 967aad3..2b605f7 100644 > > --- a/arch/s390/include/uapi/asm/virtio-ccw.h > > +++ b/arch/s390/include/uapi/asm/virtio-ccw.h > > @@ -1,13 +1,9 @@ > > -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > > +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */ > > /* > > * Definitions for virtio-ccw devices. > > * > > * Copyright IBM Corp. 2013 > > * > > - * This program is free software; you can redistribute it and/or modify > > - * it under the terms of the GNU General Public License (version 2 only) > > - * as published by the Free Software Foundation. > > - * > > * Author(s): Cornelia Huck > > */ > > #ifndef __KVM_VIRTIO_CCW_H > > diff --git a/arch/s390/kernel/setup.c b/arch/s390/kernel/setup.c > > index 164a1e1..c022007 100644 > > --- a/arch/s390/kernel/setup.c > > +++ b/arch/s390/kernel/setup.c > > @@ -60,7 +60,6 @@ > > #include > > #include > > #include > > -#include > > #include > > #include > > #include > > Shouldn't the second hunk already be in master? > > Still ack otherwise. The removal of the license text is also already in linux-next, however no need to send an updated version. I'll fix it up ;)