[coreboot-gerrit] Change in coreboot[master]: [WIP] mainboard: Add Acer 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] mainboard: Add Acer ES1-572
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38978/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/38978/5//COMMIT_MSG@32 
PS5, Line 32: Keyboard
keyboard works fine

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

https://review.coreboot.org/c/coreboot/+/38978/5/src/mainboard/acer/es1-572/... 
PS5, Line 113: PchSerialIoPci
> However, since you had no payload, it might as well have died there

As already stated on IRC: this is not true. no payload means coreboot would stop at "no payload found."