diff options
author | Richard Spiegel <richard.spiegel@silverbackltd.com> | 2018-01-16 13:25:40 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2018-01-22 21:37:33 +0000 |
commit | a318d2812dd2af2087557cd2a8172b38bfc589e9 (patch) | |
tree | ef72631425dbde47b3489318e8df42a13e40bbaa /src/security | |
parent | 2ce90903b0302d3b225973ea65402653a5cf3fb0 (diff) |
AMD/stoneyridge: Fix SATA reset inconsistency
At AGESA AmdInitReset, SATA enable and IDE enable (elements of
FCH_RESET_INTERFACE) are programmed twice (before calling AGESA
for AmdInitReset and from said AGESA function call out), using
different functions with different results. The first would result
in TRUE/FALSE, the second set would result in TRUE/TRUE. Use the
functions of the second set within the first set, and remove them
from the second set.
BUG=b:71754828
TEST=Build kahlle without the change, boot and record output. Rebuild
kahlee with the change, boot and record output. Compare both outputs,
the should be no change except in timing.
Change-Id: I326fcc8801542aa7feef286d02abdfe63354cdd0
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/23295
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/security')
0 files changed, 0 insertions, 0 deletions