Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29754 )
Change subject: soc/intel/common/acpi: Add common CPU methods ......................................................................
Patch Set 1:
Patch Set 1:
It is a good idea, but I think it is a much larger change than just fixing the cannonlake bug where these methods don't exist, which I could do instead of refactoring it into common code if we want to instead push for a change in this direction.
However, I think calling back into an SSDT from the DSDT is still not allowed by the linux kernel, and currently all the DPTF code is in the DSDT...
calling SSDT from DSDT works just fine.