Attention is currently required from: Intel coreboot Reviewers, Patrick Rudolph, Vladimir Serbinenko.
Jérémy Compostella has posted comments on this change by Vladimir Serbinenko. ( https://review.coreboot.org/c/coreboot/+/86247?usp=email )
Change subject: intel/acpi: Put BSP as the first entry ......................................................................
Patch Set 4:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86247/comment/7d10ce88_b67d3133?usp... : PS4, Line 9: Linux complains in dmesg as a firmware bug that BSP is not the first entry. Could you include requirement link?
File src/soc/intel/common/block/acpi/cpu_hybrid.c:
https://review.coreboot.org/c/coreboot/+/86247/comment/c2e1dd3b_05ba15d8?usp... : PS4, Line 22: * then followed by Efficeint Cores's APIC IDs. Could you please fix the typo on Efficient since you are editing this comment?
https://review.coreboot.org/c/coreboot/+/86247/comment/bfd99c51_5cd0460a?usp... : PS4, Line 30: * This will be used Please fill paragraph. Line length limit is 96-character.