Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47666 )
Change subject: ACPI: Define acpi_get_preferred_pm_profile() ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1:
Felix, would you like CB:42140 restored instead, with existing comments mostly addressed?
If you think that that would be a better idea, please do so and I'll abandon this one. Compared to the change I pulled from the review system I did some improvement here https://review.coreboot.org/c/coreboot/+/47666/1/src/arch/x86/acpi_pm.c#9 (avoid casting the enum value to int before the switch case that sued the enum values again)
ah, that one wasn't deleted like most of your patches. yeah then it would be better to have your patch restored and rebased instead