2 comments:
File src/soc/amd/picasso/acpi.c:
Patch Set #6, Line 50: populates this
To Marshall's comment - is there any power implication by keeping it on all the time? I'm assuming n […]
Eric, I have a question in re. power. I assume it's negligible.
Patch Set #6, Line 50: FSP populates this generates this HOB if it initializes second IOAPIC
I'd be fine with passing the address in and forcing FSP to obey the UPD. In that case, I'd probably take FSP's default back to disabled and make coreboot enable it intentionally.
Also, probably we want to also declare it as reserved resource here: https://review.coreboot.org/cgit/coreboot.git/tree/src/soc/amd/common/block/lpc/lpc.c?id=refs/heads/master#n125?
Hmm, I thought we were already reserving a big chunk of space that should've covered it. Maybe I'm thinking of how Stoney Ridge though. You wouldn't put it in picasso/root_complex.c?
To view, visit change 45056. To unsubscribe, or for help writing mail filters, visit settings.