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

[Benjamin Doron (Code Review)]

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 160:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35523/145/src/mainboard/acer/aspire... 
File src/mainboard/acer/aspire_vn7_572g/acpi/battery.asl:

https://review.coreboot.org/c/coreboot/+/35523/145/src/mainboard/acer/aspire... 
PS145, Line 135: Arg1
> `EB0S`? Only by context, if possible at all.

Done

https://review.coreboot.org/c/coreboot/+/35523/144/src/mainboard/acer/aspire... 
File src/mainboard/acer/aspire_vn7_572g/acpi/ec.asl:

https://review.coreboot.org/c/coreboot/+/35523/144/src/mainboard/acer/aspire... 
PS144, Line 177: SMI
> I suspect that some use of SMM here could be DTS ("digital thermal sensor?"). […]

I've marked this as "not implemented" in the commit message. It does not seem strictly necessary.