linux-kernel - Re: [PATCH v14 1/2] drm: add kms driver for loongson display controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <74b273dd-4310-0106-a24c-c1b15db8ead9@xen0n.name>
Date:   Mon, 22 May 2023 16:19:39 +0800
From:   WANG Xuerui <kernel@...0n.name>
To:     Sui Jingfeng <15330273260@....cn>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Maxime Ripard <mripard@...nel.org>,
        Thomas Zimmermann <tzimmermann@...e.de>,
        David Airlie <airlied@...il.com>,
        Daniel Vetter <daniel@...ll.ch>,
        Sui Jingfeng <suijingfeng@...ngson.cn>,
        Li Yi <liyi@...ngson.cn>,
        Sumit Semwal <sumit.semwal@...aro.org>,
        Christian Koenig <christian.koenig@....com>,
        Emil Velikov <emil.l.velikov@...il.com>
Cc:     linaro-mm-sig@...ts.linaro.org, loongson-kernel@...ts.loongnix.cn,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        Javier Martinez Canillas <javierm@...hat.com>,
        Nathan Chancellor <nathan@...nel.org>,
        Liu Peibao <liupeibao@...ngson.cn>, linux-media@...r.kernel.org
Subject: Re: [PATCH v14 1/2] drm: add kms driver for loongson display
 controller

On 2023/5/22 16:14, Sui Jingfeng wrote:
> Hi,
> 
> On 2023/5/21 20:21, WANG Xuerui wrote:
>>> +
>>> +      If in doubt, say "N".
>>> diff --git a/drivers/gpu/drm/loongson/Makefile 
>>> b/drivers/gpu/drm/loongson/Makefile
>>> new file mode 100644
>>> index 000000000000..9158816ece8e
>>> --- /dev/null
>>> +++ b/drivers/gpu/drm/loongson/Makefile
>>> @@ -0,0 +1,20 @@
>>> +# SPDX-License-Identifier: GPL-2.0
>>> +
>>> +loongson-y := \
>>> +    lsdc_benchmark.o \
>>> +    lsdc_crtc.o \
>>> +    lsdc_debugfs.o \
>>> +    lsdc_device.o \
>>> +    lsdc_drv.o \
>>> +    lsdc_gem.o \
>>> +    lsdc_gfxpll.o \
>>> +    lsdc_i2c.o \
>>> +    lsdc_irq.o \
>>> +    lsdc_output_7a1000.o \
>>> +    lsdc_output_7a2000.o \
>>> +    lsdc_plane.o \
>>> +    lsdc_pixpll.o \
>>> +    lsdc_probe.o \
>>> +    lsdc_ttm.o
>>> +
>>> +obj-$(CONFIG_DRM_LOONGSON) += loongson.o
>>> diff --git a/drivers/gpu/drm/loongson/lsdc_benchmark.c 
>>> b/drivers/gpu/drm/loongson/lsdc_benchmark.c
>>> new file mode 100644
>>> index 000000000000..82961531d84c
>>> --- /dev/null
>>> +++ b/drivers/gpu/drm/loongson/lsdc_benchmark.c
>>> @@ -0,0 +1,133 @@
>>> +// SPDX-License-Identifier: GPL-2.0+
>>
>> Is it GPL-2.0, GPL-2.0-only, or GPL-2.0+? Please make sure all license 
>> IDs are consistent. 
> 
> 
> I see drm/vkms is also writing the copyrights similar.
> 
> with "# SPDX-License-Identifier: GPL-2.0-only" in the Makefile,
> 
> while "// SPDX-License-Identifier: GPL-2.0+" in the C source file.
> 
> Sorry, I'm stupid, I can't figure out the difference between them.
> 
> Personally, I really don't care about this as along as checkpatch.pl 
> don't complain.
> 
> I respect the maintainers of DRM, they didn't told me to change it.
> 
> I assume there is no problem.

It's your work after all, so you get to license the work however you 
want (inside the kernel project's licensing requirements, of course), so 
maintainers won't interfere with that.

I'm suggesting the license double-check because the whole driver is one 
piece of work, so it's better to be extra clear (you DO want to make 
sure things are clear when it comes to copyright, compliance & etc.) and 
make it as consistent as possible, but ultimately it's of course down to 
you. I think you may keep things as-is if others don't voice their 
concerns in the coming days.

-- 
WANG "xen0n" Xuerui

Linux/LoongArch mailing list: https://lore.kernel.org/loongarch/