Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35864 )
Change subject: mb/lenovo/{t60,r60}: Add ThinkPad R60 support as variant board ......................................................................
Patch Set 7:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35864/7//COMMIT_MSG Commit Message:
PS7: Please obey the 72 chars per line rule of Git.
https://review.coreboot.org/c/coreboot/+/35864/7//COMMIT_MSG@9 PS7, Line 9: - It can be 100 % Open Source. Please clarify what `It` is in this context. I suppose `coreboot` if you exclude microcode updates?
https://review.coreboot.org/c/coreboot/+/35864/7//COMMIT_MSG@12 PS7, Line 12: black on 1400x1050 IPS display[1]. Display works fine on Linux. I don't why it appears like that. Please mention that this is with native graphics init. I assume it is?