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

[Nico Huber (Code Review)]

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

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

Patch Set 174:

(2 comments)

File src/mainboard/acer/aspire_vn7_572g/hda_verb.c:

https://review.coreboot.org/c/coreboot/+/35523/comment/ade8ee44_3871795e 
PS174, Line 36: 	/* Undocumented settings, see data blob in
Not easy to decode but this is usually documented in the codec's datasheet.
Note, there are two types of verbs: 4 bit and 12 bit. The former are stored
in bits 19..16 and 15..0 are used for data. The latter use 19..8 with only
8 bits of data.

For instance, the first `0x02050038` is
* codec 0x0
* node 0x20
* verb 0x5
* data 0x0038

https://review.coreboot.org/c/coreboot/+/35523/comment/fe3440fd_3fb1d93d 
PS174, Line 90: 0
This should be `2`, shouldn't it? AFAIK, the code processing this table
will take this literally and send it to codec #0.