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:
If after a sufficiently long time no-one seems interested, it might be better to simply drop the platform.
Agree it would be best to test these types of changes on actual hardware, and understand the need to drop specific platforms if they are no longer in use. However, the email thread is talking about dropping AGESA entirely, which I don't want to happen because I use it. If I triage the Coverity issues for platforms I don't have access to with a proposed bug fix and email to the list, will that be good enough to keep AGESA alive, at least for the platforms still in use? I'm hoping the majority of issues won't be actual bug fixes like this one, just code cleanup, which could make it somewhat moot.