[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 146:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35523/119/src/mainboard/acer/aspire... 
File src/mainboard/acer/aspire_vn7_572g/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/35523/119/src/mainboard/acer/aspire... 
PS119, Line 68: # TODO:	register "DspEndpointDmic" = "3"
> Ack. Is it possible to change that ArrayType value?

In coreboot, it's rather straightforward. See CB:43072. With the vendor firmware, it was easier to patch the kernel.

Based on https://bugzilla.kernel.org/show_bug.cgi?id=201251, I've made some progress. A Kconfig symbol had to be enabled in the kernel and an ALSA topology file has to be added. Unfortunately, I cannot use the default, likely because "minimum channels"="2"