Jason Glenesk uploaded patch set #4 to this change.

View Change

soc/amd/picasso/acpi: Add MADT entry for NB ioapic, if initialized.

MADT does not contain an entry for the second ioapic described in the IVRS.
Check hob produced by FSP to detect if fsp has initialized second ioapic and
use base address assigned to create MADT ioapic entry.

Cq-Depend: chrome-internal:3247427, chrome-internal:3247427
BUG=b:166519072
TEST=Boot fully to morphius board with and without amd_iommu kernel
parameter. Dump MADT and IVRS tables. Cross check ioapic entries in
MADT against IVRS. Confirm IVRS contains no alignment gaps/corruption.

Signed-off-by: Jason Glenesk <jason.glenesk@amd.corp-partner.google.com>
Change-Id: Ic4a2e9b71dba948e8a4907e5f97131426d8a4a3e
---
M src/soc/amd/picasso/acpi.c
M src/vendorcode/amd/fsp/picasso/FspGuids.h
2 files changed, 30 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/45056/4

To view, visit change 45056. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic4a2e9b71dba948e8a4907e5f97131426d8a4a3e
Gerrit-Change-Number: 45056
Gerrit-PatchSet: 4
Gerrit-Owner: Jason Glenesk <jason.glenesk@gmail.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk@amd.corp-partner.google.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-Reviewer: Raul Rangel <rrangel@chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newpatchset