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

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35523 )

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

Patch Set 54:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35523/45//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35523/45//COMMIT_MSG@28 
PS45, Line 28: - Remaining RAM slots (need other SPD addresses)
> The vendor firmware and dmidecode list 4, but the internet says that there are only 2. […]

Um, RAM slots are a physical thing you can count. For example, a random board off the internets:

https://us.v-cdn.net/6029997/uploads/editor/2l/a82ghszhap9b.jpg

There's two slots, each can take a SO-DIMM. On laptops, there's usually only two SO-DIMM slots.

If there's onboard memory (memory-down), there's usually the equivalent of one or two DIMMs.