Patch Set 1:
I feel like two comments right above each other looks kinda ugly. I also don't think we'll be able to maintain it unless you put in a linter check for it.

I think we already have four different ways of doing it in the tree: every combination of one or two comments, spdx first or second.

Do we even need that "part of the coreboot project" line? Does anyone care about that being there? For all I care, I'd be fine with just taking it out completely...

I think it's needed so that if one of our files ends up being used somewhere else, people have a change to look up its history, authors/copyright etc.

(Also, I think ideally the SPDX identifier is supposed to be the first line, so if anything we might want to move all the coreboot project lines below that.)

My motivation for pushing now is to get the SPDX transition done for the release. Can we leave the remaining lint work for later? :-)

View Change

To view, visit change 41065. To unsubscribe, or for help writing mail filters, visit settings.