summaryrefslogtreecommitdiff
path: root/src/vendorcode/amd/fsp/phoenix
diff options
context:
space:
mode:
authorAlexander Goncharov <chat@joursoir.net>2023-03-16 14:03:20 +0400
committerLean Sheng Tan <sheng.tan@9elements.com>2023-05-18 16:05:02 +0000
commit8414eed5a04004a886cafc65763ff5448ff805dd (patch)
treedc64c9f06e77a4a65f54ec47bd278098fa83ebc1 /src/vendorcode/amd/fsp/phoenix
parent2fca0261f1c08b3f2f09b3fbddd981badb577125 (diff)
cpu/qemu-x86/cache_as_ram_bootblock: drop duplicated post code
Before the bootblock stage starts setting up the CAR mode, it sends `POST_BOOTBLOCK_CAR` POST code. However, before the definition for `POST_BOOTBLOCK_CAR` was introduced in the commit 0d34a50a360228138ade623e799b03eaba83b0a5 , the value `0x20` was used. At that point, `0x20` means "entry into CAR mode" and `0x21` means "the cache memory region is cleared". Right now we are sending the same POST code twice, which makes no sense. So we can do the following (todo: drop me after we decided which one is more appropriate): 1) Drop it (current patchset does exactly that) 2) Introduce POST code similar to POST_SOC_CLEARING_CAR and use it before the cache memory region is cleared. Change-Id: I5d9014c788abdf5a4338c9e199138d1e514450b3 Signed-off-by: Alexander Goncharov <chat@joursoir.net> Reviewed-on: https://review.coreboot.org/c/coreboot/+/73744 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Elyes Haouas <ehaouas@noos.fr> Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Diffstat (limited to 'src/vendorcode/amd/fsp/phoenix')
0 files changed, 0 insertions, 0 deletions