Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/53933 )
Change subject: soc/amd/picasso: Disable CBFS MCACHE again
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Did this ever work and break again or was it just broken the whole time?
The error message looks like the mcache just wasn't transferred correctly from PSP memory to DRAM. Please check the transfer logic (e.g. hexdump _cbfs_mcache at the end of the PSP execution and right before this assertion, they should be the same but they probably aren't). Or maybe you're trying to make a CBFS access before the code copying the transfer buffer into DRAM runs? (Interestingly, the FMAP access before that runs fine. Are you using FMAP_CACHE? If so, that would suggest that the FMAP cache was transferred correctly already...)
--
To view, visit
https://review.coreboot.org/c/coreboot/+/53933
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I68b4b73670e750207414f0d85ff96f21481be8ce
Gerrit-Change-Number: 53933
Gerrit-PatchSet: 1
Gerrit-Owner: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Jason Glenesk
jason.glenesk@gmail.com
Gerrit-Reviewer: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Julius Werner
jwerner@chromium.org
Gerrit-CC: Kangheui Won
khwon@chromium.org
Gerrit-Attention: Jason Glenesk
jason.glenesk@gmail.com
Gerrit-Attention: Raul Rangel
rrangel@chromium.org
Gerrit-Attention: Marshall Dawson
marshalldawson3rd@gmail.com
Gerrit-Comment-Date: Fri, 07 May 2021 22:48:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment