Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32235 )
Change subject: Mainboard: Add MSI MS-7707 ......................................................................
Patch Set 1:
(7 comments)
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@2 PS1, Line 2: Bluemax Please use your name or a pseudonym.
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@2 PS1, Line 2: 1403092+BlueMax@users.noreply.github.com Can people contact you using that address?
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@7 PS1, Line 7: Mainboard mainboard
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@8 PS1, Line 8: Please mention somewhere, that it is a laptop.
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@9 PS1, Line 9: MSI MS-7707 V1.1 (Medion OEM Akoya P4385D MSN10014555) : SandyBridge Intel P67 (BD82x6x) : Winbond 25Q32BV (4MB) : Fintek F71808A : Intel 82579V Gigabit : NEC uPD720200 USB 3.0 Host Controller : IME 7.0.4.1197 Please format this as a list.
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@16 PS1, Line 16: Please add the board here, you used as template, or mention you used, for example, autoport.
https://review.coreboot.org/#/c/32235/1//COMMIT_MSG@22 PS1, Line 22: coreboot 4.9 (TianoCore) TianoCore from coreboot 4.9? or your commits based on 4.9? Please test with master.