Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41885 )
Change subject: dptf: Add support for generation of Active Policies ......................................................................
Patch Set 10:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41885/10/src/acpi/acpigen_dptf.c File src/acpi/acpigen_dptf.c:
https://review.coreboot.org/c/coreboot/+/41885/10/src/acpi/acpigen_dptf.c@8 PS10, Line 8: #define GNVS_DPTE "\DPTE" I think this should be unused now.
https://review.coreboot.org/c/coreboot/+/41885/10/src/drivers/intel/dptf/dpt... File src/drivers/intel/dptf/dptf.c:
https://review.coreboot.org/c/coreboot/+/41885/10/src/drivers/intel/dptf/dpt... PS10, Line 36: it's its?