Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33919 )
Change subject: src/mainboard/asus/am1i-a/buildOpts.c: set a board type to AMD_PLATFORM_DESKTOP ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/33919/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/33919/1//COMMIT_MSG@9 PS1, Line 9: incorrect
This option seems to have some effect in the AGESA code on PCIe configuration (look for GfxCardWorka […]
Most mobile boards do not have a PCIe slot (except a MiniPCIe x1 usually reserved for WiFi card) so this workaround is disabled for them because they don't need it. But this MiniITX desktop board has a PCIe slot, so it might benefit from this workaround. Changing to AMD_PLATFORM_DESKTOP will enable some workaround code at ./src/vendorcode/amd/agesa/f16kb/Proc/GNB/Modules/GnbPcieTrainingV2/PcieTrainingV2.c , line 500 : https://review.coreboot.org/cgit/coreboot.git/tree/src/vendorcode/amd/agesa/...
https://review.coreboot.org/#/c/33919/1//COMMIT_MSG@9 PS1, Line 9: desktop
But a mobile processor, right?
CPU itself is a desktop one, Athlon 5370. It has a low TDP of 25W but couldn't be found at laptops.