Change in coreboot[master]: apple/macbook21: configure the clockgen and add C3 CPU state - coreboot-gerrit

16 Apr 2022


      Attention is currently required from: Arthur Heymans, Evgeny Zinoviev, Felix Held.
Ferass EL HAFIDI has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63587 )
Change subject: apple/macbook21: configure the clockgen and add C3 CPU state
......................................................................
Patch Set 5:
(1 comment)
File src/mainboard/apple/macbook21/cstates.c:
https://review.coreboot.org/c/coreboot/+/63587/comment/66669d05_471db712 
PS4, Line 34: 		.latency = 17,
...
is this the latency value from the vendor firmware's ACPI?
That latency value works perfectly fine. I don't think I pulled this from the vendor firmware's ACPI however.
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/63587
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib97f95599fbdf66aa55e936bdcb9cc0cd69b3824
Gerrit-Change-Number: 63587
Gerrit-PatchSet: 5
Gerrit-Owner: Ferass EL HAFIDI vitali64pmemail@protonmail.com
Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz
Gerrit-Reviewer: Evgeny Zinoviev me@ch1p.io
Gerrit-Reviewer: Felix Held felix-coreboot@felixheld.de
Gerrit-Reviewer: Nico Huber nico.h@gmx.de
Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org
Gerrit-CC: Paul Menzel paulepanter@mailbox.org
Gerrit-Attention: Arthur Heymans arthur@aheymans.xyz
Gerrit-Attention: Evgeny Zinoviev me@ch1p.io
Gerrit-Attention: Felix Held felix-coreboot@felixheld.de
Gerrit-Comment-Date: Sat, 16 Apr 2022 12:18:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Held felix-coreboot@felixheld.de
Gerrit-MessageType: comment