Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36258 )
Change subject: cpu/amd/*/*/acpi/: Use 'Device()' instead of 'Processor()'
......................................................................
Patch Set 5: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/36258/4/src/cpu/amd/pi/00630F01/acp...
File src/cpu/amd/pi/00630F01/acpi/cpu.asl:
https://review.coreboot.org/c/coreboot/+/36258/4/src/cpu/amd/pi/00630F01/acp...
PS4, Line 98: P011, /* name space name */
It looks like this processor only has 4 cores with 1 thread per core.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/36258
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4541372a98f05d3e915c74bc28f41309754014e4
Gerrit-Change-Number: 36258
Gerrit-PatchSet: 5
Gerrit-Owner: HAOUAS Elyes
ehaouas@noos.fr
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Duncan Laurie
dlaurie@chromium.org
Gerrit-Reviewer: HAOUAS Elyes
ehaouas@noos.fr
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Sun, 27 Oct 2019 18:05:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment