Hi Andi > Header files need to declare the SPDX under /* ... */ style All files modified in this patch series start with: // SPDX-License-Identifier: MIT rather than: /* SPDX-License-Identifier: MIT */ As a note: :~/linux/drm-tip/drivers/gpu/drm/i915$ rg "// SPDX-License-Identifier: " | wc -l 242 :~/linux/drm-tip/drivers/gpu/drm/i915$ rg "/* SPDX-License-Identifier: " | wc -l 696 are those styles interchangable or is one preferred over the other? I am just curious seeing both are being used. Best Regards, Krzysztof