Re: [PATCH] sound: soc: Add SPDX ids to many soc files

Piotr MasÅ‚owski <[email protected]> Tue, 17 Feb 2026 19:49:49 +0100
Newsgroups org.kernel.vger.linux-spdx,dev.linux.lists.asahi,org.kernel.vger.linux-kernel,org.kernel.vger.linux-sound
Message-ID <[email protected]>
On Tue Feb 17, 2026 at 7:10 PM CET, Tim Bird wrote:
[...]
> The kernel documentation recommends GPL-2.0 instead of GPL-2.0-only.
> See  https://docs.kernel.org/process/license-rules.html
>
> So my general guideline to myself is to use GPL-2.0 when there's no estab=
lished
> pattern in a directory.  However, when there is already code with an SPDX=
 ID in a directory
> or from an author, I use whichever one has already been assigned (if it's=
 clear).
> This is to keep consistency and honors previous decisions about which of =
the
> equivalent IDs to use. sound/soc/apple/mca.c already had a GPL-2.0-only I=
D,
> and I should have used that for both the Kconfig and the Makefile, by tha=
t guideline.
>
> However, there is also consistency by file type.  There are a few more us=
es of
> GPL-2.0 than GPL-2.0-only in Makefiles throughout the kernel.  But in thi=
s
> case I think that the directory/author consistency should take precedence=
 over
> the file type consistency.
>
> So I messed up in this case.  My recollection is that I did this SDPX app=
lication in
> two different sweeps of the directory, (that is by file type instead of a=
ll files at the
> same time in this dir.) The application of the SPDX ID for the Makefile w=
as done at
> a different time than the application of the SPDX ID for the Kconfig. [..=
.]

Oh wow! That is definitely a lot to remember each time and consider. So
delegating it to an external tool, as you say, is certainly a good idea.

> (and maybe a new tool to check for inconsistencies like this) to avoid th=
is in the future.
>
> The Makefile should have had GPL-2.0-only, to match mca.c and Kconfig.
>
> I will submit a new V2 patch with the Makefile having a GPL-2.0-only ID.
>
> Thanks for catching this!

Only by chance. This time I forgot and did not do my due dilligence
(like at least consulting the documentation, for example).

Best regards,
Piotr Mas=C5=82owski