Hi Thomas, I did a comprehensive review of this patch (it is huge), using a context of 40 lines instead of the default of 3, looking for files that might have had another licensing information attached to them. I did not find any error. - I found several cases in which the file was based upon another file (according to the comments, e.g drivers/gpio/gpio-max732x.c). The few cases I followed were consistent (the original file was also GPL-2.0-only). I assume the license of the original file does not matter. - A minor issue, some SPDX headers in C files are commented with /* */ (eg sound/soc/nuc900/nuc900-audio.h) and some with // (eg arch/arm/mach-w90x900/clksel.c). Does that matter?