Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79852?usp=email )
Change subject: mb/pcengines/apu2/BiosCallOuts: don't have binaryPI clear LPC decodes ......................................................................
mb/pcengines/apu2/BiosCallOuts: don't have binaryPI clear LPC decodes
Tell binaryPI to not disable the LPC decodes for the IO ports used by the serial ports on the Super I/O chip during the AmdInitReset binaryPI entry point. Checked the Stoneyridge binaryPI source code which is closely enough related to be reasonable sure that this option only controls which LPC decode bits get cleared and won't have any other side effects.
TEST=Now the full console output from the APU2 board gets printed on the serial console.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Suggested-by: Krystian Hebel krystian.hebel@3mdeb.com Change-Id: I91ef4423bd7bf6c1d7a175336f0f89479f2cde02 Reviewed-on: https://review.coreboot.org/c/coreboot/+/79852 Reviewed-by: Paul Menzel paulepanter@mailbox.org Reviewed-by: Krystian Hebel krystian.hebel@3mdeb.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/pcengines/apu2/BiosCallOuts.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Krystian Hebel: Looks good to me, approved Paul Menzel: Looks good to me, but someone else must approve build bot (Jenkins): Verified
diff --git a/src/mainboard/pcengines/apu2/BiosCallOuts.c b/src/mainboard/pcengines/apu2/BiosCallOuts.c index efd88d8..c78dda6 100644 --- a/src/mainboard/pcengines/apu2/BiosCallOuts.c +++ b/src/mainboard/pcengines/apu2/BiosCallOuts.c @@ -45,7 +45,7 @@ { printk(BIOS_DEBUG, "Fch OEM config in INIT RESET "); //FchParams_reset->EcChannel0 = TRUE; /* logical devicd 3 */ - FchParams->LegacyFree = CONFIG(HUDSON_LEGACY_FREE); + FchParams->LegacyFree = 0; /* don't clear LPC IO decodes for serial console */ FchParams->FchReset.SataEnable = hudson_sata_enable(); FchParams->FchReset.IdeEnable = hudson_ide_enable(); FchParams->FchReset.Xhci0Enable = CONFIG(HUDSON_XHCI_ENABLE);