[coreboot-gerrit] [L] Change in coreboot[master]: acpi/acpi.c: Reduce boilerplate

[Arthur Heymans (Code Review)]

Attention is currently required from: Elyes Haouas, Felix Singer, Kyösti Mälkki, Lance Zhao, Tim Wawrzynczak.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76127?usp=email )

Change subject: acpi/acpi.c: Reduce boilerplate
......................................................................

Patch Set 3:

(3 comments)

File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/76127/comment/aa2df4b2_797ab0c2 :
PS3, Line 1875: 	acpi_header_t *dsdt_file = (acpi_header_t *)(*(acpi_header_t **)(dsdt_file_arg));
> Is there a double cast?

Done

https://review.coreboot.org/c/coreboot/+/76127/comment/01e141fa_b1ac4fcb :
PS3, Line 1904: 	 * cbfs_unmap() uses mem_pool_free() which works correctly only
> This reversed order is no longer honoured?

Done

https://review.coreboot.org/c/coreboot/+/76127/comment/fa204bf7_2dfd413b :
PS3, Line 1914: 	acpi_header_t *slic_file = (acpi_header_t *)(*(acpi_header_t **)slic_file_arg);
> same

Done