Arthur Heymans 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 4:
(2 comments)
Patch Set 4: Code-Review+1
(3 comments)
Patch Set 1:
(3 comments)
I guess you messed up with git, but here are some comments about the gpio.c file.
yea fixed that
No, this is based on a parent that is not on gerrit...
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... File src/mainboard/lenovo/t60/variants/r60/gpio.c:
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... PS1, Line 30: .gpio21 = GPIO_MODE_GPIO, /* LCD_PRESENCE */
Do you have the "LISA-7 FOR SVT" schematics? This is not true in there.
Ack
https://review.coreboot.org/c/coreboot/+/35864/1/src/mainboard/lenovo/t60/va... PS1, Line 30: .gpio21 = GPIO_MODE_GPIO, /* LCD_PRESENCE */
This is R60, not T60. […]
Ack