Piotr Król has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80299?usp=email )
Change subject: mb/amd/birman: add Phoenix with openSIL mainboard option
......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4:
Felix, how does the Phoenix code name map to CPU models available on the market? From AMD website: https://www.amd.com/en/product/13036
7940HS is Phoenix. Will your introductory patches cover that CPU? Do you see any limitation regarding my mapping method between the code name and CPU model in light of future OpenSIL CPU support?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/80299?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: I248102e92818b2d395d561a4bf2627f80906b2f7
Gerrit-Change-Number: 80299
Gerrit-PatchSet: 4
Gerrit-Owner: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Fred Reitberger
reitbergerfred@gmail.com
Gerrit-Reviewer: Jason Glenesk
jason.glenesk@gmail.com
Gerrit-Reviewer: Matt DeVillier
matt.devillier@amd.corp-partner.google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: 9elements QA
hardwaretestrobot@gmail.com
Gerrit-CC: Piotr Król
piotr.krol@3mdeb.com
Gerrit-Comment-Date: Mon, 12 Feb 2024 14:52:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment