[coreboot-gerrit] Change in coreboot[master]: mb/acer: Add Acer Aspire VN7-572G

[Benjamin Doron (Code Review)]

Attention is currently required from:  Felix Singer, Nico Huber, Paul Menzel, Angel Pons.
Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35523 )

Change subject: mb/acer: Add Acer Aspire VN7-572G
......................................................................

Patch Set 172:

(1 comment)

Patchset:

PS172:
> The vendor firmware used some I/O port writes to seemingly 'trigger' the EC to handle commands. However, a different firmware phase (DXE) uses `Stall(15)` equivalently - in coreboot, this would be `mdelay` - and reads from the port always return 0xFF.

My mistake, microseconds: `udelay`.
> Therefore, I will remove this.

Essentially, it looks like an ugly way to pause execution. Perhaps the I/O serves some purpose, but if actual `Stall` is equivalent, I don't see it.