build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25429 )
Change subject: soc/intel/common/block/acpi: fix P-States extra entry ......................................................................
Patch Set 8:
(3 comments)
https://review.coreboot.org/#/c/25429/8/src/soc/intel/common/block/acpi/acpi... File src/soc/intel/common/block/acpi/acpi.c:
https://review.coreboot.org/#/c/25429/8/src/soc/intel/common/block/acpi/acpi... PS8, Line 330: if (((ratio_max - ratio_min) % ratio_step) > 0) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25429/8/src/soc/intel/common/block/acpi/acpi... PS8, Line 333: if (turbo) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/#/c/25429/8/src/soc/intel/common/block/acpi/acpi... PS8, Line 336: if (num_entries > PSS_MAX_ENTRIES) { braces {} are not necessary for single statement blocks