Maciej Matuszczyk 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 65:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35864/7//COMMIT_MSG Commit Message:
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.
mention it in the commit message?
Done
https://review.coreboot.org/c/coreboot/+/35864/63/Documentation/mainboard/le... File Documentation/mainboard/lenovo/r60.md:
https://review.coreboot.org/c/coreboot/+/35864/63/Documentation/mainboard/le... PS63, Line 16: ![](https://imgur.com/aiEmDa6.jpg)
Should this refer to `r60_chip. […]
Done
https://review.coreboot.org/c/coreboot/+/35864/63/src/mainboard/lenovo/t60/K... File src/mainboard/lenovo/t60/Kconfig:
https://review.coreboot.org/c/coreboot/+/35864/63/src/mainboard/lenovo/t60/K... PS63, Line 35: t60
scratch that, just realized what this is. […]
Yea those boards are 99% the same