diff options
author | Karthikeyan Ramasubramanian <kramasub@google.com> | 2023-07-21 18:05:25 -0600 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2023-08-10 16:18:18 +0000 |
commit | 5eb77928e48a37d04b4dc9f60c85e015c6a8ec2f (patch) | |
tree | 0f06751aed4541056f2c9d10b731dd1f937edec3 /src/superio/nuvoton/common | |
parent | 0154fa57ee449c97123e45132fd80bdadcf6df5f (diff) |
soc/amd/common/psp_verstage: Enable Legacy IO only on older SoCs
With reference to the Picasso PPR 55570 Rev 3.18, LegacyIoEn bit is 0 on
reset and setting it will enable the decoding of the following legacy IO
ports:
0x20, 0x21, 0xA0, 0xA1 (PIC);
0x40, 0x41, 0x42, 0x43, 0x61 (8254 timer);
0x70, 0x71, 0x72, 0x73 (RTC);
0x92.
Verstage does not use those legacy IO ports. Also newer SoCs like
Phoenix do not support Legacy I/O registers to access Power Management
registers and accessing them from PSP verstage causes a hang. Hence
enable legacy IO only on platforms that support it.
BUG=b::284984667
TEST=Build Myst BIOS image with PSP Verstage. Boot to OS successfully
with PSP verstage.
Change-Id: I5e74b4cd1fa7e942770976e5e2197ded47503660
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/76692
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/superio/nuvoton/common')
0 files changed, 0 insertions, 0 deletions