Attention is currently required from: Felix Singer, Angel Pons, Michael Büchler. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56838 )
Change subject: mb/acer/g43t-am3: Add documentation ......................................................................
Patch Set 2: Code-Review+1
(4 comments)
Patchset:
PS2: Thank you very much.
File Documentation/mainboard/acer/g43t-am3.md:
https://review.coreboot.org/c/coreboot/+/56838/comment/37ec87cb_8bc7d0cc PS1, Line 30:
Done. Should I add a note how to boot Windows 10? It needs the VGA BIOS and libgfxinit disabled. […]
It could be done in a followup. I guess a generic page for Microsoft Windows would be good, which you could link to.
https://review.coreboot.org/c/coreboot/+/56838/comment/11dc6dee_f46572b4 PS1, Line 41: + PS/2 mouse and keyboard (needs CONFIG_SEABIOS_PS2_TIMEOUT, tested: 500)
Yes there it works. […]
Perfect.
https://review.coreboot.org/c/coreboot/+/56838/comment/961cd187_a8538030 PS1, Line 49: + DDR3 memory with 512Mx8 chips (G43 limitation)
I don't have that particular RAM at hand anymore but I'm pretty sure it doesn't. […]
Thank you for the clarification. I’d just leave it.
-
Paul Menzel (Code Review)