Change in coreboot[master]: [WIP] mb/acer: Add Acer Aspire ES1-572

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38978 )

Change subject: [WIP] mb/acer: Add Acer Aspire ES1-572
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38978/7/src/mainboard/acer/es1-572/... 
File src/mainboard/acer/es1-572/devicetree.cb:

https://review.coreboot.org/c/coreboot/+/38978/7/src/mainboard/acer/es1-572/... 
PS7, Line 37: 	register "SataTestMode"       = "1"
> Because I want to test SATA? 😄

well, do you really test that low-level or did you just enable it because it says "test" and you want to test? :D SataTestMode=1 modifies HSIO settings (lane eq boost and others) and buffer settings.

Why not test it in normal mode? (I did; works fine)