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

[Benjamin Doron (Code Review)]

Attention is currently required from: Nico Huber, Eugene Myers, 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 178:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/35523/comment/b1406314_d77ad008 
PS178, Line 37:
> Thanks for the console log. […]

Sure, thanks for taking a look. I believe that Linux (not GRUB) writes a "DUMMY" variable sometime around ExitBootServices(), which would call SMMSTORE. However, in this case I think that the ringbuffer is just wrapping around. I may have removed the wrapped-around logs, thinking that they were from before a reset. In any case, that boot was successful.

What's interesting to me is that the STM hasn't been activated but the system still fails to boot with the smihandler. But still, it should probably work if the STM is enabled properly and these resources were added to its lists, and I guess not doing so is misbehaviour.

I suppose a config item could also work, instead of hooks.

Sorry for taking so long to get back to you.