build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45340 )
Change subject: /soc/amd/picasso: Generate ACPI pstate and cstate objects in cb ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/45340/2/src/soc/amd/picasso/acpi.c File src/soc/amd/picasso/acpi.c:
https://review.coreboot.org/c/coreboot/+/45340/2/src/soc/amd/picasso/acpi.c@... PS2, Line 320: bytes=0; spaces required around that '=' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/45340/2/src/soc/amd/picasso/acpi.c@... PS2, Line 398: if (CONFIG(ACPI_SSDT_PSD_INDEPENDENT)){ space required before the open brace '{'
https://review.coreboot.org/c/coreboot/+/45340/2/src/soc/amd/picasso/acpi.c@... PS2, Line 398: if (CONFIG(ACPI_SSDT_PSD_INDEPENDENT)){ braces {} are not necessary for any arm of this statement