17 Nov
2020
17 Nov
'20
4:32 p.m.
Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47649 )
Change subject: nb/amd/pi: Remove 00660F01 directory & files ......................................................................
Patch Set 1:
Patch Set 1:
The PI 00630F01 also isn't used by any mb as I can see. Another candidate to remove.
Yes, I agree. The list of files not currently used in any test build is here: https://qa.coreboot.org/job/untested-coreboot-files/86/artifact/untouched_fi...
Some of these probably just need to have config options enabled for the test process, others aren't expected to be used in the build. Some are just left-over pieces from old builds though, and should be removed.
--
To view, visit https://review.coreboot.org/c/coreboot/+/47649
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I75651d2cc53fc5a3cb3233686ad66881d129312d
Gerrit-Change-Number: 47649
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth martinroth@google.com
Gerrit-Reviewer: Marshall Dawson marshalldawson3rd@gmail.com
Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Michał Żygowski michal.zygowski@3mdeb.com
Gerrit-Comment-Date: Tue, 17 Nov 2020 15:32:50 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment