Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34122 )
Change subject: mb/lenovo/{t60,z61t}: whitespace and cosmetic ......................................................................
Patch Set 2: Code-Review-1
(1 comment)
If you're going to make these two boards use the variant mechanism, why bother withn this change?
https://review.coreboot.org/c/coreboot/+/34122/2/src/mainboard/lenovo/t60/hd... File src/mainboard/lenovo/t60/hda_verb.c:
https://review.coreboot.org/c/coreboot/+/34122/2/src/mainboard/lenovo/t60/hd... PS2, Line 3: * : * Copyright (C) 2007-2009 coresystems GmbH : * 2012 secunet Security Networks AG : * Is this correct, though? This file was only touched by CB:6840, CB:16529 and CB:26568.
IANAL, but I don't think you can just add copyright lines to a file.