build bot (Jenkins) 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 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/41885/1/src/acpi/acpigen_dptf.c File src/acpi/acpigen_dptf.c:
https://review.coreboot.org/c/coreboot/+/41885/1/src/acpi/acpigen_dptf.c@142 PS1, Line 142: static void write_active_relationship_table(const struct dptf_active_policy *policies, int max_count) line over 96 characters
https://review.coreboot.org/c/coreboot/+/41885/1/src/acpi/acpigen_dptf.c@161 PS1, Line 161: for (i = 0; i < max_count; ++i) that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/41885/1/src/include/acpi/acpigen_dp... File src/include/acpi/acpigen_dptf.h:
https://review.coreboot.org/c/coreboot/+/41885/1/src/include/acpi/acpigen_dp... PS1, Line 51: } thresholds [DPTF_MAX_ACX]; space prohibited before open square bracket '['