build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41388 )
Change subject: acpi: add a function to read a table and verify it ......................................................................
Patch Set 1:
(3 comments)
https://review.coreboot.org/c/coreboot/+/41388/1/src/acpi/acpi.c File src/acpi/acpi.c:
https://review.coreboot.org/c/coreboot/+/41388/1/src/acpi/acpi.c@55 PS1, Line 55: printk(BIOS_ERR, "Invalid %s file: file length(%d) > table_size(%ld)\n", filename, file->length, table_size); line over 96 characters
https://review.coreboot.org/c/coreboot/+/41388/1/src/acpi/acpi.c@59 PS1, Line 59: printk(BIOS_ERR, "Invalid %s file: file length(%d) < table_size(%ld)\n", filename, file->length, table_size); line over 96 characters
https://review.coreboot.org/c/coreboot/+/41388/1/src/acpi/acpi.c@63 PS1, Line 63: printk(BIOS_ERR, "Invalid %s file, signature(%s) does not match %s\n", filename, file->signature, tablename); line over 96 characters