[coreboot-gerrit] Change in ...coreboot[master]: mb/{asrock,intel,kontron}: Include missing <arch/acpi.h>

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31267 )

Change subject: mb/{asrock,intel,kontron}: Include missing <arch/acpi.h>
......................................................................

Patch Set 1:

(1 comment)

can you check if a timeless build produces the same binary with the patch applied?
I'm not 100% sure if changing the order of the includes will change things; since the code quality of x4x and sandybridge is rather good, I don't really expect breakage from this though

https://review.coreboot.org/#/c/31267/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/31267/1//COMMIT_MSG@7 
PS1, Line 7: <arch/acpi.h>
<arch/io.h>