Re: [PATCH v2 7/8] drm/vmwgfx: add SPDX idenitifier and clarify license

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dirk,

On 7 May 2018 at 00:16, Dirk Hohndel (VMware) <dirk@xxxxxxxxxxx> wrote:
> These files are licensed under GPL-2.0.
> Removing the MIT boilerplate as that really didn't make any sense for
> those two header files.
>
> Signed-off-by: Dirk Hohndel (VMware) <dirk@xxxxxxxxxxx>
> ---
>  drivers/gpu/drm/vmwgfx/Kconfig                |  1 +
>  .../vmwgfx/device_include/vmware_pack_begin.h | 25 +------------------
>  .../vmwgfx/device_include/vmware_pack_end.h   | 25 +------------------
>  3 files changed, 3 insertions(+), 48 deletions(-)
>
> diff --git a/drivers/gpu/drm/vmwgfx/Kconfig b/drivers/gpu/drm/vmwgfx/Kconfig
> index 8c308dac99c5..6b28a326f8bb 100644
> --- a/drivers/gpu/drm/vmwgfx/Kconfig
> +++ b/drivers/gpu/drm/vmwgfx/Kconfig
> @@ -1,3 +1,4 @@
> +# SPDX-License-Identifier: GPL-2.0
>  config DRM_VMWGFX
>         tristate "DRM driver for VMware Virtual GPU"
>         depends on DRM && PCI && X86 && MMU
> diff --git a/drivers/gpu/drm/vmwgfx/device_include/vmware_pack_begin.h b/drivers/gpu/drm/vmwgfx/device_include/vmware_pack_begin.h
> index 7e7b0ce34aa2..75308bd0d970 100644
> --- a/drivers/gpu/drm/vmwgfx/device_include/vmware_pack_begin.h
> +++ b/drivers/gpu/drm/vmwgfx/device_include/vmware_pack_begin.h
> @@ -1,25 +1,2 @@
> -/**********************************************************
> - * Copyright 2015 VMware, Inc.  All rights reserved.
> - *
> - * Permission is hereby granted, free of charge, to any person
> - * obtaining a copy of this software and associated documentation
> - * files (the "Software"), to deal in the Software without
> - * restriction, including without limitation the rights to use, copy,
> - * modify, merge, publish, distribute, sublicense, and/or sell copies
> - * of the Software, and to permit persons to whom the Software is
> - * furnished to do so, subject to the following conditions:
> - *
> - * The above copyright notice and this permission notice shall be
> - * included in all copies or substantial portions of the Software.
> - *
> - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> - * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> - * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> - * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS
> - * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
> - * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> - * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> - * SOFTWARE.
> - *
> - **********************************************************/
> +/* SPDX-License-Identifier: GPL-2.0 */


IIRC "LFC191: Compliance Basics for Developers" was pretty clear that
the copyright line should stay.
If so, vmware_pack_end.h should also have it's copyright line reinstated.

Mildly related:
This patch adds the one-line license identifier and removes the bulky
license text.
While a later patch, does the former and not the latter. Any reason
why they're not consistent?

HTH
Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux