[coreboot-gerrit] Change in coreboot[master]: mb/supermicro/x10slm-f: Fix IPMI KCS

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34905 )

Change subject: mb/supermicro/x10slm-f: Fix IPMI KCS
......................................................................

Patch Set 12:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34905/8/src/mainboard/supermicro/x1... 
File src/mainboard/supermicro/x10slm-f/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/34905/8/src/mainboard/supermicro/x1... 
PS8, Line 5: ## Copyright (C) 2019 Hosting.de GmbH
> Usually we argue that copyright only applies to code and not to "config" files that have a predefine […]

Well, it wouldn't be any problem if there was a Copyright on the original work. Then this Copyright would be copied, too, and another Copyright added. What I actually do not know is, if it is valid to add a Copyright header when there was none before, even when the code/config generally is GPL. TL;DR I really have no clue but I would remove the whole GPL header for the reason Patrick mentioned. AFAIK config can't be licensed at all (I may be wrong here...)