[coreboot-gerrit] [S] Change in coreboot[master]: mb/amd/chausie: change AMD_FWM_POSITION_INDEX for non-chromeos case

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/70133 )

(

2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: mb/amd/chausie: change AMD_FWM_POSITION_INDEX for non-chromeos case
......................................................................

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.

TEST=Non-chromeos chausie build now boots and doesn't fail any more
before releasing the x86 cores from reset

Signed-off-by: Felix Held 
Change-Id: I89fe1d0672139e04070f05c6c8fa8955edcfc7ee
Reviewed-on: https://review.coreboot.org/c/coreboot/+/70133
Reviewed-by: Fred Reitberger 
Tested-by: build bot (Jenkins) 
---
M src/mainboard/amd/chausie/Kconfig
1 file changed, 25 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Fred Reitberger: Looks good to me, approved

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