[coreboot-gerrit] Change in coreboot[master]: acpi: add a function to read a table and verify it

[ron minnich (Code Review)]

ron minnich 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 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41388/2/src/acpi/acpi.c 
File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/41388/2/src/acpi/acpi.c@268 
PS2, Line 268: 	file = acpi_read_table(CONFIG_CBFS_PREFIX "/madt.aml", "APIC");
> I didn't know we were doing that for dsdt and slic. […]

so, for now, I'd rather not take on fixing all those problems, but if I take it as an action to fix them *along with this one* can we proceed?

What's the recommended function to use here, btw? I'm happy to make that fix everywhere it's used in this file.

What's your favorite new config variable name? I'm not a huge fan of continuing to add more config variables but if it's seen to be needed I guess we can do it.