Attention is currently required from: Tarun Tuli, Nick Vaccaro.
Kevin Chiu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69013 )
Change subject: mb/google/brya/var/lisbon: Enable SaGv
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/69013/comment/4654e777_91659e87
PS1, Line 12: TEST=FW_NAME=lisbon emerge-brask coreboot
Can we add any details on how we confirmed SaGv is operating as we expect (freq/volt change)?
Hi Tarun,
we could assure you that the RMT verification is passed after enabling SaGv, thank you!
--
To view, visit
https://review.coreboot.org/c/coreboot/+/69013
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic7d3203bfe06973b023a38d1aa3d69cce5c3a60c
Gerrit-Change-Number: 69013
Gerrit-PatchSet: 2
Gerrit-Owner: Kevin Chiu
kevin.chiu.17802@gmail.com
Gerrit-Reviewer: Nick Vaccaro
nvaccaro@google.com
Gerrit-Reviewer: Ricky Chang
rickytlchang@google.com
Gerrit-Reviewer: Tarun Tuli
taruntuli@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Tarun Tuli
taruntuli@google.com
Gerrit-Attention: Nick Vaccaro
nvaccaro@google.com
Gerrit-Comment-Date: Tue, 01 Nov 2022 01:35:26 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Tarun Tuli
taruntuli@google.com
Gerrit-MessageType: comment