On Fri, May 12, 2023 at 05:06:11PM +0700, Bagas Sanjaya wrote: > I trigger this patch series as a response to Didi's GPL full name fix > patches [1], for which all of them had been NAKed. In many cases, the > appropriate correction is to use SPDX license identifier instead. > > Often, when replacing license notice boilerplates with their equivalent > SPDX identifier, the notice doesn't mention explicit GPL version. Greg > [2] replied this question by falling back to GPL 1.0 (more precisely > GPL 1.0+ in order to be compatible with GPL 2.0 used by Linux kernel), > although there are exceptions (mostly resolved by inferring from > older patches covering similar situation). > > The series covers the same directories touched as Didi's ones, minus > Documentation/ (as should have been inferred by SPDX tags on respective > docs). > I'm glad to take these types of changes through the SPDX tree, but please break them up into smaller changes that show the justification for each type of change in each subsystem, so that we can evaluate them on an individual basis. As you did here, you are lumping things together only by the existance of the file in the tree, not by the logical type of change happening, which isn't ok. Also, you can send them as subsystem-specific series, so as to not have to cross-post all of the changes all over the place. I doubt the drm developers care about ethernet driver license issues :) thanks, greg k-h