Attention is currently required from: Nick Vaccaro, Nick Vaccaro, Shelley Chen.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78335?usp=email )
Change subject: mb/google/brya: fix MRC cache failure for hynix parts
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/78335/comment/3730318c_cff2278c :
PS2, Line 10: Failure to
: set cs_pi_start_high_in_ect when using these parts will result in an
: MRC cache failure and DUT will fail to boot.
Thank you for debugging. Is that problem known to the manufacturer, or is it the correct behavior and documented in a datasheet?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/78335?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Icb34c7e093e9f030820de7ca627848f0b767fcc4
Gerrit-Change-Number: 78335
Gerrit-PatchSet: 2
Gerrit-Owner: Nick Vaccaro
nvaccaro@google.com
Gerrit-Reviewer: Nick Vaccaro
nvaccaro@chromium.org
Gerrit-Reviewer: Shelley Chen
shchen@google.com
Gerrit-Reviewer: Subrata Banik
subratabanik@google.com
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-CC: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Shelley Chen
shchen@google.com
Gerrit-Attention: Nick Vaccaro
nvaccaro@chromium.org
Gerrit-Attention: Nick Vaccaro
nvaccaro@google.com
Gerrit-Comment-Date: Fri, 13 Oct 2023 06:46:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment