build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40937 )
Change subject: acpi: Move ACPI table support out of arch/x86 (2/5) ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/40937/2/src/include/acpi/acpigen.h File src/include/acpi/acpigen.h:
https://review.coreboot.org/c/coreboot/+/40937/2/src/include/acpi/acpigen.h@... PS2, Line 384: void acpigen_write_dsm(const char *uuid, void (**callbacks)(void *), function definition argument 'void' should also have an identifier name
https://review.coreboot.org/c/coreboot/+/40937/2/src/include/acpi/acpigen.h@... PS2, Line 384: void acpigen_write_dsm(const char *uuid, void (**callbacks)(void *), function definition argument 'void *' should also have an identifier name
https://review.coreboot.org/c/coreboot/+/40937/2/src/include/acpi/acpigen.h@... PS2, Line 438: int get_cst_entries(acpi_cstate_t **); function definition argument 'acpi_cstate_t **' should also have an identifier name