awokd@danwin1210.me has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36000 )
Change subject: vendorcode/amd/agesa/f12/Proc/CPU: Fix set MTRR6 and MTRR7 to default
......................................................................
Patch Set 2:
Patch Set 2:
I think it is better to test this on hardware
FWIW, this effort was spawned by a mailing list thread wanting AGESA cleanup due to hte quantity of Coverity issues: https://www.mail-archive.com/coreboot@coreboot.org/msg54032.html. I have access to only one of the models in question, so won't be able to test the majority of corrections on actual hardware. How would it be best to proceed?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/36000
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I78061491c16fbb51acfd8347e1eae011ff19a390
Gerrit-Change-Number: 36000
Gerrit-PatchSet: 2
Gerrit-Owner: awokd@danwin1210.me
Gerrit-Reviewer: Jacob Garber
jgarber1@ualberta.ca
Gerrit-Reviewer: Michał Żygowski
michal.zygowski@3mdeb.com
Gerrit-Reviewer: awokd@danwin1210.me
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Arthur Heymans
arthur@aheymans.xyz
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Mon, 14 Oct 2019 17:48:18 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment