Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39136 )
Change subject: mb/google/dedede: Add memory initialization support for dedede ......................................................................
Patch Set 18:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39136/5/src/mainboard/google/dedede... File src/mainboard/google/dedede/romstage.c:
https://review.coreboot.org/c/coreboot/+/39136/5/src/mainboard/google/dedede... PS5, Line 4: * Copyright (C) 2020 Intel Corporation. : * : * This program is free software; you can redistribute it and/or modify : * it under the terms of the GNU General Public License as published by : * the Free Software Foundation; version 2 of the License. : * : * This program is distributed in the hope that it will be useful, : * but WITHOUT ANY WARRANTY; without even the implied warranty of : * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the : * GNU General Public License for more details.
I think this header we are using since a while, is there any change happen lately ? […]
Please refer to 4.11 release notes - https://doc.coreboot.org/releases/coreboot-4.11-relnotes.html#shorter-file-h...
The authors file has been created and Intel Corporation is already part of the coreboot authors. The recommendation is to mention as Copyright 2020 The coreboot project Authors.
SPDX license header is also mentioned as a future effort in the same file.