build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43286 )
Change subject: SRAT revision to 3 ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/c/coreboot/+/43286/1/src/acpi/acpi.c File src/acpi/acpi.c:
https://review.coreboot.org/c/coreboot/+/43286/1/src/acpi/acpi.c@547 PS1, Line 547: int acpi_create_srat_gen_init_struct_(acpi_srat_gen_init_struc_t *gen, u8 node, u8 cpu) 'struc' may be misspelled - perhaps 'struct'?
https://review.coreboot.org/c/coreboot/+/43286/1/src/acpi/acpi.c@550 PS1, Line 550: gen->length = sizeof(acpi_srat_gen_init_struc_t); 'struc' may be misspelled - perhaps 'struct'?
https://review.coreboot.org/c/coreboot/+/43286/1/src/include/acpi/acpi.h File src/include/acpi/acpi.h:
https://review.coreboot.org/c/coreboot/+/43286/1/src/include/acpi/acpi.h@202 PS1, Line 202: typedef struct { please, no space before tabs
https://review.coreboot.org/c/coreboot/+/43286/1/src/include/acpi/acpi.h@270 PS1, Line 270: typedef struct acpi_srat_gen_init_struc { 'struc' may be misspelled - perhaps 'struct'?
https://review.coreboot.org/c/coreboot/+/43286/1/src/include/acpi/acpi.h@279 PS1, Line 279: } __packed acpi_srat_gen_init_struc_t; 'struc' may be misspelled - perhaps 'struct'?