Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47982 )
Change subject: soc/amd: move bootblock inside main SoC directories
......................................................................
Patch Set 1: Code-Review+1
I'm ok with this if it's what you want, but having the bootblock directory is pretty standard. While it might not be required, having the bootblock directory is pretty expected. Is there actually a reason to get rid of the bootblock directory?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/47982
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I78a9ce1cd0d790250a66c82bb1d8aa6c3b4f7162
Gerrit-Change-Number: 47982
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:29:46 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment