2 comments:
File src/soc/amd/picasso/acpi.c:
Patch Set #6, Line 50: FSP populates this generates this HOB if it initializes second IOAPIC
What are the conditions under which the second IOAPIC won't be initialized?
With the current patches in flight, it'll be enabled all the time. However I can't think of any reason to not enable it. I'd be fine with calling it a standard behavior of the FSP. Then we'll treat it more like an error if it's not found.
nit: space not required.
I suspect it was meant to be indented, e.g.
ACPI: * MADT
NB_IOAPIC_BASE HOB was not found.
Hmm, well some of our other messages aren't indented. I'd kind of prefer they be somewhat consistent. I'd suggest a follow-on change for formatting, unless there's a new PS for l.50.
To view, visit change 45056. To unsubscribe, or for help writing mail filters, visit settings.