Attention is currently required from: Hung-Te Lin, Jianjun Wang, Nico Huber, Paul Menzel, Yidi Lin, Yu-Ping Wu.
Yidi Lin has uploaded a new patch set (#15) to the change originally created by Jianjun Wang. ( https://review.coreboot.org/c/coreboot/+/78044?usp=email )
The following approvals got outdated and were removed: Code-Review+1 by Nico Huber, Code-Review+2 by Yu-Ping Wu, Verified+1 by build bot (Jenkins)
Change subject: soc/mediatek: PCI: Fix translation window ......................................................................
soc/mediatek: PCI: Fix translation window
Dojo fails to boot from NVMe with CONFIG_RESOURCE_ALLOCATION_TOP_DOWN enabled. The root cause is using __fls() will get a smaller value when the size is not a power of 2, for example, __fls(0x3000000) = 25. Hence the PCIe translation window size is set to 0x2000000. Accessing addresses higher than 0x2300000 will fail.
Fix translation window by splitting the MMIO space to multiple tables if its size is not a power of 2.
Resolves: https://ticket.coreboot.org/issues/508.
TEST=Build pass and boot up to kernel successfully via SSD on Dojo board, it can boot with and without the CONFIG_RESOURCE_ALLOCATION_TOP_DOWN option.
BUS=b:298255933 BRANCH=cherry
Change-Id: I42b0f0bf9222d284dee0c29f1a6ed6366d6e6689 Signed-off-by: Jianjun Wang jianjun.wang@mediatek.com --- M 3rdparty/amd_blobs M 3rdparty/arm-trusted-firmware M 3rdparty/blobs M 3rdparty/fsp M 3rdparty/intel-microcode M 3rdparty/libgfxinit M 3rdparty/libhwbase M 3rdparty/qc_blobs M 3rdparty/vboot M src/soc/mediatek/common/pcie.c M util/goswid M util/nvidia/cbootimage 12 files changed, 69 insertions(+), 29 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/78044/15