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

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Arthur Heymans, Elyes Haouas, Felix Singer, Lance Zhao, Tim Wawrzynczak.

Kyösti Mälkki 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/18f060f7_08ea6c15 :
PS3, Line 1875: 	acpi_header_t *dsdt_file = (acpi_header_t *)(*(acpi_header_t **)(dsdt_file_arg));
Is there a double cast?

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

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