Felix Held has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/70133 )
Change subject: [UNTESTED] mb/amd/chausie: change AMD_FWM_POSITION_INDEX for non-chromeos case ......................................................................
[UNTESTED] mb/amd/chausie: change AMD_FWM_POSITION_INDEX for non-chromeos case
Commit 2c102232e8f7 ("mb/amd/chausie,google/skyrim: increase RW_MRC_CACHE size to 120 kByte") increased the MRC cache size, but with the change the default AMD_FWM_POSITION_INDEX which is 5 for the 16MByte flash size, the amdfw part won't be placed on the expected position, since the cbfs header is in that exact location and cbfstool places the amdfw part right after that. Change the AMD_FWM_POSITION_INDEX to 4 for the non-chromeos builds to work around this.
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I89fe1d0672139e04070f05c6c8fa8955edcfc7ee --- M src/mainboard/amd/chausie/Kconfig 1 file changed, 19 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/70133/1
diff --git a/src/mainboard/amd/chausie/Kconfig b/src/mainboard/amd/chausie/Kconfig index da0e0c4..ad4d530 100644 --- a/src/mainboard/amd/chausie/Kconfig +++ b/src/mainboard/amd/chausie/Kconfig @@ -29,6 +29,7 @@ config AMD_FWM_POSITION_INDEX int default 3 if CHROMEOS + default 4 help TODO: might need to be adapted for better placement of files in cbfs