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